Как Вы говорите кому-то, что они пишут плохой код? [закрытый]

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

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

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

Наконец, я добавил бы, что существуют многочисленные книги, которые стоит исследовать по теме - моим фаворитом в этой точке является "Руководство по проектированию Платформы" Brad Abrams и Krystof Kwalina (и др.).NET команда BCL в Microsoft. Это делает удивительное задание обсуждения и объяснения решений, которые были приняты, и места витрин, где инструкции не сопровождались внутренне и осадки, которые закончились.

Перед ним просто осуществляют рефакторинг его код и показывают различие между этими двумя версиями. Определенно ему понравится это.

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

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

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

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

Мне действительно понравилось ответ EnderMB , но я хотел добавить к этому:

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

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

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

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

Другие программисты должны баловаться немного больше. Я нахожу сообщение их, что-то плохо, хорошо; "это не хороший способ записать код", сопровождаемый небольшим количеством тренировки "здесь, видеть, делаем ли мы это, это - больше читаемых/меньших количеств предупреждений/независимо от того, что". Это - конструктивная критика, которая помогает; если Вы не можете поместить свои деньги, где Ваш рот, и на самом деле сделайте это лучше, Вы являетесь лучшими не прокомментировать, даже если Вы знаете, что это плохо.

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

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

Дружеское электронное письмо тоже никогда не помешает.