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`](https://en.cppreference.com/w/cpp/io/basic_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](https://es.stackoverflow.com/q/47839/2742) para saber por qué.