Need help straightening this noob code

[matrim]

I have trouble knowing how to call the function and passing parameters. Can someone help point out where things need to go? It's a basic program to figure out the cost per inch of pizza of either a round(diameter) pizza or square one.

using namespace std;
 
char pizzaType;
double getData(int length, int width, int price);
double getData(int diamter, int price);
double computeUnitCost(int diameter, int price);
double computeUnitCost(int length, int width, int price);
 
int main()
{
        bool okay = true;
        do {
        cout << "Please enter in the type of pizza: " << endl;
        cout << "r - a round pizza" << endl;
        cout << "s - a square pizza" << endl;
        cin >> pizzaType;
 
        cin.ignore(80, '\n');
 
        okay = (pizzaType == 'r' || pizzaType == 's');
 
            cout << "\nYou entered an invalid pizza type!" << endl;
        }while(!okay);
        if (pizzaType == 'r')
 
        double roundPrice = getData;
        cout << "The price per square inch for this pizza is " << roundPrice << endl;
    return 0;    
}
 
double getData(int& length, int& width, int& price)
    {
 
        cout << "Enter in the length of the pizza: " << endl;
        cin >> length;
        cout << "Enter in the width of the pizza: " << endl;
        cin >> width;
        cout << "Enter in the price of the pizza: " << endl;
        cin >> price;
 
    }
double getData(int& diameter, int& price)
    {
        cout << "Enter in the diameter of the pizza: " << endl;
        cin >> diameter;
        cout << "Enter in the the price of the pizza: " << endl;
        cin >> price;
    }
 
double computeUnitCost(int diameter, int price)
    {
        int roundCost = diameter / price;
    return roundCost;
    cout << "The price per square inch for this pizza is: " << roundCost << endl;
}
double computeUnitCost(int length, int width, int price)
    {
        squareCost = ((lenth * width))/ price;
    return squareCost;
    cout << "The price per square inch for this pizza is: " << squareCost << endl;
    }

[Banfa]

Comments in bold

using namespace std;
 
char pizzaType;
 /* The 2 declarations do not match there function definitions below */
double getData(int length, int width, int price);
double getData(int diamter, int price);
 
 /* Currently you don't call these functions anywhere. */
double computeUnitCost(int diameter, int price);
double computeUnitCost(int length, int width, int price);
 
int main()
{
        bool okay = true;
        do {
        cout << "Please enter in the type of pizza: " << endl;
        cout << "r - a round pizza" << endl;
        cout << "s - a square pizza" << endl;
        cin >> pizzaType;
 
        cin.ignore(80, '\n');
 
        okay = (pizzaType == 'r' || pizzaType == 's');
 
            cout << "\nYou entered an invalid pizza type!" << endl;
        }while(!okay);
        if (pizzaType == 'r')
 /* referencing getData like this just gives you a function pointer.
   You need to call the function by calling it as getData(diameter, price);
   Obviously you will need to declare the variables diameter and price first.*/
        double roundPrice = getData;
 /* You don't have any handling here for the square type pizza. */
        cout << "The price per square inch for this pizza is " << roundPrice << endl;
    return 0;    
}
 
double getData(int& length, int& width, int& price)
    {
 /* You should indicate what units you need the input in.
   "Enter in the length of the pizza in inches: " */
        cout << "Enter in the length of the pizza: " << endl;
        cin >> length;
        cout << "Enter in the width of the pizza: " << endl;
        cin >> width;
        cout << "Enter in the price of the pizza: " << endl;
        cin >> price;
 
 /* I think you probably should have called and returned the value 
   return value of computeUnitCostSize here.  What ever you do
   this function is defined as returning a double and so must return 
   a value*/
    }
double getData(int& diameter, int& price)
    {
 /* You should indicate what units you need the input in.
   "Enter in the length of the pizza in inches: " */
        cout << "Enter in the diameter of the pizza: " << endl;
        cin >> diameter;
        cout << "Enter in the the price of the pizza: " << endl;
        cin >> price;
 
 /* I think you probably should have called and returned the value 
   return value of computeUnitCostSize here.  What ever you do
   this function is defined as returning a double and so must return 
   a value*/
    }
 
