Size: a a a

Боль Тимлида

2021 May 08

VF

Victor Fabrichenko in Боль Тимлида
Это вопрос примерно такой: если у нас медленно работает код и мы хотим его переписать, то это рефакторинг? (Передекомпозиция)
источник

MF

Magnus Frej in Боль Тимлида
Я считаю, что да, является. Любой обзор кода, по моему мнению, есть код-ревью. По определению. Блокирование коммита - особенность конкретной имплементации.
источник

СА

Сергей Аксёнов... in Боль Тимлида
Первый это линтеры. Парная работа - я слышал про такое, у меня даже инженер пришёл из такой команды, но это должно быть примерно вдвое менее эффективно.
источник

AK

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

VF

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

MF

Magnus Frej in Боль Тимлида
Если цель ускорить - то нет, не считаю. Это оптимизация. Если цель изменение структуры - то рефакторинг.
источник

AK

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

VF

Victor Fabrichenko in Боль Тимлида
Скорость написания в паре падает на 20% зато качество кода (время потраченного на ревакторинг) и количество ошибок падает минимум в два раза
источник

AS

Artem Shpynov in Боль Тимлида
Парная работа вдвое менее эффективна? Вы точно работали с такими парами?
источник

VF

Victor Fabrichenko in Боль Тимлида
Вот пусть апологеты кодревью договорятся что это такое, тогда и будет смысл что-то обсуждать, а пока это чаще всего произвол каждого конкретного тимлида
источник

AS

Artem Shpynov in Боль Тимлида
С первой частью предложения не соглашусь, если конечно под скоростью написания не подразумевался тупой набор кода
источник

MF

Magnus Frej in Боль Тимлида
На мой взгляд, зависит от контекста. Есть разные мнения, разные подходы к обучению. Кто-то считает, что надо дать набить шишки и только после этого показать правильный ответ. Кто-то, что всю информацию нужно дать сразу. Зависит от личности и культурных особенностей участников.

Второй момент - полнота информации. На этапе дизайн-ревью присутствует неполнота информации разной степени, что оставляет пространство для появления "черных лебедей".
источник

VF

Victor Fabrichenko in Боль Тимлида
Но при этом, мой аргумент про информацию остаётся в силе, потому что схема "делаем, а потом думаем", очевидно приводит к полному или частичному выбрасыванию первого решения
источник

C

Ctor in Боль Тимлида
Я не совсем понял, что предлагается вместо ревью - парное программирование или только парный дизайн или вообще ничего
источник

AK

Anton Kucherov in Боль Тимлида
Автор же написал: Он только слышал. Если только слышал, спорить не о чем. Потому что когда опыта нет, спор по умолчанию будет бессмысленным
источник

MF

Magnus Frej in Боль Тимлида
Вот именно к этому я и веду. Апологеты и противники обсуждают разные вещи в разных контекстах
источник

AS

Artem Shpynov in Боль Тимлида
По моей оценке наоборот раза в полтора скорость реализации растет, причем сразу делают что надо и все эти тормоза про ревью уходят за ненадобностью
источник

MF

Magnus Frej in Боль Тимлида
Схалтурю и сошлюсь на авторитет. Вместо ревью предлагают формальные инспекции :)
источник

VF

Victor Fabrichenko in Боль Тимлида
Количество кода произведенного парой или двумя в отдельности. Просто пара работает в день 5-6 часов, да ещё и кодит со скоростью набора текста, потому что фактически происходит печать под диктовку.
источник

AK

Anton Kucherov in Боль Тимлида
Только часть "противников" они на самом деле не противники, они просто прошли этапы:
1. Еще не делаем ревью
2. Делаем ревью
3. Отказались от ревью сознательно
источник