0

所以我正在尝试实现一个具有记分牌和两名球员的程序，我试图让两名球员使用单例模式共享一个记分牌。但是，当我尝试在玩家类中定义的全局记分板上使用方法时，我总是会收到“运行失败”消息。

这是我的两个头文件，如果有必要我可以提供完整的实现。

#ifndef PLAYER_H
#define PLAYER_H
#include "scoreboard.h"
#include <string>
#include <fstream>


class Player{
private:
        std::ifstream file1;
        std::ifstream file2;
        static Scoreboard* _game;
    public:
        static Scoreboard* Game();
        void makeMove(const char,const std::string);
};


#endif

#ifndef SCOREBOARD_H
#define SCOREBOARD_H

class Scoreboard{
    private:
        int aWin;
        int bWin;
        int LIMIT;
        int curCounter;

    public:
        void resetWins();
        void addWin(char);
        void makeMove(const int, char);
        void startGame(const int, const int);
        int getWin(char);
        int getTotal();
        int getLimit();
};

#endif  /* SCOREBOARD_H */

在 player.cc 中

Scoreboard* Player::_game = 0;

Scoreboard* Player::Game(){
    if (_game = 0)
    {
        _game = new Scoreboard;
        _game->resetWins();
    }
    return _game;
}

连同 makeMove 方法

4

2 回答 2

1

您的Scoreboard实例不需要是指针：

static Scoreboard _game;
// ...
static Scoreboard& Game() { return _game; }

或者，只需省略以下的类声明_game：

// you can either make this function static or non-static
Scoreboard& Game()
{
    static Scoreboard game; // but this variable MUST be static
    return game;
}

这将在没有内存管理问题的情况下做同样的事情。

Scoreboard这将创建一个for all的单个实例Players。如果你只想拥有一个实例（Scoreboard例如，如果你有一个Referees类也需要查看记分牌），你可以修改你的记分牌类：

class Scoreboard
{
private:
    // all your data members
    Scoreboard() {} // your default constructor - note that it is private!
public:
    // other methods
    Scoreboard& getInstance()
    {
        static Scoreboard instance;
        return instance;
    }
};

然后，要在其他类中访问它，您将包含记分牌标题并将其用作：

#include "Scoreboard.h"

void my_func()
{
    Scoreboard& scoreboard = Scoreboard::getInstance();
    scoreboard.DoSomething();
}
于 2013-11-01T15:11:05.213 回答
0

在Player::Game中，你写了

if (_game = 0)

即设置_game = 0和评估为假，因此您实际上不会创建记分牌。将其更改为：

if (_game == 0)
于 2013-11-01T15:21:39.493 回答