Запитання з тегом «code-reviews»

Цей тег призначений для запитань щодо практики перегляду коду та покрокових інструкцій. Для огляду існуючого, робочого коду, перегляньте сторінку http://codereview.stackexchange.com

19
Що ви говорите в огляді коду, коли інша людина створила надто складне рішення? [зачинено]
Днями я переглянув код, який хтось із моєї команди написав. Рішення було не повністю функціональним, і дизайн був надто складним - це означає, що зберігалася непотрібна інформація, вбудовані непотрібні функції, і в основному код мав безліч зайвих складностей, таких як позолочення, і він намагався вирішити проблеми, які не існують. У …

5
Чи доречно мати позитивні коментарі в оглядах коду чи це виключно для конструктивної критики?
Останнім часом я багато перевіряю код, і я не впевнений у позитивних та негативних ефектах та професіоналізмі розміщення позитивних та / або смішних коментарів у оглядах коду. Ми використовуємо Github як нашу платформу для перегляду коду в моїй команді, тому коментарі може бачити кожен. Я, як правило, намагаюся використовувати цю …

10
Як боротися з відсутністю оглядів коду в моєму новому місці, коли я приїхав з цієї практики?
Команда моєї нової компанії не має процесу перегляду коду. Я приїжджаю з компаній, які переглядають код як культуру обов'язкових, і тому мені не зручно вводити код, не перевіряючи його хтось. Я твердо вірю, що перегляд коду - це спосіб покращити якість та заощадити час, оскільки він вирішує потенційні проблеми раніше …

11
Коли робити огляд коду під час постійної інтеграції?
Ми намагаємось перейти до середовища безперервної інтеграції, але не впевнені, коли робити огляд коду. З того, що я читав про безперервну інтеграцію, ми повинні намагатися перевіряти код так часто, як кілька разів на день. Я припускаю, що це означає навіть для функцій, які ще не завершені. Тож питання полягає в …

9
Що мені робити, чекаючи перегляду?
Перш ніж ставити своє запитання, я повинен пояснити ситуацію. Я працюю в компанії молодшим програмним інженером. Один із людей похилого віку завжди зупиняє мене, коли я закінчую розвиток і хочу взяти на себе зобов’язання. Він завжди хоче, щоб я дочекався його перегляду. Це нормально, тому що зазвичай він знаходить деякі …

12
Чи перегляд коду є хорошою практикою?
Коли компанія, в якій я працюю, найняла нових менеджерів, вони пропонували нам переглянути чийсь код на кожній зустрічі. У нас зустрічі проводяться кожні два тижні, тому кожен раз, коли один із розробників мав показувати його / її код на проекторі, а інші збиралися обговорювати його. Я думав, що це було …

17
Зв'язок з колегами, які не мають послідовного стилю кодування?
Що ви робите, коли працюєте з тим, хто схильний писати стилістично поганий код? Код, про який я говорю, зазвичай технічно правильний, розумно структурований і може бути навіть алгоритмічно елегантним, але він виглядає просто некрасиво . У нас є: Суміш різних угод про іменах і назвах ( underscore_styleі , camelCaseі , …

11
Чи повинні ваші найкращі програмісти перевіряти код усіх інших у контролі джерела?
Однією з відмінностей svn і git є можливість контролювати доступ до сховища. Важко порівняти ці два, тому що існує різниця в поглядах щодо того, кому взагалі слід дозволити вносити зміни! Це питання стосується використання git як централізованого сховища для команди в компанії десь. Припустимо, що члени команди мають різний рівень …

7
Як ефективно контролювати перегляд коду?
Я підозрюю, що в моїй команді приховується головний перегляд коду. Занадто багато оглядів коду об'єднано без коментарів. Мені здається, що немає такого поняття, як огляд коду без жодного коментаря. Як я можу як команда вести належний моніторинг того, що моя команда виконує належний процес перегляду коду і як я можу …

7
Фрустрація з рецензування експертного коду
Я б не називав себе суперзірковим розробником, але відносно досвідченим. Я намагаюся підтримувати якість коду на високому рівні, і завжди прагну вдосконалити мій стиль кодування, намагаюся зробити код ефективним, читабельним та послідовним, а також заохочую команду дотримуватися шаблонів та методологій, щоб забезпечити послідовність. Я також розумію необхідність балансу між якістю …

17
Який хороший, стислий спосіб пояснити небезпеку програмування копіювальної пасти для непрограмістів? [зачинено]
Закрито . Це питання має бути більш зосередженим . Наразі відповіді не приймаються. Хочете вдосконалити це питання? Оновіть питання, щоб воно зосередило увагу на одній проблемі, лише відредагувавши цю публікацію . Закрито 6 років тому . Я шукаю хорошу аналогію чи метафору, яка могла б проілюструвати проблеми програмування копіювальної пасти …

5
Як поступово вводити огляди коду?
Я веду в команді з півдесятка старших інженерів. Я дуже вірю, що нам було б дуже корисно робити огляд коду з усіх стандартних причин. Не обов’язково кожну зміну, але хоча б постійний потік фонових оглядів. Тож люди принаймні бачать зміни інших і починають говорити про них. Чи є хороший спосіб …

8
Як поводитися з тим, хто не любить ідею перегляду коду?
Очевидно, якщо керівництво купує витрачати час на огляди коду, то всі повинні це робити. Але завжди є такі хлопці (або гелі), які чинять опір кожному унції свого буття. Як ви ефективно керуєте цим сценарієм, коли вирішуєте його як рецензента?

9
Що має бути першим: тестування чи перегляд коду?
Я зовсім новачок у програмуванні моделей дизайну та життєвих циклів, і мені було цікаво, що слід зробити спочатку, перегляд коду чи тестування щодо того, що це роблять окремі люди? З одного боку, навіщо турбувати перегляд коду, якщо ніхто не перевіряв, чи він навіть працює? З іншого, деякі помилки можна знайти …

12
Як переглянути код, який ви не розумієте?
Мені відведено роль покращити розвиток нашої компанії. Перше, що я хотів почати - це огляди коду, оскільки цього раніше ніколи не робилося. В нашій компанії є 3 програмісти. Я веб-програміст, мої відомі мови - це переважно PHP, ActionScript та JavaScript. Інші 2 розробники пишуть внутрішні додатки на VB.net Ми проводимо …

Використовуючи наш веб-сайт, ви визнаєте, що прочитали та зрозуміли наші Політику щодо файлів cookie та Політику конфіденційності.
Licensed under cc by-sa 3.0 with attribution required.