如何正确缩短 C++ 中的 if 语句？

Question

好的，我有一个 if 语句，其中多次比较特定值。我敢打赌有一种更短的方法可以做到这一点：

if(letter == "%" || letter == "+" || letter == "-" || letter == "!")

我尝试将其写为：

if(letter == "%" || "+" || "-" || "!")

但它工作不正确。

Answer 1

像这样的事情可能会起作用：

string s("%+-!");
if (s.find(letter) != string::npos) { ... } 

Answer 2

这是尽可能短的。不过你可以重写它，在 Boost 的帮助下，它可能是

if( boost::is_any_of( "%+-!" )( letter ) )

Answer 3

尝试使用 switch 语句代替 if。这并没有减少打字量，但它确实允许您只写一次“字母”。一些程序员喜欢 switch 语句的视觉布局。请注意，switch 仅适用于 char 等标量类型（不适用于 std::string）。

switch (letter)
{
  case '%':
  case '+':
  case '-':
  case '!':
     // if letter is %,+,-,!
     // code goes here for matching letter 
     break;
  default:
     // else do something different
     // code goes here for letter that doesn't match
}

请参阅页面底部 http://www.cplusplus.com/doc/tutorial/control/< /a> 类似的例子。

Answer 4

问题在于 || 运算符的任何一方彼此都不了解。它们是两个完全独立的表达方式。因此 "+" 等总是评估为 true （想想如果你只写 if ("+") 会发生什么 - 它衰减为一个非指针NULL）

有时我发现使用 std::find 和一对迭代器来表达此类问题更清晰：

#include <iostream>
#include <algorithm>

void test(const char letter) {
  static const char c[] = {'%', '+', '-', '!'};
  static const char *begin=c, *end=&c[sizeof(c)/sizeof(*c)];
  if (std::find(begin, end, letter) != end) {
    std::cout << "Matched " << letter << std::endl;
  }
}

int main() {
  test('a');
  test('%');
  test('!');
}

Answer 5

我想最终的结果是尽可能少地输入内容，并尝试使用熟悉的语言（例如英语）来快速理解代码。

你的代码读起来像“我的信等于这个和这个和这个和这个”。可能有点啰嗦，但是很快就能理解。

不幸的是，C++ 编译器并没有真正做到“我的字母等于这些”。这对我们大多数人来说可能是英语，但对糟糕的编译器来说却不是:)

作为一名程序员，你有能力隐藏编译器素养的缺陷，保持你的代码易于理解并减少输入。

 if( letterIsOneOfThese( letter, "%+-!") )
 {
    // do something

 }


bool letterIsOneOfThese( const char letter, const char* letterList ) 
{
    int len = strlen( letterList );
    while( len-- )
    {
        if( letterList[len] == letter )
        {
            return true;
        }
    }

    return false;

}

Answer 6

char op[4] = { '%', '+', '-', '!' };
char * opEnd(op + 4);
if (opEnd != find(op, opEnd, letter))

或者：

if (string("%+-!").find(letter) != string::npos)