std ::只讀位置的地圖分配

Question

我有一個static std::map<std::string, CreateGUIFunc>在基本上持有字符串標識gui類型的類，CreateGUIFunc是對工廠函數的引用。

在我的構造函數中我有

if (m_factoryRef.size() == 0) { 
    m_factoryRef["image"] = &CreateGUI<Image>; 
    m_factoryRef["button"] = &CreateGUI<Button>; 
} 
... 

然而，這給了我一個錯誤說assignment of read-only location ‘GUIManager::m_factoryRef.std::map<_Key, _Tp, _Compare, _Alloc>::operator[] [with _Key = std::basic_string<char, std::char_traits<char>, std::allocator<char> >, _Tp = GUI*(), _Compare = std::less<std::basic_string<char, std::char_traits<char>, std::allocator<char> > >, _Alloc = std::allocator<std::pair<const std::basic_string<char, std::char_traits<char>, std::allocator<char> >, GUI*()> >](((const std::basic_string<char, std::char_traits<char>, std::allocator<char> >&)(& std::basic_string<char, std::char_traits<char>, std::allocator<char> >(((const char*)"image"), ((const std::allocator<char>&)((const std::allocator<char>*)(& std::allocator<char>())))))))’|

我不知道爲什麼這是一個只讀的分配。我也嘗試將其更改爲普通成員，以查看是否它可能是靜態的，但同樣的事情。

什麼問題？

編輯：一些定義，以使事情多一點明確

// these are all private 

typedef GUI* CreateGUIFunc(); 

template<class T> 
GUI* GUIManager::CreateGUI(std::string &filepath, int x, int y) { 
    return new T(filepath, x, y); 
} 

static std::map<std::string, CreateGUIFunc> m_factoryRef; 

附：如果有更清晰的方式來初始化靜態地圖請讓我知道。

Answer 1

在C++中，typedef GUI* CreateGUIFunc();不是具有未指定參數的函數，而是具有NO參數的函數。所以你的功能都不符合這種類型。你想要的是typedef GUI* (*CreateGUIFunc)(std::string &filepath, int x, int y);

接下來，嘗試使用地圖的insert成員函數而不是下標運算符，這樣你就不會偶然調用常量版本。

Answer 2

以下是對當前C++的一些替代初始化。當你說你的代碼在構造函數中時，我不確定你的意思是「靜態」初始化。我有一些猜測，但沒關係。選擇對你來說「更清潔」的東西。

if (m.empty()){ 
    m.insert (map<K,V>::value_type (k1, v1)); 
    m.insert (map<K.V>::value_type (k1, v1)); 
} 

，或者

map<K,V>& getmap() { 
    static map<K,V>* m = 0; 
    if (!m){ 
     m = new map<K,V>(); // never deleted. 
     // insert. 
    } 
    return *m; 
} 

，或者

namespace /*anon*/ { 
     map<K,V>* init_map() { 
     map<K,V>* m = new map<K,V>(); 
     // insertions here. 
     return m; // return by val. can move in c++0x. 
     } 
} 

map<K,V> Foo::m = *init_map(); 

，或者

namespace /*anon*/ { 
      map<K,V>* init_map() { 
      map<K,V>* m = new map<K,V>(); 
      // insertions here. 
      return m; // return by val. can move in c++0x. 
      } 
} 

map<K,V>& Foo::get_map() { /* static singleton accessor */ 
    static map<K,V> * m = init_map(); 
    return *m; 
} 

的圖案應該是顯而易見的。你必須解決兩個問題：僅僅一次初始化該死的東西這個微不足道的問題，以及防止靜態初始化命令失敗的棘手問題。我的人的偏好就像上面的第二種情況。