Ad:

C++ Discussion Thread
Unsolved
Views: 822

Aug 21st, 2007

Please help newbie

Expand Post »

I am not sure why I am getting syntax errors in lines 29 and 31. When I run the console my return values are correct, but have syntax errors.

C++ Syntax (Toggle Plain Text)
include <iostream>
#include <iomanip>
 
using std::cout;
using std::cin;
using std::endl;
using std::setprecision;
using std::ios;
using std::setiosflags;
 
int main()
{
    //declare constants and variables
	const double COMM_RATE1 = .02;
	const double COMM_RATE2 = .05;
	int sales = 0;
	double commission = 0.0;
 
	//get input item
	cout << "Enter the sales amount: "; 
	cin >> sales; 
 
	//calculate commission
	switch (sales)
	{
	case   100000.0 <= : cout << commission = sales * COMM_RATE1 << end1;
		break;
	case 100000.0 > : cout << commission = sales * COMM_RATE2 << end1;
		break;
 
	default: cout  << "error" << endl;
	//end switch
	}
	//display commission
	cout << setiosflags(ios::fixed) << setprecision(2);
	cout << "Commission: $" << commission << endl;
 
	return 0;
}	//end of main function
include <iostream>
#include <iomanip>

using std::cout;
using std::cin;
using std::endl;
using std::setprecision;
using std::ios;
using std::setiosflags;

int main()
{
    //declare constants and variables
	const double COMM_RATE1 = .02;
	const double COMM_RATE2 = .05;
	int sales = 0;
	double commission = 0.0;
	 
	//get input item
	cout << "Enter the sales amount: "; 
	cin >> sales; 

	//calculate commission
	switch (sales)
	{
	case   100000.0 <= : cout << commission = sales * COMM_RATE1 << end1;
		break;
	case 100000.0 > : cout << commission = sales * COMM_RATE2 << end1;
		break;

	default: cout  << "error" << endl;
	//end switch
	}
	//display commission
	cout << setiosflags(ios::fixed) << setprecision(2);
	cout << "Commission: $" << commission << endl;

	return 0;
}	//end of main function

Last edited by Ancient Dragon; Aug 21st, 2007 at 9:52 pm. Reason: add code tags

Similar Threads

Book For Newbie in C++
Best free C/C++ compiler for a newbie? in C++
As a newbie, where i should start from in linux? in Getting Started and Choosing a Distro
Questions about building a system (was: newbie) in Troubleshooting Dead Machines
help newbie alert needs help with login page in ASP.NET

shannonpaul

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

10 posts
since Aug 2007

Permalink

Aug 21st, 2007

Re: Please help newbie

I think you're trying to use a switch to handle a range of values. That's not possible without one case for each value in the range. You should be using an if statement instead.

C++ Syntax (Toggle Plain Text)
//calculate commission
if ( sales <= 100000.0 ) {
  commission = sales * COMM_RATE1;
  cout << commission << endl;
} else {
  commission = sales * COMM_RATE2;
  cout << commission << endl;
}
//calculate commission
if ( sales <= 100000.0 ) {
  commission = sales * COMM_RATE1;
  cout << commission << endl;
} else {
  commission = sales * COMM_RATE2;
  cout << commission << endl;
}

I fixed two other bugs. I don't think an assignment operation is legal in a cout, and it's endl with a lower case L, not end1 with the number 1.

Hamrick

Reputation Points: 180

Solved Threads: 34

Posting Whiz

Offline

322 posts
since Jun 2007

Permalink

Aug 21st, 2007

Re: Please help newbie

Yes I know the if statement would be a better choice but I must you use the switch for an assignment. I am no slacker, and have fought this for 3 hrs. And finially decided to try for help.

shannonpaul

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

10 posts
since Aug 2007

Permalink

Aug 21st, 2007

Re: Please help newbie

Quote ...

but I must you use the switch for an assignment.

I don't think your teacher really wants you to have 100,000 separate cases, and that's exactly what you need to do what it looks like you're trying to do.

To get it to work, this is what you have to do.

C++ Syntax (Toggle Plain Text)
switch ( sales ) {
  case 0:
  case 1:
  case 2:
  case 3:
  case 4:
  case 5:
  //...
  case 100000:
    commission = sales * COMM_RATE1;
    cout << commission << endl;
    break;
  default:
    commission = sales * COMM_RATE2;
    cout << commission << endl;
    break;
}
switch ( sales ) {
  case 0:
  case 1:
  case 2:
  case 3:
  case 4:
  case 5:
  //...
  case 100000:
    commission = sales * COMM_RATE1;
    cout << commission << endl;
    break;
  default:
    commission = sales * COMM_RATE2;
    cout << commission << endl;
    break;
}

You may not be a slacker, but that's still excessively tedious.

