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

55
Должны ли младшие программисты участвовать в качестве рецензентов кода в проектах старших программистов?

Один из членов моей команды, младший программист, обладает впечатляющими навыками программирования для своего уровня опыта. И во время проверок кода я верю в то, чтобы подчеркнуть обучение, а не указывать на ошибки. Но должны ли младшие программисты участвовать в обзорах кода для более старших...

53
Что такое код «зависти к особенностям» и почему он считается запахом кода?

Этот вопрос о SO говорит о том, чтобы исправить то, что OP считал кодом зависти к функциям . Другой пример, где я видел, как цитируется эта изящная фраза, - недавно полученный ответ здесь, в programmers.SE. Несмотря на то, что я добавил комментарий к этому ответу с просьбой предоставить информацию,...

53
Agile Practices: проверка кода - провалить проверку или поднять проблему?

В конце двухнедельного спринта, и у задачи есть проверка кода, в обзоре мы обнаруживаем функцию, которая работает, доступна для чтения, но она довольно длинная и имеет несколько запахов кода. Простая работа рефакторинга. В противном случае задача соответствует определению выполненного. У нас есть...

51
Выпуск проекта с открытым исходным кодом без смущения [закрыто]

Я уже давно работаю над довольно крупным проектом с открытым исходным кодом, и он близок к тому моменту, когда я хотел бы выпустить его. Тем не менее, я самоучка, и я действительно не знаю никого, кто мог бы адекватно рассмотреть мой проект. Несколько лет назад я выпустил небольшой кусочек кода,...

48
Насколько важны положительные отзывы в обзорах кода?

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

47
Какова фактическая ценность согласованного стиля кода

Я являюсь частью команды консультантов, внедряющих новое решение для клиента. Я отвечаю за большинство обзоров кода на клиентской кодовой базе (React и javascript). Я заметил, что некоторые члены команды используют уникальные шаблоны кодирования до такой степени, что я могу выбрать файл случайным...

43
обзор кода с помощью git-flow и github

С обычными git и github я могу сделать обзор кода, просто создав запрос на извлечение из ветви функций, над которой я работаю, в главную ветку. Как бы я делал обзоры кода с помощью git-flow? С таким рабочим процессом, как «Завершение функции git flow», я не понимаю, где на самом деле происходит...

39
В какой момент «конструктивная» критика вашего кода становится бесполезной?

Я недавно начал как младший разработчик. Помимо того, что я являюсь одним из наименее опытных людей в команде, я также женщина, которая сталкивается со всеми своими проблемами, работая в среде, где доминируют мужчины. У меня были проблемы в последнее время, потому что я чувствую, что получаю...

39
Нужны ли обзоры кода для начинающих разработчиков?

Я работал в двух компаниях, у каждой из которых была своя методология, когда дело дошло до анализа кода: В первой компании руководители группы проводили проверку кода, которая требовалась после завершения каждого модуля. Однако во второй компании руководителям группы не требовалось проводить...

39
Работа в качестве единственного разработчика: просмотр кода

У меня нет иного выбора, кроме как работать самостоятельно, и я не могу найти адекватного решения для того, чтобы осмотреть мою работу, проверить работоспособность, найти кого-то для мозгового штурма идей, обсуждения лучших практик и так далее. Я думал, что получу ответ через статью Джеффа Этвуда:...

37
Что вы скажете в обзоре кода, когда другой человек построил слишком сложное решение? [закрыто]

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

36
Уместно ли иметь положительные комментарии в обзорах кода или это исключительно для конструктивной критики?

В последнее время я много пересматривал код, и я не уверен в положительных и отрицательных эффектах и ​​профессионализме добавления положительных и / или забавных комментариев в обзоры кода. Мы используем Github в качестве нашей платформы для проверки кода в моей команде, поэтому комментарии могут...

33
Как справиться с проверкой кода на моем новом месте, когда я пришел из этой практики?

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

33
Когда делать проверки кода при непрерывной интеграции?

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

32
Что я должен делать, ожидая обзора?

Прежде чем задать свой вопрос, я должен объяснить ситуацию. Я работаю в компании младшим инженером-программистом. Один из старших всегда останавливает меня, когда я заканчиваю свое развитие и хочу посвятить себя. Он всегда хочет, чтобы я подождал, пока он его рассмотрит. Это нормально, потому что...

32
Является ли проверка кода хорошей практикой?

Когда в компании, где я работаю, были наняты новые менеджеры, они предлагали нам просматривать чей-то код на каждой встрече. У нас есть встречи каждые две недели, поэтому каждый раз один из разработчиков должен был показывать свой код на проекторе, а другие собирались его обсудить. Я подумал, что...

30
Работа с коллегами, у которых нет единого стиля кодирования?

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

29
Должны ли ваши лучшие программисты проверять чужой код в системе контроля версий?

Одно из различий между svn и git - это возможность контролировать доступ к хранилищу. Сложно сравнивать эти два, потому что есть разница во взглядах на то, кому вообще разрешено вносить изменения! Этот вопрос касается использования git в качестве централизованного хранилища для команды где-то в...

28
Как эффективно отслеживать рецензирование кода?

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

27
Peer / Code Review разочарования

Я бы не назвал себя суперзвездным разработчиком, но относительно опытным. Я стараюсь поддерживать качество кода на высоком уровне и всегда стараюсь улучшить мой стиль кодирования, стараюсь сделать код эффективным, читаемым и последовательным, а также побуждаю команду следовать шаблонам и...