对象的C ++实例无法读取内存

Question

我需要有关c ++和object \\ references \\ pointer的帮助。 我正在编写一个简单的Chess程序，其中包含几个类。 特别是以下两个类别的Board和Cell有问题：

板上有一个由单元向量组成的向量（基本上是一个矩阵）。 每个单元格都有一个指向当前正在占用它的指针。

class Board
{
public:
    Board();
    void drawBoard();
    bool makeMove(int startRow, int startCol, int destRow, int destCol);
private:
    vector< vector<Cell> > _board;
    void initBoard();
    void initPieces();
};

Board::Board()
{
    initBoard();
}

void Board::initBoard()
{

    for (int i = 0; i < BOARD_SIZE; i++)
    {
        vector<Cell> row; //create empty row
        for (int j = 0; j < BOARD_SIZE; j++)
        {
            row.push_back((Cell(i, j)));
        }
        _board.push_back(row);
    }

    initPieces();
}

void Board::initPieces()
{
    //Set Pawns
    for (int i = 0; i < BOARD_SIZE; i++)
    {
        _board[1][i].setOccupying(new Pawn(White));
        _board[6][i].setOccupying(new Pawn(Black));
    }
}

void Board::drawBoard()
{

    drawLettersCoord();
    for (int i = BOARD_SIZE-1; i >= 0; i--)
    {
        std::cout << i + 1 << " ";
        for (int j = 0; j < BOARD_SIZE; j++)
        {       
            _board[i][j].draw();
        }
        std::cout << " " << i + 1 << std::endl;
    }
    std::cout << "\n";

}

bool Board::makeMove(int startRow, int startCol, int destRow, int destCol)
{
    _board[startRow][startCol].getOccupyingPiece()->isLegalMove(
        startRow, startCol, destRow, destCol);
    return true;
}

这是Cell类：

class Cell
{
public:
    Cell();
    Cell(int row, int col);
    Cell(int row, int col, ChessPiece * occupying);
    void draw();
    ChessPiece * getOccupyingPiece();
    void setOccupying(ChessPiece *occupying);
private:
    int _row;
    int _col;
    bool _occupied;
    ChessPiece * _pieceOccupying;
};

Cell::Cell()
    :Cell(0, 0)
{
    setColour();
}

Cell::Cell(int row, int col)
    : _row(row), _col(col), _occupied(false)
{
    setColour();
}

Cell::Cell(int row, int col, ChessPiece * occupying)
    : _row(row), _col(col), _pieceOccupying(occupying), _occupied(true)
{
    setColour();
}

void Cell::setOccupying(ChessPiece *occupying)
{
    _occupied = true;
    _pieceOccupying = occupying;
}
void Cell::draw()
{

    int foregroundText;
    if (!_occupied)
    {
        foregroundText = 37;
    }
    else
    {
        foregroundText = _pieceOccupying->getPlayer();
    }
    cout << "\33[" << foregroundText << ";" << _colour << "m"
         << (_occupied ? _pieceOccupying->getPieceCode(): " ") << "\33[0m";
}

ChessPiece * Cell::getOccupyingPiece()
{
    return _pieceOccupying;
}

在运行游戏时，我通过调用以下内容来创建并绘制新的游戏板

Board _board;
_board.drawBoard();

绘图似乎可以正常工作，没有任何问题。

但是，当我打电话

_board.makeMove(1,0,2,0);

检查棋子是否可以执行此操作，出现内存错误。 调试后，我看到正在调用的Cell对象中有垃圾，而不是实际数据。 当我尝试查看指向占位符的指针时，它显示“无法读取内存”，因此当我调用占位符的isLegalMove函数时，它崩溃了。

我似乎无法找出问题所在。 我不明白为什么向量中的一个单元格里面会有垃圾，它是在类中定义的（没有新的），所以据我的理解，只要当前的电路板实例还活着，它就应该可用。

谁能说明我做错了什么？

Answer 1

1. 并非所有Cell初始化的构造函数_occupied和_pieceOccupying都初始化，这意味着它们在某些Cell对象中可能具有垃圾值。

2. 此外，您的makeMove无条件地取消引用getOccupyingPiece() -它不会检查null。

3. 最后，您的棋盘上只有棋子，这意味着(0, 0)不包含片段-由于1， Cell对象的_pieceOccupying包含垃圾值，因此您可以访问随机内存并崩溃。