Один из моих товарищей по команде-мастер на все руки в магазине и я уважаю его понимание.
Однако, иногда он просматривает мой код (он's во-вторых в команду для лидера нашей группы, так что'ы ожидалось) без головы. Поэтому иногда он комментарии мои изменения, прежде чем они завершить конечной цели и вносит изменения сразу же... и даже однажды разбила мою работу.
Другие времена, он сделал ненужными улучшения в некоторые из моих код, который находится в 3+ месяца.
Это раздражает меня по нескольким причинам:
В любом случае, я сказал ему, в прошлом пожалуйста, держите меня в курсе, если он видит что-то в моей работе, которую он хочет изменить, так что я мог взять мой код (может быть, я бы сказал "недостатки", У) и он's не реагирует.
Я боюсь, что мне может показаться агрессивным, когда я просил его объяснить свои изменения для меня.
Он's просто спокойный человек, который держит при себе, но его действия по-прежнему. Я не'т хотим, чтобы изгнать его из внесение изменений в код (не могу), потому что мы команда, но я хочу внести свой вклад, чтобы помочь нашей команде.
Добавлены разъяснения:
Вы и большинство ответами подход в этом вопросе связь между двумя коллегами, но я Дон'т действительно думаю, что это. То, что вы говорите, это ужасно нарушен процесс обзора кода, чем все остальное. Во-первых, вы упоминаете, что ваш коллега является вторым в команде, и это'ожидали, что он'будете проверять ваш код. Что's просто не так. По определению экспертных оценок код не иерархическая, и они, конечно, не просто о поиске дефектов. Они также могут обеспечить обучения (для всех участников), возможность для социального взаимодействия, и оказаться ценным инструментом для создания коллективного владения кодом. Вы должны также проверить свой код, время от времени, учиться у него и поправлять его, когда он'ы не так (никто не получает это право время любую ранее). Кроме того, вы упомянули, что ваш коллега вносит изменения сразу. Что's также неправильно, но, конечно, вы уже знаете это; вы не'т уже задал этот вопрос, если его подход настырная была'т проблема. Однако я думаю, что вы ищете решение в неположенном месте. Честно говоря, ваш коллега немного напоминает мне... меня, и то, что работало для меня в подобной ситуации был вполне определенный и твердый процесс обзора и комплектом инструментов. Вы Don't действительно хотите остановить ваш коллега из рассмотрения код, а прошу его остановиться и поговорить с вами, прежде чем каждое незначительное изменение на самом деле не собирается работать. Возможно, на некоторое время, но он'скоро достигнет точки, когда он просто слишком раздражает и вы'll быть там, где вы начинали, или хуже: он'МР просто остановить просмотр кода. Ключ к решению здесь может быть инструментом рецензирования кода. Я обычно избегаю рекомендаций продукта, но по отзывам код компании Atlassian's на горнило действительно жизнь заставка. Что это может показаться очень простым, и это, но это вовсе'Т означает, что она's не удивительно удивительным. Он подключается к вашему репозиторию и дает вам возможность обзор отдельных изменений, файлы или группы файлов. Вы Don'т получить, чтобы изменить какой-либо код, вместо этого вы прокомментируете все то, что не'т чувствовать себя достаточно хорошо. И если вы абсолютно должны изменить кого-то другого's код, вы можете просто оставить комментарий с набором объясняя изменения. Вступительный ролик в тигель'страница продукта стоит смотреть, если вы хотите более детально. Тигель'ценообразование s не для всех, но существует множество свободно доступных инструментов коллегиального обзора. Один я'вэ работал и наслаждался Совета и я'м уверена, что вы'll найти много других с помощью простого поиска Google. Какой бы инструмент вы не выбрали, она будет полностью изменить свой процесс. Не нужно останавливаться, слезать с кресла, прервать другого человека и обсудить изменения; все, что вам нужно сделать, это установить выходной каждую неделю и проходим в комментарии (один раз в неделю-это просто предложение. Вы знаете свой график и распорядок дня лучше, чем я). Более главное ядро комментарии хранятся в базе данных где-то и вы можете получить их в любое время. Они не'т эфемерные дискуссии вокруг охладитель воды. Мой любимый вариант использования старых отзывов, когда появление нового члена команды для нашего кода. Это's всегда приятно, когда я могу ходить с кем-то новым в кодовой базе указывая, где именно мы застряли, где мы разошлись во мнениях и т. д. Двигаясь дальше, вы учтите, что вы Дон'т всегда найти этот коллега's код читаемым. Что позволяет мне знать, что вы Дон'т имеют общий набор стандартов кодирования, и что'ы плохо. Опять же можно подойти к этому как проблема людей, или вы можете подойти к этому как к проблеме процесса, и снова я настоятельно рекомендую последний. Получите вашу команду вместе и выработать единый стиль кодирования и набор стандартов, как можно скорее. Это вовсе't действительно важно, если вы выбрали набор стандартов, которые'распространенные в своем развитие экосистемы, Или вы придумали свой собственный. Что действительно важно, для ваши стандарты должны быть последовательными и придерживаться их. Много-много инструментов там могут помочь вам, но, что'ы совершенно другой дискуссии. Просто, чтобы вы начали, очень простая вещь, чтобы сделать, это иметь пре-коммит Хук запустить какой-то стиль форматирования кода. Вы можете продолжать, однако писать свой код вы как и на "исправить", которая автоматически пока никто не увидел. Наконец вы упоминаете в comment](https://softwareengineering.stackexchange.com/questions/193802/how-do-i-take-responsibility-for-my-code-when-colleague-makes-unnecessary-improv#comment374326_193802), что руководство не верит отдельных отраслей Дэв необходимы. Ну, там'Почему мы называем их "в ветвях Дев" и не "Управление филиалами." Я'МР остановиться здесь, как там's нет причин разглагольствовать, что'с, образуя в моей голове, чтобы выбраться наружу. Все, что говорил, знал, что я Дон'т сомневаюсь, что ваш коллега (немного) вина. Что's не моя точка зрения, моя точка зрения заключается в том, что весь процесс развития тоже виноват, и что's что-то, что'ы легче исправить. Вооружить себя с правильными инструментами, изучить многочисленные формальные и неформальные процедуры и выбирайте те, которые подходят вашей команды. Скоро вы'МР достигаете точки, где вы'МР понять, что большинство ваших "Люди проблемы на" Дон'т больше не существует. И, пожалуйста, Дон'т слушать никого (включая себя), что приносит "Мы'снова небольшая команда, мы не'т нужно все, что" извините. Опытные разработчики могут создать необходимые инструменты в менее чем за неделю, автоматизировать все что можно автоматизировать, и никогда не смотреть назад. ПС. "в собственности код" это в туманной перспективе, постоянно спорят, и это означает разные вещи для разных людей. Вы можете найти блестящую коллекцию самых разных (иногда противоположных) мнений о С2.
Я думаю, что большинство разработчиков, оказавшиеся в таком положении в какой-то момент, и я надеюсь, что каждый разработчик, который'ы счел себя жертвой понимает, как неприятно это будет, когда он или она становится старше и чувствует себя обязанным очистить код, написанный юниоров.
Для меня, как избегать конфликтов в данной ситуации сводится к двум вещам:
Вежливость. Говорить кому-то о его/ее код позволяет Дэв знал, что вы'повторно заинтересованы, и вы можете обсудить его как выросли профессионалы.
Забыть о том, что "собственности код и" - команда владеет кодом. Другие люди хотят, чтобы сделать изменения это хорошо. Если старший Дэв делает изменения, то "нечитаемые" или еще хуже, потом обратно их. Вы Don'т должны быть агрессивными, просто дайте редактор знает, что его изменения не't работа, и вы'вновь больше, чем рады обсудить ваше возвращение.
Помните, команда владению кодом является большим, и это палка о двух концах. Если вы видите что-то, что не'т смысл на кого-то другого's код, а затем восстановить ее. Слишком назойливым и недостаточно общительный-это верный способ создать ядовитой среде разработки.
О чем это процесс, который делает вы хотите брать на себя ответственность за фильм "код" и ? У вас есть ответственность, чтобы сохранить определенные функции работают? Сделал ведущий говорит: "Майкл, я хочу, чтобы вы взять на себя ответственность за ..." и ? Или подразумеваемого ответственности, в том, что свинец и остальные команды смотрят на вас каждый раз, когда некоторые функции не нарушилась?
В любом случае, если у вас есть ответственность, то вам нужна власть над кодом. В следующий раз, когда другой человек делает односторонние изменения и лидерство возвращается к вам, чтобы исправить их, вы должны сесть с ведущим и попросить, чтобы ваш авторитет и ответственность выравнивается.
Все implicitely 'владеет собственный код', независимо от политики, legalistics, или экономика - это'ы 'природе вещей' - вы, естественно, чувствуют личную связь с вашей собственной работы.
Если ваш сотрудник является вовлечение в поведение, которое вы описали и по-прежнему не отвечает, Когда вы просите для головы, что коллега-это невежливо, по меньшей мере, и, возможно, пытается подорвать вас (чтобы не сказать хуже...) - совсем не звук как командный игрок.
Хороший сотрудник будет связаться с вами и указать, проблема с ваш код для тебя - и пусть Вам исправить/изменить его, или адекватно реагировать. Я очень благодарен, что даже когда я был Новичок, мои наставники всегда говорили мне, что я делаю неправильно, объяснить, почему и пусть (или составила) мне это исправить. Это сделало меня лучшим программистом, и все выиграли. И что's то, что я всегда делал при анализе работы других участников. Тогда вы, (или кто там) на самом деле что-то узнает от ваших 'мастера на все руки', и код и команды все наладится, в том числе и ваш преподаватель: преподавание помогает пониманию.
Если это'ов на все возможные, я хотел бы обсудить этот вопрос один на один с руководителем группы. Судя по вашему описанию ситуации, хороший лидер команды примет вашу сторону - плохо выиграл'т.... Очевидно, что это требует осторожности - вам придется судить, что для себя.
Не то чтобы это решит всю ситуацию, но вы можете попробовать добавить больше комментариев в исходный код.
Все и все, попробовать сделать вместо лимонада тратить время сосал лимонов. Как сказал Михаил, в общем, товарищей нет'т заставить тебя выглядеть плохо. Стараюсь учиться на своих ошибках и применять их в будущих версиях.
Если вы считаете, что его изменения оказывают негативное влияние, пожалуйста, голос (дипломатично). Если бы это был я, я бы просто спросить, почему были сделаны определенные изменения и посмотреть, если вы можете отстоять свои оригинальные изменения. Ваши старшие коллеги тоже люди. Это'вполне возможно, что он что-то пропустил и/или не знает о каких-либо негативных последствий он.
Если вы пишете код, то я должен рассмотреть его.
Если я изменить свой код в комментарий, то код-это не код больше, что я просмотрел, но код, который я изменил. Поэтому он должен быть пересмотрен. Вероятно, вами.
Если я предаю ваш новый код с моими изменениями без кто-то рассматривает мои изменения, то я совершил (1) изменение проверены, и (2) худший грех, если комментарии код воспринимать всерьез.
Я думаю, что вы не обработка это правильный путь сейчас - но скоро будет переломный момент, где она отвлекает вас, насколько можно быть счастливым, кодирование этот способ на всех.
Если я был вами, я хотел бы попросить быстрый один-на-один с этим человеком и спокойно, но твердо объяснить свою пов. Владение команды кода и т. д. Это все хорошо, но если вы не дадите каждому разработчику достаточно места, чтобы положить его работать, делать ошибки и улучшить вам никогда не построить хороший код. Это может быть зона трения скорее раньше, чем позже.
(Там совершенно другой ответ, если это было на рабочем месте.клиент StackExchange. Выяснить, правильный способ делать обзоры кода-это проще всего. Убедить вашего сотрудника соблюдать это намного сложнее).