Skip to main content
Commonmark migration
Origen Enlace

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

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:

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:

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:

se añadieron 1746 caracteres en el cuerpo
Origen Enlace
PaperBirdMaster
  • 48.1k
  • 6
  • 45
  • 86

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é.

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

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é.

Origen Enlace
PaperBirdMaster
  • 48.1k
  • 6
  • 45
  • 86

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