станд :: вектор, не сохраняя сдвинуться назад объекты инстанцированы в реализации

Question

у меня есть вектор и объект, объявленный в plane.h:

extern Plane Default; 
extern std::vector<Plane *>universe; 

Они определены в plane.cpp:

Plane Default("Default"); 
std::vector<Plane *>universe; 

Plane конструктор:

Plane::Plane(const std::string &label) { 
    /* check universe to ensure uniqueness */ 
    std::cout << this << std:endl; //DEBUG CHECK to see what I push_back 

    universe.push_back(this); //ACTION to keep track of the planes 

    std::cout << universe.back() << std::endl; //DEBUG CHECK to ensure that it was stored correctly 
} 

выход подтверждает, что самолет действительно находится в векторе.

В главном:

if(universe.empty()) cout << "EMPTY UNIVERSE" << endl; 

показывает, что вектор не сохраняет значение. Я ожидал, что значение Default (как определено в plane.cpp) будет сохранено в юниверсе.

Demo

Однако Вселенная сохраняет значения, когда я инстанцирую самолеты из главного

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

Я также попытался объявить вектор от кучи, так как:

extern std::vector<Plane *> *universe; 

и определяется следующим образом:

std::vector<Plane *> *universe = new std::vector<Plane *>; 

Это просто разбило программу. Я использую Mingw32 4.9 на 64-битной Vista, машины в коде: Блоки 16,01 и -std = C++ 11

Answer 1

Вы попали в глобальный облом порядок инициализации:

Plane Default("Default"); 
std::vector<Plane *>universe; 

Default имеет зависимость от universe, но когда строительство Default, universe еще не построено.

Вот почему вы должны избегать глобального в целом.

обходным в вашем случае было бы изменить порядок, предполагая, есть в то же ЕП:

std::vector<Plane *>universe; 
Plane Default("Default"); 

Answer 2

Если у вас возникли проблемы со сферой ваших указателей, возможно, с помощью shared_ptr будет достаточно? Однако я не знаю вашего общего дизайна, если вас беспокоит постоянство указателей, которые могут быть удалены или перемещены и по-прежнему нуждаются в данных; умным указателем может быть ответ. Если есть проблема с циклами в любом месте, используйте weak_ptr, чтобы сломать его.

Удачи вам!

Answer 3

answer by Jarod42 должен решить возникшую проблему. Я собираюсь предложить некоторые рекомендации по предотвращению таких проблем.

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

Вы можете изменить

extern std::vector<Plane *>universe; 

к

extern std::vector<Plane *>& getUniverse(); 

и реализовать его как:

extern std::vector<Plane *>& getUniverse() 
{ 
    static std::vector<Plane *>universe; 
    return universe; 
} 

С, что на месте, вы можете изменить конструктор Plane к:

Plane::Plane(const std::string &label) { 
    getUniverse().push_back(this); 
} 

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

Вы также можете изменить

extern Plane Default; 

к

extern Plane& getDefaultPlane(); 

и реализовать его как

extern Plane& getDefaultPlane() 
{ 
    static Plane Default("Default"); 
    return Default; 
} 

Если по умолчанию Plane должны быть построены во время инициализации, вы можете использовать что-то вдоль линии:

// Use a helper struct in an anonymous namespace to initialize 
// whatever needs to be initialized. 
namespace 
{ 
    struct Initializer 
    { 
     Initializer(); 
    }; 
} 

// Make sure that Initializer::Initializer() gets called 
// at initialization time. 
static Initializer initer; 

Initializer::Initializer() 
{ 
    // Trigger construction of the default Plane 
    getDefaultPlane(); 
}