为什么代码分析警告“使用逻辑 &&”当按位 &可能是故意的”被抚养？

Question

代码：

BOOL CCreateReportDlg::CanSwapBrothers()
{
    BOOL            b1in2 = FALSE, b2in1 = FALSE;
    CStringArray    aryStrNames;

    // Must have valid data
    if (!IsSwapBrotherInit())
        return FALSE;

    // Get cell pointers
    auto pCell1 = GetSwapBrotherCell(1);
    auto pCell2 = GetSwapBrotherCell(2);
    if (pCell1 != nullptr && pCell2 != nullptr)
    {
        // Look for brother (cell 1) in cell 2 array
        auto strName = pCell1->GetText();
        pCell2->GetOptions(aryStrNames);
        const auto iNumNames = aryStrNames.GetSize();
        for (auto iName = 0; iName < iNumNames; iName++)
        {
            if (aryStrNames[iName] == strName)
            {
                b1in2 = TRUE;
                break;
            }
        }

        if (b1in2)
        {
            // Look for brother (cell 2) in cell 1 array
            auto strName = pCell2->GetText();
            pCell1->GetOptions(aryStrNames);
            const auto iNumNames = aryStrNames.GetSize();
            for (auto iName = 0; iName < iNumNames; iName++)
            {
                if (aryStrNames[iName] == strName)
                {
                    b2in1 = TRUE;
                    break;
                }
            }
        }
    }

    return b1in2 && b2in1;
}

感兴趣的行是 return 语句：

return b1in2 && b2in1;

我收到代码分析警告：

lnt 逻辑按位不匹配 当可能需要按位 & 时，使用逻辑 &&。

就我而言，我的代码是正确的。为什么会提出这个问题？

Answer 1

编译器认为 && 适用于整数操作数以及将结果隐式转换为整数类型。 BOOL 有多个位；它与内置类型 bool 不同。

正如您链接到的页面中所述，“逻辑运算符与整数值一起使用”将导致此警告，并且此处肯定存在该情况。

“MFC”编码风格在很多方面违反了现代建议，使用非标准布尔类型只是较小的问题之一。 CStringArray 也是一种代码味道，现代 C++ 使用模板化容器并具有强大的算法来操作它们，您永远不应该自己编写搜索代码。