Comp Sci Can someone look over my C++ code - Simple short code

[nukeman]

Homework Statement

My computer crashed, so I can't check my code I wrote, and the school linux lab is full, so I really need some help.

I already wrote the code (ill post it below) so I can't compile it and check for some errors. Can anyone quickly glance it over and see any mistakes?

The program will read a regular price from standard input and output a saleslip.

Here is the code:

//This program computes a regular price from standard input
//and outputs a saleslip

#include (isostream>
using namespace std;


int main()

{

const int HST_RATE = 12;
double priceItem, total, per, sp, hst, dis;
char sale;

cout << " " << endl;
cout << " This program calculates the total cost of some items " << endl;
cout << " Enter the Regular Price " ;
cin >> priceItem;
Cout << " Is this item on Sale (Y/N)? "
cin << sale;


if ( sale == 'y' or sale == 'Y')
{
cout << "Enter the percent discount: "
cin << per;
dis = (per/100) * price;
sp price - dis;
hs = (HST_RATE/100) * sp;
total = sp + hst;
cout << "Sale Slip" << endl,
cout << "===" << endl;
cout "Regular Price $: " << price << endl;
cout << "Discount $: " << dis << endl;
cout << " Sale Price $: " << so << endl;
cout << "HST $: " << hst << endl;
cout << "Total $: " << total << endl;
}


else
{

hst = (HST_RATE/100) * price;
total = price + hst;
cout << "Sale Slip" << endl;
cout << "===" << endl;
cout << "Regular Price $ " << price << endl;
cout << "HST $ << hst << endl;
cout << "Total $ " << total << endl;
}

cout << " " << endl;

return 0;

}

Homework Equations

The Attempt at a Solution

 

[Mark44]

Use (code) and (/code) tags (inside brackets, not parentheses) to preserve your formatting.

Homework Statement

My computer crashed, so I can't check my code I wrote, and the school linux lab is full, so I really need some help.

I already wrote the code (ill post it below) so I can't compile it and check for some errors. Can anyone quickly glance it over and see any mistakes?

The program will read a regular price from standard input and output a saleslip.

Here is the code:

//This program computes a regular price from standard input
//and outputs a saleslip

#include (isostream>
using namespace std;


int main()

{

	const int HST_RATE = 12;
	double priceItem, total, per, sp, hst, dis;
	char sale;

	cout << " " << endl;
	cout << " This program calculates the total cost of some items " << endl;
	cout << " Enter the Regular Price   " ;
        cin >> priceItem;
	Cout << " Is this item on Sale (Y/N)? "
	cin << sale;


	if ( sale == 'y' or sale == 'Y')
	{
	cout << "Enter the percent discount:  "
	cin << per;
	dis = (per/100) * price;
	sp  price - dis;
	hs  = (HST_RATE/100) * sp;
	total = sp + hst;
	cout << "Sale Slip" << endl,
	cout << "===" << endl;
	cout "Regular Price   $: " << price << endl;
	cout << "Discount     $: " << dis << endl;
	cout << " Sale Price  $: " << so << endl;
	cout << "HST          $: " << hst << endl;
	cout << "Total        $: " << total << endl;
	}

	
	else
	{
	
	hst = (HST_RATE/100) * price;
	total = price + hst;
	cout << "Sale Slip" << endl;
	cout << "===" << endl;
	cout << "Regular Price $ " << price << endl;
	cout << "HST 	$ << hst << endl;
	cout << "Total  $ " << total << endl;
	}

	cout << " " << endl;

	return 0;

}

There are several things wrong that will keep this from compiling.
1) You have Cout where it should be cout.
2) hs is undeclared. The variable you have is hst.

When you get those fixed, there is at least one other problem. (I have only scanned your code, so there might be more.)
HST_RATE/100 == 0, because both are integer constants. Have you learned that there are two kinds of division in C (and in C++, C#, etc.)? Because HST_RATE/100 isn't what you thought, some tax is going to be computed incorrectly.

Other comment
Some of your variables have terrible names: HST_RATE, per, sp, hst, dis. What do these mean? You should give your variables meaningful names so that someone else looking at your code has some idea of what they're supposed to represent. Your instructor might deduct points from your code for this. I would.

 

[Jokerhelper]

Some errors I've noticed:
- To call the iostream library, you should use <iostream>
- Many of the lines are missing a colon at the end and some have a comma instead
- You've messed up some of the operators. Remember, it's cout << and cin >>
- Some of the lines are meaningless, like "sp price - dis;" (Did you miss an equal sign?)

Some of your variables have terrible names: HST_RATE, per, sp, hst, dis. What do these mean? You should give your variables meaningful names so that someone else looking at your code has some idea of what they're supposed to represent. Your instructor might deduct points from your code for this. I would.

In some Canadian provinces there is the Harmonized Sales Tax that combines federal and provincial sales taxes, and is commonly known as the HST. So I'd say HST_RATE and hst are fine, and the others should be changed.

 

[Mark44]

Some errors I've noticed:
- To call the iostream library, you should use <iostream>

The OP has (isostream>, which makes two errors.

- Many of the lines are missing a colon at the end and some have a comma instead

Make that semicolon ;
This is a colon :

- You've messed up some of the operators. Remember, it's cout << and cin >>
- Some of the lines are meaningless, like "sp price - dis;" (Did you miss an equal sign?)



In some Canadian provinces there is the Harmonized Sales Tax that combines federal and provincial sales taxes, and is commonly known as the HST. So I'd say HST_RATE and hst are fine, and the others should be changed.

 

[Jokerhelper]

Make that semicolon ;
This is a colon :

I always get those terms mixed up