Resetting variables

Hey all :)

Ok the situation is that for a college assinemnt i am required to create an application that calculates whether or not the user is allowed credit based on three variables, age, income and hours of work.....now these are set as variables and i have several functions.

The thing I am wanting to do is set them to a Null length when the function has finished as at the moment the user cant enter variables for the first attempt, but when the second attempt comes it skips all of the while loops (while(variable==0)) which isnt what i want, im wanting it to request the variables again, these are char variables.

The reason for me using while loops is because my lecturer gave me the evils when using the goto command >.> any help would be appreciated :) thanks

PS - wasnt going to post all of code as it is all complete, just touching up and dont want to spam the forum with it, feel free to be technical......programming is my love and joy

But you're more than welcome to post your code snippet in [code] [/code] tags.

I'm a little confused by the description but it sounds like you should just set them to 0 at the end of your loop, as it sounds like your code might be within one. But I'm just guessing.

Quote:

---

Originally Posted by hirsty

...three variables, age, income and hours of work.....

...these are char variables.

---

Really? How do you stuff income into a char variable? (Must not be too much of an income)

ive tried resetting them using the end of the variabe last i remember, tried an inbuilt reset function, the way in which i set a value to them is using atoi(variable) = integer

Please provide code example.

Also explain what happens and what you would like to happen.

think i have just found my solution :) Thanks for offering to help to divide and conquer, on calling back to the Menu from the function I have used memset(variable,0,sizeof(variable))

unlesss there is an easier then feel free t oadvise currently this is my code for my menu and the other function:

void Menu()
{

int mchoice;
system("cls");

while(mchoice!=3)
{
memset(birthyr, 0, sizeof(birthyr)); // resets char variable to 0
memset(income, 0, sizeof(income));
memset(wrkhrs, 0, sizeof(wrkhrs));
choice='/0';

credit=0;
cout<<"Welcome to the Working Tax Credit Calculator. In order to process your enquiry"<<endl;
cout<<"further we need to know whether you are enquiring as a couple. Please choose "<<endl;
cout<<"from the following options:"<<endl<<endl;
cout<<"1. Enquiry for Singles"<<endl;
cout<<"2. Enquiry for Couples"<<endl;
cout<<"3. Exit"<<endl<<endl;
cout<<"Menu choice: ";
cin>>mchoice;
cout<<endl;
if (mchoice==1)
..........
else
cout<<"Please enter a valid menu choice"<<endl<<endl;


}
}


