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.

Quote:

---

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

Quote:

---

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.