0. ivanov660 1626 17.07.19 04:18 Сейчас в теме

Как завести у себя в команде код-ревью. Отвечаем на вопросы

Дадим советы как начать использовать у себя в команде код-ревью (code-review), а также ответим на вопросы читателей.

Перейти к публикации

Комментарии
Избранное Подписка Сортировка: Древо
1. kuzyara 797 17.07.19 05:27 Сейчас в теме
[КОД-РЕВЬЮ]
1. Изменить форматирование
...
Сам процесс внедрения и улучшение процесса вызывает определенные «сложности».
Неудивительно.
2. ivanov660 1626 17.07.19 07:03 Сейчас в теме
(1) Коллега есть комментарии по существу?
Или какие-нибудь дельные предложения, советы. Хотите поделиться опытом?
3. Stepa86 1255 17.07.19 08:15 Сейчас в теме
> Здорово если бы для 1С появился мощный статический анализатор кода типа PVS-studio, но пока его нет.

Я аж кофем подавился. https://infostart.ru/public/1089670/
kalyaka; JohnyDeath; olegtymko; NeviD; Iscarimet; +5 Ответить
7. ivanov660 1626 17.07.19 10:52 Сейчас в теме
(3) А от 1С есть что-то? Слишком много разных инструментов используется и для новичков это довольно напряжно будет применить и настроить.
11. Stepa86 1255 17.07.19 10:58 Сейчас в теме
(7) от 1С - АПК.

Новичок и не должен это все настраивать. Настроить это все должен девопсер или тот, кто взял на себя эту роль, а остальные этим пользуются.

У нас есть конфигурация для проведения код-ревью. Там сперва каждый самостоятельно проводит анализ куска кода и пишет замечания, а потом вместе собираются и быстренько пробегаются по замечаниям с обсуждениями.
В эту конфигурацию мы подгружаем все нарушения, которые нашли Сонар+АПК+EDT и аудиторы их сразу видят. И не тратят время на них. А автор кода получает по башке за то, что сам не прогнал свой код через сонар.
12. ivanov660 1626 17.07.19 11:04 Сейчас в теме
(11) Не все имеют большую и продвинутую команду с хорошим набором инструментов и специалистов, как у вас.
Есть много начинающего уровня специалистов и команд, для которых на первом шаге многие вещи из вашей статьи будут непосильны. Поэтому начинать надо мелкими шагами. А вот потом, когда уже поднимется уровень, тогда можно будет смотреть в сторону вашего окружения.
Rustig; Stepa86; +2 Ответить
13. Stepa86 1255 17.07.19 11:08 Сейчас в теме
(12) Вот прям вообще ДА. Если у вас 1.5 программиста - вам не нужен инструмент по автоматическому статическому анализу кода. А если у вас большой и серьезный проект, то обязательно нужно и двойное чтение кода, и статистический анализ кода, и гит, и тесты, и покрытие, и CI/CD, и разные контуры.

Не существует единый истины. Всегда нужно головой думать в каждой конкретной ситуации. И взвешивать профит и затраты.
RSConsulting; +1 Ответить
14. ivanov660 1626 17.07.19 11:12 Сейчас в теме
(13) Может вас и удивит, но я разговаривал с коллегами на конференции, которые не были новичками и конфигурации у них были относительно большие и серьезные, но кто-то даже не использовал хранилище для разработки, что там говорить про какие-то техники. Поэтому действительно истина у каждого своя)
15. Stepa86 1255 17.07.19 11:18 Сейчас в теме
(14) "не использовал хранилище для разработки" можно двояко понять. Есть те, кто еще не использует, а есть те, кто уже не использует.

Вы же статьи тоже пишете для того, чтоб другие почитали и улучшили процессы у себя?
16. TODD22 18 17.07.19 11:21 Сейчас в теме
(11)
А автор кода получает по башке за то, что сам не прогнал свой код через сонар.

