View more

View more

View more

### Image of the Day Submit

IOTD | Top Screenshots

### The latest, straight to your Inbox.

Subscribe to GameDev.net Direct to receive the latest updates and exclusive content.

# C++ skips a statement in it's first run

Old topic!

Guest, the last post of this topic is over 60 days old and at this point you may not reply in this topic. If you wish to continue this conversation start a new topic.

9 replies to this topic

### #1Squiffyk7  Members

Posted 13 September 2013 - 02:04 PM

Hello, i've been learning how to write in C++ using the tutorials on www.cplusplus.com for currently 3 days, so ofcourse my code will look very ammaturish. As I'm getting to grips with the syntax, i'm working on a simple program that lets you name how many films are your favourite, and specify what those films are called. Obviously its rather pointless, but as I said I'm practicing what i've learnt from the C++ syntax so far.

The error in my code is not preventing execution, it's just not doing what I want it to do.

// A program that lets you name "n" films.

#include <iostream>
#include <string>
#include <sstream>
using namespace std;

int main ()
{
int n, i;
string quant;

cout << "How many favourite films do you have?" << endl;
cin >> quant;
stringstream(quant) >> n;
cout << "What are your " << n << " favourite films?" << endl;
string n_movie[n];

for (i = 0; i < n; i ++)
{
getline(cin, n_movie[i]);
cout << "Movie " << i << " is: " << n_movie[i] << endl;
}

return 0;
}

Once I run the code, I can enter my number but the console straight away also prints "Movie 0 is: " without allowing me to specify what movie movie 0 is. I can enter all movies up to movie n-1 as i should be able to.

### #2SiCrane  Moderators

Posted 13 September 2013 - 02:18 PM

Short answer: change cin >> quant; to getline(cin, quant);. Using >> to extract from the stream leaves a carriage return hanging around in the stream that the first getline() tries to read as the first movie title.

### #3Squiffyk7  Members

Posted 13 September 2013 - 02:26 PM

Short answer: change cin >> quant; to getline(cin, quant);. Using >> to extract from the stream leaves a carriage return hanging around in the stream that the first getline() tries to read as the first movie title.

And it works! Thanks, although I'm not to sure what's changed, I think I need to look further into what streams are before I move on.

Posted 13 September 2013 - 03:41 PM

Short answer: change cin >> quant; to getline(cin, quant);. Using >> to extract from the stream leaves a carriage return hanging around in the stream that the first getline() tries to read as the first movie title.

And it works! Thanks, although I'm not to sure what's changed, I think I need to look further into what streams are before I move on.

This documentation/tutorial can be a good read to refresh on the basic knowledge of what is happening: http://www.cplusplus.com/doc/tutorial/basic_io/

Basically what is happen is that cin>> extraction stops as soon as it finds a carriage return or even a blank space.  That is left inside the stream.  Then the next time you get input getline will see that still in the stream.  So it reads it in.

So the program is technically doing what it is "supposed" to do.  Just not what you expected it to do.  It does go through the for loop properly.  It just so happens that there is something left in the input stream the first time through.

Edited by Chad Smith, 13 September 2013 - 03:42 PM.

### #5Khatharr  Members

Posted 13 September 2013 - 06:20 PM

cin is kind of a pain in the butt this way. I actually made a thread to complain about it not long ago.

void hurrrrrrrr() {__asm sub [ebp+4],5;}

There are ten kinds of people in this world: those who understand binary and those who don't.

### #6SeraphLance  Members

Posted 14 September 2013 - 04:01 PM

In languages like C/C++, I/O is always one of the hairier things to mess with.  It's always worthwhile to check the nuances of the particular functions you're using in that area.

### #7Aspirer  Members

Posted 17 September 2013 - 02:13 PM

This must be a common issue... I had the same problem with C#'s Console.Read() method...

cin>>


reads <b>UP TO</b> the first enter/return, leaving the return itself in the input stream, to be read by your next input function.

getLine()


reads <b>and includes</b>the first enter/return, removing it from the input stream, so your next input function will have an empty input stream.

### #8RobTheBloke  Members

Posted 17 September 2013 - 02:35 PM

string n_movie[n];

Or perhaps.....

vector<string> n_movie;

n_movie.resize(n);

??

Posted 17 September 2013 - 03:12 PM

string n_movie[n];

Or perhaps.....

vector<string> n_movie;
n_movie.resize(n);

??

What's the question? Is there one? Yes I'd use a vector but it is likely the OP hasn't studied vectors yet so an array is fine with him learning.

### #10SiCrane  Moderators

Posted 17 September 2013 - 03:46 PM

He's probably trying to indicate that initializing an array with local storage with a non-const array extent is not legal standard C++. It would be easier to tell if he employed the communication device commonly known as the complete sentence.

Old topic!

Guest, the last post of this topic is over 60 days old and at this point you may not reply in this topic. If you wish to continue this conversation start a new topic.