Мой совет: передать это вашему менеджеру.

Конструктивная обратная связь в обзоре кода — это одно. Вызывать вас до нескольких раз в день перед сверстниками — это подталкивание к издевательствам.

Скажи своему боссу прямо

Это вредит моему моральному духу. Раньше я получал удовольствие от работы. Конструктивная структурированная обратная связь — это здорово, но звонить мне до нескольких раз в день перед коллегами для меня неуместно.

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

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

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

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

После этого, в следующий раз, когда он сделает это, прекратите то, что вы делаете, и немедленно окликните его:

Эй, это то, о чем мы только что говорили. Мне не нужно, чтобы вы критиковали мой выбор синтаксиса здесь, и, как я уже сказал, мне нужно, чтобы вы воздерживались от этого в будущем. Вы можете это сделать?

Это может занять несколько попыток, но это должно сработать для всех, кроме самых глупых коллег. Если у вас есть один из последних в этот момент, вам нужно поднять это как проблему с производительностью с вашим / его менеджером.

Софи, ты сказала, что ничего не работает. На самом деле, твои очень вежливые попытки изменить его поведение не сработали.

Вы пытались «проигнорировать это и заняться своими делами, вежливо, но твердо объясняя, почему он не прав, просить его развеяться и с юмором указать на нелепость некоторых из его наиболее придирчивых критических замечаний». Вы не сделали одного: вы не сказали ему СТОП. В следующий раз, когда он придет, скажите ему ОСТАНОВИТЬСЯ и уйти. Этот человек заставляет вас не любить свою работу, нет причин быть вежливым. Он не понимает вас, когда вы вежливы. Нет необходимости обсуждать его предложения и критические замечания.

Не бойтесь задеть его чувства. Он вызывает у вас желание бросить работу, поэтому он растаптывает ваши чувства. Не бойтесь быть невежливым. Он вызывает у вас желание бросить работу, это самая невежливая вещь, которую только можно себе представить, так что он не заслуживает от вас никакой вежливости.

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

Давайте рассмотрим это по частям. Я думаю, у вас есть ценная возможность здесь.

Я здесь разработчик-самоучка.

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

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

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

Так... чего ты хочешь?

• Вы хотите, чтобы все было так, как было до того, как он присоединился к компании?
• Вы хотите продемонстрировать руководству, что вы материал для руководства?
• Вы хотите, чтобы он уважал вас как сверстника?

• Вы хотите, чтобы он уважал вас как начальника?

  1. Этот корабль, вероятно, уплыл.
  2. Возможный.
  3. Возможный.
  4. Возможный.

Мне интересно, есть ли у кого-нибудь совет о том, как обращаться с коллегой (со стажем на моем рабочем месте около 1 месяца по сравнению с моими 18 месяцами), который использует любую возможность, чтобы настойчиво критиковать мою работу на каждом этапе процесса, от выбора программного обеспечения. пакетов вплоть до того, ставить ли пробелы между знаком равенства и присваиванием переменной (я не шучу).

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

...Если он не проявляет такого такта при обсуждении « вещей, о которых вы не знаете », тогда вам действительно нужно обсудить его манеры с вашей командой менеджеров.

Это происходит где-то от нескольких раз в неделю до нескольких раз в день. Это происходит все чаще и чаще на глазах у коллег, которые из-за этого начинают иначе относиться ко мне.

Это та часть, которая делает его подход таким неразумным, но также и то, что представляет для вас наиболее интригующую возможность ценности. Его «нападения» видны всем. Я бы вообще не стал спорить с ним по этому поводу, а вместо этого проконсультировался бы с руководством, чтобы объяснить ситуацию и обратиться к ним за советом о том, как вы можете реагировать таким образом на его невольный подход, который демонстрирует

1. ваше стремление найти способ продуктивно сосуществовать в компании
2. ваши лидерские качества. ( рискованно, если вы чувствуете, что еще не готовы к этой задаче )

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

Поздравляем. Это означает, что вашей работой довольны нужные люди.

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

Хороший. Сохраняйте эту точку зрения, даже несмотря на то, что он ведет себя бестактно в своем выступлении. Попытайтесь прорваться через его наглый подход и посмотреть, есть ли реальная ценность за его утверждениями. Я всегда отвечал на его вызовы задумчивыми «почему?», а не резким отказом из-за его тона. Но будьте осторожны, требуйте твердых ответов; спрашивая «почему» что-то обыденное и простое, может быть воспринято как незнание основных требований вашей работы. Поэтому я бы даже не привлекал его до разговора об этом с руководством.

