Использование операторов сравнения за пределами условными

Question

Например

int f(int a) { 
    ... 
    return a > 10; 
} 

является то, что считается приемлемым (не юридическое, я имею в виду, это `` хороший код «»), или же она должна всегда быть в условном, как это

int f(int a) { 
    ... 
    if (a > 10) 
     return 1; 
    else 
     return 0; 
} 

Answer 1

Это абсолютно приемлемо! Фактически, Джоэл упомянул об этом в последнем подкасте stackoverflow. Он сказал, что это единственное, что ему пришлось показать почти каждому программисту, который начинается в Fog Creek.

Answer 2

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

Answer 3

Первый предпочтительнее для меня, поскольку он более краток. (И он избегает множественных возвратов :)

Answer 4

Я обычно делаю первое над последним.

Answer 5

Я думаю, что это вполне приемлемо, если вы обеспечите, чтобы вы приложили дополнительные усилия для обеспечения удобочитаемости. Например, я бы удостоверился, что имя метода очень однозначно, и вы используете хорошие имена переменных.

Вторая альтернатива, которую вы предоставили, я думаю, это почти хуже, потому что она включает в себя инструкцию ветвления и несколько операторов возврата, и эти вещи увеличивают сложность метода, а сами уменьшают его читаемость.

Answer 6

Было бы приемлемо - если ваш тип возврата был bool.

Answer 7

Я бы предпочел написать bool f(int);, а первая форма как bool - это тип boolean в C++. Если мне действительно нужно вернуть int, я бы написать что-то вроде

int f(int) { 
    ... 
    const int res = (i>42) ? 1 : 0; 
    return res; 
} 

Я никогда не понимал, почему люди пишут

if (expr == true) 
    mybool = true ; 
else 
    mybool = false; 

вместо простого

mybool = expr; 

булевой алгебры является инструментом, который любой разработчик должен иметь возможность инстинктивно обрабатывать

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

Answer 8

Первый случай отлично, намного лучше второго, ИМХО. По сути читаемости, лично я бы

return (a > 10); 

но это незначительные нит, а не один все будут согласны.

Answer 9

return a > 10 ? 1 : 0; 

... имеет больше смысла, потому что вы возвращаете int, а не bool.

Answer 10

Этот синтаксис не только допустим на 100%, вы также можете свободно использовать булевы выражения вне операторов if, то есть int x = i && (j || k); (или возвращать такие значения).

Answer 11

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

Я думаю, что это вполне приемлемо, если язык позволяет это и использование является частью парадигмы этого языка

Answer 12

Я просто попытался три различных варианта с GCC:

int one(int x) { return (x > 42) ? 1 : 0; } 
int two(int x) { return x > 42; } 
int thr(int x) { if (x > 42) return 1; else return 0; } 

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