2

在那之前我习惯了应付，在这里我正在尝试使用向量构建我自己的容器类。我的工作需要它。我正在使用代码块 10.05

class myclass
{
    public :
      vector<myclass> array;
      char * classname;
      ...

问题是，如果在析构函数中没有删除我的班级数据，我的班级数据会很好地显示在屏幕上。如果我删除类名，我的函数 show() 会显示奇怪的字符。我想它来自我构建对象的方法和当我将它们作为参数传递时的范围问题。

 myclass::~myclass()
 {
    //if(classname) delete [] classname;
 }

这就是它在构造函数中的初始化方式：

 myclass::myclass(long lvl = 0, const char name[] = "undefined")
    :ID(++ ID_counter)
    {
       level = lvl;

       int namelength = strlen(name);
       classname = new char[namelength + 1];
       strcpy(classname, name);
    }

add_content(const myclass & c)应该复制 c.array 的元素并在 this->array 中“push_back”它们我发现我们有可能将一个没有名称的对象作为参数：mycontainer.add_content （我的班级（3,5））； 它有效，但我对它应该具有的范围持怀疑态度

> int main()
>     {
>        myclass mycontainer(0);
>        mycontainer.add_content(myclass(3,5));
>     ...

这是完整的代码：

#include <vector>
#include <iostream>
using namespace std;

class myclass
{
  public :

   vector<myclass> array;
   static long ID_counter;
   long ID;
   long level;
   char * classname;

   myclass(int n, long lvl, const char name[]); //push_back n elements, lvl = 0, name = "undefined"
   myclass(long lvl, const char name[]); //lvl = 0, name = "undefined"
   myclass::~myclass();

   void set_level(long lvl); //recursive function, go down the tree
   void add(int n); //push_back n elements
   void add(const myclass & c); //push_back and set back the levels
   void add_content(const myclass & c); //push_back all the c.array[i] and set back the levels

   void show();

   template <typename T> myclass & operator[](const T it){ return array[it]; }
};

long myclass::ID_counter = 0;

myclass::myclass(long lvl = 0, const char name[] = "undefined")
:ID(++ ID_counter)
{
   level = lvl;

   int namelength = strlen(name);
   classname = new char[namelength + 1];
   strcpy(classname, name);
}
myclass::myclass(int n, long lvl, const char name[] = "undefined")
:ID(++ ID_counter)
{
   level = lvl;

   int namelength = strlen(name);
   classname = new char[namelength + 1];
   strcpy(classname, name);

   for(int i = 0; i < n; i++) array.push_back( myclass(this->level + 1) );
}
myclass::~myclass()
{
   //if(classname) delete [] classname; //there is the point !
}

void myclass::add(int n = 1)
{
   for(int i = 0; i < n; i++) array.push_back( myclass(level + 1) );
}

void myclass::add(const myclass & c)
{
   array.push_back(c);
   array[array.size() - 1].set_level(level + 1);
}

void myclass::add_content(const myclass & c)
{
   for(int i = 0; i < c.array.size(); i++)
   {
      array.push_back(c.array[i]);
      array[array.size() - 1].set_level(level + 1);
   }
}

void myclass::set_level(long lvl)
{
   level = lvl;
   for(int i = 0; i < array.size(); i++) array[i].set_level(level + 1);
}

void myclass::show()
{
   cout << "ID : " << ID << "\tLvl : " << level << "\t Classname : " << classname << endl;
}

int main()
{
   myclass mycontainer(0); //ground level
   mycontainer.add_content(myclass(3,5)); //the 3 elements level 5 should be reset to 0+1

   mycontainer.show();

   for(int i = 0; i < mycontainer.array.size(); i++)
   {
      mycontainer[i].show();

      for(int j= 0; j < mycontainer[i].array.size(); j++)
         mycontainer[i][j].show();
   }
   cout << "\ncheckpoint\n";
   system("PAUSE"); //press any key to crash!
   return 0;
}

这里没有在析构函数中删除 *classname，程序会这样：

ID : 1  Lvl : 0  Classname : undefined
ID : 3  Lvl : 1  Classname : undefined
ID : 4  Lvl : 1  Classname : undefined
ID : 5  Lvl : 1  Classname : undefined

checkpoint
Appuyez sur une touche pour continuer...

删除和一些坏字符出现崩溃：

ID : 1  Lvl : 0  Classname : undefined
ID : 3  Lvl : 1  Classname : ░(?
ID : 4  Lvl : 1  Classname : ░(?
ID : 5  Lvl : 1  Classname : ░(?

checkpoint
Appuyez sur une touche pour continuer...

Process returned -1073741819 (0xC0000005)   execution time : 29.651 s
Press any key to continue.

我试图研究一些关于论点和指针的原则，但这种语言太经验主义了，没有意义。感谢您的帮助。

4

2 回答 2

7

您有一个原始指针，并且没有定义复制构造函数或赋值运算符。这意味着您将获得自动生成的版本，这是您不想要的。回顾三法则。

解决方案 #1：实现正确的复制构造函数和赋值运算符。

解决方案 #2：更改char * classname为std::string classname. 现在自动生成的复制构造函数和赋值运算符就可以工作了。额外的好处：你不需要使用new，你不需要使用delete。

于 2012-12-19T19:42:26.647 回答
1

您应该定义复制构造函数。

我的调试器在这里引发错误：

for(int i = 0; i < n; i++) array.push_back( myclass(this->level + 1) );

我们在这里看到，myclass创建新对象，然后复制（克隆）到array中，然后像任何局部变量一样销毁原始对象。

因此，我们有两个具有相同指针的实例：一个被销毁，另一个在内部array，由默认复制构造函数创建，它只复制原始内存。所以，里面的实例array是无效的，因为析构函数已经在它上面运行了。

编码

下面的附加构造函数将解决这种情况：

myclass::myclass(const myclass& other) 
:ID(++ ID_counter) 
{
   level = other.level;

   int namelength = strlen(other.classname);
   classname = new char[namelength + 1];
   strcpy(classname, other.classname);
}

（上面是所谓的“复制构造函数”；是的，当然你应该遵守“三规则”以避免所有潜在的错误）

于 2012-12-19T19:48:16.547 回答