void SingleEnq()
{



while((atoi(birthyr)) == 0)
{
cout<<"Please enter your year of birth(YYYY): ";
cin>>birthyr;
if((atoi(birthyr))==0)
{
cout<<"Birth Year is not valid"<<endl;
cin.clear();
}
.....continues to check variables in same way, then asks if correct, if so carrys on to calculate.....
}

what was happening, on second entry of the SingleEnq() it was using the previous choices and i did not want that to happen

Why are you using char arrays to enter numeric data?

Instead of memsetting your arrays, you could change your while to a do/while.

Verifying the year of birth using atoi isn't particularly robust. 99934CD will convert to a non-zero value but it isn't a valid year of birth.

Quote:

---

Originally Posted by GCDEF

Why are you using char arrays to enter numeric data?

Instead of memsetting your arrays, you could change your while to a do/while.

Verifying the year of birth using atoi isn't particularly robust. 99934CD will convert to a non-zero value but it isn't a valid year of birth.

---

Well, I am a college student and basically googled how to do it as i was wanting to do nnumeric calculations on it as well as display it to the screen, ive done VB but only just started coding C++ so yea found this and applied it as like a qucik solution...-shrugs-

Quote:

---

Originally Posted by hirsty

Well, I am a college student and basically googled how to do it as i was wanting to do nnumeric calculations on it as well as display it to the screen, ive done VB but only just started coding C++ so yea found this and applied it as like a qucik solution...-shrugs-

---

If you're working with numeric data, you should use numeric data types.

Quote:

---

Originally Posted by GCDEF

If you're working with numeric data, you should use numeric data types.

---

how would i then detect if text has been input and how would i limit the amount of characters??? (wanting to make fool proof)

Quote:

---

Originally Posted by hirsty

how would i then detect if text has been input and how would i limit the amount of characters??? (wanting to make fool proof)

---

The std iostreams come with operator<< and operator>>, which basically take the input, and convert it to the corresponding numeric tipe, like this:

Code:

---

float a;
cin >> a;

---

This will even work if your input is something like 0.0f, or an hex integer, provided you told your stream you are reading an hex (it can't guess for you)

As for checking for text, if the user types "apple", then the read will "fail" (this is normal), and you will be able to tell your user to try again.

Why would you want to limit the amount of characters? If your user needs 278 characters to input his value, who are you to say it is wrong? And if it is, you should just handle the fault, then tell your user to try again.

Once you are done reading and validating user input, you are 100% guaranteed to have perfectly valid input. It is part of standard methods to never do anything before you have validated the objects you are working with. You can code away without worries :)

std iostream s are standard, expandable, error tolerant, and safe. atoi is non of those things.

Quote:

---

Originally Posted by hirsty

how would i then detect if text has been input and how would i limit the amount of characters??? (wanting to make fool proof)

---

Typically that would be done something like this....

Code:

---

string strval;
int val;
cin >> strval;
istringstream valstream(strval);
if (!(valstream >> val))
    cout << "bad input";

---

The problem with simply testing "cin >> val" directly is that if it fails, the bad input will remain in the stream, and you'll continue to fail until you either .ignore() it or you try to read something from cin which is not an integer.

(Using cin.ignore() is another option.)

I wouldn't use operator >>.

Learn how to grab input 1 line at a time as a string, and then parse from the string what you need using stringstreams. This is the C++ version of the old fgets/sscanf combo used by C programmers. It sounds a lot more complicated than it is and there are many examples on these boards. A simple search will throw up a few results.

This is such a valuable technique that its well worth learning earlier rather than later imo. It also avoids nicely the problem of mixing calls to getline and operator >> which is fraught with trouble.

Quote:

---

Originally Posted by monarch_dodra

The std iostreams come with operator<< and operator>>, which basically take the input, and convert it to the corresponding numeric tipe, like this:

Code:

---

float a;
cin >> a;

---

This will even work if your input is something like 0.0f, or an hex integer, provided you told your stream you are reading an hex (it can't guess for you)

As for checking for text, if the user types "apple", then the read will "fail" (this is normal), and you will be able to tell your user to try again.

Why would you want to limit the amount of characters? If your user needs 278 characters to input his value, who are you to say it is wrong? And if it is, you should just handle the fault, then tell your user to try again.

Once you are done reading and validating user input, you are 100% guaranteed to have perfectly valid input. It is part of standard methods to never do anything before you have validated the objects you are working with. You can code away without worries :)

std iostream s are standard, expandable, error tolerant, and safe. atoi is non of those things.

---

wanting to limit characters for things such as Date of birth and some cant have an incom of £555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555 etc

Quote:

---

Originally Posted by hirsty

wanting to limit characters for things such as Date of birth and some cant have an incom of £555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555 etc

---

How about an income of £00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000005?

or that lol

Quote:

---

Originally Posted by hirsty

wanting to limit characters for things such as Date of birth and some cant have an incom of £555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555555 etc

---

Yes, but that is not an input problem, it is a data validity problem.

Don't tell your user "You have entered an amount of dollars which is written with 52 chars, and that is too long" but tell him "cannot process orders of more than 1 million dollars"

The user input is perfectly legal. Then it is you responsibility to make sure the the input is relevant. The difference is subtle, but important.

On a side note, if you create a "date" class, you can then write "cin >> myDate", if you redefine the operator>>. Then, you can manually define that operator to reject an input which is not a valid date, but valid numeric (like if the user enters 52/17/19997).

Quote:

---

Originally Posted by Russco

I wouldn't use operator >>.

Learn how to grab input 1 line at a time as a string, and then parse from the string what you need using stringstreams. This is the C++ version of the old fgets/sscanf combo used by C programmers. It sounds a lot more complicated than it is and there are many examples on these boards. A simple search will throw up a few results.

This is such a valuable technique that its well worth learning earlier rather than later imo. It also avoids nicely the problem of mixing calls to getline and operator >> which is fraught with trouble.

---

I completely agree with this method, and have used it severall times, but isn't it essentially the same thing, with more safety? You say "parse from the string what you need using stringstreams", aren't you just going to use operator>> on your stringstream?

I was trying to explain that using operator>> is usually safer than combining string search and calls to things like atoi. Calling it on cin directly, or on a stringstream built from a getlined string is essentially the same thing, IMO.

Just seems weird that you would advise against calling operator>>. Did you mean "calling operator>> directly on cin"?

lol anyways thanks guys, i guess its working atm so going to leave it be lol as i said :) thanks

