Поместите проверку условия и присвоение переменной в один оператор if

Question

Я смотрю на какой-то устаревший код C и запутался, это что-то вроде:

UINT A, B = 1;
if((A = B) == 1){
   return(TRUE);
} else {
   return(FALSE);
}

Мы все знаем, что будет предупреждение компилятора, если мы сделаем if (A = B), но здесьПохоже, «если» проверяет А против 1, я прав?

Answer 1

Сначала присваивается значение B A (A = B), затем проверяется, равен ли результат этого присваивания, равный A и оценивается как 1, 1.

Итак, технически вы правы: на пути проверки A против 1.

Чтобы упростить чтение, код эквивалентен:

UINT A, B = 1;
A = B;
if(A == 1){
   return(TRUE);
} else {
   return(FALSE);
}

Answer 2

Скорее, ваш код всегда присваивает B A, и он также проверяет, равно ли значение B (и, следовательно, A) 1.

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

int result;
if ((result = foo()) != -1)
{
  printf("The result is: %i\n", result);
}
else
{
   // panic
}

Answer 3

Если вы хотите сохранить его в 1 строке:

if ((A = B), A == 1)

делает то же самое.

Answer 4

Мы стараемся избегать, если операторы делают код более читабельным.

UINT A, B = 1;
bool fResult = false;

fResult = (A == B);

return(fResult);

И если должно быть условие для действия (не) равенства, см. Этот пример.

UINT A, B = 1;
bool fResult = false;

fResult = (A == B);

if(fResult)
{
  doThis();
}
else
{
  doThat();
}

return(fResult);

Answer 5

Правильно. Значение A после присвоения будет сравниваться с 1.

Этот пример кода эквивалентен просто:

return (TRUE);