Skip to main content
Commonmark migration
Origen Enlace

Problema.

###Problema. NosNos encontramos ante un claro ejemplo de los problemas que pueden surgir al tener un código mal organizado. En tu caso, la función main tiene tanta información que hace que sea más complicado de lo necesario analizarla.

El problema parece estar ya resuelto por las pesquisas de gbianchi y federhico pero yo quisiera compartir una aproximación adicional. ###Propuesta. Te

Propuesta.

Te aconsejo seguir el Principio de responsabilidad Única: separa el pintado de cabeceras de la obtención de datos y la presentación de los mismos.

###Problema. Nos encontramos ante un claro ejemplo de los problemas que pueden surgir al tener un código mal organizado. En tu caso, la función main tiene tanta información que hace que sea más complicado de lo necesario analizarla.

El problema parece estar ya resuelto por las pesquisas de gbianchi y federhico pero yo quisiera compartir una aproximación adicional. ###Propuesta. Te aconsejo seguir el Principio de responsabilidad Única: separa el pintado de cabeceras de la obtención de datos y la presentación de los mismos.

Problema.

Nos encontramos ante un claro ejemplo de los problemas que pueden surgir al tener un código mal organizado. En tu caso, la función main tiene tanta información que hace que sea más complicado de lo necesario analizarla.

El problema parece estar ya resuelto por las pesquisas de gbianchi y federhico pero yo quisiera compartir una aproximación adicional.

Propuesta.

Te aconsejo seguir el Principio de responsabilidad Única: separa el pintado de cabeceras de la obtención de datos y la presentación de los mismos.

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

###Problema. Nos encontramos ante un claro ejemplo de los problemas que pueden surgir al tener un código mal organizado. En tu caso, la función main tiene tanta información que hace que sea más complicado de lo necesario analizarla.

El problema parece estar ya resuelto por las pesquisas de gbianchi y federhico pero yo quisiera compartir una aproximación adicional. ###Propuesta. Te aconsejo seguir el Principio de responsabilidad Única: separa el pintado de cabeceras de la obtención de datos y la presentación de los mismos.

Para ello empezaremos por definir una estructura de datos que contenga la información del trabajador:

struct Trabajador
{
    std::string nombre;
    int cedula;
};

He escogido std::string como tipo para almacenar el nombre del trabajador porque de esta manera evitamos que tenga una longitud fija; en tu caso estabas limitando el nombre del trabajador a 9 caracteres (8 si contamos el carácter nulo de finalización de cadena) así que un trabajador llamado "Teodoredo" ya no te cabría1.

Seguimos con una función que, dado un trabajador, lea sus datos:

void leer_datos(Trabajador &trabajador)
{
    std::cout << "\n\nIngrese el Nombre del Trabajador: ";
    std::getline(std::cin, trabajador.nombre);
    std::cout << "\nCedula del Trabajador: ";
    std:: cin >> trabajador.cedula;
}

A continuación, una estructura que almacene la producción de una semana y nos devuelva datos sobre la misma:

struct Semana
{
    float produccion[5u]{};

    float forros() const
    {
        return std::accumulate(std::begin(produccion), std::end(produccion), 0.f);
    }

    float total_a_pagar(float sueldo, float bono) const
    {
        float resultado = 0.f;
        for (const auto &produccion_dia : produccion)
        {
            resultado += (sueldo + (produccion_dia < 80.f ? 0.f : bono));
        }
        return resultado;
    }
};

He asignado la responsabilidad de calcular los salarios en base a datos de la semana a la propia semana, pero puedes escoger otra aproximación.

Con esta estructura podemos crear una función que, dada una semana, lea sus datos:

void leer_datos(Semana &semana)
{
    for (auto dia = 0u; dia < 5u; ++dia)
    {
        std::cout << "\nCantidad de Forros Producidos en el Dia: " << (dia + 1) << ". "; 
        std::cin >> semana.produccion[dia];
    }
}

Para acabar, parece que necesitamos una estructura Recibo:

struct Recibo
{
    Trabajador trabajador;
    Semana periodo[5u];
};

Que podemos leer con esta función:

void leer_datos(Recibo &recibo)
{
    leer_datos(recibo.trabajador);
    for (auto semana = 0u; semana < 5u; ++semana)
    {
        std::cout << "\n\nSemana " << (semana + 1)<< ".\n";
        leer_datos(recibo.periodo[semana]);
    }
}

De esta manera puedes tener un main tan limpio como este:

int main()
{
    Recibo recibos[9u];

    std:: cout << "< = = = = = F A B R I C A de F O R R O S para V E H I C U L O S = = = = = >";

    for (auto &recibo : recibos)
        leer_datos(recibo);

    std::cout << "\n\n< = = = = = F A B R I C A de F O R R O S para V E H I C U L O S = = = = = >";
    std::cout << "\n< = = R E C I B O  S E M A N A L = = >";

    for (const auto &recibo : recibos)
    {
        std::cout << "\n\nTrabajador: " << recibo.trabajador.nombre
                  << "\nCedula: "       << recibo.trabajador.cedula;

        for (auto semana = 0u; semana < 5u; ++semana)
        {
            std::cout << "\nForros Producidos en la Semana " << (semana + 1) << ": " << recibo.periodo[semana].forros()
                      << "\n\nTotal a Pagar: Bs. " << recibo.periodo[semana].total_a_pagar(15'000.f, 8'250.f);
        }
    }

    return 0;
}

1Y si añadimos los apellidos, nuestro amigo Teodoredo Atanagildo-Gundemaro de las Mercedes queda fuera del todo.