这段c++代码有什么问题?

whats wrong with this c++ piece of code?

本文关键字：问题 什么 c++ 代码 这段      更新时间：2023-10-16

在这段代码中，在while循环的每次迭代中，从文件中读取一行。该行类似于:

13, 4636137, 29464742, 29464746, 995560164

对于以粗体指定的每个数字，在for循环中检查该数字是否作为std::map中的键。如果键存在，则将值附加到字符串。然后将字符串写入文件。

如果映射中不存在其中一个键，则不应将该行写入文件。(bool is_points_in_range)

但在实践中，如果最后一个点(在for循环之外)不在键列表中，程序逻辑执行得很好。

为什么在for循环中布尔运算符不改变?

vector<string> res;
ifstream infile;
map<string, string> m;
while(infile.getline(buffer, LINE_BUFFER_LEN))
{
    line=string(buffer);
    res = SplitBySep(line, ",");
    bool is_points_in_range=true;
    string geo_file_line;
    for (int i=2;i<res.size()-1;i++){
        if ( m.find(res[i]) == m.end() ) {
            is_points_in_range=false;
            break;
        } else {
            geo_file_line= geo_file_line.append(m[res[i]]).append("^");
        }   
    }
    if ( m.find(res[res.size()-1]) == m.end() ) {
        is_points_in_range=false;    
    } else {
        geo_file_line= geo_file_line.append(m[res[res.size()-1]]).append("n");
    }
    if (is_points_in_range){
        fprintf(fp_geo, "%s",geo_file_line.c_str());
    }
}

res.size()-1 极其危险。res.size()是无符号类型，当res.size()为0时，从它减去1将得到一个非常大的无符号整数，因为环绕。

因此，当res未填充时，您的程序本质上是未定义的。

我很惊讶你的编译器没有警告你。你把警告关掉了吗?

看起来m.find(res[i]) == m.end()从来都不是真的。你确定这张桌子已经坐满了吗?res[i]

最好问一下为什么这个条件语句不能很好地工作?我们知道，要获取std::map中存在的键，可以使用:

if ( m.find("f") == m.end() ) {
    // not found
} else {
    // found
}
如问题所示

但

map<string, string> m;

这个语句不能很好地工作。我没有任何29464742键(我确定)，但是当我搜索这个键时，else部分被执行。同样，当我使用m.count('29464742')时，返回1。我不知道为什么?

我用这个条件语句纠正了我的代码:

if ( m["f"] == "" ) {
    // not found
} else {
    // found
}
相关文章：