cin.getline() Not Working?

[The Hobo]

It's been awhile since I've used C++. Why doesn't this work?

#include <iostream>
using namespace std;

int main() {

	//input variables:	
	char destination[20], datedepart[8], datereturn[8];
	double ticketprice;
	int ticketcount;

	cout << "Destination: ";
	cin.getline(destination, 20);

	cout << "Departure Date: ";
	cin >> datedepart;
	
	cout << "Return Date: ";
	cin >> datereturn;
	cout << "Number of Tickets: ";
	cin >> ticketcount;
	cout << "Price per Ticket: ";
	cin >> ticketprice; 

	cout << endl << destination << endl << endl;

	return 0;

}

For destination it outputs nothing?

Example Input would be:

Destination: Florida
Depart Date: 12/01/02
Return Date: 12/11/02
Number of Tickets: 2
Price per Ticket: 345.00

[The Hobo]

Anybody?

[Daok]

flush what it have in cin with cin.flush(); before using getline... But I am not sure about .flush() but I know that you have to clear what cin have to use getline properly.

[Rick Bull]

It seems to work if you make the destination array 21 not 20 Is that because you need a \0 there are well?

[The Hobo]

.flush doesn't exist...I tried .clear but it didn't help at all. I put it right before the .getline.

Any other ideas?

[The Hobo]

When I change it to this:

cout << "Destination: ";
cin.getline(destination, 20);
cout << destination << endl;

It outputs fine. So at what point is the value being erased?

[Daok]

Check what's goind on in debug, I will check why it does that when I will come back of the gym.

[The Hobo]

I need to go to work now. Thanks for your help so far!

I'll try a few things when I get home and maybe one of us can find a solution. I've never encountered anything like this before, though.

[Zaei]

Try something along the lines of "cin.ignore('\n', 100)".

Z.

[The Hobo]

Originally posted by Zaei
Try something along the lines of "cin.ignore('\n', 100)".

Z.

I tried. Where exactly should I put it? And don't the arguments go the other way around?

[The Hobo]

Okay, this is my full code. Just run it to see what's wrong. It's very obvious:

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

int main() {
	
	double flightfee = 10.00, taxrate = 0.06;

	//input variables:	
	char destination[20], datedepart[8], datereturn[8];
	double ticketprice = 0, ticketcount = 0;
	
	double totaltax = 0, tickettotal = 0, grandtotal = 0;
	
	//*****THIS IS WHERE WE GET IT********
	cout << "Destination: ";
	cin.getline(destination, 20, '\n');

	cout << "Departure Date: ";
	cin >> datedepart;
	
	cout << "Return Date: ";
	cin >> datereturn;

	cout << "Number of Tickets: ";
	cin >> ticketcount;
	cout << "Price per Ticket: ";
	cin >> ticketprice; 
	
	//tickettotal = ticketprice * ticketcount;
	//totaltax = tickettotal * taxrate;
	//grandtotal = tickettotal + totaltax + flightfee;

	cout << endl;
	cout << "********************************************" << endl;
	cout << "Destination: " << destination << endl; //*****DOESN'T WORK
	cout << "Departure Date: " << datedepart << endl;
	cout << "Return Date: " << datereturn << endl << endl;
	cout << "Number of Tickets: " << ticketcount << endl;

	cout << setprecision(2) << setiosflags(ios::fixed);
	cout << "Total Ticket Price: $" << tickettotal << endl;
	cout << "Flight Fees: $" << flightfee << endl;
	cout << "Tax: $" << totaltax << endl;
	cout << "Grand Total: $" << grandtotal << endl;
	cout << "********************************************" << endl << endl;


	return 0;

}

However when you comment out the two lines:

cout << "Departure Date: ";
cin >> datedepart;

It works perfectly!?

[cpradio]

This works for me:

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

int main() {
	
	double flightfee = 10.00, taxrate = 0.06;

	//input variables:	
	char destination[20], datedepart[11], datereturn[11];
	double ticketprice = 0, ticketcount = 0;
	
	double totaltax = 0, tickettotal = 0, grandtotal = 0;
	
	//*****THIS IS WHERE WE GET IT********
	cout << "Destination: ";
	cin.getline(destination, 20, '\n');

	cout << "Departure Date: ";
	cin.getline(datedepart, 10, '\n');
	
	cout << "Return Date: ";
	cin.getline(datereturn, 10, '\n');

	cout << "Number of Tickets: ";
	cin >> ticketcount;
	cout << "Price per Ticket: ";
	cin >> ticketprice; 
	
	tickettotal = ticketprice * ticketcount;
	totaltax = tickettotal * taxrate;
	grandtotal = tickettotal + totaltax + flightfee;

	cout << endl;
	cout << "********************************************" << endl;
	cout << "Destination: " << destination << endl; //*****DOESN'T WORK
	cout << "Departure Date: " << datedepart << endl;
	cout << "Return Date: " << datereturn << endl << endl;
	cout << "Number of Tickets: " << ticketcount << endl;

	cout << setprecision(2) << setiosflags(ios::fixed);
	cout << "Total Ticket Price: $" << tickettotal << endl;
	cout << "Flight Fees: $" << flightfee << endl;
	cout << "Tax: $" << totaltax << endl;
	cout << "Grand Total: $" << grandtotal << endl;
	cout << "********************************************" << endl << endl;


	return 0;

}

Using:
Florida
12-01-02 (or 12/01/02)
12-11-02 (or 12/11/02)
2
345.00

[parksie]

Try using this:

#include <iostream>
#include <string>

void code() {
    string lin;

    getline(cin, lin);
}

This way you don't need to worry about array sizes

[Zaei]

Know what mine does when I try that, parksie?

"Blch Blah Blah, function getline() does not take 2 arguments... blah blah blah".

So, I go and add an argument:
"blah blah blah, function getline() does not take 3 arguments....blah blah"

=).

So, we just write our own =).

Z.

[parksie]

Hmmm. It should be a global function that takes an istream& and a string& ...

[The Hobo]

Thanks a lot, Matt. That works great!

parksie: I have to use char arrays for now because my teacher says so.

[parksie]

Char arrays are for C. In C++ you should learn strings first, THEN move to the arrays once you understand why you don't normally use them anyway.

*sigh*

[The Hobo]

Originally posted by parksie
Char arrays are for C. In C++ you should learn strings first, THEN move to the arrays once you understand why you don't normally use them anyway.

*sigh*

Well the class is called C/C++ Programming.

[parksie]

There is no C/C++. You either write C or you write C++.

And people wonder why I hate programming courses

[The Hobo]

Originally posted by parksie
There is no C/C++. You either write C or you write C++.

And people wonder why I hate programming courses

I didn't name it. Not my fault.

[parksie]

I'm not blaming you. I'm blaming people who teach things or write courses on them without knowing the details first.

It's the same pattern everywhere, and it's beginning to *really* annoy me

[The Hobo]

Originally posted by parksie
I'm not blaming you. I'm blaming people who teach things or write courses on them without knowing the details first.

It's the same pattern everywhere, and it's beginning to *really* annoy me

I'm just annoyed by the fact that they're making me take this course. I already took an almost identical one in high school, plus I had previous knowledge.

Damn counselors

[Zaei]

Yeah... Lets teach the kiddies C, and then just add classes on top. Anyone who takes a course is nearly guarenteed to not know how to use classes effectivly.

Z.