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

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

7
Яка найкраща практика для "перегляду" вихідного коду у сховищі управління джерелом?
Що буде найкращим способом управління переглянутим вихідним кодом у сховищі управління джерелом? Чи повинен вихідний код пройти процедуру огляду перед тим, як перевіритись, або чи слід переглянути код після введення коду? Якщо огляд відбувається після реєстрації коду у сховищі, то як це слід відстежувати?

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

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

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

10
Кожен оператор SQL повинен переглядатись за допомогою DBA - загального?
Я маю на увазі все, а не лише зміни схем. Навіть простий SELECT на первинному ключі не може розпочати виробництво, навіть якщо його переглянули коди інших розробників (у контексті), без огляду DBA кожного твердження, вилученого з коду та поданого з висновком EXPLAIN, подробиці про те, як часто його називатимуть тощо, …
11 code-reviews  dba 

4
Як документувати та навчати інших "обчислюваним до невпізнання" обчислювально інтенсивним кодом?
Інколи є 1% коду, який обчислювально досить інтенсивний, що потребує найважчого виду оптимізації низького рівня. Приклади - це обробка відео, обробка зображень і всі види обробки сигналів загалом. Цілі полягають у тому, щоб документувати та навчити методи оптимізації, щоб код не став неможливим та схильним до видалення новішими розробниками. (*) …

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

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

3
Як надати відгук після перегляду коду
В даний час я переглядаю деякі з кодексів молодших розробників, які щойно приєдналися до моєї команди, мені цікаво, як мені забезпечити результати цього огляду: Чи повинен я сам виправити код? Чи повинен я надати їм відгуки про процес перегляду та дозволити їм виправляти згідно моїх інструкцій? І якщо так, то …

4
Переробка функції, що повертає цілий код, який представляє багато різних статусів
Я успадкував якийсь жахливий код, до якого я включив короткий зразок нижче. Чи існує назва цього анти-шаблону? Які рекомендації щодо цього рефакторингу є? // 0=Need to log in / present username and password // 2=Already logged in // 3=Inactive User found // 4=Valid User found-establish their session // 5=Valid User …

1
Хто може переглянути мій код, якщо я єдиний, хто знає технологію?
Зараз я працюю над впровадженням React у компанії, в якій працюю. Як єдиний, хто пройшов курси з цієї теми, хто можу попросити рецензію, щоб переконатися, що код дотримується "найкращих практик" та чи правильно його застосовується?

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

3
Чи є огляди коду у проектах з відкритим кодом? Якщо так, то які інструменти використовуються для цього?
Я знаю, що в комерційній розробці є великий поштовх для перегляду коду. Однак чи використовуються огляди коду у програмному забезпеченні з відкритим кодом або ґрунтуються на довірі? Якщо так, то як вони виконуються? [Чи є відкладений фіксатор, "середовище, яке попередньо виконує", чи є інструмент, який дозволяє відправити виправлення на інший …

5
Чи слід проводити перевірку коду до або після тестових одиниць
Я обговорюю з колегою питання про те, коли пройти перевірку коду - до або після тестування одиниць. Яка найкраща практика? Деякі фактори, які нам можуть знадобитися враховувати (їх може бути більше): Розмір зміни коду - велика зміна означає, що в результаті огляду коду вийде більше змін. Якщо ці зміни є …

3
Готуєтесь до огляду коду як розробника?
Я шукаю тут кілька ідей. Я прочитав статтю Як випливає перевірка коди Здійснюватиметься і код огляди, які переваги? які були дуже інформативними, але я все ще потребую більшої чіткості щодо наведеного нижче питання. Моє запитання: Будучи цільовим розробником, ви можете запропонувати деякі найкращі практики, які розробник може включити до того, …
10 c#  code-reviews 

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