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

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

336
Как бы вы узнали, если вы написали читаемый и легко обслуживаемый код?

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

188
Как мне справиться с разногласиями в обзоре кода относительно маловероятного крайнего случая?

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

183
Как найти положительные моменты в обзоре кода?

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

177
Как я могу просмотреть свой собственный код? [закрыто]

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

144
Что вы делаете, когда проверка кода слишком сложна?

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

130
Как я могу тактично предложить улучшения плохо продуманного кода других во время проверки?

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

115
Что делать, если код, представленный для проверки кода, кажется слишком сложным?

Код трудно следовать, но он (в основном) работает хорошо, по крайней мере, при поверхностном тестировании. Здесь и там могут быть небольшие ошибки, но по коду кода очень трудно определить, являются ли они симптомами более глубоких проблем или простых исправлений. Проверка общей правильности вручную...

106
Должен ли я указать на ошибки, связанные с правописанием / грамматикой, в чьем-то коде? [закрыто]

Просматривая код сотрудника, я столкнулся с некоторыми орфографическими ошибками в именах функций, а также грамматическими ошибками, такими как «doesUserHasPermission ()» вместо «doUserHavePermission ()» в именах функций и переменных. Должен ли я указать ему на это или я слишком педантичен, замечая...

93
В обзорах кода должен ли рецензент всегда представлять решение проблем? [закрыто]

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

89
Как отказаться от проверки кода, которая, по вашему мнению, не нужна?

Я нахожусь в состоянии, когда меня попросили просмотреть некоторый код, который устраняет проблему, которая, по моему мнению, не существует. Фиксатор, который старше меня, настаивает на том, что его исправление необходимо, но для меня это не более, чем софистика С ++. Частью нашего процесса...

86
Как бы вы отреагировали, если бы кто-то сказал вам, что ваш код - беспорядок?

Я хороший программист, или я так думал раньше. Я всегда люблю программировать. И я хочу узнать много нового о программировании, чтобы сделать меня лучшим программистом. Я изучал программирование в течение 1 года, а сейчас я работаю программистом почти 2 года. Короче говоря, у меня почти 3 года...

85
Насколько важно уменьшить количество строк в коде?

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

76
Какова цель проверки кода

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

71
Обзор до или после принятия кода, что лучше?

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

71
Какой самый эффективный способ проверки кода? [закрыто]

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

64
Есть ли у нас ответственность за улучшение старого кода?

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

64
Наличие флага, указывающего, должны ли мы выдавать ошибки

Я недавно начал работать в месте с некоторыми намного более старшими разработчиками (приблизительно 50 лет). Они работали над критическими приложениями, касающимися авиации, где система не могла выйти из строя. В результате старший программист имеет тенденцию кодировать этот путь. Он имеет...

55
Согласование правил бойскаутов и оппортунистического рефакторинга с обзорами кода

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

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

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