K
Ему ревью кода не надо проводить по всем. Есть правила написания кода. За соблюдением правил кто-то должен смотреть
Size: a a a
2020 February 27
K
Если продвинутые тестеры - могут они
K
Именование компонент/процедур/функций - это чья вотчина у вас?
RT
мы стараемся максимум решений спустить на уровень команд,
все решения должны принимать самы команды.
Архитекторы задают правила, помогают советами, запрещают (с обоснованием) некоторые решения.
Когда у вас уже сотни сервисов, до именования функций дело не доходит. Контролируются только публичные внешние API.
все решения должны принимать самы команды.
Архитекторы задают правила, помогают советами, запрещают (с обоснованием) некоторые решения.
Когда у вас уже сотни сервисов, до именования функций дело не доходит. Контролируются только публичные внешние API.
DK
Тимлиды были спущены в биореактор первым делом, так как такой роли в скраме нет. Кто такой тимлид в скрам-аджайле?
Другие коллеги тогда. Все ревьюят друг друга. Мир, позитив_))
DK
Kamilla
Тогда арх :) арх надзор никто не отменял, а что в него включено - никто не декларировал. Изначально - правила сдачи - код покрыт комментами. Если никто за разработчиками и кодом не смотрит, то ожидать чуда не надо :) для начала - снять стек-трейс процесса и начать с верхнеуровневого покрытия комментами процедур/функций/апи разработчиком
Как арх всех отревьюит, он вообще может не шарить в каком-то коде.
RT
Хоть я тут и ныл два дня про плохой скрам, стоит отметить что разбирается кейс только одной из команд. У нас есть очень хорошие команды.
K
если нет правил написания кода, если нет ревью кода, то какое качество ждете? разработчику предметка не нужна - ему бы в доставшейся куче разобраться :)
PD
до уровня аналитика и не надо. но иметь общее прелставление о процессах и причинах событий разработчик обязан иметь
Какого аналитика? Бизнес-аналитика, эксперта-предметника, ещё кого? Их много разных ролей.
PD
Системный аналитик описывает алгоритмы преобразования данных, но на верхнем уровне. Естественно эти алгоритмы не будут оптимальными (у СА ограниченные знания в алгоритмах и опыт их применения + недостаток знаний свойств технологий). Именно в момент чтения спецификации разработчик должен понять, чего добивается аналитик и предложить альтернативу, пускай на словах, после чего имплементировать её в код.
Аналитик протаптывает дорожку для разработчика, но последний должен тоже держать на себе часть ответственности, где надо сокращать, а где надо усложнять маршрут. Из разработчика должны порождаться требования, иначе всё будет как в институте: "если аудитория не задаёт вопросов, то ей либо всё понятно, либо ничего не понятно".
Аналитик протаптывает дорожку для разработчика, но последний должен тоже держать на себе часть ответственности, где надо сокращать, а где надо усложнять маршрут. Из разработчика должны порождаться требования, иначе всё будет как в институте: "если аудитория не задаёт вопросов, то ей либо всё понятно, либо ничего не понятно".
Разработчик не должен понимать 'что добивается аналитик', он должен прочесть явную стори с мотивацией и ограничениями.
PD
Чем опытней аналитик, тем с большей вероятностью он предложит оптимальный алгоритм разработчику (либо научится согласовывать заранее).
Да никогда так не бывает )
PD
Что и почему - это про бизнес-аналитика. Системный аналитик - это про "как", но используя чужую экспертизу. Он собирает мнения, убирает противоречия и складывает всё в решение.
Угу. Поэтому СА - это антипаттерн, говорящий о проблемах в проекте.
PD
Тимлиды были спущены в биореактор первым делом, так как такой роли в скраме нет. Кто такой тимлид в скрам-аджайле?
Ну, верните их обратно. Дефакто эти роли все равно есть. Как и техлиды.
PD
Kamilla
Тогда арх :) арх надзор никто не отменял, а что в него включено - никто не декларировал. Изначально - правила сдачи - код покрыт комментами. Если никто за разработчиками и кодом не смотрит, то ожидать чуда не надо :) для начала - снять стек-трейс процесса и начать с верхнеуровневого покрытия комментами процедур/функций/апи разработчиком
Если код приходится покрывать комментариями, то его нужно переписать.
AK
Тимлиды были спущены в биореактор первым делом, так как такой роли в скраме нет. Кто такой тимлид в скрам-аджайле?
Чувак, который проводит статус и перекладывает цветные бумажки, попутно отпуская комментарии.
RT
Если код приходится покрывать комментариями, то его нужно переписать.
Не согласен: в комментариях должны быть ссылки на документы по предметной области, то есть указание на то где почитать что это такое, откуда, зачем
K
Если код приходится покрывать комментариями, то его нужно переписать.
+1 пока будет писать комменты - поймет, что надо переписать :)
PD
Не согласен: в комментариях должны быть ссылки на документы по предметной области, то есть указание на то где почитать что это такое, откуда, зачем
Не в комментариях, а в Jira task, связанной с коммитом )
PD
Или вообще в арх.описании компоненты. Или в документации для новичка.
PD
В коде искать сложно