Total not adding correctly

[Geordienev]

Hey all
I've written a program that is designed to add records which include, itemId, itemPrice, discount, quantity. It then adds the Item prices of all the records together (Well that's what it should be doing), but the problem i'm experiencing is that i'm not getting the total of all records. Any help would be appreciated.

#include <iostream>
using namespace std;
 
int readSaleRecord(unsigned&, double&, char&, unsigned&, double&);
double calculateItemCost(double itemPrice, double& totalCost, char discountType, unsigned quantity);
void displayTotalCost(double totalCost, unsigned recordNum, bool aborted);
 
int main()
{
 unsigned itemId, quantity;
 double itemPrice, totalCost, recordNum= 0, discount;
 char discountType;
 bool aborted=false;
 
readSaleRecord(itemId, itemPrice, discountType, quantity, recordNum);        
calculateItemCost(itemPrice, totalCost, discountType, quantity);          
displayTotalCost(totalCost, recordNum, aborted);
 
system("pause");
return 0;
}
 
void displayTotalCost(double totalCost, unsigned recordNum, bool aborted)
     {
      cout << "The total cost is $" <<totalCost<<"\n";
      cout << "from " <<recordNum<<" record(s) \n";
   return;
     }
 int readSaleRecord(unsigned & itemId, double & itemPrice,char & discountType, unsigned & quantity, double& recordNum)
{
    int anotherSale = 0;
    cout << "Enter item ID: ";
    cin >> itemId;
        if (itemId==0 || itemId >1000000)
           {
           cout << "incorrect id";
           system("pause");
           exit(0);
           }
   
    cout << "Enter full item price: ";
    cin >> itemPrice;
 
        if (itemPrice<0)
           {
           cout << "please enter postive integer";
           system("pause");
           exit(0);
           }
 
    cout << "Enter discount type: ";
    cin >> discountType;
 
        if (discountType!='N' && discountType!='D' && discountType!='T' && discountType!='B')
           {
           cout << "invalid entry";
           system("pause");
           exit(0);
           }
 
    cout << "Enter quantity: ";
    cin >> quantity;
        if (quantity<0)
           {
           cout << "please enter postive integer";
           system("pause");
           exit(0);
           }
 
    cout << "would you like to enter another sale? (1 for yes and 0 for no)";
    cin >> anotherSale;
    recordNum++;
 
 
         while(anotherSale==1)
             {
             cout << "Enter item ID: ";
             cin >> itemId;
 
                 if (itemId==0 || itemId >1000000)
                    {
                     system("pause");
                      exit(0);
                     }
   
             cout << "Enter full item price: ";
             cin >> itemPrice;
 
                  if (itemPrice<0)
                   {
                   cout << "please enter postive integer";
                   system("pause");
                   exit(0);
                   }
 
             cout << "Enter discount type: ";
             cin >> discountType;
 
                 if (discountType!='N' && discountType!='D' && discountType!='T' && discountType!='B')
                 {
                   cout << "invalid entry";
                   system("pause");
                   exit(0);
                   }
 
              cout << "Enter quantity: ";
              cin >> quantity;
 
                  if (quantity<0)
                      {
                     cout << "please enter postive integer";
                     system("pause");
                     exit(0);
                      }
 
              cout << "Enter another record? (1 for yes and 0 for no)";
              cin >> anotherSale;
              recordNum++;
              }
 
return (itemId, itemPrice, discountType, quantity, recordNum);
 
}

 
double calculateItemCost(double itemPrice, double& totalCost, char discountType, unsigned quantity){
double discount;
   switch (discountType){
 
       case 'N': discount= 1; break;
       case 'B': discount = 0.9; break;
       case 'D': discount = 0.8; break;
       case 'T': discount = 0.7; break;
     
        default: discount =1;
         }
totalCost=itemPrice*discount*quantity;
   
return totalCost;
}

[S_M_A]

