Thread: Understanding functions

I am trying to write a program that will remove vowels from an input string and output only consanants. I've looked all over for resources and coding examples that would point me in the right direction. Currently my function is written with a case statement, but as this is my first attempt at integrating a function into a program, I am missing something. Unfortunately, I dont know what that is.

What I am thinking at the moment is I have my function built, and I currently coded a while statement to capture the text and output back to the screen. The problem is I have no idea how to properly have the function intercept each character and remove a vowel.

Here is my code such as it is:

Code:

#include <stdio.h>

int isvowel()
{
	//  Switch on the character

    switch (c) {
    
    //  Character is a vowel
    
        case 'A': case 'E': case 'I': 
        case 'O': case 'U': case 'Y':
		case 'a': case 'e': case 'i':
		case 'o': case 'u': case 'y':
            return true;
    
    //  Character is not a vowel
    
        default:
            return false;

}

int main(void)
{
	int c; // character variable integer initialized
	printf("This program will strip all vowels from input text and print the output\n"); // printed messages
	printf("Enter some text ;-)\n\n");
	while ((c = getchar()) != EOF) { // while statement to capture characters
	// my function goes here but I dont know yet how to code it properly 

		printf("\n"); // prints out characters
		putchar(c);
		
	
}

I think I should be using an if statement to seperate the vowels, but at this point, Im not sure how that is accomplished.

Code:

int isvowel()
{
	//  Switch on the character

    switch (c) {

Where did 'c' come from? Don't you want to pass a arguemtn to 'isvowel'? How else would it know what you want to test for a value?

In your while loop you probably want to test to see if isvowel returned true or false right and then take an action depending on that?

If these don't answer your question please ask a more direct and thought out question. Consult:
http://www.catb.org/~esr/faqs/smart-questions.html
For a reference on how to ask questions.

I apologize for my "stupiditiy" but Im very new to programming languages. Sometimes you dont know the right way to ask a question. I slept on my code and decided that I shouldnt tackle two concepts when I need to concentrate on one. I decided to re-do the code to use a compound if statement verses learning case and function in one program.

My new code works perfectly. It just doesnt have a defined function which is the concept I am trying to learn. So I am reposting the working code and hope someone can take a few minutes to explain to a newbie how to shift the if statement into a defined function.

Code:

#include <stdio.h> //Include Files


int main(void)  //Main Program
{
	int c, // i/o character initialized
	int	n; // null character variable integer initialized
	printf("This program will strip all vowels from input text and print the output\n"); // printed messages
	printf("Enter some text ;-)\n\n");
	while ((c = getchar()) != EOF) //{ // while statement to capture characters
		if (c == 'a' || c == 'A'   //Compound if statement to capture vowels to null
			|| c == 'e' || c == 'E'
			|| c == 'i' || c == 'I'
			|| c == 'o' || c == 'O'
			|| c == 'u' || c == 'U'
			|| c == 'y' || c == 'Y') 
		n = c; //  captures vowels to a null character
	//return 0;
		else 
		putchar(c); //prints non-vowels
return 0;				
}

Originally Posted by itsme86

What is it about defining a function you don't understand? You call functions all the time. putchar() is a function and you call it. main() is a function and you define it. Defining another function is exactly the same. I described in my previous post what the 3 different parts of a function definition are (the part before the actual body of the function anyway.)

Your question needs to be more specific. Right now there's no way of answering you without actually doing it all for you. And we don't do homework for people here. We're here to answer specific questions that you're having trouble with.

I apologize. It looks like I was making this out to be harder than it was. But sometimes when you bang your head against the wall too many times, things look blurry after while. Im teaching myself and dont have an instructor to tell me where Im messing up. I took your advice and beat my compiler until it worked right. Something tells me it wont be the last time I have a silly question. I appreciate everyones advice and patience.

Here is an excerpt of my code redone as a defined function:

Code:

int isvowel(int c)
{	
	int n; //initialize local 'null' character variable
		if (c == 'a' || c == 'A'   //Compound if statement to capture vowels to null
		|| c == 'e' || c == 'E'
		|| c == 'i' || c == 'I'
		|| c == 'o' || c == 'O'
		|| c == 'u' || c == 'U'
		|| c == 'y' || c == 'Y') 
		n = c; //  captures vowels to a null character
		else 
			putchar(c); //puts non-vowels to variable c
			return(c); //returns c to calling program	
}

Originally Posted by Prelude

>int isvowel(int c)
This is a reserved name. Anything that starts with "is" and a lower case letter is reserved for future library extensions. You can fix it by saying:

Code:

int is_vowel ( int c )

Code:

Im using the book c by dissection and the exercise specifically asked you to write the function isvowel.  I do however understand IRL you would do it as you asked.

>//initialize local 'null' character variable
That's a confusing comment because you don't use the variable as a 'null' character by any defintion of the term, and // style comments are only legal in C99.

Im using visual studio to write/compile and I happened to discover that commenting style. Its much quicker to use it. I know its not 100 percent legal ;-) The n variable is best described as a memory buffer. I needed n to give the if statement something to do with a vowel. As one of the persons suggested, my earlier simple program already had a function, I just didnt define it.

>putchar(c);
I don't like this. Your function suggests that it tests a character and returns a boolean value consistent with the standard library. But what the function actually does is inconsistent because it has an unexpected side effect. I would expect something more like this:

Code:

int is_vowel ( int c )
{
  c = tolower ( c );
  return c == 'a'
      || c == 'e'
      || c == 'i'
      || c == 'o'
      || c == 'u'
      || c == 'y';
}

Here I dont follow you. I dont understand how the way you suggested would work in my program without major changes. How does 'tolower' work? I thought it was a function that would convert an uppercase letter to a lower one. As the function was supposed to take an input file and return an output file stripped of vowels, if there were capitals, it wouldnt remove them, right?

The way I originally had the function coded, the putchar(c) wasnt in there. The function returned everything, not just non-vowels. Once I added that line into the function, everything worked. I realize my function would be best coded to return a boolerian t/f condition and print only if the condition returned true. Unfortunately, I spent a lot of time trying other things. I decided once I scrapped my original program, to modify my simple program with a non-defined function and see if I could fit the original code into a defined function. I ported the code and adjusted until it compiled, ran and worked. Once I get a some free time, Ill come back to this one and see if I can code it differently.

Thanks for clearing that up!