1. Это легко продемонстрирует, что вы готовы слушать
2. Это легко продемонстрирует, что вы ищете вескую причину, чтобы сделать то, что он предлагает.
3. Это был бы простой способ извлечь платное образование из его мозга.
4. Когда вас доставили к вашей управленческой команде, у вас есть о чем поговорить: «Он был неубедительным»; " Ну... у него мог быть хороший ответ, почему я должен слушать его, но я не мог услышать его из-за его грубости. "

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

То, что произошло за последний месяц с тех пор, как мой новый коллега начал работать, — это нечто совершенно другое. Я испробовал несколько стратегий, чтобы справиться с этим: игнорировать это и заниматься своими делами, вежливо, но твердо объяснять, почему этот человек не прав, просить этого человека развеяться и использовать юмор, чтобы указать на нелепую природу некоторых гнид. -придирчивая критика. НИЧЕГО НЕ РАБОТАЕТ.

Итак... вот где "он сказал/она сказала" можно частично рассмотреть. Результат, который вы, кажется, ищете, это «оставь меня в покое», а результат, которого он, кажется, ищет, это «чтобы ты изменился». Вы не указали, есть ли у вас основания требовать, чтобы вас оставили в покое, и у нас недостаточно подробностей, чтобы понять, прав ли он, ожидая, что вы изменитесь.

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

В конце концов, это повышение на 18%, о котором вы упомянули, и рассмотрение кандидатур на руководящие должности следует рассматривать как подтверждение того, что вы следуете тому пути, который хочет для вас клиент; это поляризует ваши позиции: «Вы правы, а он мелочен». Это было бы вопиющим преувеличением вашей руки... просто сказать это как предупреждение. Легко думать, что такие вещи обеспечивают ранг, когда на самом деле это не так. Они являются свидетельством того, что вы хорошо выполняете свою работу. Не меняйте этот послужной список.

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

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

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

Вам почти всегда придется находить способы работать с трудными людьми, если только вы не достигнете положения, когда вы контролируете свою рабочую силу.

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

Обстоятельства того, где и когда он это делает, могут повлиять на ваш ответ.

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

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

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

Если он начинает это на ровном месте или только тогда, когда другие люди входят в зону слышимости, то остановите его чем-то вроде:

• Это не имеет отношения к обсуждаемой теме
• это не твое дело
• Это не текущий стандарт
• я не работаю на тебя
• Я не знал, что ты бог кодирования, который может что-то диктовать. Можете ли вы показать мне электронное письмо, в котором вам была назначена эта обязанность? (Сарказм иногда творит чудеса)
• Можем ли мы вернуться к соответствующему вопросу, например... (Помогает, если вы поднимете вопрос, который ему нужно улучшить в данный момент)
• Уже было решено двигаться в этом направлении, и мы не собираемся возвращаться к нему на данный момент (хорошо, когда он критикует программное обеспечение / метод, который вы используете для решения проблемы, а не придирчивые вещи).
• Это неправильно.

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

Мне тоже хочется добавить свои 5 ручек.

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

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

Так что, если вы можете быть слишком самоуверенны в своих навыках? Все, что я прочитал до сих пор, это то, что вы пробовали:

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

Все это означает, что он ошибается или даже обесценивает свои мысли. Но что, если на самом деле вы были неправы? Или, может быть, даже никто из вас обоих не ошибается, и вы оба просто ожидаете, что друг у друга будут разные ожидания от обсуждения, чем у вас на самом деле?

Теперь позвольте мне рассказать, что может быть на самом деле, поскольку это то, с чем я сталкиваюсь не так уж редко:

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

Мне действительно нравится просвещать других своими усилиями, которые я вкладываю в изучение C, чтобы максимально соответствовать ISO/IEC, и мне это доставляет удовольствие. Иногда, просматривая чужой код, я уведомляю их о том, что что-то не совсем соответствует C. И здесь обычно возникали 2 ситуации, одна из которых мне очень нравилась, а другая ситуация ^меня немного не устраивала.

Итак, первый вариант, который обычно бывает:

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

а второй это:

• Человек чувствует себя обиженным, просто начинает защищаться, требуя от меня доказательств того, что он неправ, поскольку то, что он делает, на самом деле работает и выполняет работу. Хотя здесь я легко мог бы доказать это, я просто отказываюсь, так как я не собираюсь разоблачать их за неправоту и то, что я должен был узнать, как отмечалось ранее, в этой ситуации больше нет возможности объяснить: «Эй, я просто пытаюсь чтобы помочь», так как в любом случае это будет воспринято как предлог для их разоблачения.

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

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

