Блог

Я пытаюсь сохранить несколько разных производных классов из одного и того же базового класса в векторе указателей. Попытка вызвать функцию одного из этих объектов приводит к ошибке сегментации. Я не очень часто использовал наследование, но я перепробовал все версии, которые смог найти, и они либо приводят к ошибке сегментации, либо просто вызывают функцию базового класса.

Я относительно новичок в C и не публиковал много раньше, поэтому, пожалуйста, дайте мне знать, если я делюсь слишком большим количеством кода, пропускаю что-то важное или что-то путаю в любом другом отношении (стиль, эффективность и т. Д.).

Редактировать: вместо того, чтобы пытаться вернуть случайное или человеческое значение, функция getPlayer теперь просто возвращает значение int, которое указывает, какой тип проигрывателя создавать. Новый код по-прежнему приводит к ошибке seg в той же точке. (исключен getPlayer, поскольку он просто возвращает значение int и больше не может быть причиной проблемы.)

Здесь я определяю базовый класс (Player) и производные классы (Human и Random):

 #include <string>
#include <iostream>
#include <limits>
#include <time.h>
#include "Othello.h"
using namespace std;

// Player interface to choose game moves
class Player {
public:
    // Selects and returns a move 
    virtual int getMove(Othello amp;game) {
        return 0;
    }
};

// User-operated player
class Human: public Player {
public:
    int getMove(Othello amp;game) {
        int move = 0;
        bool valid= false;
        
        while (!valid) {
            cout << "Select a move: " << endl;
            cout << "-> ";
            cin >> move;
            
            if (cin.good()) { valid = true; }
            
            else {
                cin.clear();
                cin.ignore(numeric_limits<streamsize>::max(),'n');
                cout << "Invalid input. Try again.n" << endl;
            }
        }
        return move;
    }
};

// Basic bot - selects a move at random
class Random: public Player {
public:
    int getMove(Othello amp;game) {
        srand(time(NULL));
        return ( 1   rand() % game.n_moves );
    }
};
  

Вот основная функция, которая приводит к ошибке seg в строке move = players[state-1]->getMove(game) :

 int main() {
    
    // Select players (human, AI, random, etc.)
    // players[0] is left empty to keep the index consistent with the player id
    vector<Player*> players(2);
    int type;
    for ( int i : {1, 2} ) {
        type = getPlayer(i);
        if (type == 1) { players.push_back( new Human() ); }
        else if (type == 2) { players.push_back( new Random() ); }
    }
    
    // Load and start the game
    Othello game = loadBoard();
    int state = game.getState(1); // 1 or 2 for turn, or 0 for game over
    
    // Continue making moves until the game ends
    int move;
    int legal;
    while(state != 0) {
        game.print();
        legal = 0;
        cout << "PLAYER " << game.turn << endl;
        while (legal == 0) {
            move = players[state-1]->getMove(game);
            legal = game.doMove(move);
        }
        state = game.getState();
    }
    
    game.print();
    game.score();
    
    return 1;
}
  

vector<Player*> players(2); объявляет вектор с двумя элементами, которые оба будут инициализированы по умолчанию nullptr .

Позже вы добавляете еще два элемента, так что players получается 4 элемента.

Когда state имеет значение 1 или 2 , вызов players[state-1]->getMove(game); разыменовывает нулевой указатель, что приводит к ошибке сегментации.

Вероятно, вы хотите определить players , чтобы изначально быть пустым ( vector<Player*> players; ) , и обновить комментарий в строке перед этим определением. (Этот комментарий в его текущей форме, похоже, не имеет никакого смысла в том, как вы позже получите доступ к вектору players.)