Yes of course I meant not to call operator >> on cin. I find it much more preferable to get all input as a string and parse accordingly. It certainly makes error checking easier and cin is always in a usable state. I know some favour checking the flags, then ignoring chaff left in the buffer and dealing with mixing of operator >> and getline and clearing error flags, but for me thats just too much hassle when there is a simpler typesafe alternative available. With templates this can almost become a write once and reuse for the rest of your coding life option. Those templates would deal with the general cases and you would only need to supplement the code rarely for more special situations.

You dont always use operator >> on stringstreams to extract what you want but its the most common option. The templates mentioned above would probably use it, mine do.

Quote:

---

Originally Posted by hirsty

think i have just found my solution :) Thanks for offering to help to divide and conquer, on calling back to the Menu from the function I have used memset(variable,0,sizeof(variable))

---

For POD integer types (char, short, long, int, etc), there's basically no difference between

Code:

---

memset(var, 0, sizeof(var));

---

and

Code:

---

var = 0;

---

I note that Lindley's example would not regard input such as "123hello" as bad input. If you want to detect such cases of valid followed by invalid input, then you should change it to:

Code:

---

if (!(valstream >> val) || !valstream.eof())
    cout << "bad input";

---

Quote:

---

Originally Posted by laserlight

I note that Lindley's example would not regard input such as "123hello" as bad input. If you want to detect such cases of valid followed by invalid input, then you should change it to:

Code:

---

if (!(valstream >> val) || !valstream.eof())
    cout << "bad input";

---

---

Wouldn't that also not work? I thought eof was set to true only once a read past eof is attempted?

I might be wrong, but wouldn't the code be something like this? where we create a dummy string to do one last read before checking for EOF?

Code:

---

string isEOFString;
if (!(valstream >> val) || (valstream >> isEOFString, !valstream.eof()))
    cout << "bad input";

---

Quote:

---

Originally Posted by monarch_dodra

Wouldn't that also not work? I thought eof was set to true only once a read past eof is attempted?

---

Yes, but for reasons that I have not fully researched into, the behaviour is slightly different for stringstreams.

ack and it has just gone to pot again >< mk here is the whole code.......wanting to complete it as soon as possible as my assignment is due in knowing me probs made it more long winded than it should be......

Code:

---

//This program shows the person whether or not they are eligible for Working Tax credit
//based on their income how many hours they work and how old they are.
//If a couple is checkin whether they are eligible it will combine both of their income.
//After checking if eligible, it will then give them an estimated value.
//Created on 23rd November 2009 by Daniel Hirst
#include <iostream>
#include <conio.h>
#include <time.h> // Allows detection of current time
#include <stdio.h>
#include <stdlib.h>

using namespace std;
//Variable Declarations
int age;
char birthyr[4];
char income[7];
char wrkhrs[2];
char choice;
int credit; //Working Tax Credit
int curryr; //Current Year



//functions

void Menu();

