Thread: Update on previous post; new problem

Hi all,

I've come a little farther in my C studies since my last post, and I decided to try my hand at building a custom function. The code is compiling, but the counters no longer count past 1. I cannot for the life of me figure out the problem. Here is my updated code:

Code:

#include<stdio.h>
#include <ctype.h>

void count(int counter, char symbol) {
  counter++;
  for (int i = 0; i < counter; i++) {
    putchar(symbol);
  }
  puts("\nPlease enter a letter: ");
}

int main(void) {
  char c;
  int vowelCount = 0, conCount = 0;
  puts("\nPlease enter a letter: ");
  do {
    if (isalpha(c = getchar()) || c == '0') {
      switch (tolower(c)) {
        case '0':
          puts("\nThank you for counting on me.");
          break;
        case 'a':
        case 'e':
        case 'i':
        case 'o':
        case 'u':
          count(vowelCount, '*');
          break;
        default:
          count(conCount, '!');
      }
    }
    else if (!isspace(c)) {
      puts("\nAny character not in the English alphabet is not an accepted input for this application.\n\nPlease enter a letter: ");
    }
  } while (c != '0');
  return 0;

}

What it should do:

Code:

>>>c
!
>>>u
*
>>>l
!!
>>>8
Any character not in the English alphabet is not an accepted input for this application.
>>>r
!!!

What it does:

Code:

>>>c
!
>>>u
*
>>>l
!
>>>8
Any character not in the English alphabet is not an accepted input for this application.
>>>r
!

Any help appreciated!

Hi Salem,

Returning the counter (and changing void to int accordingly) did not fix the problem. I'm not sure what I'm supposed to do with the returned value anyway.

Actually, I spoke too soon. I just had to store the result of calling the function back into the appropriate counter variable. Thank you!

Hi, laserlight! Unfortunately, my programming background is very surface level, and until now I've never really cared about what is good practice and what is not. Now that I'm starting a CS degree, this is the kind of stuff to which I definitely need to pay more attention, so thank you for educating me a little bit! Your explanation of the single responsibility principle makes perfect sense to me and I will definitely keep it in mind for future projects.