写了一个正则表达式查找代码库里边 if(a == b)写成if(a = b)的错误

[asdftu]

附上：

\bif\s?\([^=!<>\r\n]*=\s?[^=!<>\r\n]*\s?\)\s*\{

长时间不写c++代码，今天竟然不知怎么回事写出了if（a = b）的错误代码，于是写了一个正则表达式来查询整个代码库里面有没有其他人有这样的错误。

最后发现有一些第三方的库喜欢if（a=b）这样子用，
意思为
a=b;
if(a)
{

}
是他们的省一行代码的简写，不是错误。但，如果不是第三方库，自己人写的这样的代码，八成是错的。

[skydiver]

对这种的，编译器本来就有warning吧。。

[shiny]

我就经常这么写：
if($result = fetch()){
}

但是有人批评不够直观。

[asdftu]

@shiny 确实是不太直观...不过习惯就好

[zhujinliang]

中枪，我也经常这样写，if可能意义不大，但是while会方便不少

[Mutoo]

实际上这还是一种引入后门的方式，要注意了。

http://www.infoq.com/cn/news/2013/12/Linux-Backdoor

[loseblue]

一般这种东西良好的写法是
if(1 == a)
如果不小心写成=，会由于给常数赋值而报错。