Часть 1 | "Часть 2"

«Придурок», «Мудак», «Тьфу, что теперь?», «Сделай сам!», «Чего ты такой чертовски придирчивый?»

Похоже на то, о чем вы думали, когда Сет отправлял свой отзыв на ваш последний запрос на включение? Сет — рок-звезда, всезнайка, претенциозный парень с двумя пробелами вместо четырех и определенно без вкладок. Он хранитель всего кода. Он просматривает все пул-реквесты, и без его одобрения в кодовую базу ничего не попадает. Это его вина, что вы не можете уложиться в сроки. Почему он не может просто одобрить это, чтобы вы могли перейти к следующему заданию?

Ну, он мог. Но есть несколько очень веских причин, по которым он этого не хочет.

Оказывается, Сет, вероятно, не такой плохой парень, как вы думаете. Причина, по которой вы так себя чувствуете, может усугубляться тоном или, что более важно, отсутствием тона, который использовался в его комментариях при просмотре вашего кода. То, что его комментарии заставили вас чувствовать себя дерьмово, является комбинацией нескольких факторов:

  • тон (или отсутствие тона), который использовал Сет
  • насколько хорошо ты знаешь Сета
  • ваши отношения с Сетом
  • ваше настроение и уровень энергии при чтении комментариев
  • Настроение и уровень энергии Сета при написании комментариев

Все эти факторы могут в совокупности разрушить ваши отношения с Сетом. Они также могут превратить ваш «должен быть классным» день в день «трахни-это-вытащи-меня-сюда», когда на следующий день вы изо всех сил пытаетесь выползти из постели, просто чтобы плюхнуться перед компьютер, только чтобы перечитать комментарии и снова погрузиться в то грязное и депрессивное настроение.

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

Ты Сет? Тогда рекомендую прочитать Часть 2.. А пока мы поговорим о том, каково быть получателем комментариев Сета.

Утро вторника. Открывая Gmail, вы видите ветку, содержащую комментарии к запросу на вытягивание, который вы отправили вчера. Это от Сета, конечно. Вы стискиваете зубы, раздумывая, не надеть ли вам каппу обратно. Такими темпами ваши коренные зубы сотрутся к тому времени, когда вам исполнится сорок. Напрягая мышцы, вы открываете нить.

убить это

это должно быть опечатка.

мне это не нравится. Почему это было сделано?

Да, звучит так, как будто Сет — это его обычное, угрюмое «я». Но подождите, есть еще:

не могли бы вы просто изменить его так, чтобы он использовал атрибуты?

почему вы не можете сделать это так?

можете объяснить это изменение?

И у вас есть ежедневный стендап за десять минут! Теперь вы думаете, что, может быть, было бы лучше просто заявить о болезни и играть в видеоигры весь день.

Это похоже на тебя? Хоть немного? Читать дальше. Даже если это не похоже на вас, вы можете столкнуться с этим в будущем, поэтому вам следует помнить об этом.

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

Ну не совсем. У этой истории может быть совершенно другая сторона: Сет спешит, и у него есть всего несколько минут, чтобы провести проверку вашего кода, одну из многих, которые ему приходится делать каждый день. Неудивительно, что его комментарии лаконичны, например, «уберите это». Сет на самом деле вовсе не ненавидит вас, он просто хочет закончить проверку кода, чтобы приступить к исправлению проблемы, которая привела к падению базы данных на прошлой неделе.

Как это вообще могло испортить вам день?

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

«Это здорово, но что я могу с этим поделать?» ты спрашиваешь.

Что ж, оказывается, вы можете изменить то, как вы интерпретируете такие сообщения. В книге Выученный оптимизм Мартин Селигман описывает технику, которую люди могут использовать, чтобы перестать интерпретировать комментарии как личные, общие и постоянные, а интерпретировать их как безличные, конкретные и временные. Реджинальд Брейтуэйт дает здесь гораздо лучший обзор. Иди прочитай.

http://braythwayt.com/homoiconic/2009/05/01/optimism.html

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

Если невозможно встретиться с ними лично, постарайтесь организовать совместные занятия в Интернете. Сыграйте вместе в несколько онлайн-игр с включенными микрофонами или устройте «счастливый час» для команды, когда все прыгают по тусовкам и вместе выпивают виртуальный напиток.

Кроме того, если вас это устраивает, лучшим способом действий может быть попытка поговорить с Сетом напрямую. Не начинайте со смелого заявления вроде «Почему ты меня ненавидишь?» Это означает, что Сет на самом деле ненавидит вас, хотя это, вероятно, далеко от истины. Попробуйте что-то вроде этого: «Когда вы делаете X, я чувствую Y, и это делает меня Z». Следуя этой формуле, сосредоточьтесь на своих чувствах и влиянии, которое они на вас оказывают, а не на нападках на Сета. Попробуйте: «Когда вы говорите «убей это», я чувствую, что вы разочарованы мной, и мне становится труднее прийти в офис».

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

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

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

Как вы можете продолжать работать с Сетом? Подводя итог, вот как вы можете сделать его более приятным:

  • Установите взаимопонимание: проводите время за другими интересными занятиями; личная встреча; если удаленно, играйте в игры или общайтесь онлайн
  • Узнайте, как управлять тем, как вы читаете тоны в сообщениях (полезно даже за пределами этой области)
  • Посочувствуйте Сету: он хочет, чтобы в коде не было ошибок
  • Общайтесь: поговорите с Сетом о тоне в его комментариях.

Как рецензент, Сет также приложил к этому руку. Он может приложить немного усилий, чтобы добавить тон своим комментариям. Во второй части вы узнаете, как не быть придурком при код-ревью.

Первоначально опубликовано на www.okgrow.com.