Skip to main content
2 de 2
Commonmark migration

Tengo un atributo en mi clase libro llamado author.

Author & author;

Ya empezamos mal. No tienes un atributo sino una referencia... fíjate en ese &. Las referencias pueden ser utilizadas como miembros de una clase, pero deben inicializarse sí o sí en la parte de inicialización del constructor:

Book::Book(const string & name, const Author & auth, double price, int    qtyInStock)
  : author(author) // <<--- AQUI
{
  this->name=name;
  //this->author=author; <<--- esto sobra
  setPrice(price);
  setQtyInStock(qtyInStock);
}

Ahora bien, una referencia no implica una copia de objetos, luego ese constructor tiene un problema y es que estás intentando asignar una referencia constante a una referencia que no es constante... luego el programa va a seguir sin compilar...

Lo normal en estos casos es que author deje de ser una referencia... así en vez de copiar una referencia se realizará una copia del objeto y el programa compilará sin problemas:

class Book
{
  Author author; // sin &
};

NO QUIERO RESOLVERLO INICIALIZANDO LOS PARAMETROS EN LA LISTA DEL CONSTRUCTOR.

Si quitas la referencia podrás inicializarlo donde te de la gana (siempre y cuando Author implemente de forma implícita o explícita el constructor por defecto)... no obstante piensa que inicializar los miembros dentro de las llaves del constructor tiene cierto impacto en el código ya que primero se va a llamar al constructor por defecto de cada miembro y después al operador de asignación... en vez de únicamente al constructor copia... es un dos por uno.

eferion
  • 52.6k
  • 5
  • 35
  • 75