为什么在这里出现矢量下标超出范围错误？

Question

我正在创建某种rpg战斗，程序从.txt文件读取输入。 我创建了代码，但是当我想开始战斗时，它给了我错误矢量下标超出范围的信息。 谁能帮我解决这个问题？ 非常感谢:)这是代码。 我包括了所有内容，以便您可以获取完整的上下文信息，但是我想认为主要的问题是主cpp中的while循环，如果您想跳到那里的话。

为了使我们处在同一轨道上，拉曼石txt文件的内容（命中点和再生点）为

8 2

7 3

6 1

对于它的软玉

10 3

12 4

11 5

这是我的warrior.h文件

#pragma once
#include <string>

using namespace std;

class warrior
{
public:

    warrior ();
    warrior (int h, int r);
    int getDamage() const;
    void takeDamage(int damage);
    int getCurrentHP() const;
    void regenerate();
    string tostring(int h, int r);

private:
    int HitPoints;
    int RegPoints;
    int damage;


};

这是我的战士cpp

#include "warrior.h"
#include <string>
#include <iostream>

warrior::warrior(int h, int r)
{
    HitPoints = h;
    RegPoints = r;
}

int warrior::getDamage() const
{
    int damage = rand () % HitPoints;
    return damage;
}

void warrior::takeDamage(int damage)
{
    HitPoints = HitPoints - damage;
}

int warrior::getCurrentHP() const
{
    return HitPoints;
}

void warrior::regenerate() 
{
    HitPoints = HitPoints + rand () % (RegPoints);
}

string warrior::tostring(int h, int r)
{
    return 0;
}

我的主要档案

#include <iostream>
#include <string>
#include <vector>
#include <fstream>
#include <sstream>
#include <cstdlib>
#include <ctime>

#include "warrior.h"

using namespace std;

void main ()
{
    srand(time(0));
    ifstream input1;
    cout << "input file name nephite: ";
    string filename;
    cin >> filename;

    input1.open(filename);

    int HP1, RP1;
    vector <warrior*> nephites;

    while (input1 >> HP1 >> RP1)
    {
        nephites.push_back(new warrior(HP1, RP1));
    }

    cout << nephites.size() << endl;

    ifstream input2;
    cout << "input file name lamanite : ";
    string filename2;
    cin >> filename2;

    input2.open(filename2);
    int HP2, RP2;
    vector <warrior*> lamanites;

    while (input2 >> HP2 >> RP2)
    {
        lamanites.push_back(new warrior(HP2, RP2));
    }
    cout << lamanites.size() << endl;

    cout << endl << "Battle" << endl;

    warrior nephitesw  = warrior (HP1,RP1);
    warrior lamanitesw = warrior (HP2,RP2);

    while ((nephites.size() > 0) && (lamanites.size() > 0))
    {

        int rN = rand () % nephites.size();
        int rL = rand () % lamanites.size();
        cout << rN << "xx" << rL << endl; // so that i know what rN and rL is

        while((nephites[rN]->getCurrentHP() > 0) && (lamanites[rL]->getCurrentHP() > 0)) // the program can't execute this part of the code
        {
            nephites[rN]->takeDamage(lamanites[rL]->getDamage());
            lamanites[rL]->takeDamage(nephites[rN]->getDamage());

            if(lamanites[rL]->getCurrentHP() > 0)
            {
                lamanites[rL]->regenerate();
            }
            else
            {
                lamanites.erase(lamanites.begin() + (rL));
            } 

            if(nephites[rN]->getCurrentHP() > 0) 
            {
                nephites[rN]->regenerate();
            } 
            else
            {
                nephites.erase(nephites.begin() + (rN));
            }
        }

        cout << "NEP HP: " << nephites[rN]->getCurrentHP() << " " << "LAM HP: " << lamanites[rL]->getCurrentHP() << endl;
    }

    system ("Pause");
}

Answer 1

您有一个while循环，用于测试nephites[rN]和lamanites[rL]的某些属性：

while((nephites[rN]->getCurrentHP() > 0) && (lamanites[rL]->getCurrentHP() > 0)
{
    // ...
}

但是在该循环中，您可能会删除这些元素：

{lamanites.erase(lamanites.begin() + (rL));} 

// ...

{nephites.erase(nephites.begin() + (rN));}  

至少，在执行了这些擦除操作之一之后，您将在下一个循环迭代中测试不同的软云母或拉曼石对象（可能是您想要的，也可能不是您想要的），但是如果您已经擦除了容器，您有索引现在超出范围的问题。

Answer 2

您一直循环播放，直到nephites[rN]->getCurrentHP() <= 0或lamanites[rL]->getCurrentHP() <= 0 。 但是，从1降到0任何一个都将从vector删除：

// ...
{lamanites.erase(lamanites.begin() + (rL));} 

// ...            
{nephites.erase(nephites.begin() + (rN));}

如果rN == nephites.size()或rN == lamanites.size() （大小为1时肯定会发生，并且可能会更早随机发生），这将导致您在测试循环时从vector索引。

为了快速解决此问题，请将从vector中删除warrior的代码移出循环：

while((nephites[rN]->getCurrentHP() > 0) && (lamanites[rL]->getCurrentHP() > 0))
{
    nephites[rN]->takeDamage(lamanites[rL]->getDamage());
    lamanites[rL]->takeDamage(nephites[rN]->getDamage());

    if(lamanites[rL]->getCurrentHP() > 0)
    {
        lamanites[rL]->regenerate();
    }

    if(nephites[rN]->getCurrentHP() > 0) 
    {
        nephites[rN]->regenerate();
    } 
}

cout << "NEP HP: " << nephites[rN]->getCurrentHP() << " " << "LAM HP: " << lamanites[rL]->getCurrentHP() << endl;

// *****
// Move the erasures out of the loop
// *****

if(lamanites[rL]->getCurrentHP() <= 0)
{
    lamanites.erase(lamanites.begin() + (rL));
} 

if(nephites[rN]->getCurrentHP() <= 0)
{
    nephites.erase(nephites.begin() + (rN));
}