Question

晚上好

我的构造函数有一个小问题。我试图建立一个我的董事会不同可能的游戏树（之后做一个DepthFirstSearch）。在我的节点类构造函数中，我想将当前板复制为树的叶子。但是我得到了一个错误＆＃34;未定义的参考`Board :: Board（）＆＃39; |＆＃34;当我尝试使用我的Board类实例作为Node构造函数的参数时。

如果你知道如何正确地做到这一点，我正在倾听，我真的没有看到任何问题：（

这是我的班级节点：

class Node
{
private :
    Board      state;
    list<Node>  sons;
public :
                Node(Board&);
    void        addNode(Board&);
};

在执行Node构造函数时，我这样做：

Node::Node(Board& tab)
{
    state = tab;
    sons = NULL;
}

我的董事会课程是：

class Board {

private :
    int**           tab;
    int             nbline;
    int             nbcolumn;
    Position        emptyspot;

public  :
                    Board(int, int, Play&); // initialised with random positions
                    Board(int, int); // just the good size, no values inside. Node::node during constructor.
    void            setValue(Position&, int);
    void            setNbline(int m);
    void            setNbcolumn(int n);
    int             getValue(Position&);
    int             getNbline();
    int             getNbcolumn();
    int             getEmptyline();
    int             getEmptycolumn();
    void            setEmptySpot(Position&);
    Position&       getEmptySpot();

    Board&          operator=(Board& source);
};

编辑：由于有人问，这里是董事会的建设者：

    Board::Board(int m, int n, Play& jeu) : tab{new int*[m]}, nbline{m}, nbcolumn{n}, emptyspot{n-1,m-1}{

       int              x(1);

       for (int i = 0; i < m; ++i){
            tab[i] = new int[n];

            for(int j = 0; j < n; ++j) {
                tab[i][j] = x; x++;}}

       tab[n-1][m-1]=0;
       x=0;

       while (x!=1000)
       {
        int numbers[] = { UP, DOWN, LEFT, RIGHT };
        int length = sizeof(numbers) / sizeof(int);
        int randomNumber = numbers[rand() % length];

        jeu.moves(*this, randomNumber);
        x++;
       }
    }

    Board::Board(int m, int n) : tab{new int*[m]}, nbline{m}, nbcolumn{n}, emptyspot{n-1,m-1}  {
      for (int i = 0; i < m; ++i){
        tab[i] = new int[n];

        for(int j = 0; j < n; ++j) {
            tab[i][j] = 0;}}
}

Answer 1

由于state不在Node的构造函数初始化列表中，因此将使用其默认构造函数创建它。问题是，Board没有默认构造函数。要解决此问题，请使用初始化列表。这假定Board具有适当的复制构造函数。

Node::Node(const Board& tab)
    : state(tab)
{
}

我制作了参数const。我还删除了列表分配。我不知道将NULL分配给列表意味着什么，但是如果它是std::list那么它的默认构造函数无论如何都会将它设置为空。

Answer 2

这并不奇怪。在进行分配之前，Node::Node(Board& tab)状态是默认初始化的。由于您没有明确地执行此操作，因此编译器调用Board::Board()，这是未定义的。将Node::Node()更改为：

Node::Node(Board& tab)
: state(tab)
, sons(NULL)
{
}

请记住，必须在构造函数的主体执行之前完全初始化所有成员。因此，包含引用成员或类实例的类（未定义默认/复制构造函数）必须始终通过初始化列表初始化它们。

另外，如果您在operator=()中定义了Board，请定义复制构造函数（记住Rule of Three的记忆）：

class Board
{
//...

  Board(const Board&)
  {
    //make a copy
  }

};

修改

我认为Board的复制构造函数应该以这种方式实现：

Board(const Board& origin) : tab(NULL) , nbline(origin.nbline) , nbcolumn(origin.nbcolumn) , emptyspot(origin.emptyspot) { this->tab = new int*[this->nbline]; for (int i = 0; i < m; ++i) this->tab[i] = new int[this->nbcolumn]; //Tab contains int's, so let's do simple memcpy() memcpy(this->tab, origin.tab, sizeof(int) * this->nbline * this->nbcolumn); }

对参数的构造函数的未定义引用

2 个答案: