Имеет ли смысл анализировать логический результат присваивания переменной?

Question

Имеет ли смысл анализировать логический результат присваивания переменной?

0

Я читал код только сейчас, и я нашел очень странную строку кода, написанную другим программистом:

if ((socket_obj->client_fd = accept(socket_obj->server_fd, (struct sockaddr *)&socket_obj->client_address, &fromlen)) < 0)

Если я правильно понял, строка вызывает функцию socket.h accept которая возвращает целочисленное значение. По завершении функции это значение присваивается socket_obj->client_fd а затем операция присваивания вычисляется, если она меньше 0.

Но не всегда ли это было бы выше 0, поскольку логический результат присвоения значений всегда истинен? Я имею в виду, не будет ли следующая строка всегда возвращать true?

if (myVariable = 0)

Сначала я подумал, что это socket_obj->client_fd кодирования и изменил код, поэтому сначала происходит назначение, а затем значение внутри socket_obj->client_fd оценивается с 0, но затем я нашел другое место в коде, где происходит такая же ситуация, поэтому я могу Я предполагаю, что это изолированная ошибка.

Тогда я хотел бы знать, если моя интерпретация верна.

Momergil 22 окт. 2014, в 18:23

Источник

2

Нет. Вы должны прочитать о назначении оператора. Или попробуйте свой пример и посмотрите, что произойдет.
i_am_jorf 22 окт. 2014, в 16:21
1

Кстати, вы имели в виду if (myVariable == 0) вместо if (myVariable = 0) ? (myVariable = 0) всегда ложно.
Jabberwocky 22 окт. 2014, в 16:22
0

@MichaelWalz нет, я говорю об оценке оператора присваивания, а не о сравнении. Так что это именно так, как я написал. (и не будет ли он возвращать всегда true вместо false как вам грустно? По крайней мере, это то, что я помню, как однажды сказал компилятор)
Momergil 22 окт. 2014, в 16:23
3

@Momergil if (myVariable = 0) помещает 0 в myVariable и условный оператор, который следует после, никогда не будет выполнен.
Jabberwocky 22 окт. 2014, в 16:24

Показать ещё 2 комментария

Теги:

c++

c

boolean-operations

assign

2 ответа

5

В C и C++ встроенный оператор присваивания возвращает свой левый операнд после назначения. Таким образом, линия

if ((socket_obj->client_fd = accept(...)) < 0)

делает следующее:

Вызовы accept()
Присваивает свой результат socket_obj->client_fd.
Сравнивает socket_obj->client_fd с 0.

Anton Savin 22 окт. 2014, в 14:53

0

спасибо за ответ, Савин! Теперь я понимаю, где была моя ошибка.
Momergil 22 окт. 2014, в 16:34

Ещё вопросы

Нет. Вы должны прочитать о назначении оператора. Или попробуйте свой пример и посмотрите, что произойдет.
Кстати, вы имели в виду if (myVariable == 0) вместо if (myVariable = 0) ? (myVariable = 0) всегда ложно.
@MichaelWalz нет, я говорю об оценке оператора присваивания, а не о сравнении. Так что это именно так, как я написал. (и не будет ли он возвращать всегда true вместо false как вам грустно? По крайней мере, это то, что я помню, как однажды сказал компилятор)
@Momergil if (myVariable = 0) помещает 0 в myVariable и условный оператор, который следует после, никогда не будет выполнен.
спасибо за ответ, Савин! Теперь я понимаю, где была моя ошибка.

Svalorzen · Accepted Answer · 2014-10-22T14-07-00.000Z

Назначение не возвращает true. Попробуй сам. Назначение возвращает значение, которое было назначено.

int main() {
    int a;
    std::cout << ( (a = 0) ? "true" : "false" ) << '\n';
    return 0;
}

Обратите внимание, что результирующее значение не должно быть true или false. Например, что вы можете сделать:

int x, y;
x = y = 4; // both x and y are 4 now

Это эквивалентно:

x = ( y = 4 );

И так как y присваивание возвращает значение, присвоенное y, которое равно 4, x также получает его.

@ haccks Что? Вопрос в том I mean, won't the following line always return true? , Прочти снова.
Тернарный оператор не обязан объяснять это, я думаю, что пример без него будет легче понять.
@Svalorzen Запустив твой код, я заметил правду. Спасибо за ответ! (теперь я задаюсь вопросом, почему я не проверил этот простой код самостоятельно, прежде чем прийти в SO ...)