如何调试运行时检查失败#2：变量'newString'周围栈损坏问题

首先，咱们直接揪出触发这个错误的核心原因：你定义的char newString[] = "";是一个长度仅为1的字符数组（只包含终止符'\0'）。当你在后续循环里往newString[j]赋值时，只要j大于0，就会越界写入栈内存，直接破坏栈结构，触发这个运行时检查错误。

除此之外，代码里还有另一个严重的内存安全隐患：你直接修改了传入的inString（也就是s）的末尾，添加空格和终止符。这里的问题在于：

• 如果传入的是字符串字面量（比如convertStrgToStrVect("hello world")），它存储在只读内存中，修改会直接导致未定义行为（崩溃或奇怪的错误）。
• 即使传入的是堆/栈分配的字符数组，你也没确认它有足够空间容纳额外字符，同样会引发越界访问。

调试与修复步骤 #

1. 替换不安全的char数组为std::string

手动管理char数组极易出现内存越界，直接用std::string存储临时字符串，它会自动处理内存扩容：

// 替换char newString[] = "";为：
std::string newString;

然后可以直接用std::string的构造函数截取子串，代替繁琐的循环赋值：

// 代替原循环里的字符赋值逻辑：
newString = std::string(s, *itr1, *itr2 - *itr1);
stringVect.push_back(newString);

2. 不要修改输入字符串，用副本操作

创建输入字符串的可修改副本，避免破坏原输入的内存空间：

// 替换char *inStr = inString;为：
std::string inputStr(inString);
inputStr += ' '; // std::string会自动扩容，不用手动处理终止符
const char* s = inputStr.c_str();

3. 简化代码逻辑（可选但推荐）

你用三个vector存起始/结束索引和长度的逻辑太繁琐，其实可以直接遍历字符串分割单词，代码更简洁且不易出错：

std::vector<std::string> convertStrgToStrVect(char *inString) {
    std::vector<std::string> stringVect;
    std::string inputStr(inString);
    std::string currentWord;

    for (char c : inputStr) {
        if (c != ' ') {
            currentWord += c;
        } else {
            if (!currentWord.empty()) {
                stringVect.push_back(currentWord);
                currentWord.clear();
            }
        }
    }
    // 处理原字符串末尾无空格的情况
    if (!currentWord.empty()) {
        stringVect.push_back(currentWord);
    }

    // 打印输出（可选）
    for (const auto& str : stringVect) {
        std::cout << "stringVect: " << str << std::endl;
    }

    return stringVect;
}

实用调试技巧 #

• 断点调试：在写入newString的循环处设置断点，查看j的值和newString的内存大小，能直观看到越界情况。
• 开启编译器警告：Visual Studio开启/W4等级，GCC/Clang用-Wall -Wextra，编译器会主动提示数组长度不足、可能的越界等问题。
• 内存检测工具：比如Visual Studio的AddressSanitizer，开启后能精准定位内存越界的位置，快速排查问题。

内容的提问来源于stack exchange，提问作者Manasa