Ошибка сегментации с доступом к полю с использованием виртуальной чистой функции

Я много читал об ошибке с помощью чисто виртуального вызова, но я не понял, что не так с моим кодом:

Я запрограммировал игру в шахматы, используя полиморфизм.

вот некоторый код:

Piece.hpp (Родитель для всех игровых фигур)

class Piece
{
private:
char _player;
virtual int isMoveLegalForSpecificPiece(int positionRow, int positionCol, int targetRow,
int targetCol, Piece *board[8][8]) = 0;
public:
~Piece()
{
}
Piece(char player);
virtual int isMoveLegal(int positionRow, int positionCol, int targetRow, int targetCol,
Piece *board[8][8]);
char getPlayer();
virtual char getSign() const = 0;
virtual std::string getUnicodeSymbol() const = 0;
};

Rook.hpp (например)

#include "Piece.hpp"
class Rook : public virtual Piece {
private:
std::string _unicode = "265C";
virtual int isMoveLegalForSpecificPiece(int positionRow, int positionCol, int targetRow,
int targetCol, Piece *board[8][8]);
public:
Rook(char player) : Piece(player){}
~Rook() {}
virtual std::string getUnicodeSymbol() const;
char getSign() const;
};

и Rook.cpp

#include "Rook.hpp"char Rook::getSign() const {return 'r';}
int Rook::isMoveLegalForSpecificPiece(int positionRow, int positionCol, int targetRow,
int targetCol, Piece *board[8][8]) {
if (positionRow == targetRow) {
int rightOrLeft = (targetCol - positionCol > 0) ? 1 : -1;
for (int i = positionCol + rightOrLeft; i != targetCol; i += rightOrLeft) {
if (board[positionRow][i] != 0) {return 0;  }
}
return 1;
}
else if (positionCol == targetCol)  {
int upOrDown = (targetRow - positionRow > 0) ? 1 : -1;
for (int i = positionRow + upOrDown; i != targetRow; i += upOrDown) {
if (board[i][positionCol] != 0) {return 0;}
}
return 1;
}
return 0;
}
std::string Rook::getUnicodeSymbol() const {return _unicode;}

Совет :: Совет ()

Board::Board()  {
for (int i = 0; i < 8; i++) {
for (int j = 0; j < 8; j++) {
board[i][j] = 0;
}
}
for (int i = 0; i < 8; i++) {
board[1][i] = new Pawn('w');
board[6][i] = new Pawn('b');
}
board[7][0] = new Rook('b');
board[0][0] = new Rook('w');
board[7][1] = new Knight('b');
board[0][1] = new Knight('w');
board[7][2] = new Bishop('b');
board[0][2] = new Bishop('w');
board[7][3] = new King('b');
board[0][3] = new King('w');
board[7][4] = new Queen('b');
board[0][4] = new Queen('w');
board[7][5] = new Bishop('b');
board[0][5] = new Bishop('w');
board[7][6] = new Knight('b');
board[0][6] = new Knight('w');
board[7][7] = new Rook('b');
board[0][7] = new Rook('w');
}

Совет :: печать

void Board::print() {
printRowLetters();
for (int i = 7; i >= 0; i--){
cout << (char) ('1' + i) << " ";
for (int j = 7; j >= 0; j--) {
string isColor = "0";
string pieceUnicode = " ";
if (board[i][j])    {
isColor = (board[i][j]->getPlayer() == 'w') ? "37" : "30";
pieceUnicode = board[i][j]->getUnicodeSymbol();
}
//some more code..
}
}
}

я застрял в этой строке:

pieceUnicode = board [i] [j] -> getUnicodeSymbol ();

я получил:

чисто виртуальный метод называется

я не вызывал функцию внутри конструктора или деструктора

После сдачи virtual ключевое слово к вышесказанному ~Piece() деструктор, эта проблема была решена. Но теперь на той же линии я получаю

Сигнал: SIGSEGV (ошибка сегментации)

есть идеи?

вот еще немного информации:

я объявляю доску внутри main шахмат.

Board boardGame;

Затем я отправляю void playGame (Board BoardGame) следующим образом:

пустяки (BoardGame);

и внутри я отправляю void getNextMove (доска, строка whitePlayer, строка blackPlayer) следующим образом:

getNextMove (boardGame, whitePlayer, blackPlayer);

и тогда я использую:

boardGame.print ();

0

Решение

Я предполагаю, что у вас есть объект типа Board и вы делаете его копии, но у вас нет конструктора копирования, и я также предполагаю, Board::board динамически распределяется.

Это вызовет проблемы с копированием, назначением или уничтожением. (Прочти меня)

Вы должны подчиняться правилу трех и предоставить допустимые реализации для конструктора копирования, оператора присваивания и деструктора Board,

Или вы могли бы объявить Board::board как вектор указателей и не беспокоиться об управлении памятью.

Все это основано на предположении, если вы уже делаете это, то этот ответ не применяется.

2

Другие решения


По вопросам рекламы [email protected]