Home Sign Up Sign In
爱意满满的作品展示区。
Advertisement
asdftu

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

  •   
  •   asdftu · Dec 30, 2013 · 3980 views
    This topic created in 4525 days ago, the information mentioned may be changed or developed.
    附上：

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

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

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

    }
    是他们的省一行代码的简写，不是错误。但，如果不是第三方库，自己人写的这样的代码，八成是错的。
  • 代码
  • 正则表达式
  • 错误
    6 replies  •  1970-01-01 08:00:00 +08:00
    skydiver
        1
    skydiver  
       Dec 30, 2013
    对这种的，编译器本来就有warning吧。。
    shiny
        2
    shiny  
    PRO
       Dec 30, 2013
    我就经常这么写：
    if($result = fetch()){
    }

    但是有人批评不够直观。
    asdftu
        3
    asdftu  
    OP
       Dec 30, 2013
    @shiny 确实是不太直观...不过习惯就好
    zhujinliang
        4
    zhujinliang  
       Dec 30, 2013 via iPad
    中枪，我也经常这样写，if可能意义不大，但是while会方便不少
    Mutoo
        5
    Mutoo  
       Dec 30, 2013
    实际上这还是一种引入后门的方式，要注意了。

    http://www.infoq.com/cn/news/2013/12/Linux-Backdoor
    loseblue
        6
    loseblue  
       Dec 31, 2013
    一般这种东西良好的写法是
    if(1 == a)
    如果不小心写成=，会由于给常数赋值而报错。
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Solana   ·   4103 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 59ms · UTC 04:14 · PVG 12:14 · LAX 21:14 · JFK 00:14
    ♥ Do have faith in what you're doing.