hi,
I have this code:
the output is:PHP Code:...
string BookName, Author;
cout << "> Enter book title: ";
getline(cin, BookName); // this does not work
cout << "> Enter author of book: ";
getline(cin, Author); // but this work!?
...
and then waiting for something to write!> Enter book title: > Enter author of book:
I don't know what's wrong?
Last edited by behzad_shabani; 06-08-2009 at 05:14 PM.
Is that ALL of the code?
I suspect you have something like:
If so, you will need to get rid of the newline that was left behind by cin.Code:int x; cin >> x;
--
Mats
Compilers can produce warnings - make the compiler programmers happy: Use them!
Please don't PM me for help - and no, I don't do help over instant messengers.
This is part of a class method.
BookName and Author are the attribute of the class!
this is the class:
and this is setBookInfo declaration:PHP Code:class Book
{
public:
Book();
Book(string);
void setBookInfo(void);
friend void split(Book&, string, const char);
string BookName;
string Author;
int pages;
};
PHP Code:void Book::setBookInfo()
{
cout << endl << "-------------------------------------------------------------------" << endl
<< "> Enter book title: ";
getline(cin, BookName);
cout << "> Enter author of book: ";
getline(cin, Author);
cout << "> Enter number of Book pages: ";
cin >> pages;
cout << "-------------------------------------------------------------------" << endl << endl;
}
Hey look! It's cin >> (some integer thing)! You can't mix >> and getline without thinking about it. In this case you may want to use cin.ignore() after the int, or use getline+strtol to read the int as well.Code:cin >> pages;
What prize do I get? Can I have a soft toy?Originally Posted by tabstop
--
Mats
Compilers can produce warnings - make the compiler programmers happy: Use them!
Please don't PM me for help - and no, I don't do help over instant messengers.
I didn't understand, I don't think it's related to:
because the first getline didn't work even when I remove cin >> pages; lineCode:cin >> pages;
Is this getline the very first thing you do? You don't, say, have a menu where you have to type 1 to add a book or anything?
And there isn't another cin >> choice to select the "setBookInfo"? Like a menu for example?
--
Mats
Compilers can produce warnings - make the compiler programmers happy: Use them!
Please don't PM me for help - and no, I don't do help over instant messengers.
oh yeah!
thanks, it works now
thank you all
I'll just throw in something, too, to show my good intentions.
First off myfunc(void) is the same as myfunc(), so you can drop the "void" part in your class.
And secondly, you are stripping the names of your parameters to your functions. This is bad practice for a number of reasons. More info here: SourceForge.net: Do not remove parameter names - cpwiki
If you're mixing cin >> with getline, a good solution is to always call cin.ignore() after a call to cin >> (but never after a call to getline).
