[/b/] [/d/] [/tu/] [/a/] [/ph/] [/wa/] [/cg/] [/t/] [/p/]
Помимо другой моды, есть такая мода - мода на code review. Глядя на других успешных кодревьюеров, программисты решают устраивать бизнес-линч.В чём проблема? Проблема в том, что такие ревьюеры почему-то вместо объективных методов оценки кода начинают городить чушь. То придерутся к неймингу переменных, то попросят сделать ломающие изменения, то совсем очевидно полезут подстраивать код под свой стиль, то дадут совет добавить код на эджкейс туда, где он не выполнится.Получается, что от такого code review больше вреда, чем пользы, не так ли?
Помимо другой моды, есть такая мода - мода на code review. Глядя на других успешных кодревьюеров, программисты решают устраивать бизнес-линч.
В чём проблема? Проблема в том, что такие ревьюеры почему-то вместо объективных методов оценки кода начинают городить чушь. То придерутся к неймингу переменных, то попросят сделать ломающие изменения, то совсем очевидно полезут подстраивать код под свой стиль, то дадут совет добавить код на эджкейс туда, где он не выполнится.
Получается, что от такого code review больше вреда, чем пользы, не так ли?
Ну, во-первых, ревью нужено хотя бы для того, чтобы поглядеть, чего там коллеги накуролесили. Чтобы потом не оказалось неожиданностью. Во-вторых, его отдают как правило сеньёру или ещё какому главнюку. Впрочем, я только в мелких фирмах работал.>полезут подстраивать код под свой стильВ нормальных проектах есть соглашения о форматировании, неймингах и тому подобном.
Ну, во-первых, ревью нужено хотя бы для того, чтобы поглядеть, чего там коллеги накуролесили. Чтобы потом не оказалось неожиданностью. Во-вторых, его отдают как правило сеньёру или ещё какому главнюку. Впрочем, я только в мелких фирмах работал.
>полезут подстраивать код под свой стиль
В нормальных проектах есть соглашения о форматировании, неймингах и тому подобном.
>>203160>Ну, во-первых, ревью нужено хотя бы для того, чтобы поглядеть, чего там коллеги накуролесили. Чтобы потом не оказалось неожиданностью.>Во-вторых, его отдают как правило сеньёру или ещё какому главнюку.Получается, код-ревью нужен сеньору (тимлиду), чтобы оставаться сеньором (тимлидом), так? А остальные, не проводящие в код-ревью, играют в царя горы с перечитыванием лога или каждый раз матерятся на чужие изменения.>В нормальных проектах есть соглашения о форматировании, неймингах и тому подобном.Это-то понятно. Я говорю как раз о случаях вне соглашений, когда человек суёт в код-ревью именно своё субъективное мнение. Просто каких-то регламентов на код-ревью мне видеть не приходилось. Есть ли такая практика - составлять регламенты код-ревью?И, что самое интересное, мнение на счёт того, каким должен быть хороший (читаемый etc) код, у всех разное. Почему-то каждый это своё мнение в процессе ревью пропихивает как непреложную истину.
>>203160
>Ну, во-первых, ревью нужено хотя бы для того, чтобы поглядеть, чего там коллеги накуролесили. Чтобы потом не оказалось неожиданностью.
>Во-вторых, его отдают как правило сеньёру или ещё какому главнюку.
Получается, код-ревью нужен сеньору (тимлиду), чтобы оставаться сеньором (тимлидом), так? А остальные, не проводящие в код-ревью, играют в царя горы с перечитыванием лога или каждый раз матерятся на чужие изменения.
>В нормальных проектах есть соглашения о форматировании, неймингах и тому подобном.
Это-то понятно. Я говорю как раз о случаях вне соглашений, когда человек суёт в код-ревью именно своё субъективное мнение. Просто каких-то регламентов на код-ревью мне видеть не приходилось. Есть ли такая практика - составлять регламенты код-ревью?
И, что самое интересное, мнение на счёт того, каким должен быть хороший (читаемый etc) код, у всех разное. Почему-то каждый это своё мнение в процессе ревью пропихивает как непреложную истину.
>>203164Когда код написан в куче разных стилей то его читать намного сложней.
>>203165А как к этому относится субъективное мнение смотрящего, учитывая то, что в кодовой базе уже есть код, дёргающий TIMTOWTDI направо и налево?
>>203171Очевидное желание сделать кодовую базу лучше чем она есть очевидно. То что там есть говнокод скорее означает что его нужно оттуда вычистить по мере наличия свободного времени, чем то, что все остальное тоже нужно превращать в свалку говнокода.
>>203172>Очевидное желание сделать кодовую базу лучше чем она есть очевидно.Нет, ни разу не очевидно. Зато есть желание протолкнуть свои субъективные принципы и попутно натыкать палок в колёса другому. Напомню, что регламента code review и законодательно оформленного стиля смысла кода нет, при этом каждый тянет в свою сторону - это не выглядит как желание сделать код лучше.
>>203172
>Очевидное желание сделать кодовую базу лучше чем она есть очевидно.
Нет, ни разу не очевидно. Зато есть желание протолкнуть свои субъективные принципы и попутно натыкать палок в колёса другому. Напомню, что регламента code review и законодательно оформленного стиля смысла кода нет, при этом каждый тянет в свою сторону - это не выглядит как желание сделать код лучше.
У тебя просто нормального ревьюера не было. Нормальные люди отмечают только настоящие проблемы или аккуратно предлагают свои идеи в том или ином месте.
- wakaba 3.0.7 + futaba + futallaby -