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

Этот тег предназначен для вопросов о практике проверки кода и пошаговых руководствах. Для обзоров существующего, рабочего кода, пожалуйста, смотрите http://codereview.stackexchange.com

4
Как обработать TODO в запросе на включение?
Этот вопрос перенесен из Exchange Stack Exchange для обеспечения качества программного обеспечения и тестирования, поскольку на него можно ответить в Exchange Stack Exchange. Мигрировал в прошлом году . Когда я просматриваю изменения в запросе на удаление, я иногда натыкаюсь на комментарий с пометкой «TODO», которая может быть там по разным …

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

4
Обзоры приложений / кодов для одиноких программистов?
Существуют ли какие-либо услуги, которые «по разумной цене» будут предоставлять и предоставлять полезные и технические рекомендации по применению. Во многих проектах я обычно являюсь единственным разработчиком, и иногда я думаю, что некоторые из моих работ необходимо улучшить для повышения эффективности, улучшения взаимодействия с MVC и т. Д. Было бы замечательно, …

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

3
Стратегия просмотра кода перед слиянием с мастером из функциональных веток
Я и моя команда используем функциональные ветви (с git). Мне интересно, какая стратегия лучше всего подходит для проверки кода перед слиянием с мастером. Я извлекаю новую ветку от мастера, давайте назовем ее fb_ # 1 Я совершаю несколько раз, а затем хочу объединить его с мастером Перед слиянием кто-то должен …

3
Github-подобные «запросы на получение» без Github
Я работаю аналитиком в финансовом учреждении, которое из-за чувствительности данных не будет хранить данные в облаке. Тем не менее, я добился некоторого успеха, заставив мою команду использовать Git для управления кодом. Мне было интересно, есть ли какой-нибудь способ реализовать Github-подобные запросы на нашем собственном сервере. Особая особенность, которая меня интересует, …
21 git  code-reviews 

17
Как убедить программистов следовать основным правилам
Есть несколько правил, которые я должен постоянно просить программистов следовать очень часто. Они пишут код и, если он работает, работа для них только что сделана. Основными правилами могут быть: Фиксация изменений Не писать проблемы с моделью в представлении или контроллерах Избегайте жесткого кодирования Можете ли вы рассказать мне о своем …

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

4
Как мне исправить код от менее опытного программиста?
Немного предыстории: я один из двух программистов для нашего отдела из 10 человек (остальные художники и менеджмент). Мы вдвоем делаем все необходимое для правильного кодирования и разрабатываем любые проекты, которые появятся. Я занимаюсь программированием около 4 лет, где это его первая «настоящая» работа (как он выразился). Обычно мы работаем над …
19 code-reviews  bug 

5
Что содержит стандартный обзор кода?
В моей компании это обсуждение по электронной почте того, какая функция реализована и какая ошибка исправлена ​​и отправлена ​​тем, кто пишет код. И рецензент, который получает почту, рассмотрит код и обсудит качество и то, как редактировать код, по его мнению. Что содержит стандартный обзор кода?

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

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

12
Должны ли мы попытаться просмотреть весь наш код?
В настоящее время мы модифицируем процесс разработки, и мне интересно, должны ли мы постараться сохранить 100% проверок наших коммитов. Каков ваш опыт в отношении обзоров кода? Вы тратите на них «много» времени (скажем, 1/2 часа в день) или просто скользите в течение максимум 5/10 минут? У вас есть фиксированное количество …

2
Рекомендуемый процесс для обзоров кода с Mercurial
Обычно мы использовали Perforce и SmartBear's Code Collaborator, Big Corpи теперь мы также собираемся использовать Mercurial для определенных проектов. Code Collaborator поддерживает Mercurial (мы используем версию 5), и я пытаюсь определить, когда лучшее время (во время фиксации / отправки на сервер) процесс является наилучшим / эффективным временем для проверки кода …

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

Используя наш сайт, вы подтверждаете, что прочитали и поняли нашу Политику в отношении файлов cookie и Политику конфиденциальности.
Licensed under cc by-sa 3.0 with attribution required.