Должен ли я соответствовать использованию фигурных скобок предыдущего автора? - PullRequest
3 голосов
/ 07 января 2011

При работе над кодом от нескольких авторов я часто сталкиваюсь с проблемой предпочтения фигурных скобок (та же строка против новой строки).Является ли это хорошей / плохой практикой или вообще не является проблемой, когда речь идет о сопоставлении существующего стиля с вашим собственным предпочтением?

Изменится ли ситуация, если вы добавляете новый код в Class против модификации существующего кода?Наконец, если стиль должен соответствовать, как далеко должно распространяться соответствие?т.е. файл, класс, подклассы и т. д.

Пример:

if(this)
{
    doThat();
}

Vs.

if(this){
    doThat();
}

Ответы [ 6 ]

6 голосов
/ 07 января 2011

Вообще говоря, непоследовательно отформатированный код будет труднее читать и понимать, чем код, отформатированный в стиле, отличном от вашего.

При работе над проектом с открытым исходным кодом считается вежливым, чтобы соответствовать используемому стилю. Для кода, которым вы не будете делиться, либо следуйте стилю, либо пропустите весь существующий код через переформататор.

3 голосов
/ 07 января 2011

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

2 голосов
/ 07 января 2011

Если вы будете делиться этим, соответствуйте его стилю для последовательности. Если нет, делайте то, что вам удобно. Я лично предпочитаю твой стиль.

0 голосов
/ 07 января 2011

Нет ничего, что раздражало бы меня больше, чем просмотр кода и нахождение множества несовпадающих стилей.Если вы собираетесь сотрудничать с кем-то в программе, то следует использовать как явные, так и неявные стандарты.Что-то простое, например, фигурная скобка, не имеет значения, но когда ваш проект проверяется руководством, и вы даже не можете подобрать эту небольшую вещь, она выглядит плохо.Или, если вы работаете над личным проектом с друзьями, а затем решаете присоединиться к другим, проект начнёт выглядеть непрофессионально.Я бы сказал, что при работе в такой группе речь идет не столько о личном мнении, сколько о стандартах группы.

0 голосов
/ 07 января 2011

Неважно (с точки зрения кода), куда вы помещаете фигурные скобки, это просто вопрос предпочтений.Более опытные программисты, как правило, помещают его в одну и ту же строку, но мне нравится помещать его в следующую строку, потому что это помогает мне легче видеть блоки кода.Если вы хотите, чтобы соглашения по кодированию доходили до фигурных скобок, проконсультируйтесь с вашей командой и выберите один из них, но они, вероятно, будут смеяться.

0 голосов
/ 07 января 2011

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

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

...