Когда вы вносите свой вклад в очень старый и, по-видимому, более не поддерживаемый проект с открытым исходным кодом или общественным достоянием, рекомендуется ли ему следовать стилю авторского кодирования, даже если это ужасно?
Что еще более важно, есть ли какие-либо причины, по которым вы хотели бы следовать оригинальному стилю кодировки автора, а не тщательно его очищать?
Следуйте оригинальному стилю кодировки. Далеко, намного лучше быть последовательным, даже если это вам не очень нравится.
Если вы решите очистить стиль кодирования, сделайте это отдельно от любых других изменений. Не загромождайте контроль источника с изменениями стиля. Сделайте одну (или несколько) проверок, где единственное, что вы делаете, - это изменение стиля кода. Не смешивайте реальные изменения с бессмысленными изменениями, это делает невозможным обнаружение соответствующих изменений при проверке фиксации источника.
Если вы знаете, что это не поддерживается, и вы вносите в него изменения, я подозреваю, что вы станете владельцем/сопровождающим дефактором в будущем.
Итак, я сказал бы по электронной почте оригинальному автору/сопровождающему, посмотрю, что они думают об изменениях форматирования или стиля, и продвигайтесь с пылающими орудиями.
Если вы не отправляете патчи, то нет. Запустите его через фильтр кода и автоматически отформатируйте код в предпочитаемом вами стиле, а затем взломайте.
Если вы проверяете исходный элемент управления, вы можете оставить его в покое, поскольку все изменения стиля сделают невозможным сравнение с предыдущими версиями.
Если вы хотите изменить стиль, сделайте это с базовым кодом, прежде чем выполнять какие-либо другие изменения, а затем проверьте его.
Затем проверьте его и внесите изменения в код. Таким образом, будет легче отслеживать ваши изменения с момента, когда вы его приняли.
Я лично оставил бы стиль один, если вы не внесете существенные изменения.
Как указано другими, убедитесь, что любые модификации стиля проверены в вашей системе управления версиями отдельно от ваших функциональных модификаций.
Однако мое предложение состоит в том, чтобы быть жирным и очистить код по своему усмотрению; "Оставьте уборную кемпинга, чем вы ее нашли". Чиз, но правда;)
Просто убедитесь, что вы готовы защищать свои изменения;)
Нет, нет! Что бы вы ни делали, не делайте этого хуже ради небес!
Если вы добавляете свой собственный код, сделайте это в своем собственном стиле. Это, по крайней мере, сделает часть кодовой базы легкой для вас поддержкой и пониманием. Если вы делаете небольшие обновления существующего кода, вы можете захотеть следовать этому стилю.
Я нахожусь в одной лодке, мне нужно поддерживать древнюю, критичную для бизнеса систему на основе Access 2, которая представляет собой полный мишмар стилей.
Если он даже не поддерживает, почему вы беспокоитесь об этом?
Я бы написал свой собственный предпочтительный стиль и очистил любой код, с которым мне приходилось работать. Я делаю это регулярно на работе даже...
Хорошо, я бы сказал, что если вы просто делаете небольшую модификацию, скажем, функцию, возможно, было бы более ясным идти с плохим стилем.
Если вы создаете новые методы, классы, свойства и т.д., используйте четкий, эффективный стиль. Вы можете добавить комментарий в начале этих разделов, чтобы другие могли понять, что происходит. Вы даже можете добавить короткую заметку наверху, объясняя, когда вы начали код, и что вы использовали другой стиль и т.д.
Каким был бы ваш ответ, если бы вопрос: если вы поддерживаете такое же дрянное мастерство, когда будете реконструировать свой дом?
Если вы исправляете сломанную плитку в ванной комнате, то это не стоит переделать всю стену, потому что они не использовали правильную гипсокартонную решетку.
Когда вы переделываете кухню, вы можете исправить сантехнику для этой комнаты или выровнять плохой пол. Это потому, что это облегчает вам работу. И что все это значит, какая польза от этого дает вам стихи стоимости для вас/вашего клиента.
Вы не будете переустанавливать лампу, потому что ей нужна новая лампочка.
Если вы кодируете стандартизованный компанией (или даже стандартный для команды стандарт), а код, над которым вы работаете, не соответствует стандарту (или настолько старый, что он не соответствует текущий стандарт), то, безусловно, очистить код, если вы собираетесь делать серьезные изменения в любом случае. (Как уже упоминалось, сделайте это с базовым кодом и проверьте его. Затем проверьте его и внесите изменения, которые вы должны исправить, чтобы исправить ошибки, добавить функции и т.д.)