Я, например, никогда не видел, чтобы кто-то защищался от моего совета, помещая туда пробел, и даже если они продолжали его опускать, я подправлял его всякий раз, когда мне приходилось просматривать их код (и это тоже не шутка!).

Здесь я хочу добавить еще один пример:

Я работаю в небольшой стартап-компании, которая на самом деле растет в размерах. Очевидно, что в начале не было никакой политики кодирования. Наш генеральный директор несколько месяцев назад поручил мне создать такие политики, так как я особенно хорошо осведомлен о зарезервированных выражениях, знаках и ключевых словах языка Си. Я сделал это, представил и получил одобрение. Но до сих пор это никого не волнует, и наш генеральный директор не беспокоится о том, чтобы обеспечить его соблюдение, хотя у него все еще были свои причины поручить мне это и утвердить мой дизайн, я мало что могу с этим поделать, кроме как напоминать об этом другим. . Так что, конечно, я мог бы просто встать и сказать: « Послушайте, ребята! Вы знаете политики, которые я разработал? Я ожидаю, что вы будете строго соблюдать их с этого момента, и я исправлю ваш код, чтобы он соответствовал им, если вам все еще все равно!«Наш генеральный директор, вероятно, даже поддержал бы этот шаг, но… Да ладно, каким придурком я буду выглядеть в будущем в этой компании? какой-то момент и пусть это им, зная, что они должны это сделать.Но я не работаю в том, чтобы навязывать это, поэтому, хотя мне немного больно видеть, что собственную работу не уважают таким образом, я мало что могу сделать против этого, кроме как быть придирчивым чуваком, который иногда упрекает «за несоблюдение его так называемой политики».

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

ТЛ и ДР:

Итак, каков вывод?

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

То, как вы написали ОП, позволяет вам появиться, поскольку вы предполагаете, что он делает это с плохими / эгоистичными намерениями.

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

Если это так, я думаю, это все равно изменит ваше мнение о нем.

И если первое должно иметь место .... Ну, тогда мне нечего добавить, что ни один из других ответов уже не советовал.

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

Иногда у новых сотрудников возникают очень хорошие идеи. А иногда они просто хотят, чтобы все вдруг начали работать так, как им удобнее.

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

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

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

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

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

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

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

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

Рассматривали ли вы возможность того, что у вашего коллеги могут быть действительные баллы?

Я собираюсь сосредоточиться на: "...вплоть до того, ставить ли пробелы между знаком равенства и присваиванием переменной (я не шучу)".

За годы работы разработчиком я критиковал других за:

• Непоследовательное использование пробелов.
• Использование пробелов для отступа вместо табуляции.
• ЗАГЛАВНЫЕ НАПИСАНИЯ имен переменных.
• Ошибки в именах функций.

Все это чисто косметические проблемы. Вот почему я считаю их важными:

• Когда файл редактирует один человек, затем второй, снова первый и т. д., система управления исходным кодом собирает «различия» между файлами. Если два разработчика не согласны с тем, как использовать пробелы, эти различия будут полны бессмысленных изменений пробелов, что затруднит поиск фактических изменений. Каждая команда должна определить правила использования пробелов и придерживаться их.
• В Java соглашение состоит в том, чтобы использовать ALL_CAPS для постоянных переменных уровня класса (static final) и camelCase для других переменных и функций. Большинство других языков имеют аналогичные соглашения. Если вы используете нестандартные заглавные буквы, люди могут запутаться в использовании вашей переменной и предположить, что переменная является static final, когда это не так. Это может привести к тонким ошибкам.
• Однажды я написал функцию, которая должна была переопределить функцию в суперклассе, но другой программист использовал необъяснимое написание, поэтому моя новая функция была просто новой функцией, а не переопределением (это было до того, как @Override стал обычным явлением). Очень тонкий баг, на исправление которого ушло много времени и много ругани.

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

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

Что рекомендую: Не игнорируйте критику. В любом случае это делу не поможет. Привлекать. Если критика касается интервалов или какого-либо другого аспекта стиля кодирования, найдите или создайте руководство по стилю кодирования. Каждая команда разработчиков должна иметь его. (Sonarqube предлагает автоматические проверки в стиле кодирования, которые очень хороши, даже если я не согласен с тем, где они ставят открытые скобки).

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

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

Существуют разные практики кодирования, и неправда, что кто-то с месячным опытом не сможет исправить кого-то с 18-месячным опытом, но дело в том, что он может поднять эти вопросы на сессии, когда вы или другие программисты говорите о кодировании. стили, у нас есть такая сессия, на которой некоторые старшие участники рассказывают нам о передовом опыте, а младшие также принимают участие в этом, иногда с действительными положительными моментами, вы должны предложить ему принести эти вещи на такую ​​сессию, чтобы почти каждый мог внести свой вклад. это вместо того, чтобы время от времени выбирать это, поскольку это раздражает, отвечая и объясняя каждую глупость во время разработки.

