Help Needed

Write a program that inputs a sequence of characters and outputs the number of vowels..!! Its not working.!!
if anyone can guide.!!

Code:

#include<iostream.h>
#include<conio.h>
int isVowel (char x);
int main()
{
char word;
int count=0,i;
clrscr();
cout<<"Enter sequence of characters";
cin>>word;
i=isVowel(word);
	if (i==1)
	{
	count++;
	cout<<count<<endl;
	}
		else
		{
		cout<<"No Vowels";
		getch();
		return 0;
		}
		
{
char a,e,i,o,u,x;
 if (x=='a')
{
return 1;
}
	if (x=='e')
	{return 1;
 	}
		if (x=='i')
		{return 1;
		 }
		
			if (x=='o')
			{return 1;
			}
				if (x=='u')
				{return 1;
				}
}
{
 return 0;
}
}

What errors are you getting?

All I see is ASCII art drawing of a staircase

int isVowel (char x)
undefined module..!!

where is ur isvowel() function defined? define it in the code.like this.

Code:

int isVowel(char x)
{
//do something
return something;
}

didnt get you.

A few suggestions:

• Change <iostream.h> to <iostream>
• Get rid of #include<conio.h> and getch()
• Indent your code properly
• Change the isVowel function to return a bool instead of an int
• Define the isVowel function

Originally Posted by Fatima Rizwan

didnt get you.

every function should be defined somewhere in the code to use it. You have declared isVowel before main but not defined it. So define it first.

i dun know how to indent it, i use turbo c 3..so iostream dun work on it..!!
n how to define isVowel..!!
Bool also dun work in this compiler so i have to use int..!!

Originally Posted by Fatima Rizwan

i dun know how to indent it, i use turbo c 3..so iostream dun work on it..!!
n how to define isVowel..!!
Bool also dun work in this compiler so i have to use int..!!

Change to a more standard compliant compiler, or risk acquiring an obsolete skill set.

Originally Posted by Fatima Rizwan

i dun know how to indent it, i use turbo c 3..so iostream dun work on it..!!
n how to define isVowel..!!
Bool also dun work in this compiler so i have to use int..!!

turbo c is a fossil(as they say so). It's quite an old compiler, so it doesn't copes up with the standards. That's why data-type bool and <iostream> are not compatible with it.
For definition of a function, you should refer to some functions tutorials. Here's an example.

Code:

int add(int,int);//function declared here
int main(void)
{
int a=3,b=5;
sum=add(a,b);//function add called by main
printf("%d",sum);
return 0;
}
//Here's the definition
int add(int x, int y)
{
int s;
s=x+y;
return s;
}

Code:

#include<iostream.h>
#include<conio.h>
int isVowel(char x);
int main()
{
clrscr();
int i=0;
char word[50];


cout<<"Enter characters:";
cin>>word[50];
i=isVowel(word[50]);
cout<<i;
for (i=0; i<word[50] ; i++)
if (i==1 )
{
cout<<"\nNo of vowels:"<<i;
}
getch();
return 0;
}

int isVowel(char x)
{
 if (x=='a' || x=='e' || x=='i' || x=='o'|| x=='u')
return 1;

else
     { return 0; }

}

write a sequence of letters and count no. of vowels..!!
kindly identify the logical error..!!

Originally Posted by Fatima Rizwan

kindly identify the logical error..!!

You shoud be calling isVowel() in the loop, and using word[50] is simply wrong since it is out of bounds. You should also be less self-centred and use "i" less often.