Культура ошибок: как их полюбить и извлекать пользу Хабр

Если ошибка мелкая, она, конечно, может повториться несколько раз. Но после проведенной беседы и починки процессов она тоже должна сойти на нет. Олег предлагает вопрос для дискуссии — можно ли без ошибок двигаться вперёд. Поэтому я в какой-то степени даже слово «ошибка» удалил из своего лексикона и пришел к понятию опыта. Если ты получил не тот результат, на который рассчитывал, ты сделал выводы, что в будущем нужно делать по-другому. А если ты получил тот результат, на который надеялся, ты красавчик.

• Хорошим решением будет перекалибровка шкалы оценок так, чтобы наиболее вероятные оценки уже назывались “отличным результатом” и не вызывали бы ощущение несправедливого занижения.
• Часто проводя код-ревью, разработчик воспроизводит информацию из памяти и не забывает её.
• Худший момент, который может представить себе человек, — это, по его словам, стоять перед дверью в палату, куда нужно зайти и рассказать пациенту, что ты разрушил его жизнь.
• В тестировании срабатывает принцип Парето — «20% усилий дают 80% результата».
• Для успешной разработки и внедрения процесса ревью обязательно участие менеджеров, которые определяют стратегию и ценности компании.
• На проектах, где мне доводилось работать, нужно было провести несколько код-ревью, которые оценил бы другой человек.

Badoo уже больше десяти лет, и нам важно, чтобы люди у нас росли, оставаясь с нами надолго. Некоторые ребята работают здесь по девять–десять лет. Мы стремимся всегда держать руку на пульсе и быть в курсе ожиданий сотрудников, знать, что происходит в их жизни. Конечно, не всегда их ожидания совпадают с позицией руководителей, но, по крайней мере, мы гарантируем регулярное взаимодействие и предоставляем сотрудникам возможность расти внутри компании.

Зачем бизнесу Performance Review

Один из наших подходов заключается в том, что quality и QA в целом – это часть организации, которая владеет процессами наравне с инженерами и программистами, и даже в большей степени, чем они. И ведь ABBYY – далеко не единственная компания, опыт которой приводится в качестве аргумента, есть и куда более известная история про Microsoft. Такой разговор — неформальное мероприятие, но HR придерживаются определенной структуры и дают возможность заранее подготовиться к общению по списку вопросов. Ответы вносятся в таблицу, где отмечаются болевые точки, острые моменты, плохое настроение, негативный опыт. Это необходимо для того, чтобы руководитель при дальнейшем общении знал, на что стоит обратить внимание. Между проверяющим и проверяемым нет «ступеньки опыта», которая помогает значительно улучшить код.

Далее мы столкнулись с проблемой стандартизации Performance Review. Как выяснилось, через это прошли все, кто внедрял метод. Дело в том, что оценка стандарта может быть субъективной даже при условии, что над ней работает много специалистов.

Кому и для чего нужно ревью?

Поля поддерживает — в её команде выстроена такая же культура отношения к ошибкам. Но есть ли в таком случае какой-то предел, или ошибаться можно бесконечно? Поля считает, что лучше ошибиться один раз — и в будущем такого не повторять.

В вашем распоряжении есть собственный корабль, который вы можете настраивать и модифицировать по своему усмотрению. Можно назначать членов экипажа, которые будут помогать вам в разных ситуациях. Пользователей бесит не сюжет или персонажи, а баги.

Когда не нужно проводить код-ревью?

Рецензия (англ. review, обзор) — анализ, разбор, некоторая оценка публикации, произведения или продукта, жанр газетно-журнальной публицистики и литературной критики. Рецензия может относиться к материальным вещам (приборы, аксессуары, бытовая техника), компьютерным технологиям, художественной литературе, музыке, фильмам, компьютерным играм. Рецензироваться могут также текущие события, общественные заявления и происшествия. В дополнение к критическому утверждению, автор рецензии может выставить предмету рецензирования некоторую оценку для указания относительной ценности рецензируемого предмета. Review — критическая оценка произведения (например, книги или фильма). В ревью можно описать свои впечатления, поделиться эмоциями, посоветовать или не посоветовать произведение к прочтению/просмотру.

Когда ревьюер разобрался с задачей и логикой решения, он смотрит на функции, отдельные алгоритмы и их эффективность. Проверяет, можно ли заменить их другими методами и будет ли это лучше для всего продукта. Если речь идет о понижении оценки, мы организовываем дополнительную встречу, на которой должны присутствовать руководитель, рекрутер и сотрудник.

Как понять, нужно ли вам ревью

Вы также можете пропагандировать отсутствие любых ревью, лестниц и ориентироваться на так называемые сквады, давать сотрудникам возможность периодически менять команды и работать над разными проектами. Я не очень понимаю, при каком размере компании такой подход будет эффективен, но он существует. Думаю, это несколько десятков человек – когда в компании начинаются заметными потери на взаимодействии между командами. Например, когда премия выплачена за то, что было сделано давно, уже потерялся фокус, уже нет возможности качественно ни собрать информацию, ни получить фидбэк. В этом случае люди наверняка будут считать ревью несправедливым. Заметьте, что большой процент таких людей будет всегда – полностью удовлетворить всех не получится, поэтому очень важно прийти к консенсусу.

Далее всё это собирается, и свою оценку даёт менеджер. Сам по себе процесс состоит из выдачи оценок – грейдов (grades); это могут быть грейды за ревью-период (хорошо/не очень), или глобальные грейды относительно прогресса в профессиональном росте. Существуют разные подходы к формированию шкал грейдов, но в целом они все построены вокруг неких ожиданий. Ревью – это сложный процесс, и за него должен кто-то отвечать. В идеале – эйчар-специалист, кто-то с административной функцией.

Performance Review в компании: выстраиваем систему оценки сотрудников

Такие должности не с чем сравнить, а значит, систему критериев нужно подстраивать под них. Если в предприятии десять человек, то и по ним можно понять, кто из них джун, кто мидл, а кто сеньор. В данном случае это пример простого ранжирования peer review это — мы же выстроили более сложную систему, где оценки зависят от вклада сотрудника и результата, которого мы ждем от той или иной должности. Во-вторых, нужно понимать, что размер компании никак не влияет на систему Performance Review.

У ревьюера может быть свой способ работы с кодом или другое решение для конкретной задачи. Но вся ценность его работы — предложить улучшения, ориентируясь на методы работы автора кода. Хорошо, чтобы ревьюер уже решал такие задачи, писал подобный код и был знаком с тем стеком технологий, который используют в команде.