You might be confused about what the switch needs to be used for. Can you paste the requirements for your program so I can see what needs to be done?

Hamrick

Reputation Points: 180

Solved Threads: 34

Posting Whiz

Offline

322 posts
since Jun 2007

Permalink

Aug 21st, 2007

Re: Please help newbie

Thank you so so much. Now I have a much better understanding of where and why I was unable to get it right....thanks again!

shannonpaul

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

10 posts
since Aug 2007

Permalink

Aug 21st, 2007

Re: Please help newbie

What I have to do is replace the following code with a switch. One problem I am having is switching a double....so I madw it an int, which could not be used with dollar figure.

/Ch6AppE09.cpp - calculates and displays a commission
//Created/revised by <your name> on <current date>

C++ Syntax (Toggle Plain Text)
#include <iostream>
#include <iomanip>
 
using std::cout;
using std::cin;
using std::endl;
using std::setprecision;
using std::ios;
using std::setiosflags;
 
int main()
{
    //declare constants and variables
	const double COMM_RATE1 = .02;
	const double COMM_RATE2 = .05;
	double sales      = 0.0;
	double commission = 0.0;
 
	//get input item
	cout << "Enter the sales amount: "; 
	cin >> sales; 
 
	//calculate commission
	if (sales <= 100000.0)
		commission = sales * COMM_RATE1;
	else
		commission = sales * COMM_RATE2;
	//end if
 
	//display commission
	cout << setiosflags(ios::fixed) << setprecision(2);
	cout << "Commission: $" << commission << endl;
 
	return 0;
}	//end of main function
#include <iostream>
#include <iomanip>

using std::cout;
using std::cin;
using std::endl;
using std::setprecision;
using std::ios;
using std::setiosflags;

int main()
{
    //declare constants and variables
	const double COMM_RATE1 = .02;
	const double COMM_RATE2 = .05;
	double sales      = 0.0;
	double commission = 0.0;
	
	//get input item
	cout << "Enter the sales amount: "; 
	cin >> sales; 

	//calculate commission
	if (sales <= 100000.0)
		commission = sales * COMM_RATE1;
	else
		commission = sales * COMM_RATE2;
	//end if

	//display commission
	cout << setiosflags(ios::fixed) << setprecision(2);
	cout << "Commission: $" << commission << endl;

	return 0;
}	//end of main function

Last edited by Ancient Dragon; Aug 21st, 2007 at 9:52 pm. Reason: add code tags -- please use them in future posts

shannonpaul

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

10 posts
since Aug 2007

Permalink

Aug 21st, 2007

Re: Please help newbie

Not a good idea to use doubles in switch statements because they are often inprecice due to the way they are stored in memory. If you really have to use switch instead of that if statement then you should multiply it by 100 then assigned the result to an integer. Be careful that you don't get data overflow errors doing that.

Ancient Dragon

Sponsor

Team Colleague

Featured Poster

Reputation Points: 5608

Solved Threads: 2282

Retired and Enjoying Life

Offline

21,947 posts
since Aug 2005

Permalink

Aug 22nd, 2007

Re: Please help newbie

> I know the if statement would be a better choice but I must you use the switch for an assignment
would this make your teacher happy?

C++ Syntax (Toggle Plain Text)
enum { LESS_THAN_10000, NOT_LESS_THAN_10000 };
int to_be_used_in_switch = sales < 100000.0 ? 
                       LESS_THAN_10000 : NOT_LESS_THAN_10000 ;
switch( to_be_used_in_switch )
{
       case LESS_THAN_10000:
            /* ... */ break ;
       case NOT_LESS_THAN_10000:
            /* ... */ break ;
       default:
            assert(false) ;
 }
enum { LESS_THAN_10000, NOT_LESS_THAN_10000 };
int to_be_used_in_switch = sales < 100000.0 ? 
                       LESS_THAN_10000 : NOT_LESS_THAN_10000 ;
switch( to_be_used_in_switch )
{
       case LESS_THAN_10000:
            /* ... */ break ;
       case NOT_LESS_THAN_10000:
            /* ... */ break ;
       default:
            assert(false) ;
 }

Last edited by vijayan121; Aug 22nd, 2007 at 1:35 am.

vijayan121

Reputation Points: 1159

Solved Threads: 285

Posting Virtuoso

Offline

1,606 posts
since Dec 2006

This thread is more than three months old

No one has posted to this discussion for at least three months. Please let old threads die and do not reply to them unless you feel you have something new and valuable to contribute that absolutely must be added to make the discussion complete. Otherwise, please start a new thread in this forum instead.

Previous Thread in C++ Forum Timeline: gmp little help please

Next Thread in C++ Forum Timeline: private constructors

DaniWeb Message

Cancel Changes

User Name
Password