Поговорите с ним и скажите, чтобы во время такой сессии он поднимал свои лучшие практики, а не придирался, по крайней мере, он не будет чувствовать себя полностью проигнорированным (если он был в чем-то прав).

Эффективной контрмерой является улучшение собственной работы настолько, чтобы он не смог найти достаточно реальных проблем, чтобы продолжать.

Это его не остановит, я понимаю. Но, все еще пытаясь выливать критику ведрами, он начнет ошибаться и станет уязвимым.

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

Также проверьте, действительно ли работа коллеги безупречна. Очень часто такие люди предъявляют к себе другие и гораздо более мягкие требования. Обнаружение и открытое обсуждение какого-либо сбоя на вашей стороне может сработать гораздо лучше, чем попытка снова защититься от критики.

PS Не пытайтесь отвечать с сарказмом, это относительно легко может быть обращено против вас.

Ваш вопрос

Мне интересно, есть ли у кого-нибудь совет о том, как обращаться с коллегой (со стажем работы на моем рабочем месте около 1 месяца по сравнению с моими 18 месяцами), который использует любую возможность, чтобы настойчиво критиковать мою работу на каждом этапе процесса,

Затем вы продолжаете говорить

Это работа моей мечты. В течение последних 17 месяцев, прежде чем он начал работать, я каждый день просыпался с радостью перед работой. Теперь все настолько плохо, что я серьезно подумываю о том, чтобы бросить курить.

Может быть, то, что вы ищете, — это не столько способ общаться с вашим коллегой, сколько снова наслаждаться работой своей мечты? Возможно, нет. В любом случае есть два варианта:

1. они могут что-то изменить
2. ты можешь что-то изменить
3. ни один из вышеперечисленных

1. Могут ли они что-то изменить?

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

Так каковы шансы, что ваш коллега изменится только для вашей выгоды?

Даже если они собирались измениться ради вас, у вас нет четкого способа заставить их измениться.

2. Вы можете что-то изменить?

На мой взгляд, вы уже попробовали то, что я бы назвал реалистичным вариантом (рассказывая ему, почему он объективно не прав). Есть еще два варианта (которые я вижу), и они оба одинаково действительны.

Вы: пессимист

Предполагая, что ваш коллега не изменится и никогда не изменится , и вы не меняете работу: возможно ли, чтобы вы сохранили эту работу, стала ли она снова работой вашей мечты, независимо от вашего коллеги? Если ответ «да» и вы видите четкий путь к этому, неуклонно идите по этому пути. Если вы не видите путь, взгляните на абзац ниже и посмотрите, поможет ли это. Если ответ категорически нет , подумайте об уходе. Я обещаю вам, есть не одна работа мечты. Иногда нам приходится расставаться с хорошими вещами в нашей жизни и надеяться, что мы найдем что-то еще лучшее.

Вы: оптимист

Вы пробовали с ним договориться? Независимо от того, согласны вы или нет. На мгновение войдите в его мир, где вы предполагаете, что вы действительно не правы. Оставайтесь там и укажите на то, что, по вашему мнению, он упустил. Пример:

Он: «Ты забыл поставить здесь пробел»

Вы: «Знаете что? Вы правы. И я тоже забыл сделать это здесь. Спасибо. Не могли бы вы сделать обзор кода этого патча, чтобы увидеть, есть ли еще какие-то стилистические варианты, которые я мог бы улучшить?»

Что это делает, так это освобождает вас от любой ответственности иметь дело с ним. Иметь с ним дело - не твоя работа. Ваша работа заключается в том, чтобы писать код и делать это как можно лучше.

Ведь цитирую вас:

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

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

Ох, хорошо. Я вижу, вы несколько раз указывали на это в моем коде. Где я могу прочитать больше об этом?

Это передает мяч обратно к нему. Может ли он подкрепить свою критику реальными фактами/наилучшей практикой? Может ли он быть полезным сотрудником? Быть полезным означает игнорировать критику и помогать своему коллеге становиться лучше.

3. Ничего не меняется

Предположим, что он не меняется, и вы не меняетесь с точки зрения ваших взаимодействий. Давайте немного отмотаем назад в вашем вопросе:

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

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

Удачи, надеюсь, вы снова обретете радость от работы. Каждый заслуживает этого :)

