помогите написать красивый код

return f1() && f2() && f3();

?

>f1, f2, f3 могу возвращать только PASS/FAIL

PASS/FAIL это какого типа? строки? константы? enum? и какой язык вообще? =)

если результаты булевы, то все тривиально:

return f1() && f2() && f3();

> если результаты булевы, то все тривиально:
> return f1() && f2() && f3();

тогда не обязательно вызовутся f2() и f3()

ret = 0;
ret += f1();
ret += f2();
ret += f3();
return !!ret;

res = true;
res &= f1();
res &= f2();
res &= f3();
return res;

язык С. хотелось бы решение, в котором нет никаких предположений о значении PASS/FAIL (тип int).

res = PASS;
res = ( f1() == FAIL ) ? FAIL : res;
res = ( f2() == FAIL ) ? FAIL : res;
res = ( f3() == FAIL ) ? FAIL : res;
return res;

это если надо, чтоб f1, f2, f3 обязательно выполнились

ну тогда уже:

enum { NOERR, ERR1, ERR2, ERR3 };

int errno = NOERR;

if (f1() != NOERR)
         errno = ERR1;

if (f2() != NOERR)
         errno = ERR2;

if (f3() != NOERR)
         errno = ERR3;

return errno;

а то потом не разберёшся, где лагануло

есть четкое ТЗ ;)

кстати можно короче записать

res = f1();
res = ( f2() == FAIL ) ? FAIL : res;
res = ( f3() == FAIL ) ? FAIL : res;
return res;

>тогда не обязательно вызовутся f2() и f3()

зависит от компилятора и степени оптимизации. Но грабли таки возможны, это да. "Счастливой отладки, суки" (с)

> зависит от компилятора и степени оптимизации.

Не зависит.

> тогда не обязательно вызовутся f2() и f3()

Только в том случае, если f1() будет false.

>> тогда не обязательно вызовутся f2() и f3()
> Только в том случае, если f1() будет false.

true

> true

согласен, если бы там было логическое или :-)

да ну, "если f1() будет false" - f2 и f3 не вызовутся, я это гарантирую ;)

> да ну, "если f1() будет false" - f2 и f3 не вызовутся, я это гарантирую ;)

Я это и утверждаю. :-)

>> тогда не обязательно вызовутся f2() и f3()
> Только в том случае, если f1() будет false.

или у меня парсер сломался, или вы утверждали обратное :)

> или у меня парсер сломался, или вы утверждали обратное :)

Ты сказал, что f2() и f3() не обязательно будут вызваны... а я и добавил, что они не будут вызваны в том случае, если f1() будет false. Или я что-то не так понял? :-)

> не обязательно вызовутся f2() и f3(), только в том случае, если f1() будет false.

так читается вместе, вообщем все понятно - хватит нам тут умничать :)

умничайте. главное, не подеритесь

> умничайте. главное, не подеритесь

Ну и к чему это было сказано? :-))))

> Ну и к чему это было сказано? :-))))

очевидно намек на детсад, что мы тут устроили

> очевидно намек на детсад, что мы тут устроили

Ну почему ж детсад. Нормально все, выяснили проблему. :-)

>>return f1() && f2() && f3();
>тогда не обязательно вызовутся f2() и f3()

Если результат - bool, и f не возвращают больших чисел, то можно f1() * f2() * f3().

Хотя, я бы так писать не стал...

А еще можно следать ф-цию conjunction(...) :)

res1 = f1();
res2 = f2();
res3 = f3();
...
return res1 && res2 && res3;


> хотелось бы решение, в котором нет никаких предположений о значении PASS/FAIL (тип int).

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

Юзать монаду maybe :-)

http://blog.sigfpe.com/2007/02/monads-in-c-pt-ii.html

А почему нельзя f1() & f2() & f3() ? И аналог с переменной: res = f1(); res &= f2(); ...

Как разновидность (если не нравится битовая арифметика): fail = !f1(); fail |= !f2();

> А почему нельзя f1() & f2() & f3() ?

man operator &

int rv = PASS;
res1 = f1();
if (res1 != PASS) {
  rv = FAIL;
}
res2 = f2();
if (res2 != PASS) {
  rv = FAIL;
}
res3 = f3();
if (res3 != PASS) {
  rv = FAIL;
}
return rv;

>> А почему нельзя f1() & f2() & f3() ?

> man operator &

К сожалению, такой скупой ответ приемлем только при чётко поставленной задаче. Здесь же из постановки задачи не следует даже неприемлемость оператора && (это уже потом поясняется в треде отдельно). В общем, вопрос остаётся в силе.

return (PASS==f1()==f2()==f3())?PASS:FAIL;

Вру. Ночью писал. :)
Вот так надо:

return ( (PASS==f1())==(PASS==f2())==(PASS==f3()) )? PASS: FAIL;

P.S. А если булевы:
return (true==f1()==f2()==f3())?true:false;

int conj (int ... values)
{
   for value in values
   {
       if (value==FAIL)
          return FAIL;
   }
   return PASS;
}


someFunc()
{
...
  int test_result = conj (test1(), test2(), test3(), test4());
...
}

>res = true;
>res &= f1();
побитовые операции над логической константой? мрак...

ЗЫ: нафиг вообще красивый код? заказчик на него будет пялиться? красивый не всегда просто отлаживать.
имхо, красивый код - тот, который проще отлаживать.

return ( PASS==f1() && PASS==f2() && PASS==f3() )? PASS: FAIL;

логика очевидна.