C++のシングルトンからRapidjson値にアクセスする

Question

私はCocos2dx用に書かれたコードのsingletonからrapidjson :: Valueを取得しようとしています。

これはシングルトンから私の抜粋です： AppData.h

class AppData { 

private: 
    AppData() { }; // Constructor 

    AppData(AppData const &) = delete; 
    void operator=(AppData const &) = delete; 

public: 
    static AppData *getInstance() { // Get instance 
    static AppData instance; // Instantiated on first use 
    return &instance; 
    }; 

    void getData(); // Get Data from JSON 
    rapidjson::Value& getCurrentLocation(); // Get Current Location Branch 

    rapidjson::Value locations; // JSON locations tree 
    int selectedLocation = 1; // Cursor to current location 
    rapidjson::Document doc; // JSON document 
}; 

とAppData.cpp

void AppData::getData() { 
    auto fileutil = FileUtils::getInstance(); 
    std::string path = fileutil->fullPathForFilename("data/locations.json"); 
    std::string data = fileutil->getStringFromFile(path.c_str()); 

    if (!doc.Parse<0>(data.c_str()).HasParseError()) { 
    locations = doc["locations"]; 
    } 
} 

// Get Current Location (Public) 
rapidjson::Value& AppData::getCurrentLocation() { 
    return locations[selectedLocation]; 
} 

とメインコードCPPから、その後の抽出物：

auto DATA = AppData::getInstance(); 

    rapidjson::Value& loc0 = DATA->getCurrentLocation(); 

    CCLOG("loc0: %s", loc0["title"].GetString()); 

これを実装する正しい方法がわからない場合でも、これは機能します。

しかし、私は、これは書く場合：

rapidjson::Value& test0 = DATA->locations[0]; 

私はプライベートGenericValue :: GenericValue（定数GenericValue &）は

アクセスできません。しかし場所はpublicとして宣言されているというエラーを得ました。

私はあまりにもエラーを得た：

auto test0 = DATA->locations[0]; 

しかし、それは私が2行に分割書く場合は大丈夫です：

rapidjson::Value t1; 
    t1 = DATA->locations[1]; 

私は別のものを使用する場合、これはOKですが、私は別の問題を抱えて

rapidjson::Value t1; 
    t1 = AppData::getInstance()->locations[2]; 

    rapidjson::Value t2; 
    t2 = AppData::getInstance()->locations[2]; 

：私が書く場合、私は、rapidjson値にISOBJECTエラー を得た同じ値を持つ変数場所から同じ値を使用して

は、[2] ISOBJECTエラーに

を与えるだけで、他の、それはシングルトンの実装に問題がある場合、一部の人々は、ポインタのアプローチを使用しているため、私は、知らない、私を助けてください参照。 しかし、他の方法と変数は、このタイプのシングルトンでは問題ありません、ただrapidjsonは私にrapidjsonを正しく理解していないかもしれないかもしれないかもしれません。

ありがとうございます。

[EDIT] 私はrapidjson値を再利用したコードの一部を追加します。

Location.h

class Location : public Layer { 

public: 
    Location() { }; // Constructor 
    static Scene *createScene(); // Create a scene with this class inside 
    CREATE_FUNC(Location); // Implement create with macro 
    bool init() override; // Initialize after creation 

}; 

Location.cpp（これはCocos2Dxで標準のシーン作成です）

Scene *Location::createScene() { 
    auto scene = Scene::create(); 
    auto layer = Location::create(); 
    scene->addChild(layer); 
    return scene; 
} 

// Called by create for initialize 
bool Location::init() { 
    if (!Layer::init()) return false; // Super init 

    const rapidjson::Value* t1; 
    t1 = &AppData::getInstance()->locations[1]; 

    CCLOG("t1: %s", (*t1)["title"].GetString()); 

    auto locationPanel = Layout::create(); 
    locationPanel->setLayoutType(Layout::Type::RELATIVE); 
    // Back Button 
    auto btnBack = Button::create("gfx/button.png", "gfx/buttonHighlighted.png"); 
    btnBack->setScale9Enabled(true); 
    btnBack->setContentSize(Size(180, btnBack->getContentSize().height * 1.2f)); 
    btnBack->setTitleText("Back"); 

    btnBack->addTouchEventListener([=](Ref *sender, Widget::TouchEventType type) { 
    if (type == Widget::TouchEventType::ENDED) { 
     Director::getInstance()->replaceScene(TransitionFade::create(0.2f, Location::createScene())); 
    } 
    }); 
    locationPanel->addChild(btnBack); 
    addChild(locationPanel); 
    return true; 
} 

AppDele gate Location.cpp start before私はのAppData :: getInstance（）でJSONを読んでいます。getData（）;上記の

私はいくつかの不要なコードを削除しましたが、たとえば、t1 ["title"]を印刷するとエラーが発生します。最初はGetString（）です。

プロセスラインは： AppDelegate - > AppData - > getData - >ロケーションシーンの作成 - >ボタンをクリックしてシーンを置き換える - >新しいシーンが同じAppData :: getInstance（） - > locations [1 ]私は2、3、4のような別のブランチにアクセスするとエラーが発生します。これは大丈夫ですので、ゲーム内で連続したシーンしか再生しない場合、jsonの情報は良い方法で読み込まれます。同じ変数t1でもうアクセスできません。

ありがとうございました。

Answer 1

シングルトンの実装は問題ありません。ただ、あなたはrapidjsonがどのように動作するのか分かりません。値を変更する必要がない場合は、const auto&を使用してこれらの変数を宣言できます。

プライベートGenericValue :: GenericValue（定数GenericValue &は）それはコピーコンストラクタがアクセス不可能であることを意味

アクセスできません。必要に応じてrapidjson::Value& test0 = DATA->locations[0];をconst rapidjson::Value& test0 = DATA->locations[0];に変更するか、const auto& test0に変更してください。