Thread: Program thrown into loop when character entered instead of flaot

I am trying to get my program to check the validity of a entry. When the user enteres a character rather than a float, I want the program to ask them to check their entry and enter it again. I have tried the getchar(), and isdigit commands, but can't figure it out. I am completely new to this. Right now when the user enters a character rather than a float, it throws the program into a continuous loop.

Code:

#include <system.h>
#include <stdio.h>
#include <ctype.h>
main()
{

//   Variable declarations and initializations
	int iResponse = 0;
	float fDel_Mar_Tax = 7.25;
	float fEncinitas_Tax = 7.5;
	float fLa_Jolla_Tax = 7.75;
	float fPurchase_Amount = 0.0;
	
//   Program Header
	printf("\n\t\t             Tax Calculator 2 by Aleksandr Babich\n\n\n");

//   Enter purchase amount and verify purchase amount
	do { // do-while loop start
	printf("\n\nEnter the total purchase amount: $");
	scanf("%f", &fPurchase_Amount);	
	if (fPurchase_Amount <= 0.0)
		printf("\nInvalid entry, try again");
		printf("invalid");
	} //end do-while loop	
	while (fPurchase_Amount <= 0.0);
	printf("\n\nTotal purchase is $ %5.2f", fPurchase_Amount);
	

	do { //start of do-while loop

// Store selection
	printf("\n\n1\tDel Mar\n");
	printf("2\tEncinitas\n");
	printf("3\tLa Jolla\n");
	printf("\nPlease select your shopping location (1-3): ");
	scanf("%d", &iResponse);
	
// User Store selection validity
	if (iResponse <= 0 || iResponse >= 4) 
		printf("\n\nInvalid entry - Please try again\n\n");
	else
		printf("\n\n\t\t\t\t   TOTALS");
	


	
	switch (iResponse){ 

	case 1:
	printf("\n ------------------------------------------------------------------------------\n");
	printf("\n\t\tTotal purchase amoune is $%5.2f\n",fPurchase_Amount);
	printf("\n\t\tThe total tax on your purchaes is $%5.2f\n", fDel_Mar_Tax/100.0 * fPurchase_Amount);
	printf("\n\t\tYour total amount due is $%5.2f\n", fPurchase_Amount + (fDel_Mar_Tax/100.0 * fPurchase_Amount));
	printf("\n\t    Thank you for shopping at Kuddler Fine Foods Del Mar");
	printf("\n ------------------------------------------------------------------------------\n");
	break;
	case 2:
	printf("\n ------------------------------------------------------------------------------\n");
	printf("\n\t\tTotal purchase amoune is $%5.2f\n",fPurchase_Amount);
	printf("\n\t\tThe total tax on your purchase is $%5.2f\n", fEncinitas_Tax/100.0 * fPurchase_Amount);
	printf("\n\t\tYour total amount due is $%5.2f\n", fPurchase_Amount + (fEncinitas_Tax/100.0 * fPurchase_Amount));
	printf("\n\t    Thank you for shopping at Kuddler Fine Foods Encinitas");
	printf("\n ------------------------------------------------------------------------------\n");
	break;
	case 3:
	printf("\n ------------------------------------------------------------------------------\n");
	printf("\n\t\tTotal purchase amoune is $%5.2f\n",fPurchase_Amount);
	printf("\n\t\tThe total tax on your purchase is $%5.2f\n", fLa_Jolla_Tax/100.0 * fPurchase_Amount);
	printf("\n\t\tYour total amount due is $%5.2f\n", fPurchase_Amount + (fLa_Jolla_Tax/100.0 * fPurchase_Amount));
	printf("\n\t    Thank you for shopping at Kuddler Fine Foods La Jolla");
	printf("\n ------------------------------------------------------------------------------\n\n\n\n");
	
	} //end switch
	
	} while (iResponse <= 0 || iResponse >= 4); 


	
//Initiate a stop and wait for user input to continue
	printf("\nPress any key to continue...\n");
	getch();


}//end main

any ideas???

please ignore the printf("invalid") after the first printf("invalid entry, try again");

Maybe you could pick-up some idea with this code.

Code:

#include <ctype.h>
#include <stdio.h>
#include <conio.h>

int main()
{
    char input;

    while( 1 )
    {
        input = getch();
        if( isdigit( input ) )
          printf( "\n%c is a digit", input );
        else if( isalpha( input ) )
          printf( "\n%c is an alpha", input );
        else if( 27 == ( int ) input ) //press escape to exit
          break;
    }

    return 0;
}

Thanks,

I'll play with it, I entered it in to start and it still throws the program in a continuous loop if a character is entered instead of a number. I'll play with your suggestions. Thanks.

An alternative to scanf() would be to use a combination of fgets() and strtod(), as in

Code:

char line[BUFSIZ];
double fPurchase_Amount = 0.0;
fgets(line, sizeof(line)-1, stdin);
fPurchase_Amount = strtod(line, NULL);
...

Caveat with this approach is that input errors like 123q will be parsed correctly so that fPurchase_Amount has 123.0 in it.

char line[BUFSIZ];
double fPurchase_Amount = 0.0;
fgets(line, sizeof(line)-1, stdin);
fPurchase_Amount = strtod(line, NULL);
...

I'm new to the whole C programming, can you explain this code line by line?

Thanks

Code:

char line[BUFSIZ];  // Creates a character array BUFSIZE characters long
double fPurchase_Amount = 0.0; // Creates a float variable and initializes to 0.0
fgets(line, sizeof(line)-1, stdin);  // sizeof(line) - 1 is the number of characters that can fit into line.  So fgets gets that many characters from stdin (the keyboard) and puts them into the variable line
fPurchase_Amount = strtod(line, NULL); // converts the character array line into a double value and assigns this value to fPurchase_Amount

Maybe I'm overcomplicating the problem. I simply only want to allow numric entries whre I ask the user to enter something. Does "C" have an easy way of doing this?

Thanks,

When I woke up this morning, thats what I was thinking (isdigit). Then I logged on here and saw your comment, I entered it and it works well. Thanks.

Can you impose a time limit on a command like getchar() ???

I got the program to stop and not loop if the user enters a character rather than a number, but now the program stops and waits for the user to hit a key when they enter a number.

Code:

//   Enter purchase amount and verify purchase amount
	do { // do-while loop start
		printf("\n\nEnter the total purchase amount: $");
		scanf("%f", &fPurchase_Amount);
	if (fPurchase_Amount <= 0.0)
		printf("\nYou did not enter a valid number, please try again");
	if (isdigit(fPurchase_Amount));
	else
		getchar();
	} //end do-while loop	
	while (fPurchase_Amount <= 0.0);

Originally Posted by laserlight

That said, you can check for such an input error by passing a non-null pointer as the second argument to strtod() and then checking to see that it points to the first element of an empty string after the conversion.

Yep! the second (non-NULL) argument to strtod() can be used to check if there are any unwanted characters left behind.
The caveat was relevant only to the code snippet I had provided but the o/p will do well to use a non-NULL 2nd argument.