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

Каков будет лучший способ управления проверенным исходным кодом в репозитории исходного кода? Должен ли исходный код пройти процесс проверки перед регистрацией, или же проверка кода должна произойти после того, как код будет зафиксирован? Если проверка происходит после регистрации кода в репозитории, то как это следует отслеживать?

12 version-control  code-reviews  svn 

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

12 code-reviews 

Распространенным сценарием в разработке программного обеспечения является проверка кода чужого кода. Распространенным инструментом для этого является открытие запроса на извлечение. Мой вопрос, когда проблемы обнаружены в обзоре, должны ли изменения быть зафиксирован отдельно (новый коммит) или если существующий коммит будет изменен (при условии, что никто не выходит из вашего предыдущего …

11 git  code-reviews  repository 

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

11 teamwork  code-reviews 

Я имею в виду все, а не только изменения схемы. Даже простой SELECT по первичному ключу не может быть запущен в производство, даже если он был проверен кодом другими разработчиками (в контексте), без проверки DBA каждого оператора, извлеченного из кода и переданного с выводом EXPLAIN, подробности как часто это будет …

11 code-reviews  dba 

Иногда есть 1% кода, который является достаточно интенсивным в вычислительном отношении и требует самого тяжелого вида низкоуровневой оптимизации. Примерами являются обработка видео, обработка изображений и все виды обработки сигналов в целом. Цель состоит в том, чтобы документировать и обучать методам оптимизации, чтобы код не становился неуправляемым и не склонным к …

11 documentation  code-reviews  optimization 

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

11 code-reviews 

У нас есть зависимость от стороннего сервиса, который предоставляет гигантский интерфейс, который нам нужен только как 3 метода. Кроме того, интерфейс часто меняется ... Я решил обернуть интерфейс в класс в нашем проекте и раскрыть только те методы, которые нам нужны. Но я не уверен, как я должен обрабатывать возвращаемые …

11 c#  design-patterns  programming-practices  code-reviews  wrapper 

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

10 code-reviews  process 

Я унаследовал некоторый ужасный код, который я включил короткий пример ниже. Есть ли название для этого конкретного анти-паттерна? Какие рекомендации по рефакторингу это? // 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 …

10 coding-style  code-quality  coding-standards  code-reviews  code-smell 

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

10 code-reviews 

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

10 code-quality  interview  code-reviews 

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

10 open-source  code-reviews  source-code 

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

10 language-agnostic  code-reviews 

Я ищу некоторые идеи здесь. Я прочитал статью. Как должны проводиться проверки кода и какие проверки кода, в чем преимущества? которые были очень информативными, но мне все еще нужно больше ясности в вопросе ниже. Мой вопрос Будучи целевым разработчиком, можете ли вы предложить некоторые передовые практики, которые разработчик может использовать, …

10 c#  code-reviews