Поскольку наш предыдущий топик с описанием процесса внедрения инспекций кода в нашей компании вызвал определенный интерес у хабрасообщества, мы решили написать внеочередное продолжение. Сегодня мы рассмотрим пример внедрения данной практики на конкретном проекте.Читать полностью »
Рубрика «code review» - 9
Практический пример внедрения инспекции кода
2012-08-09 в 6:03, admin, рубрики: code review, Блог компании Positive Technologies, инспекции кода, разработка, метки: code review, инспекции кода, разработкаАвтоматическая проверка кода для PHP
2012-08-08 в 5:48, admin, рубрики: code review, continuous integration, php, Песочница, тестирование, метки: code review, continuous integration, PHP Разрешите представить Вам перевод статьи Johannes Schmitt Automated Code Reviews for PHP. Лично мне она помогла несколько иначе взглянуть на процесс разработки и тестирования своих приложений. А оригинальный подход автора к тестированию, как минимум, заслуживает внимания.
Если вам тоже интересно, добро пожаловать под кат.
Читать полностью »
How it’s made для программистов. Серия №2
2012-08-05 в 12:59, admin, рубрики: code review, java, mockito, netty, open source, spring, блог, Программирование, Совершенный код, метки: code review, java, mockito, netty, open source, spring, блогСлышали ли вы о Queuepy? А слышали ли вы об Open Source проектах, написанных на Java? А интересно ли вам узнать как они работают?
Если ваш ответ на последние два вопроса положителен, то неважно, слышали или нет вы о Queuepy до сих пор. Далее нам по пути.
Читать полностью »
Внедрение инспекций кода в процесс разработки
2012-08-01 в 12:26, admin, рубрики: code review, Блог компании Positive Technologies, инспекции кода, разработка, метки: code review, инспекции кода, разработкаВнимание! Данная статья рассчитана на людей, имеющих представление о том, что такое инспекции кода, и желающих внедрить эту методику в своей компании.
Когда мы начинали заниматься процессом внедрения инспекций кода (code review) в своих проектах, то были неприятно удивлены отсутствием толковых материалов по организации этого процесса с нуля. Еще один крайне скудно освещенный аспект — это масштабирование процесса инспекций.
Восполняя этот пробел, мы хотим поделиться опытом внедрения этой замечательной практики в нашей команде. Конструктивная критика в комментариях приветствуется.
Итак, начнем.Читать полностью »
«Gerrit Code Review»: краткое руководство с картинками
2012-07-12 в 8:59, admin, рубрики: badoo, code review, gerrit, Git, баду, Блог компании Badoo, разработка, метки: badoo, code review, gerrit, Git, баду
Red and Blue Chair by Gerrit Rietveld (1918)
В компании «Badoo» есть отдел C/C++-программистов. Отдел довольно небольшой, и потому его сотрудники обычно работают над разными проектами, которые между собой пересекаются только в исключительных случаях.
Одним из негативных последствий такой ситуации является bus factor, который стремится к единице. Для решения этой и других проблем было решено в порядке эксперимента внедрить систему ревизии кода (англ. code review): назначить одного разработчика ревизором у другого и таким образом познакомить его с кодом, а заодно и повысить качество последнего.
Code Review Open Source проектов
2012-06-26 в 8:25, admin, рубрики: code review, java, open source, блогосфера, Программирование, метки: code review, open source, блогосфераАнонсирую собственный эксперимент — Queuepy.com — сборник обзоров исходного кода Open Source проектов. В первом приближении, проекты ограничиваются JVM платформой. Формат обзоров — цикл сухих статей, объединенных рассматриваемым проектом. В обзор попадают архитектурные особенности, ошибки разработчиков и другие интересные детали реализации.
Проект первого цикла статей — Mockito. Под его капотом много интересных библиотек, взаимодействие с которыми будет освещено в деталях.
Еще одна статья о code review
2012-04-22 в 15:03, admin, рубрики: code review, Программирование, разработка, метки: code review, ПрограммированиеЧто такое code review
Code review - инженерная практика в терминах гибкой методологии разработки. Это анализ (инспекция) кода с целью выявить ошибки, недочеты, расхождения в стиле написания кода, в соответствии написанного кода и поставленной задачи.
К очевидным плюсам этой практики можно отнести:
- Улучшается качество кода
- Находятся «глупые» ошибки (опечатки) в реализации
- Повышается степень совместного владения кодом
- Код приводится к единому стилю написания
- Хорошо подходит для обучения «новичков», быстро набирается навык, происходит выравнивание опыта, обмен знаниями.
Ревизия кода / Повторная проверка проекта Notepad++
2012-02-13 в 10:28, admin, рубрики: c plus plus, code review, notepad++, pvs-studio, обзор кода, ошибки в коде, статический анализ кода, метки: c plus plus, code review, notepad++, pvs-studio, обзор кода, ошибки в коде, статический анализ кода
Прошло более года, как мы проверили Notepad++ с помощью PVS-Studio. Интересно посмотреть, насколько анализатор PVS-Studio стал лучше, и что было исправлено в Notepad++ из прежних ошибок.
Введение
Итак, мы проверили проект Notepad++ взятый из репозитория 31 января 2012. Для проверки использовался анализатор PVS-Studio версии 4.54.
Как уже было сказано, мы ранее проверяли этот проект. Ошибок нашли не много, но всё-таки что-то нашли. В новой версии проекта часть старых ошибок исправлена, а часть нет. Это странно. По всей видимости, прежняя заметка осталась незамеченной авторами Notepad++ и они не воспользовались PVS-Studio для проверки проекта.Читать полностью »