Как убедить разработчиков принять единый код стайл?

Question

[Василий Мельников]

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

Завтра у меня запланировано собрание с подведением итогов. Там все будет официально утверждено. В итоге выйдет приказ либо просто по отделу, либо если надо и Генеральный утвердит.

Может нагенерите идей и аргументов? Я изобрел/скопипастил 7 тезисов, но вдруг что-то еще скажете.

Answer 1

[plasticmirror]

мы подобный вопрос решили для с# - просто большинство договорилось до формата и впилили настройки в решарпер - после чего каждый может переформатировать в одно касание хоть весь солюшн несогласные немного поворчали и привыкли

ну а карательные меры простые - если есть комиты, весь смысл которых сводится к "неправильному форматированию" - человек получает по думательному месту

учитывая ревью - это естественным образом получилось

п.с. некоторые писали по привычке - потом в один клик форматировали готовое в "как у всех", особых проблем не доставляло никому

Comments

[Василий Мельников]

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

Answer 2

[1nd1go]

Демократия редко работает на командах овер 5 человек. Она вообще впринципе редко когда работает, но в меньше 5 человек - можно еще договориться.

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

Comments

[Василий Мельников]

Не очень годится - много старых/текущих проектов. Для новых вещей наверное можно

Answer 3

[delightmanua]

В случае использования CI системы, можно поставить проверку code style наряду с тестированием и в случае проблем - реджектить

Работать в таком режиме тяжело в первый месяц, затем все втягиваются

Comments

[Василий Мельников]

До CI не дожили пока :)

Answer 4

[lexdevel]

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

Answer 5

[igordata]

Обязать.

Comments

[lexdevel]

Есть шанс потерять сотрудника(-ов). Обязаловки - в гос структурах, в компаниях - демократия :)

[Василий Мельников]

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

[igordata]

К логике взывали когда обсуждали и голосовали. Когда решение принято и все высказались, не надо устраивать детский сад. ТС указал, что устроил голосование, и со всеми обсудил. Те, кто не хотят дальше жить в таких условиях и не могут принять такое простое требование, не вписываются в коллектив и мешают продуктивной работе. У них на каждом коммите каждый файл который они открывали будет весь переформачен, и как следствие защитаны изменения в строках. Как с ними потом работать при таком раскладе - не ясно. Решение одно, да. Обязать. Карательных мер не надо. Либо работает как договорились (все вместе между прочим), либо досвидос. Просто демократию надо было строить правильно. Сначала надо было оговорить, что кто принимает голосование, тот принимает его результат. А кто голосование не принимает, тот уже явно не демократию разводит, а просто капризит.

[igordata]

Спасибо, тостер, что похерил все мои абзацы.