Skip to main content
se eliminaron 2 caracteres en el cuerpo
Origen Enlace
eferion
  • 52.6k
  • 5
  • 35
  • 75

Tienes dos errores básicamente:

Una división de enteros no genera decimales

int cal1, cal2, cal3, cal4, cal5;
float prom = (cal1 + cal2 + cal3 + cal4 + cal5) / 5;
             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~   ~
                          int                     int

Por mucho que prom sea float, la división es int / int, y esta división genera un resultado de tipo int (es decir, no tiene decimales), que posteriormente se recalcula a float. El problema de la conversión a float es que el resultado original ya ha perdido los decimales, luego prom tendrá problemas de precisión.

Este problema es sencillo de solucionar. Basta con hacer que uno de los dos términos de la operación sea de tipo float

float prom = (cal1 + cal2 + cal3 + cal4 + cal5) / 5.f;

De esta forma, el procesador realizará la operación float / float, operación que sí que generará decimales.

printf no necesita referencias

printf ("El promedio es: %f", & prom);

scanf necesita referencias a las variables porque para su correcto funcionamiento tiene que modificar dichas variables. printf, en cambio, no necesita modificar nada, luego le basta con recibir una copia de los valores a mostrar.

El uso correcto de printfsería:

printf ("El promedio es: %f", & prom);

En cualquier caso, estás programando en C++, así que sería más correcto y seguro usar los mecanismos de entrada/salida propios de C++:

#include <iostream>

int main()
{
    std::cout << "Ingrese la primera calificacion:";
    std::cin >> cal1;
    std::cout << "Ingrese la segunda calificacion:";
    std::cin >> cal2;
    // ...
}

Tienes dos errores básicamente:

Una división de enteros no genera decimales

int cal1, cal2, cal3, cal4, cal5;
float prom = (cal1 + cal2 + cal3 + cal4 + cal5) / 5;
             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~   ~
                          int                     int

Por mucho que prom sea float, la división es int / int, y esta división genera un resultado de tipo int (es decir, no tiene decimales), que posteriormente se recalcula a float. El problema de la conversión a float es que el resultado original ya ha perdido los decimales, luego prom tendrá problemas de precisión.

Este problema es sencillo de solucionar. Basta con hacer que uno de los dos términos de la operación sea de tipo float

float prom = (cal1 + cal2 + cal3 + cal4 + cal5) / 5.f;

De esta forma, el procesador realizará la operación float / float, operación que sí que generará decimales.

printf no necesita referencias

printf ("El promedio es: %f", & prom);

scanf necesita referencias a las variables porque para su correcto funcionamiento tiene que modificar dichas variables. printf, en cambio, no necesita modificar nada, luego le basta con recibir una copia de los valores a mostrar.

El uso correcto de printfsería:

printf ("El promedio es: %f", & prom);

En cualquier caso, estás programando en C++, así que sería más correcto y seguro usar los mecanismos de entrada/salida propios de C++:

#include <iostream>

int main()
{
    std::cout << "Ingrese la primera calificacion:";
    std::cin >> cal1;
    std::cout << "Ingrese la segunda calificacion:";
    std::cin >> cal2;
    // ...
}

Tienes dos errores básicamente:

Una división de enteros no genera decimales

int cal1, cal2, cal3, cal4, cal5;
float prom = (cal1 + cal2 + cal3 + cal4 + cal5) / 5;
             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~   ~
                          int                     int

Por mucho que prom sea float, la división es int / int, y esta división genera un resultado de tipo int (es decir, no tiene decimales), que posteriormente se recalcula a float. El problema de la conversión a float es que el resultado original ya ha perdido los decimales, luego prom tendrá problemas de precisión.

Este problema es sencillo de solucionar. Basta con hacer que uno de los dos términos de la operación sea de tipo float

float prom = (cal1 + cal2 + cal3 + cal4 + cal5) / 5.f;

De esta forma, el procesador realizará la operación float / float, operación que sí que generará decimales.

printf no necesita referencias

printf ("El promedio es: %f", & prom);

scanf necesita referencias a las variables porque para su correcto funcionamiento tiene que modificar dichas variables. printf, en cambio, no necesita modificar nada, luego le basta con recibir una copia de los valores a mostrar.

El uso correcto de printfsería:

printf ("El promedio es: %f", prom);

En cualquier caso, estás programando en C++, así que sería más correcto y seguro usar los mecanismos de entrada/salida propios de C++:

#include <iostream>

int main()
{
    std::cout << "Ingrese la primera calificacion:";
    std::cin >> cal1;
    std::cout << "Ingrese la segunda calificacion:";
    std::cin >> cal2;
    // ...
}
Origen Enlace
eferion
  • 52.6k
  • 5
  • 35
  • 75

Tienes dos errores básicamente:

Una división de enteros no genera decimales

int cal1, cal2, cal3, cal4, cal5;
float prom = (cal1 + cal2 + cal3 + cal4 + cal5) / 5;
             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~   ~
                          int                     int

Por mucho que prom sea float, la división es int / int, y esta división genera un resultado de tipo int (es decir, no tiene decimales), que posteriormente se recalcula a float. El problema de la conversión a float es que el resultado original ya ha perdido los decimales, luego prom tendrá problemas de precisión.

Este problema es sencillo de solucionar. Basta con hacer que uno de los dos términos de la operación sea de tipo float

float prom = (cal1 + cal2 + cal3 + cal4 + cal5) / 5.f;

De esta forma, el procesador realizará la operación float / float, operación que sí que generará decimales.

printf no necesita referencias

printf ("El promedio es: %f", & prom);

scanf necesita referencias a las variables porque para su correcto funcionamiento tiene que modificar dichas variables. printf, en cambio, no necesita modificar nada, luego le basta con recibir una copia de los valores a mostrar.

El uso correcto de printfsería:

printf ("El promedio es: %f", & prom);

En cualquier caso, estás programando en C++, así que sería más correcto y seguro usar los mecanismos de entrada/salida propios de C++:

#include <iostream>

int main()
{
    std::cout << "Ingrese la primera calificacion:";
    std::cin >> cal1;
    std::cout << "Ingrese la segunda calificacion:";
    std::cin >> cal2;
    // ...
}