C++给结构体string成员赋值导致程序崩溃的问题排查

核心崩溃原因 #

你遇到的崩溃本质是数组越界访问，和string成员本身没有直接关系：

• 你通过new dynInputs[labelsNeeded]申请的动态数组，合法下标范围是0到labelsNeeded - 1，C/C++的数组下标从0开始计数。
• 你的循环第一次访问的是dynamicInputs[labelsNeeded]，这个位置完全超出了你申请的内存块范围，属于未分配的野内存。

为什么只有给string赋值才会崩溃？ #

• 当你越界写入perCost、unitTotal这类int类型的POD成员时，只是非法修改了数组后方的内存数据，不会立刻触发操作系统的内存保护机制，所以看起来"没报错"。
• 当你给越界位置的name成员（string类型）赋值时，本质是在一个根本没有被合法构造的string对象上调用赋值操作，这个位置存储的是随机垃圾值，被当成string内部的堆内存指针去访问、写入，会立刻触发非法内存访问，直接导致程序崩溃。
• 你之前尝试把字符串先读到临时变量再赋值给结构体成员的方案无效，原因也是一样的：赋值的目标地址本身就是越界的野地址，和赋值的源数据没有关系。

代码里的其他隐藏bug #

除了越界问题，你的代码还有几个会导致异常或逻辑错误的问题：

• if (convertedResponse = 1) 是赋值操作不是相等比较，这个判断永远为真，且类成员convertedResponse没有在构造函数里初始化，初始值是随机垃圾值。
• 结构体dynInputs里的totalCost在默认构造时就会计算，此时perCost和unitTotal都是未初始化的垃圾值，后续你修改这两个成员的值时，totalCost不会自动重新计算，结果完全错误。
• 输入流处理逻辑有缺陷：cin >> labelCount读取数字后，输入缓冲区里残留的换行符没有被正确清理，容易导致后续getline读到空行；多余的cin.clear()没有实际作用，clear()只重置流状态，不会清空缓冲区内容。
• 你通过new申请的动态数组没有配套delete[]释放，会造成内存泄漏。

修复后的核心代码片段 #

// 需额外包含头文件 <limits> 用于处理输入缓冲区
struct dynInputs
{
    string name;
    int perCost;
    int unitTotal;
    // 不在构造时计算totalCost，等输入完perCost和unitTotal后手动计算
    int totalCost;
};

// 记得在类构造函数中给convertedResponse赋合理初始值，比如 convertedResponse = 1;
void populateSaveData()
{
    // 修正比较判断
    if (convertedResponse == 1)
    {
        cout << "How many labels would you like to create?" << endl;
        int labelCount;
        cin >> labelCount;
        // 读取数字后立刻清空缓冲区里的剩余换行符
        cin.ignore(numeric_limits<streamsize>::max(), '\n');
        int labelsNeeded = labelCount;
        dynInputs* dynamicInputs = new dynInputs[labelsNeeded];
        // 从合法的最大下标开始遍历，彻底避免越界
        int currentIdx = labelsNeeded - 1;
        while (currentIdx >= 0)
        {
            cout << "please type the name for this row" << endl;
            getline(cin, dynamicInputs[currentIdx].name);
            cout << dynamicInputs[currentIdx].name << endl;
            cout << "please type the cost of the unit, and the number of units" << endl;
            cin >> dynamicInputs[currentIdx].perCost;
            cin >> dynamicInputs[currentIdx].unitTotal;
            // 输入完数值后手动计算totalCost
            dynamicInputs[currentIdx].totalCost = dynamicInputs[currentIdx].perCost * dynamicInputs[currentIdx].unitTotal;
            // 读取完数字后再次清空缓冲区剩余换行符，给下一次getline做准备
            cin.ignore(numeric_limits<streamsize>::max(), '\n');
            currentIdx--;
        }
        cout << dynamicInputs[0].unitTotal << endl;

        // 用完动态数组必须释放
        delete[] dynamicInputs;
        dynamicInputs = nullptr;
    }
}

如果不想手动管理内存，建议直接用vector<dynInputs>代替手动new的动态数组，自动处理内存释放，能从根源上避免大部分内存越界、泄漏问题。

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