Need a little help with my code.

Hye there. Im working on a program, but it seems not working like i wanted to. Below is the code

Code:

class Books
{
	string isbnNo, description, title, author;
	float price, discountedprice, discountperc;

public :
	void set_Data()
	{
		cout<<"Enter ISBN \t\t: ";
		getline(cin,isbnNo);

		cout<<"Enter Title \t\t: ";
		getline(cin,title);
		cin.ignore();

		cout<<"Enter Description \t: ";
		getline(cin,description);

		cout<<"Enter Author's name \t: ";
		getline(cin,author);

		cin.ignore();
		cout<<"Enter price \t\t: RM ";
		cin>>price;

		cout<<"Enter discount (%) \t: ";
		cin>>discountperc;
	}

	void calcDiscountedPrice()
	{
		discountedprice = price - (discountperc / 100 * price);
	}

	void print()
	{
		cout<<endl;
		cout<<"ISBN             : "<<isbnNo<<endl;
		cout<<"Title            : "<<title<<endl;
		cout<<"Description      : "<<description<<endl;
		cout<<"Author           : "<<author<<endl;
		cout<<"Original Price   : RM "<<price<<endl;
		cout<<"Discounted Price : RM "<<discountedprice<<endl;
	}

};

void func(Books*);

void main()
{
	Books B1;
	
	func(&B1);
}

void func(Books *add)
{
	add->set_Data();
	add->calcDiscountedPrice();
	add->print();
}

The first few lines work well, but until enter price and enter discount, it skip and i can not type in the price and discount. It straightly printed out everything and the original price and discounted price showed sumthing like -1.(numbers)e+(numbers) When i compiled it, it didnt state any errors. So ima lil bit confuse here. Hope someone can help me with this, and so sorry if my question seems stupid. I am new in this C++ programming..
Thank you..

Worked fine for me, didn't skip those sections (price/discount).

What compiler do you have? Is it MS Visual Studio 5/6? If so you could be a known issue with the getline function.

>> cin.ignore();
Why is that there? It will ignore the first digit of the price. You should put that after all calls to cin >>, not before, and not after calls to getline.

Originally Posted by hk_mp5kpdw

Worked fine for me, didn't skip those sections (price/discount).

What compiler do you have? Is it MS Visual Studio 5/6? If so you could be a known issue with the getline function.

Hurmm.. Worked fine for u? Sigh..
Im using MS Visual C++ 6.0
does it have something to do with my compiler?
isnt getline function used for like a string?
Hurmm..so sorry.. Im clueless..

Originally Posted by Daved

>> cin.ignore();
Why is that there? It will ignore the first digit of the price. You should put that after all calls to cin >>, not before, and not after calls to getline.

I edited it but still the same output.

> Im using MS Visual C++ 6.0
Have you applied Service Pack 6 yet?

Some earlier releases had problems with the first getline call IIRC
http://www.codeguru.com/forum/showthread.php?t=266029

I juz checked out the link, and it seems too much info for a beginner like me. I couldnt understand it. a simple getline function has to fix with all that stuff..Wow.. Well .. n e way...thanx for the link. I'll get into it next time.
For now, i juz need a simple little help.
Any other ways?

1. Download and apply the service pack.


2. Find and open the <string> header file (not the string.h file, but rather the one without any extenstion) in any text editor. On my machine, this file is found in the "Program Files\Microsoft Visual Studio\VC98\Include" directory.


3. Try to find a section of code that looks like this:

Code:

else if (_Tr::eq((_E)_C, _D))
     {_Chg = true;
      _I.rdbuf()->snextc();
     break; }

4. If that section exists as written above, change it to this:

Code:

else if (_Tr::eq((_E)_C, _D))
     {_Chg = true;
      _I.rdbuf()->sbumpc();
     break; }

5. Save the file and exit.


6. Recompile/link and run your program.


7. You should also go here and try to apply the other suggested fixes to several other header files.

Thanx all.. now it works..
thank you..

What did you expect? This is a programming forum Friend of newbies, regulars and kings alike.

BTW, consider upgrading your IDE, VC++ 6 is quite old and the Express version 8 (2005) is available for free.