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

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

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

7
Каков наилучший способ для программиста справиться с проверкой кода?
Я довольно новичок в обзоре кода, но я уже много лет занимаюсь программированием, что не всегда делает вас хорошим программистом! Если рецензент изменит ваш код и построчно проведет его с вами, что вы будете делать, если не согласны? Иногда вы делали выбор X, и рецензент меняет его на Y, и …

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

3
Является ли обзор кода, который использует только комментарии кода хорошей идеей?
Предпосылками Команда использует DVCS IDE поддерживает анализ комментариев (например, TODO и т. Д.) Такие инструменты, как CodeCollaborator, дороги для бюджета Инструменты типа gerrit слишком сложны для установки или не пригодны для использования Workflow Автор публикует где-то в центральной ветке репо Рецензент получить его и начать обзор В случае возникновения какого-либо …

7
Как мне описать процесс изучения чужого кода? (В ситуации выставления счетов.) [Закрыто]
Закрыто. Этот вопрос не по теме . В настоящее время не принимает ответы. Хотите улучшить этот вопрос? Обновите вопрос, чтобы он соответствовал теме разработки программного обеспечения в стеке. Закрыто 4 года назад . Редактировать: Джастин Кейв подчеркнул, что такого рода коммуникации должны быть в моих цитатах / оценках. В этом …

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

4
Лучшая практика для хранения DateTime на основе TimeZone
Разработка веб-приложения, которое должно позволять пользователю планировать встречу на основе своего часового пояса. И я сохраняю запланированное пользователем время и дату как серверное время в поле базы данных. При отображении расписания информация извлекает значение из базы данных и преобразует в пользовательский тимзон. Обработка в базе кода Я конвертирую DateTime на …

5
Когда делать Code Review
Недавно мы перешли к скрам-процессу и работаем над задачами и пользовательскими историями внутри спринтов. Мы хотели бы делать обзоры кода часто, чтобы сделать их менее пугающими. Мы думаем, что делаем их на уровне истории пользователя, но не уверены, как разветвить наш код, чтобы учесть это. Мы используем VS и TFS …

4
Продвижение практики проверки кода и модульного тестирования
Как команда, управляющая группой разработчиков, не имеющих опыта (и не видящего необходимости) в проверке кода и модульном тестировании, как вы можете продвинуться в проверке кода и практике модульного тестирования? Как вы собираетесь создать способ, чтобы проверка кода и модульное тестирование естественно вписывались в поток разработчиков? Одним из препятствий в этих …

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

6
Чистый код - я должен изменить литерал 1 на константу?
Чтобы избежать магических чисел, мы часто слышим, что мы должны дать литералу осмысленное имя. Такие как: //THIS CODE COMES FROM THE CLEAN CODE BOOK for (int j = 0; j < 34; j++) { s += (t[j] * 4) / 5; } -------------------- Change to -------------------- int realDaysPerIdealDay = 4; …

5
Проверка кода отстает от цикла доставки / тестирования
В нашем гибком процессе у нас есть 2-недельные спринты. Задачи выполняются ежедневно (ежедневные сборки), и группа тестирования завершает свое тестирование немедленно на следующий день или даже в тот же день. У нас также есть проверки кода Dev, которые требуют некоторого времени (1-2 часа), поэтому они запланированы 3 раза в неделю: …

1
Устраняет ли парное программирование необходимость проверки кода в проекте Extreme Programming (XP)?
В экстремальном проекте программирования программисты большую часть времени занимаются парным программированием. Поскольку эти пары также вращаются, то есть вы объединяете программу с разными людьми, и возникает чувство коллективной собственности, исходный код часто пересматривается и обновляется. В таком случае, нужны ли обзоры кода? Я имею в виду, прекратить программирование и просто …

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

6
Рецензирование для тестов так же, как обзоры кода
Кто-нибудь практикует процесс «проверки кода» для функциональных тестов? Считаете ли вы это полезным? То, как мой нынешний работодатель практикует SCRUM, мы включаем функциональное тестирование как часть нашего «должно быть сделано» в любом заданном спринте.

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