C++ Карта бросает вектор подстрочный из диапазона

Question

Я пытаюсь создать класс, который хранит свои экземпляры в карте, как так:

class Apple { 
    public: 
     static Apple* getApple(const std::string &name) { 
      auto it = allApples.find(name); 

      if (it == allApples.end()) // if this apple doesnt exist, make a new one 
       return new Apple(name); 
      else // if the apple exists, return its pointer 
       return it->second; 
     } 
    private: 
     static std::unordered_map<std::string, Apple*> allApples =     
      std::unordered_map<std::string, Apple*>(); 

     Apple(const std::string &name) { 
      // create an apple 
      allApples.insert({ name, this }); 
     } 
} 

Теперь я сделал классы, которые хранят статические яблоки, как так:

class Orange { 
    static Apple *myOrangeApple; 
} 

myOrangeApple = Apple::getApple("orange"); 

Когда я запускаю программу, она сбрасывается в первой строке метода getApple() с ошибкой векторного индекса вне диапазона. Я пробовал оглядываться, но я не могу найти какое-либо решение, касающееся карт и ошибки вне диапазона. Мое лучшее предположение из некоторых исследований, которые я сделал, это то, что это что-то о инициализации статического порядка, но я действительно не слишком уверен.

Answer 1

Используйте статический объект функции-scope.

... 
private: 
    std::unordered_map<std::string, Apple*>& allApples() { 
     static std::unordered_map<std::string, Apple*> apples; 
     return apples; 
    } 

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

Этот метод не будет работать, если между объектами, управляемыми таким образом, существуют взаимные зависимости.

Answer 2

Прежде всего, ваш метод getApple должен быть статическим, и вы должны инициализировать член класса (статический член) вне класса. Попробуйте это -

class Apple { 
    public: 
     static Apple* getApple(const std::string &name) { 
      auto it = allApples.find(name); 

      if (it == allApples.end()) // if this apple doesnt exist, make a new one 
       return new Apple(name); 
      else // if the apple exists, return its pointer 
       return it->second; 
     } 
    private: 
     static std::unordered_map<std::string, Apple*> allApples; 

     Apple(const std::string &name) { 
      // create an apple 
      allApples.insert({ name, this }); 
     } 
}; 

std::unordered_map<std::string, Apple*> Apple::allApples = std::unordered_map<std::string, Apple*>(); 

class Orange { 
    static Apple *myOrangeApple; 
}; 

Apple * Orange::myOrangeApple=Apple::getApple("orange"); 

Answer 3

Вы получаете доступ allApples, прежде чем он был инициализирован. Пока вы не введете main, нет никакой гарантии, что был построен allApples, поэтому доступ к нему определенно не-нет.

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