Thread: loop not terminating

I've got a program where the user can add items to an array. All the sections of the array are error checked to make sure the user inputs numbers or letters when they are supposed to.

The problem I am having is with one field, the others are error checked and working fine. Its the price, it is defined as a float and read into the array differently so I am unsure where the problem is. Here is basically what I'm having trouble with:

Code:

     float price;
     char tempstr[25];

int check(char tempstr[25])
{
     int j, c=1;
     j = atoi(tempstr);

     if (j==0)
        {
           c = 0;
            }

     return c;
}


int main()
{
     do{
           cout << ("Enter Price: ")
           cin >> tempstr;
 
           if (check(tempstr)==0)
               {
                  cout << "not valid";
                    }
         }while (check(tempstr) !=1);
     
     scanf("%f",&slist[numitems].price);
}

With this code just now, it can detect when letters are entered and returns "Not Valid", but when numbers are entered, the line moves down and then nothing happens, it does not move onto the next section of code. Any idea?

First, your code contains errors.
There's a missing semicolon on the cout >> Price line. And why on earth do you have the text "Enter Price: " in parentheses?
There's also the issue of your check() function. Its local variable has the same name as a global one. (A global which should probably be local to main, and passed to check().)

I'm a little confused as to what your issue is... the code you posted will remain in the while loop until the user enters a number. (Except 0... and many people recommend using strtol() over atoi())

the mistakes in my code do not exist in my actual program. I hand typed that because it was easier that way to cut stuff down than to copy and paste it. The text "Enter Price: " is not in brackets in my code, but it doesn't appear to make a difference if it is or isn't. I don't see what you mean about my variables either.

My problem is that the program does not continue to do what its supposed to do. When I enter a number for the price and hit enter, it moves onto the next line and does nothing, whereas it should usually display another screen. If i remove the cin, do..while and if loop, it works fine.

I've just found out that it requires me to enter 2 sets of numbers before it continues to the next screen. Enter price comes up, say I type 57 and hit enter, it moves onto the next line and I have to hit another number, say 2, and then it goes onto the next screen like it should and saves the 57 to the array. I am stuck as to why its asking for the second number

I think I see what you're saying now.

You can do what you've been doing, but for floats, use the function atof() instead of atoi().

Do note the note that says that atof() (and atoi() ) do not detect errors, like the user entering in "foobar" for a price.