2013-11-19 3 views
1

Мне было интересно, какое соглашение лучше и понятнее читать?Какое соглашение лучше?

void someMethod() { 
    if (!condition) { 
     return; 
    } 

    // do some work 
} 

Или

void someMethod() { 
    if (condition) { 
     // do some work 
    } 
} 
+0

Я думаю, что сначала лучше – subash

+4

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

+0

В общем, я думаю, что второе лучше. – Plux

ответ

4

Зависит от имени метода. Вы не хотите делать:

if(!notEmpty())

Там нет предпочтительнее или лучший способ сделать это. Зависит от имени метода, логики.

+0

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

+0

@Teodor Зависит от многих вещей .. Для меня второй яснее .. Но это только я .. Нет предпочтительного способа .. ИМХО. – Maroun

+0

Хорошо, как я уже сказал, мне просто интересно, потому что я поймал себя на том, чтобы использовать обе эти конвенции. Спасибо за ваш ответ :) – Teodor

0

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

0

Лично я предпочитаю бывшие. Педанты скажут, что он неструктурирован, но я бы сказал, что речь идет о семантике.

Смежные вопросы