C++程序在文件读取循环中调用splitString后意外退出，求排查

核心问题分析 #

你的程序在调用splitString后直接退出，大概率是两个原因导致：

• 文件打开失败：inFile.open(filename[0])未做错误校验，如果文件路径错误、文件不存在或无读取权限，getline会直接返回失败，循环根本不会执行，看起来像程序直接退出。
• vector越界访问：如果读取的line中没有逗号（分割符），或者line为空，splitString返回的splitThis可能为空或只有单个元素，此时直接访问splitThis[0]会触发未定义行为，导致程序崩溃退出。

分步修复方案 #

1. 增加文件打开错误检查 #

在readCityLocation函数中，打开文件后立即校验是否成功：

void readCityLocation(int ** cityMap){
    ifstream inFile;
    inFile.open(filename[0]);
    // 新增：检查文件是否成功打开
    if (!inFile.is_open()) {
        cerr << "无法打开目标文件: " << filename[0] << endl;
        return;
    }
    string line;
    while (getline(inFile, line)){
        // 原有代码逻辑
    }
    // 记得关闭文件
    inFile.close();
}

2. 避免vector越界访问 #

在访问splitThis的元素前，先检查容器大小，防止越界：

while (getline(inFile, line)){
    int xCoordinate;
    int yCoordinate;

    string xValue;
    string yValue;
    
    vector <string> splitThis = splitString(line,",");
    
    // 检查分割结果是否有效
    if (splitThis.empty()) {
        cerr << "无效行数据（为空）: " << line << endl;
        continue; // 跳过当前行，继续处理下一行
    }
    xValue = splitThis[0];
    
    // 如果需要读取y坐标，还要检查是否有第二个元素
    if (splitThis.size() >= 2) {
        yValue = splitThis[1];
        // 后续坐标转换逻辑...
    } else {
        cerr << "行数据缺少Y坐标: " << line << endl;
        continue;
    }

    cout << line << endl;
}

3. 优化splitString函数（可选） #

当前splitString逻辑没问题，但可以提前处理空输入，避免不必要的计算：

vector <string> splitString (string str, string delim){
    vector <string> result; 
    // 提前处理空字符串
    if (str.empty()) {
        return result;
    }
    size_t pos = 0; 
    string token;

    while ((pos = str.find(delim)) != std::string::npos){
        token = str.substr(0, pos);
        result.push_back(token);
        str.erase(0, pos+delim.length());
    }

    if (!str.empty())
        result.push_back(str);

    return result;
}

4. 校验全局变量filename的合法性 #

确保filename[0]是正确的文件路径——可以用绝对路径（比如"C:/data/cities.txt"），或者相对于程序运行目录的相对路径，避免因路径错误导致文件无法打开。

调试小技巧 #

• 在readCityLocation的循环开头添加输出，确认是否真的读取到了文件内容：

  while (getline(inFile, line)){
      cout << "当前读取行: " << line << endl;
      // 后续代码...
  }
  

• 如果循环完全没执行，重点检查filename[0]的路径是否正确；如果执行后崩溃，说明是splitThis越界，检查对应行的内容是否符合预期（是否有逗号、是否为空）。

内容的提问来源于stack exchange，提问作者Izzat Bin Ismail