Вопросы с тегом «code-reviews»

10
Должен ли анализ кода выполняться до или после юнит-тестов

Я спорю со своим коллегой о том, когда выполнять проверку кода - до или после модульных тестов. Какова лучшая практика? Некоторые факторы, которые мы, возможно, должны принять во внимание (может быть больше): Изменение размера кода - большое изменение означает, что больше изменений будет...

10
Есть ли обзоры кода в проектах с открытым исходным кодом? Если да, какие инструменты используются для этого?

Я знаю, что есть большой толчок для обзоров кода в коммерческой разработке. Однако используются ли обзоры кода в программном обеспечении с открытым исходным кодом или основаны на доверии? Если так, то как они выполняются? [Является ли это отложенной фиксацией, «средой предварительной фиксации»,...

10
Как оценить качество кода, когда вы не знакомы с языком? [закрыто]

Закрыто . Этот вопрос основан на мнении . В настоящее время он не принимает ответы. Хотите улучшить этот вопрос? Обновите вопрос, чтобы ответить на него фактами и цитатами, отредактировав этот пост . Закрыто 4 года назад . Гипотетически, если бы я брал интервью у кого-то для новой должности...

10
Кто может просмотреть мой код, если я единственный, кто знает технологию?

В настоящее время я работаю над внедрением React в компании, в которой я работаю. Как единственный, кто прошел курсы по этой теме, кого я могу попросить о проверке, чтобы убедиться, что код соответствует «передовым методам» и правильно...

10
Переработка функции, возвращающей целочисленный код, который представляет множество различных состояний

Я унаследовал некоторый ужасный код, который я включил короткий пример ниже. Есть ли название для этого конкретного анти-паттерна? Какие рекомендации по рефакторингу это? // 0=Need to log in / present username and password // 2=Already logged in // 3=Inactive User found // 4=Valid User...

10
Как оставить отзыв после проверки кода

В настоящее время я рассматриваю код младших разработчиков, которые только что присоединились к моей команде, мне интересно, как мне представить результаты этого обзора: Должен ли я исправить код сам? Должен ли я дать им отзыв о процессе проверки и позволить им исправить в соответствии с моими...

9
Почему не рекомендуется иметь свойство set-only?

Сегодня на работе один из моих коллег просмотрел мой код и предложил удалить свойство «только для набора» и использовать вместо него метод. Поскольку мы оба были заняты другими делами, он сказал мне взглянуть на Property Designраздел из книги «Руководство по разработке структуры». В книге писатель...

9
Рекомендации по проверке кода для CSS, JS и HTML

Меня попросили создать руководство по обзору CSS, JS и HTML. Я знаю, что есть правила кодирования для JS, но я не знаю ничего о HTML и CSS. Чтобы проверить JS, я обязательно последую этим указаниям и упомяну их. Но как насчет CSS и HTML? Кроме логических ошибок и проблем с отступами, есть ли...

9
Что плохого в возвращении хеш-таблицы из открытого метода и когда имеет смысл это делать?

Каковы проблемы проектирования при возврате хеш-таблицы из открытого метода, когда вы хотите вернуть несколько элементов вместо создания класса и возврата объекта этого? Если у него есть проблемы, то при каких обстоятельствах имеет смысл это делать? Как меняется ответ на этот вопрос в зависимости...

9
В чем разница между статическим анализом кода и проверкой кода?

Я просто хотел узнать, в чем разница между статическим анализом кода и проверкой кода. Как каждый из этих двух сделан? В частности, какие инструменты доступны сегодня для обзора кода / статического анализа PHP? Я также хотел бы знать о хороших инструментах для обзора кода для любого...

9
Какие инструменты или методы полезны для описания незнакомой кодовой базы? [закрыто]

Закрыто. Этот вопрос не по теме . В настоящее время он не принимает ответы. Хотите улучшить этот вопрос? Обновите вопрос, чтобы он соответствовал теме разработки программного обеспечения в стеке. Закрыто 5 лет назад . При ручной проверке незнакомого кода (для просмотра или изменения) у меня,...