c++ - 创建对象指针列表

Question

我正在尝试创建一个对象指针列表(它是一个对象的私有(private)成员)...

所以我正在编写的程序是两组“英雄”之间的模拟。每个英雄都有特殊的力量、 body 属性等……以及目标列表。目标列表是指向英雄的指针列表。当我调用构造函数创建英雄时，除目标列表外，所有信息都使用随机值进行初始化。到目前为止，我已经创建了两个英雄列表，团队 1 和团队 2。我正在尝试在团队 1 中创建一个指向团队 2 中英雄地址的指针列表，反之亦然。 main() 中的一系列 for 循环是我拥有的最佳解决方案，但函数 Hero::setTarget 多次“pushes_back”第一个英雄的地址。任何建议将不胜感激。

这是我目前所拥有的，不包括 Powers.h 和 Powers.cpp 文件...

已编辑:删除了嵌套的 for 循环，但我仍然将相同的地址传递到我的列表中....

    #include <iostream>
    #include <list>
    #include <random>
    #include <ctime>

    #include "Hero1.h"
    #include "Powers.h"

    using namespace std;

    int main()
    {
 ///random number generator
    default_random_engine generator(time(NULL));
    uniform_int_distribution<int> numHero(1,10);

///lists of hero's, pointers to a hero, list iterators
    list<Hero> team1;
    list<Hero> team2;
    Hero * hptr1;
    Hero * hptr2;
    list<Hero>::iterator hitr1;
    list<Hero>::iterator hitr2;

/// team 1 is created
    int a =  numHero(generator);
    for(int x=0; x<a; x++)
    {
    hptr1 = new Hero();
    team1.push_back(*hptr1);
    }

/// team 2 is created
    int b = numHero(generator);
    for(int x=0;x<b;x++)
    {
    hptr2 = new Hero();
    team2.push_back(*hptr2);
    }

for(hitr2=team2.begin();hitr2!=team2.end();hitr2++)
{
        hptr1->setTarget(hptr2);
}

for(hitr1=team1.begin();hitr1!=team1.end();hitr1++)
{
    hptr2->setTarget(hptr1);
}

 ///printing results for list of targets
 int w =1;
 cout<<"target address "<<w<<endl;
        hitr1=team1.begin();
        hptr1->displayTarget();

英雄是我的英雄.h

#ifndef HERO1_H_INCLUDED
#define HERO1_H_INCLUDED

#include <iostream>
#include <random>
#include <ctime>
#include <list>

#include "Powers.h"


using namespace std;

class Power;

class Hero
{
public:

    Hero();
   // ~Hero();

    void setID();
    void setLoc();
    void setPhy();
    void setPowers();
    void setEquip();
    void setTarget(Hero *h);

    int getID(){return id;}
    int getLoc(int x);
    int getPhy(int x);

    void displayHero();
    void displayTarget();
    void displayPowers();


private:
    int id;
    int location[3];
    int physical [4];
    Power * powPtr;
    Power * equipPtr;
    Hero * targetPtr;
    list<Power> powers;
    list<Power> equipment;
    list<Hero*> target;
    list<Power>::iterator equipItr;
    list<Power>::iterator powItr;
    list<Hero*>::iterator targetItr;

};

我的英雄.cpp

#include "Hero1.h"

default_random_engine generator(time(NULL));
uniform_int_distribution<int> distribution(100000,200000);
normal_distribution<double> disto(50,10);
uniform_int_distribution<int> randPow(1,3);

Hero::Hero()
{
    setLoc();
    setID();
    setPhy();
    setPowers();
    setEquip();
}

void Hero::setLoc()
{
    location[0] = 1;
    location[1] = 2;
    location[2] = 3;
}

void Hero::setID()
{
    int a = distribution(generator);
    id = a;
}

void Hero::setPhy()
{
    double a = disto(generator);
    double b = disto(generator);
    double c = disto(generator);
    double d = disto(generator);

    physical[0] = a;
    physical[1] = b;
    physical[2] = c;
    physical[3] = d;

}

void Hero::setPowers()
{
    int a = randPow(generator);
    for(int x=0;x<a;x++)
    {
    powPtr = new Power(getPhy(3));
    powers.push_back(*powPtr);


    }
}


void Hero::setEquip()
{
    int a = randPow(generator);
    for(int x=0;x<a;x++)
    {
    equipPtr = new Power(getPhy(3));
    powers.push_back(*equipPtr);
    }

}
void Hero::setTarget(Hero *h)
{

    target.push_back(h);

}

void Hero::displayTarget()
{
    int x =1;
    for(targetItr=target.begin();targetItr!=target.end();targetItr++)
    {
        cout<<&targetPtr<<endl;
        x++;
    }
    cout<<x<<endl;

}

int Hero::getLoc(int x)
{
    int p;
    p = location[x];
    return p;
}

int Hero::getPhy(int x)
{
    int p;
    p = physical[x];
    return p;
}
void Hero::displayPowers()
{
    cout<<"Number of powers = "<<powers.size()<<endl<<endl;
    for(powItr=powers.begin();powItr!=powers.end();powItr++)
    {
        powPtr->displayEffect();
    }
}

void Hero::displayHero()
{
    cout<<"Id :\t\t\t\t\t"<<id
        <<"\nLocation:\t\t\t\t"<<location[0]<<","<<location[1]<<","<<location[2]
        <<"\nPhysical attributes:\tstrength\t"<<physical[0]<<"\n\t\t\tendurance\t"<<physical[1]
        <<"\n\t\t\tagility\t\t"<<physical[2]<<"\n\t\t\tspeed\t\t"<<physical[3]<<endl<<endl;

        displayPowers();
}

Answer 1

我在您的代码中发现了几个不同的问题，但让我们从您询问的问题开始...

你的 setTarget调用每次都在做同样的事情，因为调用它们时使用的指针在循环中根本没有更新。这是一个可能来自于在函数顶部声明所有变量的问题。超出其用途的变量很容易出现这样的错误(或被重用于多种用途，然后使代码更难理解和维护)。因此，我强烈建议将局部变量的范围限定在需要它们的地方。

例如:

/// lists of heroes
    list<Hero> team1;
    list<Hero> team2;

/// team 1 is created
    int a = numHero(generator);
    for(int x=0; x<a; x++)
    {
        Hero * hptr1 = new Hero();
        team1.push_back(*hptr1);
    }

/// team 2 is created
    int b = numHero(generator);
    for(int x=0;x<b;x++)
    {
        Hero * hptr2 = new Hero();
        team2.push_back(*hptr2);
    }

for(list<Hero>::iterator hitr2=team2.begin();hitr2!=team2.end();hitr2++)
{

    for(list<Hero>::iterator hitr1=team1.begin();hitr1!=team1.end();hitr1++)
    {
        hptr1->setTarget(hptr2);
    }
}

for(list<Hero>::iterator hitr1=team1.begin();hitr1!=team1.end();hitr1++)
{
    for(list<Hero>::iterator hitr2=team2.begin();hitr2!=team2.end();hitr2++)
    {
        hptr2->setTarget(hptr1);
    }
}

如果你那样做了，那么你可能会得到编译错误，因为 hptr1和 hptr2不存在于setTarget正在被调用。这正是您想要的 - 编译器提醒您注意错误。

那么这些循环可以更正为:

for(list<Hero>::iterator hitr2=team2.begin();hitr2!=team2.end();hitr2++)
{

    for(list<Hero>::iterator hitr1=team1.begin();hitr1!=team1.end();hitr1++)
    {
        Hero& target = *hitr2;
        hitr1->setTarget(&target);
    }
}

接下来，对于第二个问题:填充列表时内存泄漏:

Hero * hptr1 = new Hero();
team1.push_back(*hptr1);

这将动态分配一个 Hero，然后复制它的值到另一个在列表中创建的 Hero - 因为这就是 push_back 的方式。适用于 list<Hero> 之类的东西.然后指向动态分配的指针被丢弃(或在您最初发布的代码中重新使用)，因此没有任何东西可以释放该内存。

你真正想做的更像是:

team1.push_back(Hero());