C++中如何正确编写多组或与与条件组合的if判断语句

嘿，我一眼就看出你代码里的问题啦！你现在写的if语句逻辑和你想要的完全不符，核心原因是C++中逻辑与&&的优先级比逻辑或||高，导致你的条件被编译器错误解析了。

你的代码为什么不对？ #

你原本想表达的是：x属于{a,b,c} 且 y属于{d,e,f} 且 z属于{g,h,i}，但你的代码会被编译器解析成这样的逻辑：

((x==a || x==b || (x==c && y==d)) || y==e || (y==f && z==g)) || z==h || z==i

这显然和你的需求差远了——只要满足其中某一小段条件（比如z==h），整个if就会触发，完全不是三个集合条件同时满足的逻辑。

正确的写法 #

方法1：添加括号明确优先级

最直接的修复方式就是给每个集合判断加上括号，强制让||的优先级先于&&：

if( (x == a || x == b || x == c) && (y == d || y == e || y == f) && (z == g || z == h || z == i) ) {
    // 执行你的代码逻辑
}

括号会告诉编译器：先判断x是否在{a,b,c}里，再判断y是否在{d,e,f}里，最后判断z是否在{g,h,i}里，只有三者都为真时，整个条件才成立。

方法2：用辅助函数让代码更整洁

如果你的判断集合经常变化，或者想让代码更易读，可以写一个通用的辅助函数来判断元素是否在集合中：

#include <algorithm>
#include <initializer_list>

template<typename T>
bool is_in(const T& value, const std::initializer_list<T>& candidates) {
    return std::find(candidates.begin(), candidates.end(), value) != candidates.end();
}

// 调用方式
if( is_in(x, {a, b, c}) && is_in(y, {d, e, f}) && is_in(z, {g, h, i}) ) {
    // 执行代码
}

方法3：C++20及以上版本的简洁写法

如果你使用的是C++20或更高版本，可以直接用标准库的std::ranges::contains函数，代码会更简洁：

#include <ranges>

if( std::ranges::contains({a, b, c}, x) && std::ranges::contains({d, e, f}, y) && std::ranges::contains({g, h, i}, z) ) {
    // 执行代码
}

这样改完之后，逻辑就完全符合你的需求啦——只有当x、y、z分别属于各自的目标集合时，才会执行if块里的代码。

内容的提问来源于stack exchange，提问作者Daniel Rika