В рейтинге участвует:

групп:

каналов:

Виртуальный сервер на SSD - недорого!

Аренда выделенных и виртуальных серверов (VDS/VPS), хостинг, аренда IP-адресов, администрирование, круглосуточная поддержка

qwarta.ru подробнее

Резервное копирование с проверкой на вирусы!!!

Удобный сервис создания резервных копий на любой сервер сети интернет. Отслеживайте изменения, проверяйте на вирусы. Надежно защитите свой бизнес!

go.backupland.com

Выбираете сервер? Любая конфигурация на заказ!

Аренда физических серверов любых конфигураций под любые запросы - 1С бухгалтерия, игровые сервера, нагруженные проекты, интернет-магазины!

qwarta.ru подробнее

Size: a a a

Боль Тимлида

4702 membersпожаловаться на группу
2021 May 08

VF

Victor Fabrichenko in Боль Тимлида
Понял относительно чего? Код это реализация, надо не реализацию понимать, потому что это не имеет смысла, а понимать насколько реализация соответствует спецификации
источник
14:35пожаловаться#1

MS

Maxim Sosnov in Боль Тимлида
У вас неправильный критерий.
Прочитал, все понял, вернулся через полгода делать правку и все понятно

И тут всплывает то, о чем говорит Виктор. На код ревью ревьюер не входит полностью в контекст задачи и оценить понятность решения не может, только если он не участвовал изначально в проектировании решения
источник
14:35пожаловаться#2

VF

Victor Fabrichenko in Боль Тимлида
Мне кажется что вы не отвечаете на мои вопросы )) зачем читать в четвертый раз за человеком, если предыдущие три раза все было понятно?
источник
14:36пожаловаться#3

VF

Victor Fabrichenko in Боль Тимлида
И нет не перепутал, сначала проект, потом код и кода как можно меньше. Меньше кода меньше проблем с пониманием.
источник
14:36пожаловаться#4

СА

Сергей Аксёнов... in Боль Тимлида
Разумеется CR привязан к тикету, а тот к спеке.
источник
14:36пожаловаться#5

СА

Сергей Аксёнов... in Боль Тимлида
Я, наверное, плохо вас понимаю. В какой момент произошли "предыдущие три раза"?
источник
14:38пожаловаться#6

VF

Victor Fabrichenko in Боль Тимлида
Ну раз вы так боитесь не понять того, что человек пишет, вы перестраховываетесь и читаете за ним код. Прочитали три раза
источник
14:39пожаловаться#7

VF

Victor Fabrichenko in Боль Тимлида
Вообще когда работаешь над проектом, регулярно читаешь часть кода, и там если непонятно можно посмотреть блейм и увидеть автора, я обычно делаю так. Возникли вопросы, пошел спросил. Так же делают все остальные.
источник
14:40пожаловаться#8

СА

Сергей Аксёнов... in Боль Тимлида
Нет, я читаю один раз, во время code review. А вот из ваших слов я понял, что вы читаете три раза до код ревью, и не видите смысла делать это в четвертый раз.
источник
14:40пожаловаться#9

VF

Victor Fabrichenko in Боль Тимлида
Вы точно программист? Вы когда над проектом работаете, регулярно приходится читать разные куски кода, ну просто потому что задачи разные
источник
14:41пожаловаться#10

СА

Сергей Аксёнов... in Боль Тимлида
Это если автор ещё работает в команде. А блейм не испорчен каким-нибудь сквош-мержем. И каждый коммит можно смапить в соответствующий тикет.
источник
14:41пожаловаться#11

VF

Victor Fabrichenko in Боль Тимлида
Вот вы за человеком три раза прочитали, три разных код ревью, зачем читать четвертый?
источник
14:41пожаловаться#12

СА

Сергей Аксёнов... in Боль Тимлида
Да, я точно программист. А вы когда-нибудь работали с проектом, где больше 50 тысяч строк кода?
источник
14:42пожаловаться#13

AK

Anton Kucherov in Боль Тимлида
Если вы работаете в команде, вы доверяете друг другу. Если вы далаете код-ревью - это Zero Trust окружение. Там нет речи ни о какой "команде".
источник
14:42пожаловаться#14

VF

Victor Fabrichenko in Боль Тимлида
Конечно, вы хотите начать меряться кто круче? 😅
источник
14:42пожаловаться#15

СА

Сергей Аксёнов... in Боль Тимлида
Я снова вас теряю. Откуда взялись три код ревью?
источник
14:42пожаловаться#16

VF

Victor Fabrichenko in Боль Тимлида
Представим гипотетическую ситуацию. У вас новый человек и вы ему не доверяете. Делаете кодревью трёх его задач. Пока понятно?
источник
14:43пожаловаться#17

СА

Сергей Аксёнов... in Боль Тимлида
Нет, просто вы бы тогда знали, что статистически чтобы прочитать весь код проекта надо проработать над ним около года. И срок этот растёт экспоненциально от объёма кодовой базы проекта.
источник
14:45пожаловаться#18

СА

Сергей Аксёнов... in Боль Тимлида
Зачем же тогда существую QA, юнит-тесты, интеграционные тесты, e2e-тесты, acceptance-тесты, мониторинг, алертинг? Мы же все друг другу доверяем, мы команда, давайте катить как есть, всё должно просто работать.
источник
14:47пожаловаться#19

VF

Victor Fabrichenko in Боль Тимлида
Вы не отвечаете на мой вопрос и уводите разговор в другую сторону
источник
14:47пожаловаться#20