Qt失误之判断两个QString字符串是否相等,结果永远是真

⌚Time: 2022-03-08 22:32:13

👨‍💻Author: Jack Ge

最近写Qt程序运行出现bug,查找原因,发现出在了字符串比较的代码上面。

代码类似于以下结构:


    if(!QString::compare(QString("abb"),QString("abc")),Qt::CaseSensitive){

        qDebug()<<"字符串相同分支";

        ...

    }else{

        qDebug()<<"字符串不同分支";

        ...

    }


运行后进入“字符串相同分支”,并运行了其中代码。

如果不仔细看,很难发现其中错误。错误原因就是将Qt::CaseSensitive参数写在QString::compare方法外面,成为了if括号中的语句,造成if括号里面就成为了一个(xxx,xxx)形式的逗号表达式,而逗号表达式的值,永远是最后的一个值。而Qt::CaseSensitive的值恰好是真!

这就造成了if语句的值永远是真,所以会永远进入真的分支。

在编译时,由于QString::compare方法第三个参数时具有默认值的,因此可以只传递前两个参数,所以编译也不会报错

一系列的巧合,就造成了这个永远进入真分支的错误!

改正的方法也很简单,只需要把Qt::CaseSensitive参数正确的移如compare方法内就行了


    if(!QString::compare(QString("abb"),QString("abc"),Qt::CaseSensitive)){

        qDebug()<<"字符串相同分支";

        ...

    }else{

        qDebug()<<"字符串不同分支";

        ...

    }


!!!!!!!!!!!!