Thread: Switch statement

I noticed a rather unfortunate choice of variable names:

Code:

double _Num1;
double _Num2;
double _Num3;

The C++ Standard states that "each name that contains a double underscore (_ _) or begins with an underscore followed by an uppercase
letter is reserved to the implementation for any use". As such, you should use a different variable naming convention. It seems to me that something as straightforward as num1, num2 and num3 will do.

It also seems that these three variables were not really used at all. Likewise, it looks like much of your class actually functions as a namespace with the member variables left largely unused except for result. After all, although you use Num1 and Num2 in various member functions, you have parameters of the same name, thus those parameters are used and the member variable names are hidden.

I have went over my code and realised that it was a bit of a mess, thanks to the people on this bored for helping me realise this, so I have reformatted(I think that's the correct word) my code and come across a few errors, I have moved....

Code:

double num1;
double num2;
double num3;
double Num1;
double Num2;
double Num3;

....to the constructor and removed the other declarations from the program, but I don't see why i get these errors:

Code:

main.cpp          28     error: 'num1' undeclared(first use this function)

I thought that whenever an object is made that the constructor is called. So why are my variables not declared?

Am I missing something here?

I believe variables declared inside a constructor are local to that constructor, just like a regular function. You need variables that have class-wide scope to access them anywhere inside the class.

Yes: declare local and member variables as needed and keep them and their scopes straight.