Мы следим за читаемостью, ну если читаемость есть, то куда она денется? Люди тупеют у вас что-ли?

В какой момент происходит прочтение? В исходном описании сказано: "будут знать решение до написания".

Вы же код пишете, значит читаете.

А как вы следите за читаемостью, с помощью какого инструмента? Какой критерий читаемости?

Там надо кавычки поставить

Но пишет его один человек, после того как двое спроектировали. Или я неправильно понял методику?

Да неважно кто и как делал, важно что в последствии вы регулярно читаете код, когда меняете/дополняет систему

Зачем читать код, когда вам он не интересен?

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

Прочитать код модуля, который никак не соприкасается в данный момент с вашей работой и в проектировании которого вы не принимали участия, это практически гарантия того, что вы ничего не запомните и не сможете нормально проверить

Просто потому что в вас мало времени на понимание задачи

Человек пишет код, вы три раза за ним прочитали и вам все понятно, зачем читать все последующие разы? Чтобы что?

Божечки, фраза топ! у меня в проекте как раз вахтёрские методики, буду теперь отбрыкиваться фразой, что мне не интересно, я больше по сайнс фикшн

А вы как следите? Вы же делаете ревью, какие критерии читаемости?

Ну серьезно? В данный момент вам не интересен и не нужен кусок кода, какой смысл его читать? Все равно ни понять, ни тем более оценить вы его нормально не сможете

Я на полном серьезе) просто никогда в голову не приходило на просьбу провести кодревью ответить ‘мне не интересно’

Проблемы с читаемостью есть там, где нет проектирования. Если проектирование есть, то проблем с читаемостью нет. Ну будет непонятная внутри функция, ну либо не будем трогать, либо перепишем, если дизайн простой, понятный и прозрачный. Я много раз видел, когда проект говно, но код весть такой "правильный" и когда местами код странный, зато все всегда понятно. Понятность системы в ее дизайне, а не в коде конкретных методов.

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

Критерий читаемости при ревью очевидный: ревьюер прочитал и всё понял. Если что-то не понял или понял с трудом - сказал: "у тебя вот тут непонятно".

> если проектирование есть, то проблем с читаемостью нет

Мне кажется, вы перепутали необходимое условие с достаточным.