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

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

71 code-quality  code-reviews 

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

64 c#  design  object-oriented-design  exceptions  code-reviews 

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

64 code-quality  code-reviews 

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

55 teamwork  code-reviews  pair-programming 

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

55 refactoring  code-reviews 

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

55 code-reviews 

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

53 agile  code-reviews 

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

53 refactoring  code-reviews  clean-code  code-smell 

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

51 open-source  self-improvement  code-reviews  personal-projects 

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

48 code-reviews 

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

47 javascript  code-quality  code-reviews  reactjs 

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

43 code-quality  git  code-reviews  github  gitflow 

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

39 code-reviews  solo-development 

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

39 code-reviews  junior-programmer 

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

39 code-quality  self-improvement  teamwork  code-reviews