Not displaying greatest number?

[MisterTams]

Need some help with debugging why my code isn't correctly choosing the division with the highest sales figure? Thanks guys!

//Finds the highest sales figure of a company's 4 divisions
#include <iostream>
#include <string>
#include <iomanip>
using namespace std;

//function prototype
double getSales(string);
void findHighest(double, double, double, double);

int main ()
{
	double NEsales, SEsales, NWsales, SWsales;
	string division;

	division = "Northeast";
	NEsales = getSales(division);
	
	division = "Southeast";
	SEsales = getSales(division);
	
	division = "Northwest";
	NWsales = getSales(division);
	
	division = "Southwest";
	SWsales = getSales(division);

	findHighest(NEsales, SEsales, NWsales, SWsales);

	system("pause");
	return 0;
}

double getSales(string division)
{
	double sales;
	cout << "Please enter the sales figure for The " << division << " Division." << endl;
	cout << "Sales Figure: $";
	cin >> sales;
	
	while (sales < 0.00)
	{
		cout << "\nInvalid Request. Sales must be greater than $0.00" << endl;
		cin.clear();
		cin.ignore();
		cin >> sales;
	}
	return sales;
}

void findHighest(double NE, double SE, double NW, double SW)
{
	if (NE > SE && NW && SW)
	{
		cout << "\nHighest Grossing Division: Northeast Division" << endl;
		cout << "Sales Figure: $" << fixed << setprecision(2) << NE << endl;
	}

	else if (SE > NE && NW && SW)
	{
		cout << "\nHighest Grossing Division: Southeast Division" << endl;
		cout << "Sales Figure: $" << fixed << setprecision(2) << SE << endl;
	}

	else if (NW > NE && SE && SW)
	{
		cout << "\nHighest Grossing Division: Northwest Division" << endl;
		cout << "Sales Figure: $" << fixed << setprecision(2) << NW << endl;
	}

	else if (SW > NE && NW && SE)
	{
		cout << "\nHighest Grossing Division: Southwest Division" << endl;
		cout << "Sales Figure: $" << fixed << setprecision(2) << SW << endl;
	}
}

[48T7M4Gy]

my code isn't correctly choosing the division with the highest sales figure

Perhaps you might like to give a little more detail than that.

The usual way to describe a problem is to say what you input, what you got as output and why that isn't what you expect. Often an indication of what testing you have done is a good move and perhaps even your thoughts on where the problem might be. Remember, it's unlikely anyone has seen your code before and even less likely to know what it is supposed to do. It's in your interests to provide the few extra lines of explanation.

[MisterTams]

kemort,

Ok, here is what I mean. I'll give an example.
If I input $345 for Northeast, $32 for Southeast, $46 for Northwest, and $753 for Southwest. The function findHighest(), displays Northeast Division with having the highest sales figure with $345. Obviously this isn't correct. I went wrong somewhere with my if and else if statements in that particular function. I've been trying and trying to see where I went wrong. But I keep getting similar wrong calculations each time.

Thanks

[MisterTams]

Nevermind, I got it figured out on my own eventually. I was doing the if statements' conditions wrong. Here is the final product.

//Finds the highest sales figure of a company's 4 divisions
#include <iostream>
#include <string>
#include <iomanip>
using namespace std;

//function prototype
double getSales(string);
void findHighest(double, double, double, double);

int main ()
{
	double NEsales, SEsales, NWsales, SWsales;
	string division;

	division = "Northeast";
	NEsales = getSales(division);
	
	division = "Southeast";
	SEsales = getSales(division);
	
	division = "Northwest";
	NWsales = getSales(division);
	
	division = "Southwest";
	SWsales = getSales(division);

	findHighest(NEsales, SEsales, NWsales, SWsales);

	system("pause");
	return 0;
}

double getSales(string division)
{
	double sales;
	cout << "Please enter the sales figure for The " << division << " Division." << endl;
	cout << "Sales Figure: $";
	cin >> sales;
	
	while (sales < 0.00)
	{
		cout << "\nInvalid Request. Sales must be greater than $0.00" << endl;
		cin.clear();
		cin.ignore();
		cin >> sales;
	}
	return sales;
}

void findHighest(double NE, double SE, double NW, double SW)
{
	if (NE > SE && NE > NW && NE > SW)
	{
		cout << "\nHighest Grossing Division: Northeast Division" << endl;
		cout << "Sales Figure: $" << fixed << setprecision(2) << NE << endl;
	}

	else if (SE > NE && SE > NW && SE > SW)
	{
		cout << "\nHighest Grossing Division: Southeast Division" << endl;
		cout << "Sales Figure: $" << fixed << setprecision(2) << SE << endl;
	}

	else if (NW > NE && NW > SE && NW > SW)
	{
		cout << "\nHighest Grossing Division: Northwest Division" << endl;
		cout << "Sales Figure: $" << fixed << setprecision(2) << NW << endl;
	}

	else
	{
		cout << "\nHighest Grossing Division: Southwest Division" << endl;
		cout << "Sales Figure: $" << fixed << setprecision(2) << SW << endl;
	}
}

[48T7M4Gy]

Well done, MisterTams, I'm glad you were able to solve it.

Depending where you are with your C++ studies this problem lends itself to arrays which reduce the amount of duplication and make life a lot easier in finding the highest value. :)