Box dimensions problem

[FaffyWaffle]

I'm having problems with the DisplayData function in my program. I was wondering why when I put area in the function it says it's undefined, but if I take it away it says there are to few.

My program is not complete though.

#include<iostream>
#include<iomanip>
using namespace std;

double Volume (double length, double width, double height);

void DisplayData (double length, double width, double height, double volume, double area);

void BoxDimensions (double length, double width, double height, double volume);



void main()
{
	//variable declaration
	double width, length, height, volume;
	char repeat='y';

	//setprecision

	cout.setf(ios::fixed, ios::floatfield);
	cout.setf(ios::showpoint);
	cout<<setprecision(2);

	//user instructions
	cout<<"This program will accept four inputs."<<endl;
	cout<<"For the measurement you would like too find, "<<endl;
	cout<<"please enter 0."<<endl;


	do//do-while 
	{
		
		BoxDimensions(length, width, height, volume);

		//intro input
		cout<<"Please enter the width of your box";
		cin>>width;
		cout<<"Please enter the length of your box";
		cin>>length;
		cout<<"Please enter the height of your box";
		cin>>height;
		cout<<"Please enter the volume of your box";
		cin>>volume;

		//function calls and variables
		if(volume=0);
		{
			volume=width*length*height;
		}
		if(width=0);
		{
			width=volume/(height*length);
		}
		if(length=0);
		{
			length=volume/(height*width);
		}
		if(height=0)
		{
			height=volume/(length*width);
		}
		
		DisplayData(length, width, height, volume, area);



			//rerun the program
		cout<<"Enter y or Y to rerun the program, anything else to quit ";
		cin>>repeat;
	}while (repeat=='y'||repeat=='Y');//end do-while 

[Chervil]

Several issues with this code:

    if (volume=0);
    {
        volume=width*length*height;
    }

1. the assignment operator = is used instead of == for testing equality.

2. the first line should not end with a semicolon. That brings the entire if statement to an end, the rest of the code in braces is always executed.

3. it should end with an else as there follow other mutually-exclusive conditions to be tested.

Should look like this:

    if (volume == 0)
    {
        volume=width*length*height;
    }
    else 

[Yanson]

Also I don't see area declared anywhere. As in double area = 1.0;
something like that. Unless i'm missing it.

[greenleaf800073]

Um, should you only have 1 function outside of main? This program should be really simple.

[FaffyWaffle]

chervil, i changed all my if's to else except the first one and removed the semicolons but now it asks for semicolons after the else

Yanson my area is declared inside of my void DisplayData or is this wrong???

[Chervil]

There should be an else after every if except the last one.
then it goes like this:

if (  )
{

}
else if ( )
{

}

repeat lines 5 - 8 as many times as required. This is of course just an outline, you need the details inside the brackets too.

If area is declared inside function DisplayData(), then there's no need for it to be in the parameter list.