double computeUnitCost(int diameter, int price)
    {
 /* The price per square inch of a pizza is not the area / cost, it is
   the cost / area.  The diameter of a circle is not it's area, the area 
   of a circle is PI * radius squared. */
        int roundCost = diameter / price;
    return roundCost;
 /* You return before this cout statement so it is never executed */
    cout << "The price per square inch for this pizza is: " << roundCost << endl;
}
double computeUnitCost(int length, int width, int price)
    {
 /* The price per square inch of a pizza is not the area / cost, it is
   the cost / area.  Area / Cost gives square inches per $. */
        squareCost = ((lenth * width))/ price;
    return squareCost;
 /* You return before this cout statement so it is never executed */
    cout << "The price per square inch for this pizza is: " << squareCost << endl;
    }

[matrim]

Thanks for the help so far! My real problem is knowing how to call and pass parameters correctly. Are the other functions right?

#include <iostream>
 
using namespace std;
 
char pizzaType;
void getData(int length, int width, int price);
void getData(int diamter, int price);
double computeUnitCost(int diameter, int price);
double computeUnitCost(int length, int width, int price);
 
int main()
{
        bool okay = true;
        do {
        cout << "Please enter in the type of pizza: " << endl;
        cout << "r - a round pizza" << endl;
        cout << "s - a square pizza" << endl;
        cin >> pizzaType;
 
        cin.ignore(80, '\n');
 
        okay = (pizzaType == 'r' || pizzaType == 's');
 
            cout << "\nYou entered an invalid pizza type!" << endl;
        }while(!okay);
        if (pizzaType == 'r')
 
        int roundPrice = getData(int& diameter, int& price);
        cout << "The price per square inch for this pizza is " << roundPrice << endl;
    //return 0;
        if (pizzaType == 's')
 
        int squarePrice = getData(int& diameter, int& price);
        cout << "The price per square inch for this pizza is " << squarePrice << endl;
}
 
void getData(int& length, int& width, int& price)
    {
 
        cout << "Enter in the length of the pizza: " << endl;
        cin >> length;
        cout << "Enter in the width of the pizza: " << endl;
        cin >> width;
        cout << "Enter in the price of the pizza: " << endl;
        cin >> price;
 
    }
void getData(int& diameter, int& price)
    {
        cout << "Enter in the diameter of the pizza in: " << endl;
        cin >> diameter;
        cout << "Enter in the the price of the pizza in: " << endl;
        cin >> price;
    }
 
double computeUnitCost(int diameter, int price)
    {
        int roundCost = diameter / price;
    cout << "The price per square inch for this pizza is: " << roundCost << endl;
    return roundCost; // not sure if i need to return this or if its the right way
}
 
double computeUnitCost(int length, int width, int price)
    {
        int squareCost = ((length * width))/ price;
    cout << "The price per square inch for this pizza is: " << squareCost << endl;
        return squareCost; // not sure if this is the right way to return it
 
    }
 

[Banfa]

Function Parameters

function parameters can be passed either by value or by reference, in C++ if passing by reference you can use either references or pointers, in C only pointers are available.

To pass data to a function the data has to exist, here is a simple example that I hope will help

#include<iostream>
 
using namespace std;
 
static int fnPassByValue(int x);
static int fnPassByReferencePointer(int *x);
static int fnPassByReferenceReference(int &x);
 
 
int main()
{
    int x = 0;
    int old;
 
    cout << "main                      : Initial value x = " << x << endl;
 
    old = fnPassByValue(x);
 
    cout << "main                      : After calling fnPassByValue x = " << x << endl;
    cout << "main                      :   it's previous value was " << old << endl;
 
    old = fnPassByReferencePointer(&x);
 
    cout << "main                      : After calling fnPassByReferencePointer x = " << x << endl;
    cout << "main                      :   it's previous value was " << old << endl;
 
    old = fnPassByReferenceReference(x);
 
    cout << "main                      : After calling fnPassByReferenceReference x = " << x << endl;
    cout << "main                      :   it's previous value was " << old << endl;
 
    return 0;
}
 
int fnPassByValue(int x)
{
    int old = x;
 
    cout << "fnPassByValue             : Parameter Input " << x << endl;
 
    x += 10;
 
    cout << "fnPassByValue             : Parameter Now " << x << endl;
 
    return old;
}
 
