Question

我需要一些关于c ++和objects \ references \ pointers的帮助。我正在写一个简单的国际象棋程序，它有几个类。特别是我对以下两个类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);

检查pawn是否可以进行此类移动，我收到内存错误。在调试时，我看到被调用的Cell对象中有垃圾，而不是实际数据。当我试图查看指向占用件的指针时，它显示“无法读取内存”，所以当我调用占用件的isLegalMove函数时，它会崩溃。

我似乎无法找出问题所在。我不明白为什么矢量中的单元格会有垃圾，它在类中定义（没有新的），因此根据我的理解，只要当前的电路板实例存活，它就应该可用。

有人能说清楚我做错了吗？

Answer 1

并非Cell的所有构造函数都初始化_occupied和_pieceOccupying，这意味着这些构造函数可能在某些Cell个对象中具有垃圾值。
< / LI>
此外，您无条件makeMove解除引用getOccupyingPiece() - 它不会检查是否为空。
最后，你的棋盘上只有棋子，这意味着(0, 0)不包含一块 - 而且由于1. Cell对象在其{_pieceOccupying对象中包含一个垃圾值1}}，所以你访问随机内存并崩溃。

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

1 个答案: