2

我在 C++ 中遇到了一个非常奇怪的集合迭代器问题。

set<string> dict;
dict.insert("hello");
dict.insert("my");

int maxLen = INT_MIN;

set<string>::iterator itr;
for (itr=dict.begin(); itr!=dict.end(); itr++) {
    int len = (*itr).length();
    if ( len > maxLen )
        maxLen = (*itr).length();
}

这段代码帮助我将 maxLen 设置为 5，这是单词集中最长单词的长度。

set<string> dict;
dict.insert("hello");
dict.insert("my");

int maxLen = INT_MIN;

set<string>::iterator itr;
for (itr=dict.begin(); itr!=dict.end(); itr++) {
    if ( (*itr).length() > maxLen )
        maxLen = (*itr).length();
}

但是，这段代码不能给我正确的结果。运行代码后，maxLen 仍然是 INT_MIN 的值。基本上没有什么变化，只是我不再使用变量来保存 (*itr).length() 的值。

这对我来说很奇怪。我错过了什么吗？我只是想澄清我对迭代器使用的怀疑。

非常感谢！

4

1 回答 1

6

我认为这里的问题是该string::length函数返回一个unsigned类型，而int您使用的类型是signed。在有符号值和无符号值之间进行比较时，始终首先将有符号值转换为无符号值。在您的情况下，转换为INT_MIN无符号值使其具有最大可能的无符号值，因为 和 的位模式INT_MIN是UINT_MAX相同的。

第一次这样做的原因是无符号值int在分配给临时变量时被强制转换为 s 。

要解决此问题，请重新添加演员表：

for (itr=dict.begin(); itr!=dict.end(); itr++) {
    if ( int((*itr).length()) > maxLen )
        maxLen = (*itr).length();
}

只要你在它，你可以在这里做很多其他的风格修复，比如

  • 将 postincrement 转换++为 preincrement++以提高效率，
  • 使用->而不是(*).，
  • 在运算符周围添加空格，以及
  • 在循环中本地声明迭代器

这显示在这里：

for (set<string>::iterator itr = dict.begin(); itr != dict.end(); ++itr) {
    if (int(itr->length()) > maxLen) {
       maxLen = itr->length();
    }
}

或者，如果您有一个符合 C++11 的编译器，则使用基于范围的 for 循环：

for (const auto& val: dict) {
    if (int(val.length()) > maxLen) {
       maxLen = val.length();
    }
}

希望这可以帮助！

于 2013-10-22T02:27:47.203 回答