一个简单的问题，这样写是不是不太好： while (ERROR1 == (ret = do_sth() ) ) {...}

Question

问一个简单的问题，这样写是不是不太好： while (ERROR1 == (ret = do_something() ) ) {...}

比如这样：
2084     while (READ_ERR == (ret = tcp_read (g_client_socket, buf, 2, 5000)) )
2085     {
2086         usleep (500000);
2087         if (i++ == 5)
2088             break;
2089     }

Answer 1

很不好。

可以这样：

while(1){
    ret = tcp_read( ...... );
    if ( ret == READ_ERR ){
        break;
    }

    ........
}

Answer 2

是的，很想知道理由

Answer 3

问一个简单的问题，这样写是不是不太好： while (ERROR1 == (ret = do_something() ) ) {...}

比如这样： ...
reer 发表于 2011-04-15 19:30

很好的习惯，不会错的。

Answer 4

to yulihua49：
你是说这样写好吗？

Answer 5

这么些纯属装13

Answer 6

容易出错，而且代码难看！

Answer 7

理由就是读起来一句是一句，不喘气儿，雍容，从容，正确得显而易见。
而不是让你看着一愣一愣的，像捉迷藏、脑筋急转弯似的，最后突然又来一个“哦！原来如此！”

Answer 8

我自己是不会怎么写，不过也经常有人这么写，看呀看呀也习惯了