Обязательное использование брекетов - PullRequest
17 голосов
/ 16 декабря 2009

Как часть документа стандартов кода, который я написал некоторое время назад, я обязуюсь «вы всегда должны использовать фигурные скобки для циклов и / или блоков условного кода, даже (особенно), если они состоят только из одной строки».

Пример:

// this is wrong
if (foo) 
    //bar
else 
    //baz
while (stuff)
    //things

// This is right.
if (foo) {
    // bar
} else {
    // baz
}
while (things) {
    // stuff
}

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

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

Ответы [ 21 ]

0 голосов
/ 16 декабря 2009

Для таких вещей, я бы порекомендовал просто придумать шаблон конфигурации для автоформатера вашей IDE. Затем, когда ваши пользователи нажимают Alt-Shift-F (или любое другое нажатие клавиши в выбранной вами IDE), автоматически добавляются фигурные скобки. Затем просто скажите всем: «Идите вперед и измените цвет шрифта, настройки PMD или что-либо еще. Пожалуйста, не изменяйте правила отступа или автоматической скобки».

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

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