void SingleEnq()
{
       

       
        while((atoi(birthyr)) == 0)
        {
                cout<<"Please enter your year of birth(YYYY): ";
                cin>>birthyr;
                if((atoi(birthyr))==0)
                {
                        cout<<"Birth Year is not valid"<<endl;
                        (atoi(birthyr)) == 0;
                }
                       
        }
        while((atoi(income)) == 0)
        {
       
                cout<<"Please enter your yearly Income: "<<char(156);
                cin>>income;
                if((atoi(income))==0)
                {
                        cout<<"Yearly Income is not valid"<<endl;
                        (atoi(income))==0;
                }
        }
        while((atoi(wrkhrs)) == 0)
        {
                cout<<"Please enter how many hours you work in a week: ";
                cin>>wrkhrs;
                if((atoi(wrkhrs))==0)
                {
                        cout<<"Work Hours is not valid"<<endl;
                        (atoi(wrkhrs)) == 0;
               
                }
        }
       
        age = curryr - atoi(birthyr);
        cout<<"Your age is "<<age<<endl;
        cout<<"You work "<<wrkhrs<<" hours a week"<<endl;
        cout<<"Your yearly income is "<<char(156)<<income<<endl<<endl;

        while ((choice!='Y') && (choice!='y') && (choice!='N') && (choice!='n'))
        {
                cout<<"Is the above data correct? (Y/N): ";
                cin>>choice;
                cout<<endl;
                if ((choice!='Y') && (choice!='y') && (choice!='N') && (choice!='n'))
                {
                        cout<<"Choice not valid"<<endl;
                        cin.clear();
                }
                else if ((choice=='N') || (choice=='n'))
                {
                        memset(birthyr, 0, sizeof(birthyr)); // resets char variable to 0
                        memset(income, 0, sizeof(income));
                        memset(wrkhrs, 0, sizeof(wrkhrs));
                        credit=0;
                        choice==0;
                        system("cls");
                        return SingleEnq();
                }

                else if ((choice=='Y') || (choice=='y') )
                {
                        if((age>25) && (atoi(wrkhrs)>30) && (atoi(income)<11700))
                        {
                                if(atoi(income)<=8612)
                                {
                                        credit=1180;

                                }
                                else if((atoi(income)>8612) && (atoi(income)<=11700))
                                {                       
                                        credit=665;
                                }
                                cout<<"Using the data provided, we have found you are eligible "<<endl;
                                cout<<"for "<<char(156)<<credit << " of Working Tax Credit"<<endl<<endl;
                                cout<<endl;
                               

                        }       
                        else
                        {
                                cout<<"Sorry, you are not eligible for any working tax credit because: "<<endl;
                                if(age<25)cout<<"- You are under 25"<<endl;
                                if(atoi(wrkhrs)<30) cout<<"- You work less than 30 hours a week"<<endl;
                                if(atoi(income)>11700) cout<<"- You earn more than "<<char(156)<<"11700 yearly"<<endl;
                               
                        }
                }
       
        }
       
        getch();
        system("cls");
        choice==0;
        return;
       
}


