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

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

19
Як би ви дізналися, чи написали читабельний та легкодоступний код?
Звідки можна знати, чи створений код легко читабельний, зрозумілий та підтримуваний? Звичайно, з точки зору автора, код є читабельним та доступним для догляду, адже автор його написав і відредагував, для початку. Однак повинен бути об'єктивний і кількісно вимірюваний стандарт, за допомогою якого наша професія може вимірювати код. Ці цілі досягаються, …

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

20
Як знайти позитивні речі в огляді коду?
Після серйозних проблем з якістю в останній рік моя компанія нещодавно представила огляди коду. Процес перегляду коду був швидко введений, без настанов чи будь-якого виду контрольного списку. Ще один розробник і я вирішили переглянути всі зміни, внесені до систем, перш ніж їх об'єднати в магістраль. Нас також обрали "Технічним керівником". …
184 code-reviews 

19
Як переглянути власний код? [зачинено]
Я працюю над сольним проектом і маю підтримувати власний код. Зазвичай перевірку коду проводить не автор коду, тому рецензент може дивитися на код свіжими очима - однак, у мене немає такої розкоші. Які практики я можу використовувати для більш ефективного перегляду власного коду?

16
Що ви робите, коли перегляд коду надто важкий?
Добре, тому багато перевірки коду є досить рутинним. Але іноді трапляються зміни, які широко впливають на існуючий складний, крихкий код. У цій ситуації кількість часу, яке знадобиться для перевірки безпеки змін, відсутність регресу тощо, є надмірною. Можливо, навіть перевищення часу, необхідного для самої розробки. Що робити у цій ситуації? Злитися …

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

8
Що робити, коли код, поданий на перевірку коду, виявляється занадто складним?
Код важко дотримуватися, але, здається, він (в основному) працює добре, принаймні, з поверхневим тестуванням. Тут можуть бути невеликі помилки, але дуже важко сказати, читаючи код, чи вони є симптоматикою більш глибоких проблем або простих виправлень. Перевірка загальної коректності вручну за допомогою перегляду коду є дуже складною і трудомісткою, якщо це …
115 code-reviews 


16
Чи слід зазначити помилки, пов'язані з написанням / граматикою в чиємусь коді? [зачинено]
Переглядаючи код співробітника, я зіткнувся з деякими орфографічними помилками у назвах функцій, а також граматичними помилками на зразок 'didUserHasPermission ()' замість 'doesUserHavePermission ()' у функціях та назвах змінних. Чи варто вказати на нього або я занадто педантичний, помічаючи їх?

11
Чи повинен рецензент в оглядах коду представляти рішення питань? [зачинено]
Переглядаючи код, я зазвичай намагаюся дати конкретні рекомендації щодо вирішення проблем. Але через обмежений час, який можна витратити на огляд, це не завжди працює добре. У цих випадках я вважаю його більш ефективним, якщо розробник сам знайде рішення. Сьогодні я ознайомився з деяким кодом і виявив, що клас, очевидно, не …

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

21
Як би ви відреагували, якби хтось сказав, що ваш код - безлад?
Я хороший програміст, або так я думав раніше. Я завжди люблю програмувати. І я хочу дізнатися багато речей про програмування, щоб зробити мене кращим програмістом. Я навчався програмуванню 1 рік, і зараз працюю програмістом майже 2 роки. Отже, коротко, у мене майже 3 роки досвіду програмування. Наша команда складається з …

20
Наскільки важливо зменшити кількість рядків у коді?
Я розробник програмного забезпечення, який працює на J2SE (core java). Часто під час огляду коду нас просять зменшити кількість рядків у нашому коді. Йдеться не про видалення надлишкового коду, це про дотримання стилю, який зосереджений на тому, щоб робити ті самі речі з меншою кількістю рядків у коді, тоді як …

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

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

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