How do I change this code ?

Question

0.00/5 (No votes)

See more:

I'm a new pupil of C++ programming.
This is my practice code.
But I have a error on bold text(REGULAR_PAY).
What problem is on that!? I wonder...

C++

 #include <iostream>
 using namespace std;
 #define REGULAR_HOUR 125;
 #define REGULAR_PAY 5580;
 #define OVER_PAY 6500;

int main()
{
	int hour;
	int grossPay;
	int taxRate;
	cout << "Input the worked hour of this month: ";
	cin >> hour;
	if (hour<=125)	{
		grossPay = hour*REGULAR_PAY;
	}
	else
		grossPay = REGULAR_HOUR*REGULAR_PAY + (hour-125)*OVER_PAY;
	if	(grossPay<900000)
		taxRate=1;
	else if (grossPay<=1500000)
		taxRate=2;
	else if (grossPay<=2000000)
		taxRate=8;
	else if (grossPay>2000000)
		taxRate=10;
	cout << "Gross Pay = KRW "<<grossPay<<endl
		<< "Income Tax Rate = "<<taxRate<<"%"<<endl;


}

Posted 24-Aug-15 20:38pm

Member 11933200

Updated 24-Aug-15 20:45pm

CPallini

v2

Add a Solution

Comments

chandanadhikari 25-Aug-15 2:58am

well,
it is time to stop wondering and start learning debugging !!
If you are using Visual Studio IDE then you can check this link:
https://msdn.microsoft.com/en-us/library/bb384844.aspx

if you are using some other IDE then google out how to use its debugger .

also please try to be more precise with your question.
for example include details like whether you are getting some error message or the application crashes etc.

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

CPallini · Answer 1 · 2015-08-24T20:52:00

Solution 1

Quote:
#define REGULAR_HOUR 125;
#define REGULAR_PAY 5580;
#define OVER_PAY 6500;

You should remove the trailing semicolons in preprocessor defines.
Better, you should get rid of preprocessor defines and use enum instead, e.g.

C++

enum
{
  REGULAR_HOUR = 125,
  REGULAR_PAY = 5580,
  OVER_PAY =6500
};

If you have a modern compiler you may also have a look at scoped enumerations[^].

Posted 24-Aug-15 20:52pm

CPallini

Comments

Member 11933200 25-Aug-15 3:17am

Oh, semicolons are problems!! Thank you!
I changed to enum, it succeeded!
Yeah, What the differences are #define , const int , enum !?