I need some help...

Please support our C++ advertiser: Intel Parallel Studio Home
Reply
1 2

•
•
Join Date: Jun 2009
Posts: 13
Reputation: TheSilverFox is on a distinguished road 
Solved Threads: 0
TheSilverFox TheSilverFox is offline Offline
Newbie Poster

I need some help...

 
0
  #1
Jun 19th, 2009
This is what I need to do:

Use a single-subscripted array to solve the following problem: A company pays its salespeople on a commission basis. The salespeople receive $200 per week plus 9% of their gross sales for that week. For example, a salesperson who grosses $5000 in sales in a week receives $200 plus 9% of $5000 or a total of $650. Write a program (using an array of counters) that determines how many of the salespeople earned salaries in each of the following ranges (assume that each salesperson’s salary is truncated to an integer amount):
a) $200-$299
b) $300-$399
c) $400-$499
d) $500-$599
e) $600-$699
f) $700-$799
g) $800-$899
h) $900-$999
i) $1000 and over
Summarize the results in tabular format

This is what I've got:
cpp Syntax (Toggle Plain Text) 
  1. #include <iostream>
  2. using std::cout;
  3. using std::cin;
  4. using std::endl;
  5.  
  6. #include <iomanip>
  7.  
  8. void displayResults( );
  9.  
  10. int main()
  11. {
  12.     int sales;
  13.     double salary;
  14.     int counter[ 9 ] = { 0, 0, 0, 0, 0, 0, 0, 0, 0 };
  15.  
  16.  
  17.  
  18.  
  19.     cout << "Enter a sales amount: (negative to end)";
  20.     cin >> sales;
  21.  
  22.  
  23.     while (sales > 0){
  24.          salary = (sales * .09) + 200;
  25.  
  26.          if ( (salary >= 200) && (salary < 300) )
  27.               ++counter[ 0 ];
  28.          else if ( (salary >= 300) && (salary < 400) )
  29.               ++counter[ 1 ];
  30.          else if ( (salary >= 400) && (salary < 500) )
  31.               ++counter[ 2 ];
  32.          else if ( (salary >= 500) && (salary < 600) )
  33.               ++counter[ 3 ];
  34.          else if ( (salary >= 600) && (salary < 700) )
  35.               ++counter[ 4 ];
  36.          else if ( (salary >= 700) && (salary < 800) )
  37.               ++counter[ 5 ];
  38.          else if ( (salary >= 800) && (salary < 900) )
  39.               ++counter[ 6 ];
  40.          else if ( (salary >= 900) && (salary < 1000) )
  41.               ++counter[ 7 ];
  42.          else
  43.               ++counter[ 8 ];
  44.     cout << "Enter a sales amount: (negative to end)";
  45.     cin >> sales;
  46.     }        
  47.  
  48.     displayResults();
  49.  
  50.  
  51.  system("pause");
  52.  return 0;
  53. }
  54.  
  55. void displayResults( )
  56. {
  57.      int counter[ 9 ];
  58.  
  59.      cout << " Range        Number" << endl;
  60.      cout << "$200 - $299\t  " << counter[ 0 ] << endl;
  61.      cout << "$300 - $399\t  " << counter[ 1 ] << endl;
  62.      cout << "$400 - $499\t  " << counter[ 2 ] << endl;
  63.      cout << "$500 - $599\t  " << counter[ 3 ] << endl;
  64.      cout << "$600 - $699\t  " << counter[ 4 ] << endl;
  65.      cout << "$700 - $799\t  " << counter[ 5 ] << endl;
  66.      cout << "$800 - $899\t  " << counter[ 6 ] << endl;
  67.      cout << "$900 - $999\t  " << counter[ 7 ] << endl;
  68.      cout << "Over $1000 \t  " << counter[ 8 ] << endl;
  69. }



