Esa función confía en que el dato pasado a std::cout
tenga sobrecargado el operador de inyección de datos (operator <<
).
Es muy fácil de sobrecargar dicho operador:
std::ostream &operator<<(std::ostream &o, const Libro &l)
{
return o << l.nombre << ' ' << l.codigo << ' ' << l.disp;
}
El flujo de datos es un flujo de salida genérico (std::ostream
) por lo que vas a poder enviar datos de tipo Libro
a cualquier flujo de salida (std::cout
o un archivo, por ejemplo).
###Otras cosas a tener en cuenta.
Tu función de muestra de libros es semánticamente incorrecta; dado que pretendes mostrar los datos de un contenedor, se asume que no tienes intención de modificar dicho contenedor, así pues el parámetro debería ser constante:
void mostrarlibros( const list<Libro> & sList )
// ~~~~~ <--- La lista es de solo lectura
Por otro lado, los bucles for
y while
son fácilmente intercambiables, así que sería indiferente el uso de uno u otro; pero en general se sigue una guía para saber cuál escoger:
- Si se sabe cuántas iteraciones serán necesarias: se usa el bucle
for
.
- Si no se sabe las iteraciones que serán necesarias: se usa el bucle
while
.
En tu caso, lo que cabría esperar es un bucle for
:
void mostrarlibros( const list<Libro> & sList )
// ~~~~~ <--- La lista es de solo lectura
{
for (auto b = sList.begin(), e = sList.end(); b != e; ++b)
// ~~~~~ ~~~
// iteramos tantas veces como elementos existan entre begin y end
{
cout<<*b<<endl;
}
}
Pero este tipo de bucles son farragosos e incómodos, lo habitual cuando se recorren colecciones de datos es usar el bucle for
de rango:
void mostrarlibros( const list<Libro> & sList )
// ~~~~~ <--- La lista es de solo lectura
{
for (const auto &libro : sList)
// ~~~~~~~~~~~~~ <--- por cada Libro en sList...
{
cout<<libro<<'\n';
}
}
He substituido el std::endl
por '\n'
ya que suele ser mejor idea, consulta este hilo para saber por qué.