Sicuro per restituire un vettore popolato con variabili locali?

Question

È sicuro restituire un vettore che è stato riempito con variabili locali?

Per esempio, se ho ...

#include <vector> 

struct Target 
{ 
public: 
    int Var1; 
    // ... snip ... 
    int Var20; 
}; 


class Test 
{ 
public: 
    std::vector<Target> *Run(void) 
    { 
     std::vector<Target> *targets = new std::vector<Target>; 
     for(int i=0; i<5; i++) { 
      Target t = Target(); 
      t.Var1 = i; 
      // ... snip ... 
      t.Var20 = i*2; // Or some other number. 
      targets->push_back(t); 
     } 
     return targets; 
    } 


}; 

int main() 
{ 
    Test t = Test(); 
    std::vector<Target> *container = t.Run(); 

    // Do stuff with `container` 
} 

In questo esempio, sto creando molteplici Target casi in un ciclo for, spingendoli al vettore, e restituendo un puntatore ad esso. Poiché le istanze Target sono state allocate localmente, nello stack, significa che il vettore restituito non è sicuro perché fa riferimento a oggetti nello stack (che potrebbero presto essere sovrascritti, ecc.)? In tal caso, qual è il modo consigliato per restituire un vettore?

Sto scrivendo questo in C++, a proposito.

Answer 1

Gli elementi vengono copiati quando si push_back in un vettore (o si assegnano agli elementi). Il tuo codice è quindi sicuro: gli elementi nel vettore non fanno riferimento a variabili locali, sono di proprietà del vettore.

Inoltre, non è nemmeno necessario restituire un puntatore (e mai gestire i puntatori non elaborati, utilizzare invece i puntatori intelligenti). Basta invece restituire una copia; il compilatore è abbastanza intelligente da ottimizzarlo in modo tale che non venga effettuata alcuna copia ridondante.