Productivity

Существует специфическое явление, которое происходит в организациях при введении метрик: все вдруг считают, что нашли секретный ингредиент успеха. Диаграммы появляются на панелях управления, цели выгравированы на стенах командных комнат, и разговоры становятся всё более эмоциональными по поводу достижения этих чисел. Это опьяняет — обещание, что если мы просто будем измерять правильные вещи, всё волшебным образом улучшится. Но вот неудобная правда: иногда лучшее решение, которое вы можете принять относительно своих метрик, — это отказаться от них....

Your codebase is a mess. You know it. Your team knows it. That one function has 47 parameters. Variables are named after your cats. And somewhere in there, a comment from 2015 says “TODO: fix this before production” in a voice that sounds increasingly desperate. So naturally, you do what any self-respecting developer does: you commit to a massive refactoring sprint. Two weeks, you think. Maybe three. You’ll rewrite everything beautifully, apply design patterns like they’re going out of style, and emerge victorious with a codebase so clean you could eat off it....

Ваша кодовая база — это хаос. Вы это знаете, и ваша команда тоже. У одной функции 47 параметров. Переменные названы в честь ваших кошек. И где-то там комментарий 2015 года гласит: «TODO: исправить это перед запуском в производство», и голос в нём звучит всё более отчаянно. Естественно, вы поступаете как любой уважающий себя разработчик: берёте обязательство провести масштабную рефакторинг-спринт-доработку. Две недели, думаете вы. Может быть, три. Вы красиво перепишите всё, примените шаблоны проектирования, словно они выходят из моды, и выйдете победителем с кодовой базой, настолько чистой, что по ней можно будет есть....

Let me start with a controversial statement that’ll probably ruffle some feathers: code reviews, as practiced by most teams, are a colossal waste of time. Before you sharpen your pitchforks and light your torches, hear me out. I’ve been on both sides of this fence – as a reviewer drowning in diffs and as a developer waiting for approval while my brilliant code grows stale. The uncomfortable truth is that we’ve turned code reviews into a cargo cult practice....

Начну с неоднозначного утверждения, которое, вероятно, вызовет возмущение: ревью кода, как их проводит большинство команд, — колоссальная трата времени. Прежде чем хвататься за вилы и факелы, выслушайте меня. Я был по обе стороны баррикад — и в роли ревьювера, погрязшего в diff, и в роли разработчика, который ждёт одобрения, пока его блестящий код устаревает. Неудобная правда заключается в том, что мы превратили ревью кода в подобие карго-культа. Мы проводим их, потому что «лучшие практики» говорят, что так надо, а не потому, что они действительно улучшают наш код или делают наши команды продуктивнее....