EL
Код-ревью это жутко токсичная история, кстати
Size: a a a
2020 June 25
AM
Код-ревью это жутко токсичная история, кстати
смотря как проводить
EL
смотря как проводить
Как ни проводи. Если там что-то, проверку чего можно формализовать и автоматизировать, надо делать так. А если подключается человек, то это всегда будет вкусовщина и дедовщина
AL
Как ни проводи. Если там что-то, проверку чего можно формализовать и автоматизировать, надо делать так. А если подключается человек, то это всегда будет вкусовщина и дедовщина
Поэтому стиль - автоматика, а человек пусть на решение в целом смотрит.
AI
но в любом случае нельзя делать это неубираемым
Го в могилу фраербокс
EL
Andrey Listochkin
Поэтому стиль - автоматика, а человек пусть на решение в целом смотрит.
Решение либо работает, либо нет. Если работает и прошло автоматический контроль, человеку там делать нечего
AI
Там все убирается
AL
Решение либо работает, либо нет. Если работает и прошло автоматический контроль, человеку там делать нечего
Так-то оно так, но если открываешь, и видишь, что в коде N+1 запрос в базу например, и ты ему советуешь на left join переделать. Или разбор строки супом из substr и indexOf, а ты советуешь на регулярку перевести. Че б нет?
AL
Ревью не для того, чтоб с B+ код сделать A, а чтобы не пропустить случайно D, E, F
NS
если ревьючер токсичный человек то у него всегда ревью токсичное будет
AL
Это да
NS
это просто не "командный игрок" со всеми вытекающими
NS
Решение либо работает, либо нет. Если работает и прошло автоматический контроль, человеку там делать нечего
такой вариант подходит только для кода предназначенного для машин, а обычно код пишеться для других людей
MK
и как заставить не править
Поставить лучший клиент, tdesktop
AM
Решение либо работает, либо нет. Если работает и прошло автоматический контроль, человеку там делать нечего
это хорошо работает для разовых проектов
EL
Andrey Listochkin
Так-то оно так, но если открываешь, и видишь, что в коде N+1 запрос в базу например, и ты ему советуешь на left join переделать. Или разбор строки супом из substr и indexOf, а ты советуешь на регулярку перевести. Че б нет?
Ну за джуниором понятное дело нужно присматривать
AL
Ну за джуниором понятное дело нужно присматривать
Это не только джуны могут сделать. Я сам - первоклассный говнокодер.
EL
Andrey Listochkin
Это не только джуны могут сделать. Я сам - первоклассный говнокодер.
Исправишь в следующем релизе :)