А разве автоматически это не должно делаться?
17. Stepa86 1255 17.07.19 11:25 Сейчас в теме
(16) Не на всех проектах. Да и если прогнали - не всегда правят. У нас тут тоже как бы костыли и эксперименты, а не отлаженный идеальный процесс с розовыми понями.
18. TODD22 18 17.07.19 11:28 Сейчас в теме
(17)У нас было строго, проверка автоматически, пока не исправил код не принимают.
4. genayo 17.07.19 08:38 Сейчас в теме
"++ Задача-6800 Иванов" - не очень хороший комментарий. Имхо, вместо номера задачи более уместна прямая ссылка на треккер, и кроме задачи не помешало бы указывать проект/спринт, в рамках которого производятся изменения, чтобы потом было легко найти все изменения по проекту, сделанные всеми разработчиками.
6. CheBurator 3399 17.07.19 10:49 Сейчас в теме
(4) + по крайней мере как минимум в скобках-комментах д.б. маркер дата-время
ManyakRus; +1 Ответить
9. ivanov660 1626 17.07.19 10:55 Сейчас в теме
(6) Конечно под себя нужно "допилить"). Однако если у вас задача в системе баг-трекинга, то большой перечень комментарием может быть излишним.
25. skillman 21.07.19 17:29 Сейчас в теме
(9) Можете прикрепить к статье свои регламенты разработки, хотелось бы посмотреть чужую практику, возможно на будущее по взаимствовать.
27. ivanov660 1626 21.07.19 19:02 Сейчас в теме
(25)не могу, это внутренний документ, зато могу пообщаться в рамках этого вопроса.
19. genayo 17.07.19 11:41 Сейчас в теме
(6) Зачем, если есть ссылка на задачу в треккере?
5. VmvLer 17.07.19 09:37 Сейчас в теме
читал, пытался держать нить статьи в рациональном понятийном русле, в итоге пришел к мнению, что
с водой выплеснули ребенка

тут есть примеры свежих статей с хорошим стилем, где авторы просто и понятно "ведут" по теме.
не в этот раз.(
user1219528; +1 3 Ответить
8. user1219528 17.07.19 10:53 Сейчас в теме
10. ivanov660 1626 17.07.19 10:57 Сейчас в теме
(5) Коллега - я привел ответы на вопросы, которые просили. А что вы собственно хотели почерпнуть?
20. Kutuzov 564 18.07.19 09:03 Сейчас в теме
Добрый день! Хотел уточнить такой момент. Временные затраты, которые понес проверяющий - они падают на стоимость проекта? Опять же при возврате задачи на доработку программисту - исправления он делает за свой счет, или эти дополнительные часы опять же падают на стоимость проекта?
21. TODD22 18 18.07.19 09:33 Сейчас в теме
(20)
Временные затраты, которые понес проверяющий - они падают на стоимость проекта?

А на что ещё падают затраты в коммерческой организации?
ivanov660; +1 Ответить
24. Kutuzov 564 18.07.19 10:15 Сейчас в теме
(21) Ну мало ли, может это считается как инвестиции в развитие сотрудников. Я поэтому и спрашиваю, что надо узнать)
22. VmvLer 18.07.19 09:42 Сейчас в теме
(20) я полагаю, что после проверки весь отдел разработки станет полгода обучать азам тех кто наваял кривой код.
проще было бы не допускать таких к работе, но кого это волнует.
сейчас принято сначала поощрять бредо-писателей, а потом целыми командами искать в этом логику.
23. ivanov660 1626 18.07.19 09:43 Сейчас в теме
(22)Самое разумное раздавать задачи в зависимости от компетенции. А за джунами обязательно должен быть присмотр.
26. skillman 21.07.19 17:31 Сейчас в теме
(22)
проще было бы не допускать таких к работе

Не согласен по поводу допуска, учить джунов как-то нужно, а код ревью дает еще рост квалификации
ivanov660; +1 Ответить
28. ivanov660 1626 21.07.19 19:05 Сейчас в теме
(26)конечно, джун же должен же как-то расти и притом что же он будет делать. Иначе это будет напоминать анекдот про молодого специалиста, которого не берут на работу из-за отсутствия опыта)
Оставьте свое сообщение
Новые вопросы с вознаграждением
Автор темы объявил вознаграждение за найденный ответ, его получит тот, кто первый поможет автору.

Вакансии

Автор новостных обзоров на тему 1С и бухучета
Санкт-Петербург
По совместительству

Ведущий программист 1С
Санкт-Петербург
зарплата от 130 000 руб.
Полный день

Программист 1С
Москва
зарплата от 130 000 руб. до 200 000 руб.
Полный день

Бизнес-архитектор 1С, ведущий консультант
Санкт-Петербург
Полный день

Руководитель проектов 1С
Санкт-Петербург
Полный день