I'm struggling with the loop of keeping track of the number of participants per distance category, the amount received per category and total amount received and also the highest income.

Question

3.00/5 (2 votes)

See more:

#include <iostream>

using namespace std;

int main()
{

const double KM5_FEE= 65.00;
const double KM10_FEE = 100.00;
const double KM15_FEE= 150.00;
double income_5=0;
double income_10=0;
double income_15;
int count_fee5=0;
int count_fee10 = 0;
int count_fee15 =0;
char answer;
int choice;
int total_income=0;
int total_part=0;
int highest_cat;


  cout<<"Fun Run Competition"<<endl;
   cout<<" Do you want to register a participant( Y or N):";
    cin>>answer;

    while(answer != 'N' && answer != 'n')
    {
        cout<<"Categories\n"
            <<"1. 5 kilometres\n"
            <<"2. 10 kilometres\n"
            <<"3. 15 kilometres\n";
            cout<<" Your choice(1/2/3):";
            cin>> choice;

            switch(choice)
            {
                case 1:
                income_5 = income_5+ KM5_FEE;
                count_fee5++;
                break;

                case 2:
                income_5 = income_10+ KM5_FEE;
                count_fee10++;
                break;

                case 3:
                income_5 = income_15+ KM5_FEE;

                count_fee15++;
                break;
                default:
                cout<<"invalid category entered/n"
                    <<"Registration cancelled";
                break;
                cout<<"Do you want to register another participant( Y or N)";
                cin>>answer;

            }
            if(count_fee5>count_fee10&& count_fee5>count_fee15)
            {
                highest_cat = highest_cat+ 5;
            }
            else if (count_fee10>count_fee5&& count_fee10>count_fee15)
            {
                highest_cat= highest_cat+ 10;
            }
            else
            {
             highest_cat= highest_cat+ 15;
             }
            total_income = total_income+ income_15+ income_5+income_10;
            total_part = total_part + count_fee5+ count_fee10+ count_fee15;

            cout<<"Registration per category\n";
            cout<<"Category\tNumber of Athletes\tIncome";
            cout<<"5 kilometres\t"<<count_fee5<<"\t"<<income_5;
            cout<<"10 kilometres\t"<<count_fee10<<"\t"<<income_10;
            cout<<"15 kilometres\t"<<count_fee15<<"\t"<<income_15;
            
            cout<<"The total number of participants:"<<total_part;
            cout<<"Total income"<< total_income;
            cout<<"Category with the highest income"<<highest_cat;



}

What I have tried:

I have tried evrything and its giving me the wrong the output, the logic is wrong.

Posted 12-Aug-22 4:06am

Wen Z

Updated 12-Aug-22 6:27am

Add a Solution

Comments

Rick York 12-Aug-22 12:36pm

one other thing - in your default case there is a break in the middle of some logic and that makes the prompt and input statements unreachable. Also, the final listings should probably happen only after all participants are registered. This means you should rearrange your loop and prompting.

merano99 12-Aug-22 14:44pm

Wrong Output? In this case, there are still very many errors in it, which make the process almost impossible. But it would also be good if you provide data documenting the errors and also write what you would have expected as outputs.

3 solutions

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)

Patrice T · Answer 1 · 2022-08-12T04:16:00

Quote:
I have tried evrything and its giving me the wrong the output, the logic is wrong.

Look at your code !

C++

case 1:
income_5 = income_5+ KM5_FEE;
count_fee5++;
break;

case 2:
income_5 = income_10+ KM5_FEE;
count_fee10++;
break;

case 3:
income_5 = income_15+ KM5_FEE;

Be careful, everything matters.

merano99 · Answer 2 · 2022-08-12T04:27:00

Solution 2

Quote:
its giving me the wrong the output

There are already some anomalies even without data, which Patrice T has already noted.
Here, for example:

C++

double income_5=0;
double income_10=0;
double income_15;

Variable income_15 is not initialised, and also you should initialize with a double and not with an integer.

Posted 12-Aug-22 4:27am

merano99

Updated 12-Aug-22 4:29am

v2

Comments

Patrice T 12-Aug-22 11:55am

Didn't see that one :)
+5

merano99 12-Aug-22 14:32pm

Thanks!

Rick York · Answer 3 · 2022-08-12T06:27:00

I think you have the answers by now. Since this is C++, I wanted to point out something about your code. You have a set of variables declared :

C++

const double KM5_FEE= 65.00;
const double KM10_FEE = 100.00;
const double KM15_FEE= 150.00;
double income_5=0;
double income_10=0;
double income_15;
int count_fee5=0;
int count_fee10 = 0;
int count_fee15 =0;

and to me they appear to lend themselves to a class organization. Something like this :

C++

class Entry
{
public:  // members
    double  m_fee     { 0 };
    double  m_income  { 0 };
    double  m_count   { 0 };

public:  // methods
    Entry() {}  // default constructor

    // assignment constructor

    Entry( double fee, double income=0, double count=0 )
    {
        Set( fee, income, count );
    }

    void Set( double fee, double income=0, double count=0 )
    {
        m_fee = fee;
        m_income = income;
        m_count = count;
    }

    void Register()
    {
       m_income += m_fee;
       ++m_count;
    }
};

to construct some you can do this :

C++

Entry entry05( 65 );
Entry entry10( 100 );
Entry entry15( 150 );

this will make the switch statement in your code look like this :

C++

switch(choice)
{
    case 1:
       entry05.Register();
       break;

    case 2:
       entry10.Register();
       break;

    case 3:
       entry15.Register();
       break;
}

I will leave this rest of it for you to figure out.

This is just something for you to consider since you have classes available to you. I think the use of a class for this can simplify your logic considerably.

I'm struggling with the loop of keeping track of the number of participants per distance category, the amount received per category and total amount received and also the highest income.

3 solutions

Solution 1

Solution 2

Solution 3

Add your solution here

Preview 0