I am fairly new to C++, and am not sure how to fix it. I have used QBasic and C before, but not extensively. The program sort of works...but it doesn't display the proper results in the function. If I put a regular cout << counter[whatever]; into the main function, it displays the proper results...any ideas how I can transfer the correct results into the function? I have read a lot about passing arrays into functions, but can't figure out what exactly to add to my program to make it do what it is supposed to. Any help would be greatly appreciated, thank you! =)

~TheSilverFox
Last edited by Tekmaven; Jun 19th, 2009 at 7:25 pm. Reason: Code Tags
Reply With Quote Quick reply to this message  
•
•
Join Date: Jun 2009
Posts: 830
Reputation: wildgoose is a name known to all wildgoose is a name known to all wildgoose is a name known to all wildgoose is a name known to all wildgoose is a name known to all wildgoose is a name known to all 
Solved Threads: 94
wildgoose's Avatar
wildgoose wildgoose is offline Offline
Practically a Posting Shark

Re: I need some help...

 
0
  #2
Jun 19th, 2009
The slot determination is easy, below range, range, over range.

C++ Syntax (Toggle Plain Text) 
  1.  
  2. int counter[ 9 ];
  3.  
  4. i = counter[ n ] ; is the value so...
  5. int *pCnt = &counter[ n ];
  6. Gives you a pointer to the integer located at cell [n].
  7. Or try this.
  8.  
  9. int *pCnt = counter;
  10. while( n-- )
  11. {
  12.           int i = *pCnt++;
  13.  
  14. }
Reply With Quote Quick reply to this message  
•
•
Join Date: Jun 2009
Posts: 13
Reputation: TheSilverFox is on a distinguished road 
Solved Threads: 0
TheSilverFox TheSilverFox is offline Offline
Newbie Poster

Re: I need some help...

 
0
  #3
Jun 19th, 2009
I tried adding that into my code:

void displayResults( )
{

int counter[ 9 ];
int n;
int i;

i = counter[ n ] ;
int *pCnt = &counter[ n ];

cout << " Range Number" << endl;
cout << "$200 - $299\t " << &counter[ 0 ] << endl;
cout << "$300 - $399\t " << &counter[ 1 ] << endl;
cout << "$400 - $499\t " << &counter[ 2 ] << endl;
cout << "$500 - $599\t " << &counter[ 3 ] << endl;
cout << "$600 - $699\t " << &counter[ 4 ] << endl;
cout << "$700 - $799\t " << &counter[ 5 ] << endl;
cout << "$800 - $899\t " << &counter[ 6 ] << endl;
cout << "$900 - $999\t " << &counter[ 7 ] << endl;
cout << "Over $1000 \t " << &counter[ 8 ] << endl;
}

and it just caused my .exe to crash. I wasn't 100% sure how to add it in properly though. Do I need to declare n better? Where do I use i? Thank you in advance...
Reply With Quote Quick reply to this message  
•
•
Join Date: Jun 2009
Posts: 830
Reputation: wildgoose is a name known to all wildgoose is a name known to all wildgoose is a name known to all wildgoose is a name known to all wildgoose is a name known to all wildgoose is a name known to all 
Solved Threads: 94
wildgoose's Avatar
wildgoose wildgoose is offline Offline
Practically a Posting Shark

Re: I need some help...

 
0
  #4
Jun 19th, 2009
First problem that array is empty (not initialized).

Second, you only want the value, not the address for printing!

C++ Syntax (Toggle Plain Text) 
  1.  
  2. // YOU need t fill counter
  3. // and initialize n like in a loop reference or something.
  4.  
  5. //i = counter[ n ] ; 
  6. int *pCnt = &counter[ n ];
  7. for (n=0; n < 9; n++)
  8. {
  9.     counter[n] = n;
  10. }
  11.  
  12.  
  13.  
  14. cout << " Range Number" << endl;
  15. cout << "$200 - $299\t " << counter[ 0 ] << endl;
  16. cout << "$300 - $399\t " << counter[ 1 ] << endl;
  17. cout << "$400 - $499\t " << counter[ 2 ] << endl;
  18. cout << "$500 - $599\t " << counter[ 3 ] << endl;
  19. cout << "$600 - $699\t " << counter[ 4 ] << endl;
  20. cout << "$700 - $799\t " << counter[ 5 ] << endl;
  21. cout << "$800 - $899\t " << counter[ 6 ] << endl;
  22. cout << "$900 - $999\t " << &counter[ 7 ] << endl;
  23. cout << "Over $1000 \t " << counter[ 8 ] << endl;
  24. }
