New into C++ Question

[Lee M.]

Alright. I am real new at C++ and have a simple bracket { question.

I am not sure of the placement in a nested if, else if statement. Please help!
Here is an example...It gives the same response no matter what number you enter, although I don't know why:



#include <iostream.h>

void main()
{
//declare and initialize variable
short code = 0;

//get code from user
cout << "Enter the code (1 through 5): ";
cin >> code;

//display salary
if (code = 1)
cout << "$45,000" << endl;

else
if (code == 2 || code == 5)
cout << "$33,000" << endl;

else
if (code == 3 || code == 4)
cout << "$25,000" << endl;

else

cout << "Entry error" << endl;
//end if (code == 3 || code == 4)
//end if (code == 2 || code == 5)
//end if (code == 1)

} //end of main function

[Cybrg641]

//display salary
if (code = 1)
cout << "$45,000" << endl;

Use "==" instead. Here you are assigning the number 1 to code instead of checking to see if it equals 1.

[sail3005]

Also, for me at least, it is easier to read 'if' statments if the you put {} around the statment you want to be executed.

Like:

if (x==y) {do this}
else if (x==z) {do this}
else {do this}

[Lee M.]

Thanks! I spaced on the "=="
VB was a breeze, this is tough!
You guys are awesome!
Cheers!
Lee

[machine_wars]

You use the {}'s if there is more than one code line of events to do.

if( a == 4 ) {
cout << "This is a test" << endl;
cout << "This is only a test" << endl;
}
else
cout << "Bye" << endl;

[sail3005]

yeah i know, but when using nested if statments you need them and i usually use more than one conditional. So i just always put them and it seems to make it easier for me to read.

Just my opinion though

[parksie]

Also if you indent each bracket level it makes it a lot easier to follow the flow:

if(x) {
    if(y) {
        cout << "x && y" << endl;
    }
}