Thread: Just started C++ and need help on... validation.

I did have a look through previous threads about validation, but most of the posts went straight over my head. I'm making a CLI calc app in Linux, and it works fine, but there are a couple of queries:

1. My current program uses GOTOs to return to the options dialogue, so the user can enter another option to use a different operation. I have heard plenty about GOTOs being bad practice, but what I don't clearly know is why? All I know is they are decried on every corner of the globe. If I shouldn't use GOTOs, I assume a loop of some sort would do the job? I am not sure which to use or how to break out of it should the user wish to close the program.

2. I have no idea how to validate the input so that letters are rejected. I have been told a for loop of some kind is the way to do it, but I am still stumped as to how. I was also told that getting the program to check the ASCII values of the characters/digits passed through and only accept a certain range, but again, I am clueless as to how exactly.

If I could have some help on these issues, it would be much appreciated.

My program so far:

Code:

#include <iostream>

using namespace std;

int main(void)
{
    float option,operation,num1,num2;

    cout << "Welcome to Bal Jacques' calculator program. Select the operation you want from the list below, "
            "by typing in the corresponding number:\n";
    cout << "1. Addition\n2. Subtraction\n3. Multiplication\n4. Division\n5. About\n6. Quit\n";
    Start:
    cout << "Option: ";
    cin >> option;
    /*int iCount = 0
    for(iCount = 0; iCount < len(option); iCount++;)
    {

    }*/
    if (option == 1)
    {
        cout << "This operation will add two numbers.\n";
        cout << "Number 1: ";
        cin  >> num1;
        cout << "Number 2: ";
        cin >> num2;
        operation = num1 + num2;
        cout << "The answer is " << operation << "\n";
		goto Start;
    }
    else if (option == 2)
    {
        cout << "This operation will subtract two numbers.\n";
        cout << "Number 1: ";
        cin >> num1;
        cout << "Number 2: ";
        cin >> num2;
        operation = num1 - num2;
        cout << "The answer is " << operation << "\n";
		goto Start;
    }
    else if (option == 3)
    {
        cout << "This operation will multiply two numbers.\n";
        cout << "Number 1: ";
        cin >> num1;
        cout << "Number 2: ";
        cin >> num2;
        operation = num1 * num2;
        cout << "The answer is " << operation << "\n";
		goto Start;
    }
    else if (option == 4)
    {
        cout << "This operation will divide two numbers.\n";
        cout << "Number 1: ";
        cin >> num1;
        cout << "Number 2: ";
        cin >> num2;
        if (num2 == 0)
        {
            cout << "You cannot divide by zero.\n";
        }
        else
        {
            operation = num1 / num2;
            cout << "The answer is " << operation << "\n";
        }
		goto Start;
    }
    else if (option == 5)
    {
        cout << "Calculator App version 1.6.1 by Bal Jacques.\n";
		goto Start;
    }
	else if (option == 6)
	{
		cout << "Thank you for using this program, Goodbye.\n";
		cin.ignore();
		cin.get();
	}
    else
    {
        cout << "Enter a number from 1 to 6.\n";
        goto Start;
    }

}

Originally Posted by Old Marcus

If I shouldn't use GOTOs, I assume a loop of some sort would do the job? I am not sure which to use or how to break out of it should the user wish to close the program.

This is roughly what you are currently doing:

Code:

Start:
    read option
    if option is 1
        ...
        goto Start
    else if option is 2
        ...
        goto Start
    ...
    else if option is 6
        ...
    else
        ...
        goto Start

For alternatives, you could express it this way:

Code:

loop indefinitely
    read option
    if option is 1
        ...
    else if option is 2
        ...
    else if option is 6
        ...
        break
    else
        ...

or this way:

Code:

set option to 0
while option is not 6
    read option
    if option is 1
        ...
    else if option is 2
        ...
    else if option is 6
        ...
    else
        ...

To loop indefinitely, you could use a for loop for which the loop condition is empty, e.g., for (;;) or a while loop for which the condition is true, e.g., while (true).

Originally Posted by Old Marcus

2. I have no idea how to validate the input so that letters are rejected. I have been told a for loop of some kind is the way to do it, but I am still stumped as to how. I was also told that getting the program to check the ASCII values of the characters/digits passed through and only accept a certain range, but again, I am clueless as to how exactly.

Well, suppose the user enters 12 and 123abc. Are you willing to ignore the "abc"?

Regarding validation of input: try searching the board. For example: RPG troubles
(That was me. What can I say? I remember and can more easily find my own posts . . . )

Originally Posted by Daved

1. Goto is bad because it makes it more difficult to follow the control flow of the program and it doesn't express the intent of the coder as well as the equivalent loop. It makes reading and maintaining the code more difficult. Experienced coders can use goto in certain situation where they understand the pros and cons of such use, but beginners lack the knowledge and experience to use goto appropriately.

I like to compare it to a writer taking "literary license" to break the rules of English. A good writer can do it for effect, and get away with it. An inexperienced writer trying to do the same just looks ignorant.