int fnPassByReferencePointer(int *x)
{
    int old = *x;
 
    cout << "fnPassByReferencePointer  : Parameter Input " << *x << endl;
 
    *x += 100;
 
    cout << "fnPassByReferencePointer  : Parameter Now " << *x << endl;
 
    return old;
}
 
int fnPassByReferenceReference(int &x)
{
    int old = x;
 
    cout << "fnPassByReferenceReference: Parameter Input " << x << endl;
 
    x += 1000;
 
    cout << "fnPassByReferenceReference: Parameter Now " << x << endl;
 
    return old;
}
 

[matrim]

Thanks for that. However, we haven't even gone through pointers yet, so I need a way to do that referencing without having pointers in it? :)

Here's what I need:

Your program should include

1. A function named getData( )that gets all of the user input for a round pizza. For a round pizza it will ask for the diameter of the pizza and its price. Each piece of data entered by the user must be validated to ensure that it is a non-zero, positive value. If a value is zero or negative, tell the user that the input is invalid and go back and ask for that data to be input again. Somehow, this function must store the values it gets from the user where they can be accessed by other parts of the program. You cannot use global variables! The return type of the function is void.
2. Another function named getData( )that gets all of the user input for a square pizza. For a square pizza it will ask for the length and width of the pizza and its price. Each piece of data entered by the user must be validated to ensure that it is a non-zero, positive value. If a value is zero or negative, tell the user that the input is invalid and go back and ask for that data to be input again. Somehow, this function must store the values it gets from the user where they can be accessed by other parts of the program. You cannot use global data. The return type of the function is void.
3. A function named computeUnitCost( ) that takes the diameter of a round pizza and its price as parameters, and returns the cost of 1 square inch of that pizza.
4. Another function named computeUnitCost( ) that takes the length and width of a rectangular pizza and its price, and returns the cost of 1 square inch of that pizza. Note that the function computeUnitCost( ) is overloaded.

And what I have coded so far:

 
#include <iostream>
 
using namespace std;
 
void getData(int& length, int& width, int& price)
void getData(int& diameter, int& price);
double computeUnitCost(int diameter, int price);
double computeUnitCost(int length, int width, int price);
 
int main()
{
        char pizzaType;
        bool okay = true;
        do {
        cout << "Please enter in the type of pizza: " << endl;
        cout << "r - a round pizza" << endl;
        cout << "s - a square pizza" << endl;
        cin >> pizzaType;
 
        cin.ignore(80, '\n');
 
        okay = (pizzaType == 'r' || pizzaType == 's');
 
            cout << "\nYou entered an invalid pizza type!" << endl;
        }while(!okay);
        int price = 0;
 
        if (pizzaType == 'r')
        {
        int diameter = 0;    
        getData(diameter, price);
        cout << "The price per square inch for this pizza is " << computeUnitCost(diameter, price) << endl;
        }
        if (pizzaType == 's')
        {
        int length = 0;
        int width = 0;
 
        getData(length, width, price);
        cout << "The price per square inch for this pizza is " << computeUnitCost(length, width, price) << endl;
        }
        int length = 0;
        int width = 0;
 
        computeUnitCost(length, width, price);
 
        system("PAUSE");
}
 
void getData(int& length, int& width, int& price)
    {
 
        cout << "Enter in the length of the pizza: " << endl;
        cin >> length;
        cout << "Enter in the width of the pizza: " << endl;
        cin >> width;
        cout << "Enter in the price of the pizza: " << endl;
        cin >> price;
    }
 
void getData(int& diameter, int& price)
    {
        cout << "Enter in the diameter of the pizza in: " << endl;
        cin >> diameter;
        cout << "Enter in the the price of the pizza in: " << endl;
        cin >> price;
    }
 
double computeUnitCost(int diameter, int price)
    {
        int roundCost = diameter / price;
    cout << "The price per square inch for this pizza is: " << roundCost << endl;
    return roundCost; // not sure if i need to return this or if its the right way
}
 
double computeUnitCost(int length, int width, int price)
    {
        int squareCost = ((length * width))/ price;
    cout << "The price per square inch for this pizza is: " << squareCost << endl;
        return squareCost; // not sure if this is the right way to return it
 
    }

[Banfa]

And that code has all the basic elements needed. You still have not corrected the maths in the computeUnitCost functions that I pointed out originally.

Please ask a specific question or problem about this code, what are you stuck on?