Creating a user defined function to count number of vowels

[SinaSyndrome]

The output I'm getting here just counts every letter in the sentence and counts them as vowels. I'm trying to make the user defined function return the amount of vowels within the sentence.

#include <iostream>
#include <iomanip>
#include <cmath>
#include <ctime>
#include <cctype>
using namespace std;


bool isvowel(char x)
{	
	if(x == 'a' || x == 'A' ||
	   x == 'e' || x == 'E' ||
	   x == 'i' || x == 'I' ||
	   x == 'o' || x == 'O' ||
	   x == 'u' || x == 'U')
		{return true;}
	else{return false;}	
}

int main()
{
	
	int vowel = 0;
	int uppercase = 0;
	int lowercase = 0;
	char sentence;

	cout << "Enter sentence: ";
	cin.get(sentence);

	while(sentence != '\n')
	{
		if(isupper(sentence))// >= 'A' && sentence <= 'Z')
		{
			uppercase++;
		}
		if(islower(sentence))// >= 'a' && sentence <= 'z')
		{
			lowercase++;
		}
		if(isvowel(sentence));
		{
			vowel++;
		}

		cin.get(sentence);
	}

	cout << setfill('.');
	cout << "Number of uppercase letters." << right << setw(4) <<uppercase << endl;
	cout << "Number of lowercase letters." << right << setw(15) << lowercase << endl;
	cout << "Number of vowels." << right << setw(15) << vowel << endl;

	return 0;
}

[LB]

Semicolon on line 41

[SinaSyndrome]

Wow, thankyou!

[andywestken]

Also, going by line 26, I take it you just want to use single chars??

Andy

[ajh32]

Also, going by line 26, I take it just want to use single chars??

Probably better to add:

#include <string> 

and then read the whole sentence somthing like this:

#include <iostream>
#include <string>

using namespace std;

const int MAX = 100;

int main()
{
   string sentence;
   getline(cin, sentence);
   cin.ignore(MAX, '\n');
}

HTH