isspace函数的性能警告，从int到bool的转换

Question

#include <iostream>
#include <algorithm>
#include <vector>
#include <string>
#include <iterator>

using namespace std;

bool notSpace(char c) {
    return !isspace(c);
}
bool isSpace(char c) {
    return isspace(c);
}
vector<string> split(const string& s) {
    vector<string> words;
    string::const_iterator i = s.begin();
    while (i != s.end()) {
        i = find_if(i, s.end(), notSpace); // "              "
        if (i != s.end()) {
            string::const_iterator j = i;
            j = find_if(i, s.end(), isSpace);
            words.push_back(string(i, j));
            i = j;
        }
    }
    return words;
}
int main() {
    string test = "Hello world, I'm a simple guy";
    vector<string> words = split(test);
    for (vector<string>::size_type i = 0; i < words.size();i++) {
        cout << words[i] << endl;
    }
    return 0;
}

当我编译代码时，我会收到以下警告：

警告C4800：'int'：强制值为'true‘或'false’(性能警告)

在此函数返回时：

bool isSpace(char c) {
    return isspace(c);
}

好习惯会把isspace(c)变成(isspace(c) != 0)吗？还是说这只是不必要的麻烦？

Answer 1

看看下面的代码：

#include <iostream>
using namespace std;
bool f()
{
    return 2;
}
int main()
{
  cout <<f()<<endl;
  return 0;
}

当你返回2的时候，它会打印1，这就是为什么你会得到警告。有人可能会认为bool是个小整数，但事实并非如此。

如果回到C，不存在bool类型，这就是为什么许多C方法(如isspace)返回int，甚至WINDOWS类型的BOOL实际上也是整数，可以返回除TRUE (1)或FALSE (0)以外的其他值。