Reply With Quote Quick reply to this message  
•
•
Join Date: Jul 2005
Posts: 1,755
Reputation: Lerner is a name known to all Lerner is a name known to all Lerner is a name known to all Lerner is a name known to all Lerner is a name known to all Lerner is a name known to all 
Solved Threads: 283
Lerner Lerner is offline Offline
Posting Virtuoso

Re: I need some help...

 
0
  #5
Jun 19th, 2009
void displayResults( );

change this to:

void displayResults(int *, int);

Change declaration of counter to this:
const int SIZE = 9;
int counter[SIZE];

call displayResults() like this:
dipsplayResults(counter, SIZE);

Try to define it on your own.
Last edited by Lerner; Jun 19th, 2009 at 5:57 pm.
Klatu Barada Nikto
Reply With Quote Quick reply to this message  
•
•
Join Date: Jun 2009
Posts: 13
Reputation: TheSilverFox is on a distinguished road 
Solved Threads: 0
TheSilverFox TheSilverFox is offline Offline
Newbie Poster

Re: I need some help...

 
0
  #6
Jun 19th, 2009
Ok. This is what the function reads now:
cpp Syntax (Toggle Plain Text) 
  1. void displayResults(int*, int)
  2. {
  3.      int n;
  4.      const int SIZE = 9;
  5.      int counter[SIZE];
  6.      int *pCnt = &counter[ n ];
  7.      for (n=0; n < 9; n++)
  8.          {
  9.             counter[n] = n;
  10.          }
  11.  
  12.  
  13.      cout << " Range        Number" << endl;
  14.      cout << "$200 - $299\t  " << counter[ 0 ] << endl;
  15.      cout << "$300 - $399\t  " << counter[ 1 ] << endl;
  16.      cout << "$400 - $499\t  " << counter[ 2 ] << endl;
  17.      cout << "$500 - $599\t  " << counter[ 3 ] << endl;
  18.      cout << "$600 - $699\t  " << counter[ 4 ] << endl;
  19.      cout << "$700 - $799\t  " << counter[ 5 ] << endl;
  20.      cout << "$800 - $899\t  " << counter[ 6 ] << endl;
  21.      cout << "$900 - $999\t  " << counter[ 7 ] << endl;
  22.      cout << "Over $1000 \t  " << counter[ 8 ] << endl;
  23. }
The program works with no compiler errors, but the results show just 0-8 in sequential order for the Number corresponding to each range. I believe that is because somewhere, a pointer isn't pointing to the original int counter[9]; I created...I'm not sure which parts (if any) should be omitted or edited...

Thank you very much for your quick responses and help! I greatly appreciate it!!
Last edited by Tekmaven; Jun 19th, 2009 at 7:26 pm. Reason: Code Tags
Reply With Quote Quick reply to this message  
•
•
Join Date: Jun 2009
Posts: 830
Reputation: wildgoose is a name known to all wildgoose is a name known to all wildgoose is a name known to all wildgoose is a name known to all wildgoose is a name known to all wildgoose is a name known to all 
Solved Threads: 94
wildgoose's Avatar
wildgoose wildgoose is offline Offline
Practically a Posting Shark

Re: I need some help...

 
0
  #7
Jun 19th, 2009
Sorry I think I confused you

