Question about execution

My program works.... but when I execute it, and input the sentence, I have to push enter twice in order for the rest of the program to execute. What do I need to change so that I only have to push enter once for the rest to follow through?

Code:

#include <iostream>
#include <string>
#include <algorithm>

using namespace std;

int isVowel(string mystring);


int main()
{
	string vowelcheck;
	cout << "Enter a sentence: " << endl;
	getline(cin, vowelcheck, '\n');
	cout << "There are " << isVowel(vowelcheck) << " vowels in this sentence." << endl;
	return 0;
}

int isVowel(string mystring)
{
	int numvow = 0;
	transform (mystring.begin(), mystring.end(), mystring.begin(), tolower);
	string::iterator forward;
	for(forward = mystring.begin(); forward != mystring.end(); forward++)
	{
		if(*(forward) == 'a' || *(forward) == 'e' || 
			*(forward) == 'i' || *(forward) == 'o' || *(forward) == 'u' )
		{
			numvow++;
		}
	}
	return numvow;
}

I saw this in your other thread, but I didn't touch it because you're using non-standard functions... that may be the source of your problem... see below:

Code:

#include<iostream>
#include<string>
//#include<algoritm>    //my guess is to include transform?

using namespace std;

int isVowel(string mystring);


int main()
{
        string vowelcheck;
        cout << "Enter a sentence: " << endl;
        getline(cin, vowelcheck, '\n');
        cout << "There are " << isVowel(vowelcheck) << " vowels in this sentence." << endl;
        return 0;
}

int isVowel(string mystring)
{
        int numvow = 0;
        //transform (mystring.begin(), mystring.end(), mystring.begin(), tolower);      //non-standard function (AFAIK)

        int l=mystring.length();        //returns the length of the string
        char ch;                        //to hold the individual characters of the string
        for(int i=0;i<l;i++)            //loop through each character in the string
        {
                ch=mystring.at(i);      //get the character from the string
                ch=toupper(ch);         //change it to upper-case (upper-case letters have lower
                                        //ascii values than do lower-case letters, not that it
                                        //matters at all.
                if( ch == 'A' || ch == 'E' || ch == 'I' || ch == 'O' || ch == 'U' )     //check to see if it's a vowel
                {
                        numvow++;       //if it is, increase the count
                }
        }
        return numvow;                  //return the number of vowels in the string
}

Um I add a cin.get(); before return 0 in your code then compiled on devc++ and got an error because there is no such thing as transform...So the code is not portable, then in VS.net it compiled and I only had to push enter once after it found the vowels...must be a bug on ur comp

what non standard about iterator, using a iterator
is better then using that, it protect from going out of bounds.

BTW this is a strange occurrence, no one else
is having the problem with the having to press enter.

Maybe your running it in debug mode? With some parameters or something.

Originally Posted by ILoveVectors

what non standard about iterator, using a iterator
is better then using that, it protect from going out of bounds.

anyways and transform should be standard be cause the
algorithm header is a standard header.

BTW this is a strange occurrence, no one else
is having the problem, wether compiled on .NET
or devc++ which i tested it on both.

It doesnt compile on the newest version of dev-c++

Actually that is correct about transform,
that was my mistake, but you can either
convert the string tolower yourself,
or you can just search for aeiouAEIOU

Originally Posted by ILoveVectors

what non standard about iterator, using a iterator
is better then using that, it protect from going out of bounds.

I didn't mean the iterator was non-standard. the function was. I just chose not to use an iterator...

ah ok, sorry i get defensive, when its my code lol.

The problem is probably a bug in your compiler because the OP's posted code is correct and well written. There is a bug in VC++ 6 that makes you hit enter twice, see the following site and scroll to the <string> fix: http://www.dinkumware.com/vc_fixes.html

Everything in the code is standard from what I see, except potentially for the interpretation of what a vowel is.