c ++代码审查

Question

if (ad && (etag = strrcasestr_len(ad,strlen(ad),
                "</XHTML-STRIPONREPLY>", 21))) {

这里ad和etag是const char *

当自定义函数strrcasestr_len返回NULL

时

表达式将评估为（ad＆amp;＆amp; null）

Answer 1

布尔上下文中

NULL为false，因此表达式将为false。 （即不会采取if分支。）

Answer 2

...将评估为false，是的。 另请注意，如果etag = strrcasestr_len(...)为ad，则根本不会评估NULL。

Answer 3

如果我是代码审查该代码，我会告诉作者重写它，以便明确行为是什么。由于存在混淆的可能性，因此应通过编码标准排除在条件表达式中使用赋值运算符。

行为将被很好地定义，但这不足以成为写作的理由。代码需要清楚。将其分为两个陈述可能更容易理解：

etag = strrcasestr_len(ad, strlen(ad), "</XHTML-STRIPONREPLY>", 21);

[不知道发生了什么......我不能在这里写if语句，因为如果我这样做，stackoverflow会拒绝编辑]