code review

  • Хабрахабр

    А компетентен ли советчик? Проблемы рекомендации «не изобретай велосипед»

    Иногда с ярко выраженным пренебрежением и самоутверждением, иногда, якобы, как дельный совет. Довольно часто приходится встречать рекомендацию "не изобретать велоcипед". Однако, пусть и призванная быть советом, данным из благих побуждений, в ряде контекстов она лишь показывает некомпетентность говорящего. Вкладываемое назначение фразы — уберечь от выполнения бесполезной работы, призыв воспользоваться готовым решением для поставленной задачи, и с точки зрения стороннего наблюдателя…

    Подробнее »
  • Хабрахабр

    [Перевод] Как сделать код-ревью быстрее и эффективнее

    Вы отправляете пул-реквест, получаете обратную связь, вносите исправления, отправляете фиксы на повторный ревью, затем получаете одобрение, и происходит мерж. Как обычно происходит код-ревью? Звучит просто, но на деле процесс ревью бывает очень трудоемким. Ревьюер должен потратить немало времени, чтобы полностью прочитать код и понять предлагаемые изменения. Представьте, что у вас есть пул-реквест с сотнями строк изменений. И велики шансы, что…

    Подробнее »
  • Хабрахабр

    Шестая проверка Chromium, послесловие

    В начале 2018 года в нашем блоге появился цикл статей, посвящённый шестой проверке исходного кода проекта Chromium. Цикл включает в себя 8 статей, посвященных ошибкам и рекомендациям по их предотвращению. Две статьи вызвали бурное обсуждение, и до сих пор на почту мне изредка приходят комментарии касательно тем, затронутых в них. Пожалуй, следует дать некоторые дополнительные пояснения и, как говорится, расставить…

    Подробнее »
  • Хабрахабр

    [Видео] Почему взрываются ракеты, что скоро появится в Kotlin и как спасти код ревью

    Там говорили вот о чём: 6 декабря мы провели очередной Java-митап. о разработке Moira — системы экстренного реагирования на инциденты (про ракеты — здесь); о контрактах в Kotlin, задачах, проблемах и улучшениях для DSL; о том, как роботом выбирать ревьюеров в большой команде разработчиков; о том, как научить все компоненты генерировать графики и метрики на боевой среде; о правильной обратной связи для обнаружения проблемных релизов. В этом посте — пять докладов, которые сделают вашу жизнь лучше,…

    Подробнее »
  • Хабрахабр

    Я порчу разрабам жизни своими код ревью и больше так не хочу

    Каждый мой коммент на ревью к его коду был гвоздем в крышку гроба. Однажды в моей команде был настолько слабый парень, что его собирались уволить (уволить разработчика!). Он был приятным человеком, я даже переживал за него, но это не мешало мне разносить его старания в щепки. Я почти слышал стук молотка, нажимая «submit review». Все просто — я сильнее, я…

    Подробнее »
  • Хабрахабр

    Как PVS-Studio оказался внимательнее, чем три с половиной программиста

    PVS-Studio, как и другие статические анализаторы кода, часто выдаёт ложные срабатывания. Но не стоит спешить считать странные срабатывания ложными. Это короткая история о том, как PVS-Studio вновь оказался внимательнее нескольких человек. Письмо, написанное в поддержку, изначально попало к Евгению Рыжкову, который, бегло прочитав его и не заметив аномального в фидбеке, сразу переслал его ведущему разработчику Святославу Размыслову. Нам в поддержку…

    Подробнее »
  • Хабрахабр

    Все люди не умеют писать код

    В преддверии Moscow Python Conf ++ мы поговорили с Никитой Соболевым, CTO компании «Мы делаем сервисы», о глобальной проблеме управления сложностью кода в разрезе развития языков программирования. А также о том, почему тут со временем ситуация становится только хуже. Плюс расспросили, зачем ему потребовалось создавать собственный линтер. — Расскажи в двух словах о себе и своей работе Озвучивая название компании,…

    Подробнее »
  • Хабрахабр

    Процесс ревью кода в hh.ru

    Мне на глаза попался документ с правилами и рекомендациями по процессу ревью кода внутри компании. Я решил, что такой полезной информацией надо поделиться с внешним миром. С благословения автора я публикую работу. Большинство указанных правил носят рекомендательный характер и надо руководствоваться в первую очередь здравым смыслом, а не слепым соблюдением. Общие положения В hh.ru ревью проходит в формате пулл-реквестов на…

    Подробнее »
  • Хабрахабр

    Авторы игры 0 A.D. — молодцы

    0 A.D. — это трёхмерная игра в жанре исторической стратегии в реальном времени, разрабатываемая сообществом добровольцев. Размер кодовой базы маленький и я решил проверить игру в качестве отдыха от больших проектов, таких как Android и XNU Kernel. Итак, перед нами проект, содержащий 165000 строк кода на языке C++. Посмотрим, что интересного в нём можно найти с помощью статического анализатора PVS-Studio.…

    Подробнее »
  • Хабрахабр

    Команда PVS-Studio непредвзята при написании статей

    Наша команда проверяет различные открытые проекты с помощью PVS-Studio и пишет о результатах анализа кода. Время от времени мы сталкиваемся со странными обвинениями в предвзятости. Думаем, что часто это «тролли», и вступать в дискуссии с ними не имеет смысла. С другой стороны, оставлять подобные комментарии совсем без ответа тоже не хочется. Поэтому я решил написать небольшую статью, чтобы иметь возможность…

    Подробнее »


Кнопка «Наверх»
Закрыть