void CoupleEnq()
{
        int page;
        char pbirthyr[5];
        char pincome[6];
        char pwrkhrs[3];
        int incometot;
        while((atoi(birthyr)) == 0)
        {
                cout<<"Please enter your year of birth(YYYY): ";
                cin>>birthyr;
                if((atoi(birthyr))==0)
                        {
                                cout<<"Birth year not valid"<<endl;
                        }
        }

                while((atoi(income)) == 0)
                {
                        cout<<"Please enter your yearly Income: "<<char(156);
                        cin>>income;
                        if((atoi(income))==0)
                                {
                                        cout<<"Yearly Income not valid"<<endl;
                                }
                }
                while((atoi(wrkhrs))==0)
                {
                        cout<<"Please enter how many hours you work in a week: ";
                        cin>>wrkhrs;
                        if((atoi(wrkhrs))==0)
                                {
                                        cout<<"Work Hours not valid"<<endl;
                                }
                }

                while((atoi(pbirthyr))==0)
                {
                        cout<<"Please enter your partners year of birth(YYYY): ";
                        cin>>pbirthyr;
                        if((atoi(pbirthyr))==0)
                                {
                                        cout<<"Partner Birth Year not valid"<<endl;       
                                }
                }

                while((atoi(pincome))==0)
                {
                        cout<<"Please enter your partners yearly Income: "<<char(156);
                        cin>>pincome;
                        if((atoi(pincome))==0)
                        {
                                cout<<"Partners Yearly Income not valid"<<endl;
                        }
                }

                while((atoi(pwrkhrs))==0)
                {
                        cout<<"Please enter how many hours your partner works in a week: ";
                        cin>>pwrkhrs;
                        if((atoi(pwrkhrs))==0)
                                {
                                        cout<<"Partners Work Hours not valid"<<endl;
                                }
                }

                age = curryr - (atoi(birthyr));
                page= curryr - (atoi(pbirthyr));
                incometot = (atoi(income))+ (atoi(pincome));
                cout<<"Your age is "<<age<<endl;
                cout<<"You work "<<wrkhrs<<" hours a week"<<endl;
                cout<<"Your yearly income is "<<char(156)<<income<<endl;
                cout<<"Your partners age is "<<page<<endl;
                cout<<"You partner works "<<pwrkhrs<<" hours a week"<<endl;
                cout<<"Your partners yearly income is "<<char(156)<<income<<endl;
                cout<<"Your combined income is "<<char(156)<<incometot<<endl;

                while ((choice!='Y') && (choice!='y') && (choice!='N') && (choice!='n'))
                {
                        cout<<"Is the above data correct? (Y/N): ";
                        cin>>choice;
                        cout<<endl;

                        if ((choice!='Y') && (choice!='y') && (choice!='N') && (choice!='n'))
                        {
                                cout<<"Choice not valid"<<endl;
                                cin.clear();
                        }

                        else if ((choice=='N') || (choice=='n'))
                        {
                                memset(birthyr, 0, sizeof(birthyr)); // resets char variable to 0
                                memset(income, 0, sizeof(income));
                                memset(wrkhrs, 0, sizeof(wrkhrs));
                                memset(pbirthyr, 0, sizeof(pbirthyr));
                                memset(income, 0, sizeof(pincome));
                                memset(wrkhrs, 0, sizeof(pwrkhrs));
                                memset(wrkhrs, 0, sizeof(incometot));
                                credit=0;
                                choice==0;
                                cout<<endl;
                                system("cls");
                                return CoupleEnq();
                        }               
                        else if ((choice=='Y') || (choice=='y'))
                        {
                                cout<<endl;
                                if (age>= 25 && atoi(wrkhrs)>=30 && incometot < 16300 && page>= 25 && atoi(pwrkhrs)>=30)
                                {
                                        if(incometot<=8612)
                                        {
                                                credit=2880;
                                        }
                                        else if((incometot>8612) && (incometot<=11700))
                                        {                       
                                                credit=2370;
                                        }
                                        else if((incometot>11700) && (incometot<=12000))
                                        {                       
                                                credit=1630;
                                        }
                                        else if((incometot>12000) && (incometot<=14000))
                                        {                       
                                                credit=890;
                                        }
                                        else
                                        {                       
                                                credit=150;
                                        }
                                        cout<<"Using the data provided, we have found you and your partner are eligible  "<<endl;
                                        cout<<"for "<<char(156)<<credit << " of Working Tax Credit"<<endl<<endl;
                                }
                                else
                                {
                                        cout<<"Sorry, you are not eligible for any working tax credit because: "<<endl;
                                        if(age<25) cout<<"- You are under 25"<<endl;
                                        if((atoi(wrkhrs))<30) cout<<"- You work less than 30 hours a week"<<endl;
                                        if(page<25) cout<<"- Your partner is under 25"<<endl;
                                        if((atoi(pwrkhrs))<30) cout<<"- Your partner works less than 30 hours a week"<<endl;
                                        if(incometot>16300) cout<<"- Your combined wage is over "<<char(156)<<"16300 a year"<<endl;       
                                }
                        }
                }
                getch();
                system("cls");
                choice==0;
                return;
       
}

       

       

