Issue with code

I have a tiny prob with this piece of code . I want the program to compute fees for each car separately store it somewhere and then display the table will all result . the problem is that I don't now were to put the charges each time then display all of them together . the prog is displaying on each loop how can I modify that to work as I expected .

Code:

---

         
double calculate_charges ( double h ) {

  if ( h == 24 ) {

          return 10.00;
  }
  else if ( h <= 3 ) {
         
          return 2.00;
  }
  else {

          double chg = 3 + (( h - 3 ) *0.50);

      return chg;
  }

}

int main () {
int x; double hours;

do {
               
                cout << "Please enter the number of cars :\n" << endl;
                cin >> x;
               
          }while ( x <= 0 && cout <<"Wrong choice!" );

                for ( int i = 1 ; i <= x ; ++i ) {
               
            do {
                cout << "\nPlease enter the number of hours parked by car " << i << " today :\n" << endl;
                cin >> hours;

                }while ( hours > 24 && cout << "Wrong choicde! " );
               
                    for ( int j = 1 ; j <= i ; j++ )
               
                 
                        cout <<"\n"; added this in order to separate them so you can see what the problem try to compute it like this
                        cout << "Car\t\tHours\t\tCharge\n"
                            << "----\t\t------\t\t-------" << endl
                                << i <<"\t\t"<< hours << "\t\t" << calculate_charges ( hours ); // this function is out the main .
                }

            system ( " pause " );
            return 0;
}

---

thanks

[Split into new thread]

Code:

---

  double chg = 3 + (( h - 3 ) *0.50);

      return chg;

---

Why not just

Code:

---

return 3 + (( h - 3 ) * 0.50);

---

Quote:

---

compute fees for each car separately store it somewhere and then display the table will all result .

---

Use a vector to store the hours for each car. Then once all the data has been entered, iterate the vector to compute the charges and display the info for each car. Have you covered vectors yet?

Have you come across the ternary ? operator yet?

Code:

---

double calculate_charges ( double h ) {

  if ( h == 24 ) {

          return 10.00;
  }
  else if ( h <= 3 ) {
         
          return 2.00;
  }
  else {

          double chg = 3 + (( h - 3 ) *0.50);

      return chg;
  }

}

---

could be coded as

Code:

---

double calculate_charges(double h)
{
        return h == 24 ? 10.0 : (h <= 3 ? 2.0 : 3 + ((h - 3) * 0.50));
}

---

Quote:

---

Originally Posted by 2kaud

Have you come across the ternary ? operator yet?

Code:

---

double calculate_charges ( double h ) {

  if ( h == 24 ) {

          return 10.00;
  }
  else if ( h <= 3 ) {
         
          return 2.00;
  }
  else {

          double chg = 3 + (( h - 3 ) *0.50);

      return chg;
  }

}

---

could be coded as

Code:

---

double calculate_charges(double h)
{
        return h == 24 ? 10.0 : (h <= 3 ? 2.0 : 3 + ((h - 3) * 0.50));
}

---

---

For a beginner, it's probably easier to read and debug if it's broken out the way he did it.

Actually the program ask user how many car it should compute charges for . then take hours parked by each car and get the final charges which is stored is this function calculate_charges ( hours )
All I need to do is to be able to store display the table all result together . I'm not familiar with ternary operator you mentioned . Is that possible to display all result at once and not the charges of each car separately as above part is doing ?? thanks

Quote:

---

Originally Posted by david16

Actually the program ask user how many car it should compute charges for . then take hours parked by each car and get the final charges which is stored is this function calculate_charges ( hours )
All I need to do is to be able to store display the table all result together . I'm not familiar with ternary operator you mentioned . Is that possible to display all result at once and not the charges of each car separately as above part is doing ?? thanks

---

See the reply in post #3 re use of a vector.

Sadly still didn't cover vectors or arrays . is it possible without using these ?? if not, can you tell me how to do it with arrays . I have a bit of infos about it maybe I'll understand it thx

The issue with arrays is that you have to declare their size at compile time - and with vectors you don't. So using an array you need to specify the maximum size as part of the code. In this case the number of cars can't be greater than this value. So using an array, consider

Code:

---

#include <iostream>
using namespace std;

double calculate_charges(double h)
{
        return h == 24 ? 10.0 : (h <= 3 ? 2.0 : 3 + ((h - 3) * 0.50));
}

int main()
{
        const int maxcar = 20;

        int x = 0;

        do {
                cout << "Please enter the number of cars (max " << maxcar << ") :";
                cin >> x;
        } while ((x <= 0 || x > maxcar) && cout << "Wrong choice!" << endl);

        double hours = 0.0;
        double carhours[maxcar] = { 0.0 };

        for (int i = 0; i < x; ++i) {
                do {
                        cout << "Please enter the number of hours parked by car " << i + 1 << " today :";
                        cin >> hours;
                } while ((hours <= 0 || hours > 24) && cout << "Wrong choice!" << endl);

                carhours[i] = hours;
        }

        cout << "\nCar\t\tHours\t\tCharge";
        cout << "\n---\t\t-----\t\t------" << endl;

        for (int j = 0; j < x; ++j)
                cout << j + 1 << "\t\t" << carhours[j] << "\t\t" << calculate_charges(carhours[j]) << endl;

        //system(" pause ");
        return 0;
}

---

But here the number of car is fixed to 20 right ?? I asked to user to enter the number of car so it should store as many as user want can I just put it with empty brakes double car Hours [] = { 0.0 }; or is that considered to be wrong ??

or maybe I can create an array with a fixed number but instead of 20 I put 200 like this const int maxCar = 200; double [maxCar] = {0.0} which I know is bigger than and do the what the user actually want and then do the rest normally ??

Quote:

---

But here the number of car is fixed to 20 right

---

No. The maximum number is fixed at 20 - but the user enters the required number not greater than this. If the maximum is required to be more, then 20 can be changed to the required maximum number.

Quote:

---

double car Hours [] = { 0.0 };

---

No. That's not allowed. The number of elements in an array has to be specified at compile time. If you can't specify the number of elements in the array at compile time, then you need to use a vector.

ok thanks you I'll try that on my side then

Just for info. The code does have a problem. Try entering a non-numeric value for the number of cars.

Have you covered stream errors yet?

no problem everything is working fine . thanks