counting vowels in string (c language)

[ashpats]

Hey everyone,

I am supposed to write a program that counts the number of vowels in string of characters.
It must include "Struct counter_t (it must be declared as in the code i've written)" and function "int CountVowels (IN text[], OUT Count)". the OUT parameter must be of type counter_t. I've written the code but it doesn't seem to work. Can someone help me please???

-------------------------------------------------------------------------------------------------------------
#include <stdio.h>

typedef struct {
int a;
int e;
int i;
int o;
int u;
}counter_t;

int CountVowels (char text[], counter_t Count);

int main ()
{
char text[100];
counter_t Count;
Count.a = 0;
Count.e = 0;
Count.i = 0;
Count.o = 0;
Count.u = 0;
printf ("Text\n");
gets(text);
CountVowels(tex t, Count);
printf("a = %d\n", Count.a);
printf("e = %d\n", Count.e);
printf("i = %d\n", Count.i);
printf("o = %d\n", Count.o);
printf("u = %d\n", Count.u);
fflush (stdin);
getchar ();
return 0;
}

int CountVowels (char text[], counter_t Count)
{
int i;
for (i = 0; text[i] != '\0', i++)
{
switch(text[i])
{
case 'a': Count.a++; break;
case 'e': Count.e++; break;
case 'i': Count.i++; break;
case 'o': Count.o++; break;
case 'u': Count.u++; break;
}
}
return (Count);
}
------------------------------------------------------------------------------------------------------

[boxfish]

Hi,
There are two things that I see wrong offhand. Since CountVowels returns a Count object, its return type has to be Count, not int. Make it:

Count CountVowels (char text[], counter_t Count);

Also, your CountVowels function does not count uppercase vowels. The tolower function from ctype.h should be useful for that.
Hope this helps.

[ashpats]

well i've improved my code a bit. but it still has some problems. (the program doesn't have to count upper case vowels).

#include <stdio.h>

typedef struct {
int a;
int e;
int i;
int o;
int u;
}counter_t;

counter_t CountVowels (char text[], counter_t* Count);

int main ()
{
char text[100];
counter_t Count;
Count.a = 0;
Count.e = 0;
Count.i = 0;
Count.o = 0;
Count.u = 0;
printf ("Text\n");
fgets(text, sizeof text, stdin);
Count = CountVowels(tex t, Count);
printf("a = %d\n", Count.a);
printf("e = %d\n", Count.e);
printf("i = %d\n", Count.i);
printf("o = %d\n", Count.o);
printf("u = %d\n", Count.u);
getchar ();
return 0;
}

counter_t CountVowels (char text[], counter_t* Count)
{
int i;
for (i = 0; text[i] != '\0'; i++)
{
switch(text[i])
{
case 'a': Count->a++; break;
case 'e': Count->e++; break;
case 'i': Count->i++; break;
case 'o': Count->o++; break;
case 'u': Count->u++; break;
}
}
return (Count);
}

[Laharl]

What problems are you encountering? We're neither telepathic nor C compilers (either would be kinda cool, just look at code and BOOM! it's compiled), so if you can post what's going wrong, we can help you.

Also, please surround your code with [code] tags. It makes it much easier to read.

[ashpats]

[Count = CountVowels(tex t, Count);] error C2440: 'function' : cannot convert from 'counter_t' to 'counter_t *'

and
[return (Count);] error C2440: 'return' : cannot convert from 'counter_t *' to 'counter_t'

[gpraghuram]

[Count = CountVowels(tex t, Count);] error C2440: 'function' : cannot convert from 'counter_t' to 'counter_t *'

and
[return (Count);] error C2440: 'return' : cannot convert from 'counter_t *' to 'counter_t'

This line is the issue.
Count = CountVowels(tex t, Count);
Pass the Count by its address like
Count = CountVowels(tex t, &Count);


Raghu

[newb16]

Hi,
There are two things that I see wrong offhand. Since CountVowels returns a Count object, its return type has to be Count, not int.

No. Assignment specification reads "int vowelcount(IN text, OUT count);"
So second arg needs to be count* .

[Banfa]

If you are tassing a pointer to counter_t into the function CountVowels that is the location of the structure to store the results in (and therefore provides a channel to pass the results back to the calling code) then ask yourself what is the purpose of the return value of the function CountVowels, what could it return, what does it need to return?

From that decide what type it should be and change your function accordingly.

[ashpats]

well i've changed few lines:

Count = CountVowels(tex t, Count); to Count = CountVowels(tex t, Count*);

counter_t CountVowels (char text[], counter_t* Count) to
void CountVowels (char text[], counter_t* Count)

end got rid of 'return'. but now i get a syntax error:
Count = CountVowels(tex t, Count*); error C2059: syntax error : ')'

[Laharl]

If you don't return anything from the function, you can't set its return value to a variable, now can you?