void Menu()
{
       
        int mchoice=0;
        system("cls");
       
        while(mchoice!=3)
        {
                memset(birthyr, 0, sizeof(birthyr)); // resets char variable to 0
                memset(income, 0, sizeof(income));
                memset(wrkhrs, 0, sizeof(wrkhrs));

                credit=0;
                cout<<"Welcome to the Working Tax Credit Calculator. In order to process your enquiry"<<endl;
                cout<<"further we need to know whether you are enquiring as a couple. Please choose "<<endl;
                cout<<"from the following options:"<<endl<<endl;
                cout<<"1.  Enquiry for Singles"<<endl;
                cout<<"2.  Enquiry for Couples"<<endl;
                cout<<"3.  Exit"<<endl<<endl;
                cout<<"Menu choice: ";
                cin>>mchoice;
                cout<<endl;
                if (mchoice==1)
                {
                        system("cls");
                        SingleEnq();
                }
                else if (mchoice==2)
                {               
                        system("cls");
                        CoupleEnq();
                }
                else if (mchoice==3)
                {               
                        cout<<endl;
                        cout<<"Thank you for using the Working Tax Credit Calculator";
                        getch();
                        exit(1);
                }
                else
                        cout<<"Please enter a valid menu choice"<<endl<<endl;


        }
}



void main()
{
        time_t t = time(0);
        tm time = *localtime(&t);
        curryr = time.tm_year + 1900;
        Menu();
        getch();
}

---

Quote:

---

Originally Posted by hirsty

Code:

---

                cout<<"Choice not valid"<<endl;
                cin.clear();

---

---

This does not do what you think it does. "clear" merely removes the errors status from your stream. It does not remove any corrupt data, or empty your stream or anything like that. If you are trying to read an int, and the data in your stream is "apples", you will fail forever.

This is what you need to do:

Code:

---

    std::cin.clear(); //Remove error status
    std::cin.ignore(255, '\n'); //Remove any data left in the stream.

---

Or just read this guide. I really think you need it. It'll help.
http://www.parashift.com/c++-faq-lit....html#faq-15.3

On a side note, I really think you need to try to learn the "C++ way" of doing things, rather than the "C way".

Also, try concentrating on the scope of your variables, and what they should be. For example, the variable "choice" is global, but mchoice is not?

Another weird thing in your code is that every now and then you write things like:

Code:

---

            cout<<"Birth Year is not valid"<<endl;
            (atoi(birthyr)) == 0;

or

                        memset(wrkhrs, 0, sizeof(wrkhrs));
            credit=0;
            choice==0;
            system("cls");
            return SingleEnq();

---

