求大神帮忙看看关于这个类的构造函数该如何改

求大神帮忙看看关于这个类的构造函数该怎么改。

class WarSystem
{
public:
WarSystem();
void PrintSoldiers();
void startWar();
        void startRound();
public:
Soldier *pGroupA[3];
Soldier *pGroupB[3];
};
WarSystem::WarSystem()
{
Soldier A1("A1"),B1("B1");
Wizard  A2("A2"),B2("B2");
Master  A3("A3"),B3("B3");
pGroupA[0]=&A1;
pGroupA[1]=&A2;
pGroupA[2]=&A3;
pGroupB[0]=&B1;
pGroupB[1]=&B2;
pGroupB[2]=&B3;
}
求解一下，新手第一次写关于指针的构造函数，这样写貌似问题很严重。没办法构造
[解决办法]
A1，A2……等都是局部变量，构造函数执行完成后就被析构。而pGroupA和pGroupB是类成员变量，与WarSystem的对象生存期相等。于是，构造函数执行完成后，pGroupA和pGroupB中的六个指针全都指向失效的对象，自然不会正常。

至于如何修改，那要取决于你的程序逻辑是什么。最简单的修改方法就是不使用指针：

class WarSystem
{
public:
WarSystem();
void PrintSoldiers();
void startWar();
void startRound();
public:
Soldier pGroupA[3];
Soldier pGroupB[3];
};
WarSystem::WarSystem()
{
Soldier A1("A1"),B1("B1");
Wizard  A2("A2"),B2("B2");
Master  A3("A3"),B3("B3");
pGroupA[0]=A1;
pGroupA[1]=A2;
pGroupA[2]=A3;
pGroupB[0]=B1;
pGroupB[1]=B2;
pGroupB[2]=B3;
}

[解决办法]
不要返回/保存局部对象的指针或引用。
[解决办法]
如果一定要使用指针，那么就需要在构造函数中new，在析构函数中delete，并且要禁止或重写拷贝构造函数、拷贝赋值操作符：

class WarSystem
{

public:
WarSystem();
~WarSystem();//增加析构
void PrintSoldiers();
void startWar();
void startRound();
public:
Soldier *pGroupA[3];
Soldier *pGroupB[3];
private:
  WarSystem( WarSystem const & );//禁止拷贝构造
  WarSystem & operator = ( WarSystem const & );//禁止拷贝赋值
};
WarSystem::WarSystem()
{
std::fill(&pGroupA[0], &pGroupA[3], 0 );
std::fill(&pGroupB[0], &pGroupB[3], 0 );

Soldier A1("A1"),B1("B1");
Wizard  A2("A2"),B2("B2");
Master  A3("A3"),B3("B3");
pGroupA[0]=new Soldier ("A1");
pGroupA[1]=new Wizard ("A2") ;
pGroupA[2]=new Master("A3") ;
pGroupB[0]=new Soldier ("B1");;
pGroupB[1]=new Wizard ("B2") ;;
pGroupB[2]=new Master("B3")  ;
}

WarSystem::~WarSystem()
{
for( int i = 0; i < 3; ++ i)
{
  if(pGroupA[i])
  {
     delete pGroupA[i];
  }
}
for( int i = 0; i < 3; ++ i )
{
  if(pGroupB[i])
  {
    delete pGroupB[i];
  }
}
}

查看更多 下一篇