多态无法正常工作

Question

我花了几个小时找到这个错误，至少现在我知道问题出在哪里，但我不知道为什么。

所以我有：

std::vector< std::vector <Organism* > >world;

我有基层Organism和班级Beetle和Ant 。

在基层Organism我具有虚拟功能：

    private:
         bool energy;
    public:
    Organism():energy(true){}
    virtual void removeEnergy(){
        energy = false;
    }
    virtual bool hasEnergy(){
        return energy;
    }

    virtual void resetEnergy(){
        energy = true;
    }

而且我还没有在任何子类中覆盖它们。 我在子类中调用了基类构造函数。

当我调用此函数时：

void World::replaceOrganism(int x, int y, int newX, int newY){  
        world[newX][newY] = world[x][y];
        world[x][y] = new Organism;
        world[newX][newY]->removeEnergy(); // this line
}

它仅从此类对象中消除能量。 这意味着如果在向量world有多个甲虫，则通过以下行：

world[newX][newY]->removeEnergy();

能量从所有人身上移走了。

我认为它与向量有关。 有人有什么建议吗？ 如果您需要更多信息->评论，我会添加。

如果我删除该行，还有一件事：

world[newX][newY]->removeEnergy();

则Organism类对象或其子类的能量不会改变。

已编辑

创建世界向量：

world.resize(worldSizeX);

    for (int x = 0; x < worldSizeX; x++){
        world[x] = std::vector< Organism* >(worldSizeY);

        // INITATES world ARRAY
        for (int y = 0; y < worldSizeY; y++){
            world[x][y] = new Organism;
        }
        // !INITATES world ARRAY
    }

已编辑

使用以下方法更改replaceOrganism函数：

void World::replaceOrganism(int x, int y, int newX, int newY){  
        delete world[newX][newY];   
        world[newX][newY] = world[x][y];
        delete world[x][y];
        world[x][y] = new Organism;
 }

删除word [newX] [newY]指向的所有类对象，但是仅调用一次函数。

已编辑

该函数用其子代替换Organism类对象。

bool World::createOrganism(int x, int y, Organism* organism){
    Organism* empty = new Organism;
    if (world[x][y]->type() != empty->type()){
        delete empty;
        return false;
    }
    else{
        delete empty;
        world[x][y] = organism;
        return true;
    }
}

我称其为：

createOrganism(2,3,new Beetle);

已编辑

void World::generateOrganisms(Organism* organism, int amount){
    while (amount != 0){
        int x = rand() % worldSizeX;
        int y = rand() % worldSizeY;
        if (createOrganism(x, y, organism)){
            amount--;
        }
    }
}

一切都指向同一个对象。

Answer 1

在generateOrganisms(Organism* organism, int amount) ，您设置的所有多个指针均指向同一单个生物。 甲虫也一样。

您需要预见有机体的其他虚拟功能：

Organism *clone()  {
   // create a new organism of the same type 
   }   

应该为每个孩子覆盖此虚拟函数，以使用正确的构造函数创建新的有机体。

然后，您可以更新创建功能：

bool World::createOrganism(int x, int y, Organism* organism){
    Organism* empty = new Organism;
    if (world[x][y]->type() != empty->type()){
        delete empty;
        return false;
    }
    else{
        delete empty;
        world[x][y] = organism->clone();  // use a clone, not the original
        return true;
    }
}

另外，我不确定是否会根据需要删除所有未使用的对象（至少仅使用您提供的片段很难说）。 有必要调查一下shared_ptr的潜在用途，以避免内存泄漏。