Использование операторов сравнения вне условных выражений

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

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

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

Обычно я делаю первое, а не второе.

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

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

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

Я лучше напишу 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;

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

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

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

   return  (a > 10);

но это мелочь, и не все согласятся с ней.

return a > 10 ? 1 : 0;

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

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

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

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

Я просто попробовал три разных варианта с 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; }

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