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

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

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

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

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

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

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

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

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

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

А разве автоматически это не должно делаться?
17. Stepa86 1232 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 3395 17.07.19 10:49 Сейчас в теме
(4) + по крайней мере как минимум в скобках-комментах д.б. маркер дата-время
ManyakRus; +1 Ответить
9. ivanov660 1587 17.07.19 10:55 Сейчас в теме
(6) Конечно под себя нужно "допилить"). Однако если у вас задача в системе баг-трекинга, то большой перечень комментарием может быть излишним.
25. skillman 21.07.19 17:29 Сейчас в теме
(9) Можете прикрепить к статье свои регламенты разработки, хотелось бы посмотреть чужую практику, возможно на будущее по взаимствовать.
27. ivanov660 1587 21.07.19 19:02 Сейчас в теме
(25)не могу, это внутренний документ, зато могу пообщаться в рамках этого вопроса.
19. genayo 17.07.19 11:41 Сейчас в теме
(6) Зачем, если есть ссылка на задачу в треккере?
5. VmvLer 17.07.19 09:37 Сейчас в теме
читал, пытался держать нить статьи в рациональном понятийном русле, в итоге пришел к мнению, что
с водой выплеснули ребенка

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

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

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

Вакансии

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

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

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

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