Редактировать: не связанный, но важный: о придирках типа «пробелы против вкладок» должен заботиться разумный бот / pre-commit hooks, который проходит через ваши коммиты еще до того, как они будут обнародованы. Посмотрите git pre-commit hooks здесь . Предложите вашей команде сделать один для проверки стиля, потому что автоматизация важнее, чем разработчики, выражающие свое мнение о стиле.

Я знаю, что у этого уже есть много ответов, и хотя ответ Зайбайса касается этого, я хотел добавить краткое предложение, которое сработало для меня (и НА меня) в прошлом.

Некоторым людям трудно держать свои «хорошие идеи» при себе. Я знаю, что многие люди не понимают, почему я возражаю, когда люди отделяют свои переменные от своих назначений с помощью табуляции, чтобы они могли составить красивый список, выровненный по одной позиции табуляции. Если вы хотите прекратить мои придирки, просто выслушайте и поймите меня. Мне все равно, если вы продолжите делать то, что делаете (я работал над этим), но я не могу не попытаться помочь вам, если увижу, как вы можете делать что-то лучше. Я не позволю тебе после обеда ходить со шпинатом в зубах; Я скажу кое-что.

Пробовали ли вы притворяться, что это не критика, а просто полезное предложение? Иногда, если вы скажете: «О, почему вы думаете, что это важно/должно быть сделано именно так?» вы можете превратить критику в дискуссию, где вы можете продемонстрировать, что вы знаете свое дело и у вас есть причины поступать так, как вы это делаете. Когда вы начинаете обсуждать и слушать, вы подключаетесь. Людям, не являющимся социопатами, труднее уничтожать людей, с которыми они связаны. Кто знает, они могут быть инь для вашего яна, и команде будет лучше, если вы будете уравновешивать друг друга.

Это может быть слишком сложно для вас, в зависимости от того, насколько напряжённо обстоят дела, но я изменил ситуацию, подобную этой, отправив электронное письмо, просто оценив хорошее качество человека. «Шэрон, я очень ценю, что ты высказываешься, когда видишь, что что-то, по твоему мнению, можно было бы сделать лучше. Я могу не всегда с тобой соглашаться, но я уважаю то, что ты постоянно заботишься о команде». Или что-то в этом роде — это должно быть искренним, что затрудняет выполнение, если у вас к кому-то много плохих чувств.

Был в отрасли некоторое время. Так делают некоторые разработчики.

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

Я собираюсь прийти к этому с немного другого подхода.

Ошибка заключается в том, что вы (двое) плохо работаете в команде. В команде нет «я», и хотя этот парень может действовать неправильно, он на 100% прав в том, что эти решения принимаются на «командном уровне», и он должен вносить в них свой вклад.

выбор программных пакетов сводится к тому, ставить ли пробелы между знаком равенства и присваиванием переменной (я не шучу).

Линтинг вашего кода важен. пробел или не пробел - это серьезный вопрос стиля кода, который не соответствует некоторым показателям. Лучший ответ здесь — иметь руководство по стилю, на которое вы можете указать, в котором говорится, что не нужно или нужно ставить пробелы. Если вы делаете это одним способом несколько раз и другим способом в другой раз. Тогда вы «плохой разработчик», и он имеет полное право критиковать вас. Что более вероятно, так это то, что вам нравятся пробелы, а ему нет (или инверсия). Наличие руководства по стилю исправит это примерно за 30 секунд. Конечно, вы должны придерживаться этого, и это может быть отстойным, пока вы приспосабливаетесь. Почти каждый язык программирования, который я видел, имеет руководство по стилю, которое вы можете просто «использовать». Большинство, если не все, поставляются с инструментами, которые помогут вам применять это руководство.

То же самое с пакетами (скорее пакетами кода или цепочкой инструментов) должен быть хороший предопределенный набор того, что в порядке. Когда вам нужно что-то, чего нет в «утвержденном списке», пришло время для собрания команды. КОМАНДА должна выбрать, какие инструменты будут использоваться всей командой. Не вы случайно идете в одном направлении, в то время как ваш товарищ по команде идет в другом.

Короткая версия такова, что вы оба ошибаетесь. Он не должен быть таким настойчивым, и вы должны включать его в эти решения. Тебе следует говорить об этом до того, как он успеет придраться к ним. Теперь ты в команде и должен вести себя соответственно. Может быть, из-за вашего опыта ваш QB. Но ты ничего не добьешься, если твоя линия играет одну пьесу, а ты другую. Общайтесь, общайтесь, работайте вместе и ПРЕКРАТИТЕ самостоятельно принимать решения на командном уровне.