I'm not trying to be rude but this is a good opportunity to learn how to debug. MSVC has a very good debugger so start your code by pressing F10 (it will start at the very first line in main). Single step with F10 and step into a function with F11. After every line has been passed check the content of your variables by hovering with the mouse over them. Are they as you expect them to be? Why not? How can you correct the bug?

Some other commonly used keys:
Shift+F5 terminates the debugging session.
Shift+F11 steps out of the function
F9 sets a breakpoint on the current line. F9 again clears it.

[GCDEF]

I'll just add that calling exit from inside a function like that is terrible form. Break that habit now.

Also functions can only return one value.
return (itemId, itemPrice, discountType, quantity, recordNum);
Isn't doing what you think it is.

[Paul McKenzie]

Hey all
I've written a program that is designed to add records which include, itemId, itemPrice, discount, quantity. It then adds the Item prices of all the records together (Well that's what it should be doing), but the problem i'm experiencing is that i'm not getting the total of all records. Any help would be appreciated.

As S_M_A mentioned, use the debugger.

Also, your code is very poorly formatted. For the sake of the others looking at your code:

#include <iostream>
using namespace std;

int readSaleRecord(unsigned&, double&, char&, unsigned&, double&);
double calculateItemCost(double itemPrice, double& totalCost, char discountType, unsigned quantity);
void displayTotalCost(double totalCost, unsigned recordNum, bool aborted);

int main()
{
    unsigned itemId, quantity;
    double itemPrice, totalCost, recordNum= 0, discount;
    char discountType;
    bool aborted=false;

    readSaleRecord(itemId, itemPrice, discountType, quantity, recordNum);        
    calculateItemCost(itemPrice, totalCost, discountType, quantity);          
    displayTotalCost(totalCost, recordNum, aborted);

    system("pause");
    return 0;
}

void displayTotalCost(double totalCost, unsigned recordNum, bool aborted)
{
    cout << "The total cost is $" <<totalCost<<"\n";
    cout << "from " <<recordNum<<" record(s) \n";
    return;
}

int readSaleRecord(unsigned & itemId, double & itemPrice,char & discountType, unsigned & quantity, double& recordNum)
{
    int anotherSale = 0;
    cout << "Enter item ID: ";
    cin >> itemId;
    if (itemId==0 || itemId >1000000)
    {
        cout << "incorrect id";
        system("pause");
        exit(0);
    }

    cout << "Enter full item price: ";
    cin >> itemPrice;

    if (itemPrice<0)
    {
        cout << "please enter postive integer";
        system("pause");
        exit(0);
    }

    cout << "Enter discount type: ";
    cin >> discountType;

    if (discountType!='N' && discountType!='D' && discountType!='T' && discountType!='B')
    {
        cout << "invalid entry";
        system("pause");
        exit(0);
    }

    cout << "Enter quantity: ";
    cin >> quantity;
    if (quantity<0)
    {
        cout << "please enter postive integer";
        system("pause");
        exit(0);
    }

    cout << "would you like to enter another sale? (1 for yes and 0 for no)";
    cin >> anotherSale;
    recordNum++;


    while (anotherSale==1)
    {
        cout << "Enter item ID: ";
        cin >> itemId;

        if (itemId==0 || itemId >1000000)
        {
            system("pause");
            exit(0);
        }

        cout << "Enter full item price: ";
        cin >> itemPrice;

        if (itemPrice<0)
        {
            cout << "please enter postive integer";
            system("pause");
            exit(0);
        }

        cout << "Enter discount type: ";
        cin >> discountType;

        if (discountType!='N' && discountType!='D' && discountType!='T' && discountType!='B')
        {
            cout << "invalid entry";
            system("pause");
            exit(0);
        }

        cout << "Enter quantity: ";
        cin >> quantity;

        if (quantity<0)
        {
            cout << "please enter postive integer";
            system("pause");
            exit(0);
        }

        cout << "Enter another record? (1 for yes and 0 for no)";
        cin >> anotherSale;
        recordNum++;
    }
    return(itemId, itemPrice, discountType, quantity, recordNum);
}