C++ Syntax (Toggle Plain Text) 
  1. void displayResults(int*pCnt, int , nCnt)
  2. {
  3. cout << " Range Number" << endl;
  4. cout << "$200 - $299\t " << *(pCnt+0) << endl;
  5. cout << "$300 - $399\t " << *(pCnt+1) << endl;
  6.  
  7. etc.
Reply With Quote Quick reply to this message  
•
•
Join Date: Jun 2009
Posts: 830
Reputation: wildgoose is a name known to all wildgoose is a name known to all wildgoose is a name known to all wildgoose is a name known to all wildgoose is a name known to all wildgoose is a name known to all 
Solved Threads: 94
wildgoose's Avatar
wildgoose wildgoose is offline Offline
Practically a Posting Shark

Re: I need some help...

 
0
  #8
Jun 19th, 2009
Sorry let's try that again.


void displayResults(int*pCnt, )
{
cout << " Range Number" << endl;
cout << "$200 - $299\t " << *(pCnt+0) << endl;
cout << "$300 - $399\t " << *(pCnt+1) << endl;

etc.
Reply With Quote Quick reply to this message  
•
•
Join Date: Jul 2005
Posts: 1,755
Reputation: Lerner is a name known to all Lerner is a name known to all Lerner is a name known to all Lerner is a name known to all Lerner is a name known to all Lerner is a name known to all 
Solved Threads: 283
Lerner Lerner is offline Offline
Posting Virtuoso

Re: I need some help...

 
0
  #9
Jun 19th, 2009
In this program it's probably easier to write out the display as opposed to using a loop so you could follow this outline:
C++ Syntax (Toggle Plain Text) 
  1. void display(int *);
  2.  
  3. int main()
  4. {
  5.     const int SIZE = 9;
  6.     int counter[SIZE] = {0};
  7.  
  8.     //do dah
  9.  
  10.    display(counter);
  11.  
  12.    //finish up
  13. }
  14.  
  15. void display(int * counter)
  16. {
  17.    cout << " Range Number" << endl;
  18.    cout << "$200 - $299\t " << counter[0] << endl;
  19.    cout << "$300 - $399\t " << counter[1] << endl;
  20.    //etc
  21. }

You want to pass counter as an argument to display(), not declare it in display().
Klatu Barada Nikto
Reply With Quote Quick reply to this message  
•
•
Join Date: Jun 2009
Posts: 13
Reputation: TheSilverFox is on a distinguished road 
Solved Threads: 0
TheSilverFox TheSilverFox is offline Offline
Newbie Poster

Re: I need some help...

 
0
  #10
Jun 19th, 2009
Ok, I did that, wildgoose. The compiler is now giving me an error that the line

int *pCnt = &counter[ n ];

shadows a parameter. I can't delete the declaration, because that's what links it to the original array... Also, I assumed that when you typed


void displayResults(int*pCnt, )

it should be


void displayResults(int*pCnt, int )

so that's what I made it. Was that wrong?
Last edited by TheSilverFox; Jun 19th, 2009 at 6:32 pm.
Reply With Quote Quick reply to this message  
Reply
1 2

Quick Reply to Thread
This thread is more than three months old.
Perhaps start a new thread instead?
•
•
•
•
•
•
•
•
Message:
•
•
•
•



Other Threads in the C++ Forum


Views: 507 | Replies: 13
Thread Tools Search this Thread



Tag cloud for C++
6 add api array arrays assignment beginner binary bitmap c++ c/c++ calculator char class classes code compile compiler console conversion convert count data delete desktop directshow dll encryption error file forms fstream function functions game getline givemetehcodez google graph homeworkhelper iamthwee ifstream input int integer java lazy lib linkedlist linux loop looping loops map math matrix memory multidimensional newbie news node number output parameter pointer problem program programming project proxy python random read recursion recursive reference return sort string strings struct studio system template templates text tree unix url variable vector video visual visualstudio win32 windows winsock word wordfrequency wxwidgets
About Us | Contact Us | Advertise | DaniWeb | Acceptable Use Policy | RSS Feed

©2003 - 2009 DaniWeb® LLC