Skip to main content
2 de 2
se añadieron 391 caracteres en el cuerpo
MrDave1999
  • 7.6k
  • 1
  • 11
  • 22

El problema radica en esta parte del código:

ask:
    printf("Ingrese un valor numerico para calcular la raiz de A\n");
    awnser = scanf("%f", &A);
if( awnser !=1 || awnser == EOF){ // not a number
    printf("Por favor introduce un dato numerico\n");
    goto ask;
}

Por ejemplo, si el usuario ingresa Hola Mundo, la función scanf retornará 0 porqué hubo un error, entonces como el búfer de entrada estándar quedo sucio, la próxima vez que se vuelva a pedir datos al usuario, scanf no se detendrá, porqué encontró caracteres que no son válidos para un valor decimal. La solución es limpiar el búfer después del scanf.

    ask:
    printf("Ingrese un valor numerico para calcular la raiz de A\n");
    awnser = scanf("%f", &A);
    //Limpiamos el búfer hasta encontrar un caracter de nueva línea o hasta hallar el final del archivo (EOF).
    while((ch = getchar()) != '\n' && ch != EOF);
    if( awnser !=1 || awnser == EOF){ // not a number
        printf("Por favor introduce un dato numerico\n");
        goto ask;
    }

Hay una librería llamada PScanf que te permite validar los datos que el usuario ingrese. Tu código con PScanf sería así:

#include <stdio.h>
#include <pscanf.h>
int main()
{
    float A, K, i, aux;
    dataread("%f", &A, "Ingrese un valor numerico para calcular la raiz de A\n");
    if(A < 0 ){
        aux = -A;
        i = sqrt(aux);
        if(A == (int)A) printf("La raiz cuadrada de %.0f, es %.0fi", A, i);
        else printf("La raiz cuadrada de %.3f, es %.4fi", A, i);
    }else{
        K = sqrt(A);
        printf("La raiz cuadrada de A es %.2f", K);
    }
    return 0;
}

Con la macro dataread te valida todo automáticamente, no necesitas agregar ningún bucle adicional, ni limpiar el búfer del teclado.

Enlace de la librería:

Library-PScanf

EDITO:

Este código:

while((ch = getchar()) != '\n' && ch != EOF);

Es equivalente a:

while(1)
{
   ch = getchar();
   if(ch == '\n' || ch == EOF)
   {
        break;
   }
}

La diferencia es que el primero es un while sin implementación (que no tiene ningún código entre llaves), así te ahorras unas cuantas líneas y el segundo, si tiene una implementación.

MrDave1999
  • 7.6k
  • 1
  • 11
  • 22