Restituzione di un oggetto locale da una funzione

Question

È questo il modo corretto per restituire un oggetto da una funzione?

Car getCar(string model, int year) { 
    Car c(model, year); 
    return c; 
} 

void displayCar(Car &car) { 
    cout << car.getModel() << ", " << car.getYear() << endl; 
} 

displayCar(getCar("Honda", 1999)); 

Ho ricevuto un errore, "indirizzo di temporaneo". Dovrei usare in questo modo:

Car &getCar(string model, int year) { 
    Car c(model, year); 
    return c; 
} 

Answer 1

getCar restituisce un Car per valore, che è corretto.

Non è possibile passare tale valore di ritorno, che è un oggetto temporaneo, in displayCar, perché displayCar prende uno Car&. Non è possibile associare un riferimento temporaneo a un riferimento non const. Si dovrebbe cambiare displayCar di prendere un riferimento const:

void displayCar(const Car& car) { } 

In alternativa, è possibile memorizzare il temporaneo in una variabile locale:

Car c = getCar("Honda", 1999); 
displayCar(c); 

Ma, è meglio avere displayCar prendere un riferimento const dal momento che doesn modificare l'oggetto.

Non restituire un riferimento alla variabile locale Car.

Answer 2

Non è sicuro restituire il riferimento di una variabile locale da una funzione.

Quindi sì questo è corretto:

Car getCar(string model, int year) { 
    Car c(model, year); 
    return c; 
} 

Answer 3

Il tuo problema è:

void displayCar(Car &car) { 
    cout << car.getModel() << ", " << car.getYear() << endl; 
} 

si dovrebbe utilizzare un riferimento const:

void displayCar(const Car & car) { 
    cout << car.getModel() << ", " << car.getYear() << endl; 
} 

Questa funzione:

Car getCar(string model, int year) { 
    Car c(model, year); 
    return c; 
} 

è OK così com'è, ma tutto quello che sta facendo è ciò che fa il costruttore, quindi è ridondante. Il passaggio di un valore di schiena, piuttosto che un punto di riferimento, è la cosa giusta da fare per questo tipo della funzione, tuttavia, il parametro modello dovrebbe essere passato per riferimento const:

Car getCar(const string & model, int year) { 

In generale, per i tipi di classe come stringa o Car, la scelta predefinita per un parametro dovrebbe sempre essere un riferimento const.

Answer 4

Ancora meglio:

Car getCar(string model, int year) { 
     return Car(model, year); 
} 

Answer 5

Sì, è sicuramente non è sicuro per restituire un riferimento o un puntatore a un oggetto temporaneo. Quando scade (cioè, quando la funzione getCar termina), ti verrà lasciato ciò che è noto come "puntatore penzolante".

Tuttavia, se si desidera ridurre le operazioni di copia su un oggetto, è necessario controllare "Move semantics" di C++ 0x. È un concetto relativamente nuovo, ma sono sicuro che diventerà presto il flusso principale. GCC 4.4 verso l'alto supporta C++ 0x (utilizzare l'opzione del compilatore -std=c++0x per abilitare).