函数调用和函数完成后的C++，游戏完全崩溃

Question

我在学校的 C++ 游戏编程课上制作的游戏遇到了问题。出于某种原因，在调用我用来管理基于库存的东西的函数之后，该函数似乎完成并工作（我认为这是因为我在它的末尾输入了 cout 命令并且它们正确打印，也是函数连续运行两次，它们都运行），我的整个游戏崩溃并且没有到达下一行。我尝试注释掉函数中的所有代码，但它仍然崩溃。我注释掉了函数调用并且它起作用了，但我仍然不知道它有什么问题。我将把函数的代码和我调用的部分放在一起：

string inventoryFunction(int h, string ab)
{
    if(h == 1)
        inventory.push_back(ab);
    else
        if(h == 2)
        {
            for(int i=0; i < inventory.size(); i++)
            {
                if(inventory[i] == ab)
                    inventory[i].erase();
            }
        }
        else
            if(h == 3)
            {
                cout << inventory[0];
                for(int i=1; i < inventory.size(); i++)
                    cout << ",     " << inventory[i];
            }
}

函数调用：

if(answer.find("village") != string::npos)
    {
        cout << endl;
        cout << "While looking around your village,\nyou found a stone sword and a cracked wooden shield!" << endl;
        inventoryFunction(1, "stone sword");
        inventoryFunction(1, "cracked wooden shield");
        cout << "Would you like to set off on your adventure now?" << endl;
        cin >> answer2;
        capitalizeLower(answer2);

Answer 1

不确定是否存在任何可能导致崩溃的情况，我的建议是在调试器中单步执行您的代码以查看其崩溃的位置。这个错误很可能是在某个完全不同的地方，它只是被修改向量的函数调用加剧了。

不幸的是，这就是错误的本质，如果不仔细观察，您永远无法真正知道它们实际上来自哪里:-)

但是，我想指出一些代码问题。

---

首先，关于：

inventory[i].erase();

这并不像你认为的那样。 inventory[i] 是向量中的 字符串，因此它只是擦除字符串内容。

如果你想从向量中删除字符串，你需要这样的东西：

inventory.erase (inventory.begin() + i);

---

其次，我倾向于为addToInventory、removeFromInventory 和listInventory 设置三个独立函数。

这似乎有点...不直观...必须记住 h 的神奇值才能实现您想要做的事情，除了访问 @987654328 之外，这三个用例没有真正的共同点@vector（这还不足以将它们组合成同一个成员函数）。

---

最重要的是，您的函数似乎返回了 string，但您没有实际的 return 语句，事实上，您的函数的三个用例都不需要传回任何内容。

签名最好如下：

void inventoryFunction(int h, string ab)

---

就上述第二点和第三点而言，我可能会从以下内容开始：

void addToInventory (string item) {
    inventory.push_back(ab);
}

void removeFromInventory (string item) {
    for (int i = 0; i < inventory.size(); i++) {
        if (inventory[i] == ab) {
            inventory.erase (inventory.begin() + i);
            break;
    }
}

void listInventory () {
    cout << inventory[0];
    for (int i = 1; i < inventory.size(); i++)
        cout << ",     " << inventory[i];
}

您可能还想考虑将迭代器专门用于第二个和第三个函数，而不是手动使用i 遍历集合。

它会为您节省一些代码，并且更“C++ic”，是“Pythonic”概念的 C++ 版本，我希望它会流行起来并让我出名:-)

Answer 2

因此，通过将inventoryFunction 更改为@Retired Ninja 所说的void 函数，崩溃已经停止发生，现在程序运行良好。 另外，@paxdiablo 指出我错误地使用了 inventory[i].erase() 东西，所以非常感谢他，因为现在我以后不必再回到这里来尝试解决这个问题：D

Answer 3

string inventoryFunction(int h, string ab)

应该返回一个字符串，但没有任何返回语句。当然它可以工作，在你将它更改为一个 void 函数后，它正确地不会返回任何东西。有趣的是，您可以毫无错误地共同编译此代码 - 通常编译器会向您显示此问题。