[c++] return di un riferimento a un elemento di std::vector

[boots]

Ciao a tutti,
Ho questa situazione:

//components.h
typedef struct structPositionComponent{
     float x;
     float y;
     float z;
     structPositionComponent(){
       x = y = z = 0;
     }
}PositionComponent;

//componentmanager.h
template <typename T>
class ComponentManager
{
    public:
        ComponentManager(){};
        virtual ~ComponentManager(){};

        T& getComponent(Entity* e){
           if(this->_map.find(e) == this->_map.end()){
             return create(e);
           }

           return this->components[this->_map[e]];

        };
        T& create(Entity* e){
           if(this->_map.find(e) == this->_map.end()){
              T data;
              this->components.push_back(data);
              this->_map[e] = this->components.size() - 1;
              return this->components[this->components.size() - 1];
           }else{
             return this->components[this->_map[e]];
           }
        }
    protected:
    private:
        std::vector<T> components;
        std::map<Entity*, unsigned int> _map;
};

//main.cpp

int main(){
  ComponentManager<PositionComponent> posManager;
  Entity the_entity;

  PositionComponent p = posManager.create(&the_entity);
  p.x = 3.0f;
  p.y = 3.0f;
  p.z = 2.1f;

  PositionComponent ptest = posManager.getComponent(&the_entity);
  std::cout << "->" << (float)ptest.x << std::endl;
}

Io mi aspettavo che venisse stampato a video 3.0 ...invece stampa 0. Mi viene il dubbio che p è una copia invece dell'elemento dentro il vettore. Quancuno mi dice dove sbaglio ? Devo per forza usare un puntatore?

PS:
Se invece di fare

 T data;
  this->components.push_back(data);

facco

this->components.push_back(new T);

E' la stessa cosa a livello di memoria?

Grazie

[Scara95]

Nel main, quando chiami la funzione, controlla il tipo della variabile a cui assegni. Lì stai facendo una copia

[boots]

Così sembra andare

PositionComponent &p = posManager.create(&the_entity);

Una domanda

T data;
this->components.push_back(data);

la push_back, dovrebbe farsi una copia di data (che sta sullo stack), giusto? se io faccio

this->components.push_back(new T);

T viene viene allocata nel heap...ma poi lo spazio non viene più liberato?
C'è un modo più conciso (o migliore) per aggiugere un nuovo T nel vettore?

Grazie ancora

[Scara95]

Non puoi fare new, ha tipo T* non T.
Non preoccuparti della copia oppure trasforma tutto il codice per usare i puntatori.

[boots]

però potrei scrivere
this->components.push_back(*(new T));

cmq mi confermi che in questo modo alloco memoria senza poterla liberare?

[Scara95]

però potrei scrivere
this->components.push_back(*(new T));

cmq mi confermi che in questo modo alloco memoria senza poterla liberare?

Oltre ad allocare la memoria senza liberarla, effettui lo stesso la copia...

[boots]

Ma io voglio che venga fatta una copia nel vettore

mi chiedevo se ci fosse un modo più semplice, senza dichiarare T data

PS:
Ma con i template non è possibile separare la dichiarazione dal implementazione della classe (.h e .cpp) ?

[Scara95]

No, in quanto a tempo di compilazione viene generato codice specifico per ogni uso che ne fai. La separazione complicherebbe di molto il tutto, oltre al fatto che in ambiente C/C++ è impossibile collegare file header e file di implementazione se non attraverso convenzioni.

[boots]

Capito...

Grazie ancora

[Scara95]

Di niente