Thread: fgets buffer

This is my code from searching on the forums!!

Code:

 printf("Enter Message in text:");		
		fgets (message, 500, stdin ); 
		printMessage ();
		fflush(stdout);
		clearBuffer ();

void clearBuffer()
{
	while((c = getchar()) != EOF && c != '\n') {
	     ;
		}
}

However it is not clearing the buffer and is displaying the same message as i typed in previously, where have i gone wrong!!

1. Post the whole program - like how did you declare buffer anyway
2. You seldom need your flush buffer code when using fgets, since fgets reads a newline if there is room to store it (so unlike scanf, a newline is seldom ever left for later)

Code:

do
		{
		clearBuffer ();
		printf("Enter Message in text:");		
		fgets (message, 500, stdin ); 
		printMessage ();
		fflush(stdout);
		clearBuffer ();
		printf("\n");
		printf("Your message is %s\n",Result);
		printf("\n");
		printf("Would you like to do this again(Y/N):");
		fflush(stdout);
		scanf("%c", &select);
		select = toupper(select);
		printf("\n");
		}while(select != 'N');

> fgets (message, 500, stdin );
HOW DID YOU DECLARE message??????????

> scanf("%c", &select);
You only need your hacky buffer flush after scanf() calls, not before fgets() calls.

char message[500];

Sorry salem

Considering how fgets() works, the \n will either be in your buffer (which has become a string) or stdin. If it's in your buffer, then there's nothing to clear from stdin.

So try something like this:

Code:

#include <string.h>    /* for strchr() */

void
check_input (char *s)
{
  char *t;
  if ((t = strchr (s, '\n')) == NULL)
  {
    while (getchar() != '\n')
      ;
  }
  else
    *t = '\0';
}

This will also kill the \n if it's in your string, since it's common not to want it.