Size: a a a

Боль Тимлида

2021 May 08

AK

Anton Kucherov in Боль Тимлида
Там сравниваются формальные инспекции и анализ проекта/кода
источник

C

Ctor in Боль Тимлида
Ну вот последнее из моего опыта, что просто один кейс работы системы вылетел из головы автора, но ревьювер это заметил. Если это проблема дизайна или тестов, то очевидно, что никто не подумал раньше о том, что это можно упустить из виду, потому что в тот момент всё казалось понятно и просто.
источник

VF

Victor Fabrichenko in Боль Тимлида
Как выглядит книга? Подскажите плз, пойду куплю
источник

AK

Anton Kucherov in Боль Тимлида
Другими словами: Формальные инспекции противопоставляются код-ревью, на сколько я могу видеть.
источник

VF

Victor Fabrichenko in Боль Тимлида
Что было бы, если бы код в таком виде уехал на прод?
источник

AK

Anton Kucherov in Боль Тимлида
Стив Макконел "Совершенный код." Достаточно хорошая книга с кучей первоисточников которые тоже можно проанализироват в книге.
источник

VF

Victor Fabrichenko in Боль Тимлида
Ну камон, у меня действительно был лозунг и генерализация или как там это нащывается
источник

MF

Magnus Frej in Боль Тимлида
Там есть и про простое чтение кода. 486 страница.
источник

AK

Anton Kucherov in Боль Тимлида
Есть и оно противопоставляется тому примеру что вы привели. Вы использовали аргумент в пользу код-ревью подменив понятие на формальную инспекцию
источник

AS

Artem Shpynov in Боль Тимлида
Дак вот эти моменты и надо лечить. А что еще в тот момент показалось понятно и просто, но ревьювер этого не вспомнил?
источник

VF

Victor Fabrichenko in Боль Тимлида
Нене, нормальный аргумент
источник

VF

Victor Fabrichenko in Боль Тимлида
Я про проверки же говорил
источник

AK

Anton Kucherov in Боль Тимлида
Я за 15 лет ни одной компании использующей формальные инспекции не встречал. Это вообще другая методика ни какого отрошения к среднестатистическому код ревью в рогах и копытах не имеющая
источник

MF

Magnus Frej in Боль Тимлида
Еще раз. Я опровергаю лозунг Виктора о том, что

Любая проверка сделанная после работы, а не до нее никакой пользы принести не может

Я не топлю за ревью.
источник

VF

Victor Fabrichenko in Боль Тимлида
Короче, поинт в том, что очень желательно понимать чего мы хотим получить и в конце просто выполнить сверку, полученное == ожидаемое?
источник

AS

Artem Shpynov in Боль Тимлида
Я встречал. Там цена критической ощибки была - отзыв всей партии телефонов по всему миру
источник

VF

Victor Fabrichenko in Боль Тимлида
А я признаю свою неправоту 😅
источник

AK

Anton Kucherov in Боль Тимлида
источник

MS

Maxim Sosnov in Боль Тимлида
Просмотрел по диагонали главу про инспекции кода в Совершенном коде
тлдр:
- есть формальные инспекции где мы задействуем кучу ролей (автор, координатор, секретарь, руководители, инспектор), есть четкий флоу как его проводить
- есть анализ кода, когда несколько разработчиков собираются и смотрят на код\проект
- есть чтение кода. Когда автор дает код, а люди смотрят его и дают фидбек (кажется ближе всего к типичному код ревью)

в выводах:
- эффективны формальные инспекции
- парное программирование и формальные инспекции примерно одинаковы по эффективности.

Цитата:
> Парное программирование и инспекции примерно эквивалентны по показателям расходов и качества итогового кода. Парное программирование может
оказаться особенно полезным, если вы хотите сократить срок разработки системы. Некоторые разработчики предпочитают работать в паре, а не самостоятельно.
источник

MF

Magnus Frej in Боль Тимлида
А кто-то не читатель, кто-то писатель.
источник