double calculateItemCost(double itemPrice, double& totalCost, char discountType, unsigned quantity)
{
    double discount;
    switch (discountType)
    {
        case 'N': discount= 1; break;
        case 'B': discount = 0.9; break;
        case 'D': discount = 0.8; break;
        case 'T': discount = 0.7; break;
    
        default: discount =1;
    }
    totalCost=itemPrice*discount*quantity;

    return totalCost;
}

This line doesn't make sense:

    return(itemId, itemPrice, discountType, quantity, recordNum);

As GCDEF stated, only one value can be returned. If you really wanted to return 5 values, then it's time to learn what a struct is in C++, as that is the most popular way to return multiple values at once (the other is a std::tuple, but that is for later).

Right now, you have all of your related items such as itemId, itemPrice, discountType, etc. in separate variables, thereby having to write functions that take 5 arguments. What if you had to maintain 50 related items? Write all your functions with 50 parameters? That would be crazy, and that is what a struct prevents from happening.

struct SaleInfo
{
    unsigned itemId;
    unsigned quantity;
    double itemPrice;
    double totalCost;
    // add more related items.
};

Then you pass a SaleInfo around instead of 5, 6, 7, or 50 different variables. Each SaleInfo consolidates all the information for one sale. See how easier that looks?

Regards,

Paul McKenzie

[Geordienev]

Cheers for the replies guys, I'll be the first to admit that my coding is absolute shiet to read lol.
But with the struct, do you declare that at the start of the code or within the fucntion? and how exactly do you use it.
Do you replace the;

return (itemId, itemPrice, discountType, quantity, recordNum);

Or does it sit somewhere else, I can understand why its simpler to use and i thank you for that.
I'll also give it a go at debugging now.

[GremlinSA]

Geordienev: Please do not repost the same question in a different section (Non Visual C++ Issues). you are getting good help here, and almost all the same people reply in both sections..

Thanks.

[GCDEF]

Cheers for the replies guys, I'll be the first to admit that my coding is absolute shiet to read lol.
But with the struct, do you declare that at the start of the code or within the fucntion? and how exactly do you use it.
Do you replace the;

return (itemId, itemPrice, discountType, quantity, recordNum);

Or does it sit somewhere else, I can understand why its simpler to use and i thank you for that.
I'll also give it a go at debugging now.

Using a struct is one way. You'd create the struct and return it. In your case, it's not really necessary as you're passing in the parameters by reference. That means that when you set the values of the parameters in the function, they'll contain the values in the calling function when the called function returns. For example

void somefunction(int& i)
{
    i = 1;
}

int main()
{
    int nInt = 0;
    somefunction(nInt);
    //nInt = 1 now.
}

Setting up parameters like that are called output parameters. That's typically how you get a function to populate multiple variables.

[Paul McKenzie]

Cheers for the replies guys, I'll be the first to admit that my coding is absolute shiet to read lol.
But with the struct, do you declare that at the start of the code or within the fucntion?

For now, all you need to do is declare the struct right before main().

struct SaleInfo
{
    unsigned itemId;
    unsigned quantity;
    double itemPrice;
    double totalCost;
    // add more related items.
};

int main()
{
   SaleInfo mySale;  // declare a variable of type SaleInfo
}

As long as whoever needs to know about SaleInfo has it defined before use, you're OK. Note that you now treat SaleInfo just like any other type such as int, double, etc. when it comes to declaring variables, return values, etc.

Second, you use a struct (in this case), using the dot operator to operate on each item in the struct.

int main()
{
   SaleInfo mySale;  // declare a variable of type SaleInfo
   SaleInfo mySale2;  // declare a variable of type SaleInfo
   //...
   mySale.itemInfo = whatever;
   mySale.quantity = 10;
  //  etc...
   mySale2.itemInfo = another_item;
   mySale2.quantity = 4;
}

In the fake example above, there are two completely different SaleInfo items. Each one contains all of the information for a SaleInfo.

Regards,

Paul McKenzie