Hay varios errores en tu codigo, los iremos analizando paso a paso:
- No utilices gets: El uso de gets esta descontinuado en C, se recomienda utilizar mejor
printf-scanf
para impresion y captura de datos.
- El problema tuyo:
fflush
sirve para borrar el flujo saliente, pero el entrante no, segun lo que encontre aqui. Como reemplazo, se puede utilizar while(getchar()!='\n');
para borrar el flujo, se utiliza despues del scanf
para eliminar el flujo de datos de entrada.
Falla en tu logica: En el codigo que tienes tiene cosas incoherentes. Vamos por partes. La primera es la condicion nota[x] <=0 || nota[x] >= 70
. Si bien esta bien, lo mejor es invertir las condiciones, de tal manera que cuando la note este en el rango encontrado ingrese la nota en la lista, de lo contrario tendria que repetir el bloque de codigo pidiendo de nuevo la nota. Si la nota esta bien el segundo ciclo se rompe pudiendo ingresar la nota al siguiente estudiante. El codigo quedaria de la siguiente manera:
for(x=0;x<10;x++){
do{
printf("Ingrese nota de %s: ", nom[x]);
float nota_temp = 0;
scanf("%f" , ¬a_temp);
//while(getchar()!='\n');
if (nota[x] >= 0 || nota[x] <= 70){
nota[x] = nota_temp;
break;
} else{
printf("Nota mal ingresada, vuelva a introducir\n");
}
}while(true);
}
Para que funcione corerctamente debes de agregar en la cabecera de tu proyecto las siguientes macros:
#define false 0
#define true 1
- La logica de encontrar el menor esta mal: Estas metiendo en la misma logica el codigo de encontrar el menor y el mayor, cosa que no es correcta dentro de la logica. Para encontrar el mayor necesitabamos un numero lo menor posible. Como tienes un rango de notas entre 0 a 70, lo recomendable es iniciar
mayor
con el valor -1
. Para el menor es al contrario, toca iniciarlo con el numero mayor posible. Como encontramos anteriormente el mayor, podemos hacer menor = mayor
.
int x;
float nota [10], promedio = 0;
char nom [10][50];
float mayor = -1 , menor = -1, total;
mayor= nota[0];
mayor= nota[0];
for (x=0; x<10; x++){
if (nota[x]> mayor){
mayor=nota[x];
}
}
menor = mayor;
for (x=0; x<10; x++){
promedio+=nota[x];
}
for (int i = 0; i < 10; i++){
if (nota[i] < menor){
menor = nota[i];
}
}
En conclusion, el error principal, que responde a la pregunta es: Te saltaba el error debido a que fflush no borra el flujo de entrada que queda, haciendo que se salte todas las instrucciones debido a los caracteres basura que quedan.
El codigo, en su totalidad queda de la siguiente manera:
#include<stdio.h>
#include<stdlib.h>
#define false 0
#define true 1 // Option 1
int main()
{
int x;
float nota [10], promedio = 0;
char nom [10][50];
float mayor = -1 , menor = -1, total;
mayor= nota[0];
for(x=0;x<10;x++){
printf("Ingrese nombre de alumno n° %i: " , x+1);
scanf("%s", &nom[x]);
while(getchar()!='\n');
}
printf("\n");
for(x=0;x<10;x++){
do{
printf("Ingrese nota de %s: ", nom[x]);
float nota_temp = 0;
scanf("%f" , ¬a_temp);
while(getchar()!='\n');
if (nota[x] >= 0 || nota[x] <= 70){
nota[x] = nota_temp;
break;
} else{
printf("Nota mal ingresada, vuelva a introducir\n");
}
}while(true);
}
for(int i = 0; i < 10; i++){
printf("%f \n", nota[i]);
}
mayor= nota[0];
for (x=0; x<10; x++){
if (nota[x]> mayor){
mayor=nota[x];
}
}
menor = mayor;
for (x=0; x<10; x++){
promedio+=nota[x];
}
for (int i = 0; i < 10; i++){
if (nota[i] < menor){
menor = nota[i];
}
}
printf("\nEl mayor es %f\n\n", mayor);
printf("El menor es %f\n\n", menor);
promedio = promedio/10.0;
printf("El Promedio es: %.2f\n\n", promedio);
system("PAUSE");
return 1;
}
ACTUALIZACION: He actualizado el codigo para que pueda aceptar cadenas de string con especios en la entrada de flujo de la consola, esto li hice utilizando la funcion fgets
.
#include<stdio.h>
#include<stdlib.h>
#define false 0
#define true 1 // Option 1
int main()
{
int x;
float nota [10], promedio = 0;
char nom [10][50];
float mayor = -1 , menor = -1, total;
mayor= nota[0];
for(x=0;x<10;x++){
printf("Ingrese nombre de alumno n° %i: " , x+1);
fgets(nom[x], 100, stdin);
while(getchar()!='\n');
}
printf("\n");
for(x=0;x<10;x++){
do{
printf("Ingrese nota de %s: ", nom[x]);
float nota_temp = 0;
scanf("%f" , ¬a_temp);
while(getchar()!='\n');
if (nota[x] >= 0 || nota[x] <= 70){
nota[x] = nota_temp;
break;
} else{
printf("Nota mal ingresada, vuelva a introducir\n");
}
}while(true);
}
for(int i = 0; i < 10; i++){
printf("%f \n", nota[i]);
}
mayor= nota[0];
for (x=0; x<10; x++){
if (nota[x]> mayor){
mayor=nota[x];
}
}
menor = mayor;
for (x=0; x<10; x++){
promedio+=nota[x];
}
for (int i = 0; i < 10; i++){
if (nota[i] < menor){
menor = nota[i];
}
}
printf("\nEl mayor es %f\n\n", mayor);
printf("El menor es %f\n\n", menor);
promedio = promedio/10.0;
printf("El Promedio es: %.2f\n\n", promedio);
system("PAUSE");
return 1;
}
Adicional a esto, agrego un compilador y editor de codigo de C, el enlace lo puedes encontrar aqui