What exactly do you think this does? I think the first one does not do what you want (I don't know what you want though), and the second one, might be a typo'ed = turned ==.

Finally, as we said, don't work with char arrays (or string). If your data represents a number, put it in a number, and work with that. Don't atoi() every time you need the birthyear. This is just god awful:

Code:

---

    age = curryr - atoi(birthyr);

---

I can't really help you more, since you never even said what was wrong, which is basically the first thing you should have told us...

Quote:

---

Originally Posted by monarch_dodra

This does not do what you think it does. "clear" merely removes the errors status from your stream. It does not remove any corrupt data, or empty your stream or anything like that. If you are trying to read an int, and the data in your stream is "apples", you will fail forever.

This is what you need to do:

Code:

---

    std::cin.clear(); //Remove error status
    std::cin.ignore(255, '\n'); //Remove any data left in the stream.

---

Or just read this guide. I really think you need it. It'll help.
http://www.parashift.com/c++-faq-lit....html#faq-15.3

On a side note, I really think you need to try to learn the "C++ way" of doing things, rather than the "C way".

Also, try concentrating on the scope of your variables, and what they should be. For example, the variable "choice" is global, but mchoice is not?



Another weird thing in your code is that every now and then you write things like:

Code:

---

            cout<<"Birth Year is not valid"<<endl;
            (atoi(birthyr)) == 0;

or

                        memset(wrkhrs, 0, sizeof(wrkhrs));
            credit=0;
            choice==0;
            system("cls");
            return SingleEnq();

---

What exactly do you think this does? I think the first one does not do what you want (I don't know what you want though), and the second one, might be a typo'ed = turned ==.

Finally, as we said, don't work with char arrays (or string). If your data represents a number, put it in a number, and work with that. Don't atoi() every time you need the birthyear. This is just god awful:

Code:

---

    age = curryr - atoi(birthyr);

---

I can't really help you more, since you never even said what was wrong, which is basically the first thing you should have told us...

---

Sorry, the main reason for the == on the atoi's, they are mainly copied and pasted and as for the == on the choice, moans at me if i use a single which confuses me as it should work :/ but meh sorry, first yr on this HND and last time i coded was in VB, DarkBasic, HTML etc etc....

What im wanting (as seen with the size of the char variables) is to LIMIT the variables to a certain amount of characters so the user cannot exceed it, i also want to display the variable, on the second time of antering an illegal date of birth, the age tends to go to 6 figures or so......-sigh-.......sorry :( just want it to work, and as u said all ive rlly done is naffed it up so to speak :/. The other thing is im not wanting this code so my tutor can sense that I havent coded it etc...

firstly, sorry for the double post

birthyr

i want limite to 4 DIGITS, such as 1979 and not 1974538134753 or fsdfsdff or 19ddsfdsfd

income

i want to be limited to 6 or 8 digits, not sure, dont think i will allow 2 decimal places as that does complicate things

wrkhrs

2 digits, as 1 person isnt really going to work over 100 hours a week (correct me if im wrong)

choice

either "Y","y","N" or "n"

credit

is ok as is as ppl dont mess with this, calculated automatically

curryr

is ok as is as ppl dont mess with this, calculated automatically

edit: as for the scope of my choice variable, reason i made it global is because it is used in more than one function

Quote:

---

Originally Posted by hirsty

firstly, sorry for the double post

birthyr

i want limite to 4 DIGITS, such as 1979 and not 1974538134753 or fsdfsdff or 19ddsfdsfd

income

i want to be limited to 6 or 8 digits, not sure, dont think i will allow 2 decimal places as that does complicate things

wrkhrs

2 digits, as 1 person isnt really going to work over 100 hours a week (correct me if im wrong)

choice

either "Y","y","N" or "n"

credit

is ok as is as ppl dont mess with this, calculated automatically

curryr

is ok as is as ppl dont mess with this, calculated automatically

edit: as for the scope of my choice variable, reason i made it global is because it is used in more than one function

---

Did you even look at the link?

Code:

---

int age = 0;
while ((std::cout << "How old are you? ")
          && (!(std::cin >> age) || age < 1 || age > 200))
{
    std::cout << "That's not a number between 1 and 200; ";
    std::cin.clear();
    std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
}
std::cout << "You have entered a valid age of " << age << ";" << std::endl;

---

In this basic example, you declare an age, as an integer. Your program will keep asking the user his age until a number is provided AND that number is actually smart.

Do the same thing for revenue. And all your other variables.

I know you want to "code yourself", but that shouldn't keep you from doing things the right way. The above code is pretty self forward, and easy to understand. I'd just replace "std::numeric_limits<std::streamsize>::max()" with 255.

for your Yes/No:

Code:

---

char answer;
while ((std::cout << "Y/N? ")
          && (!(std::cin >> answer) || !(answer=='Y' || answer=='y' ||answer=='N' ||answer=='n') ))
{
    std::cout << "Please provide Y or N; ";
    std::cin.clear();
    std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
}
bool isYes = (answer=='y' || answer=='Y');

---

Once done, you have a simple boolean variable that represents yes or no.

Where I work, 50% of our activity consists in parsing, validating and using user provided data. Not with simple user interface of course, they are actually more complex binary files, but the way it is done is ALWAYS in the same 3 steps, which you should follow too:

1-Read user input into an adequate container: int for numbers, strings for text.
2-Validate the numeric validity of the number: for example, an age cannot be negative.
3-Repeat until all data is valid.

If you mix these steps, your code will become un-maintainable.

If you do these steps, then step 2 will guarantee your "birthyear" is a number between 1 and 200, and you will not have to worry every step of your code if it is "5 digits long" or "apple". You have already done step 2 and step 3, so you know it is a normal birthyear.