Devolución de un objeto local desde una función

Question

¿Es esta la forma correcta de devolver un objeto desde una función?

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)); 

Me aparece un error, "tomando la dirección de temporal". Debería usar de esta manera:

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

Answer 1

getCar devuelve un Car por valor, lo cual es correcto.

No puede pasar ese valor de retorno, que es un objeto temporal, en displayCar, porque displayCar toma Car&. No puede vincular una referencia temporal a una referencia no constante. Debe cambiar displayCar para tomar una referencia constante:

void displayCar(const Car& car) { } 

O bien, puede almacenar el temporal en una variable local:

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

embargo, es mejor tener displayCar tomar una referencia constante, ya que doesn no modifica el objeto

No devolver una referencia a la variable local Car.

Answer 2

No es seguro devolver la referencia de una variable local desde una función.

Así que sí, esto es correcto:

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

Answer 3

Su problema es:

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

se debe utilizar una referencia constante:

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

Esta función:

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

está bien tal como está, pero todo lo que está haciendo es lo que hace el constructor, por lo que es redundante. El paso de un valor de nuevo, en lugar de una referencia, es lo que hay que hacer para este tipo de la función, sin embargo, los parámetros del modelo se debe pasar por referencia const:

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

En general, para los tipos de clases como cadena o automóvil, su elección predeterminada para un parámetro siempre debe ser una referencia constante.

Answer 4

Aún mejor:

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

Answer 5

Sí, es definitivamente no es seguro para devolver una referencia o un puntero a un objeto temporal. Cuando caduque (es decir, cuando finalice la función getCar), se quedará con lo que se conoce como "puntero colgante".

Sin embargo, si está interesado en reducir las operaciones de copia en un objeto, debe verificar C++ 0x "Mover semántica". Es un concepto relativamente nuevo, pero estoy seguro de que se convertirá en main-stream lo suficientemente pronto. GCC 4.4 en adelante admite C++ 0x (use la opción del compilador -std=c++0x para habilitar).