5

I'm trying to use gets() to get a string from the user, but the program seems to be passing right over gets(). There is no pause for the user to give input. Why is gets() not doing anything?

char name[13]; printf("Profile name: "); gets(name); printf("\n%s", name);
Improve this question
10
  • 1
    @DerekDrummond 12 characters of input, the last has to be '\0'. Commented Oct 30, 2013 at 18:32
  • 2
    please use fgets(name, sizeof name, stdin) instead of gets() issue is answered many times see one of the answer stackoverflow.com/questions/7231349/… Commented Oct 30, 2013 at 18:33
  • 2
    Don't use scanf for reading strings either. Just use fgets. Commented Oct 30, 2013 at 18:36
  • 1
    I believe you can leave a white-space at the end of the scanf format to make it consume any white-space (such as the trailing new-line) in the input. But don't quote me on that. Commented Oct 30, 2013 at 18:53
  • 1
    I tried fgets(name, sizeof name, stdin) but it get passed over. It's sad that gets() used to be so simple and now is replaced by something bulky. Commented Feb 20, 2016 at 9:28

7 Answers 7

Reset to default
7

Call getchar() before you call gets() or fgets(). Since gets() or fgets() is getting skipped due to an already present '\n' from previous inputs in stdin, calling getchar() would lead to itself getting skipped instead of gets() or fgets() or any other similar function. But remember its more of a hack and not a standard solution (I think so), and also use of gets() is forbidden.

 printf("\nEnter a String: "); getchar(); //fgets(inputString, 100, stdin); gets(inputString); printf("\n%s", inputString);
Improve this answer
Sign up to request clarification or add additional context in comments.

Comments

5

It's because gets() it's so incredibly dangerous to use, that some C libraries have removed it completely and replaced it with a version that does nothing.

Use fgets() instead.

Improve this answer

1 Comment

gets has also been officially removed in C11 and depricated in C++.
3

Use fflush(stdin); before gets()

Improve this answer

2 Comments

This is the solution that worked for me, the other didn't. thank you @Aadil Ansari
That's even worse, since now you have two problems: the undefined behaviour of fflush(stdin) and the security vulnerabilities of gets()!
2

You get lot of troubles using gets()

Instead go for fgets()

fgets(name,13,stdin);

See this SO question Why is the gets function so dangerous that it should not be used?

The reason why fgets() does not work, may be you are not handling the newline left behind by scanf in your previous statements.

You can modify your scanf format string to take it into account: scanf("%d *[^\n]", &N);

*[^\n] says to ignore everything after your integer input that isn't a newline, but don't do anything with the newline (skip it).

When you use scanf("%d",&num) you hit 13 and enter and 13 is stored in num and the newline character is still in the input buffer when you read fgets from stdin it treats \n as the data you have entered and the fgets() statement is skipped

You cannot flush input buffer however you can do this fseek(stdin,0,SEEK_END); add this before your every fgets statement

Improve this answer

7 Comments

@Andre could you post us your complete code? its working here
void createProfile() { char name[13]; printf("Profile name: "); fgets(name,13,stdin); printf("\n%s", name); }
did u use scanf in any part of your code @Andre because scanf() reads exactly what you ask it to; it's leaving everything else unread, notably the newline following the data.
Yes I used it a few times earlier. So are you saying I should only use fgets?
Suggest fgets(name, sizeof(name), stdin).
|
1

Take a look at gets() reference

Get string from stdin

Reads characters from the standard input (stdin) and stores them as a C string into str until a newline character or the end-of-file is reached.

The newline character, if found, is not copied into str.

A terminating null character is automatically appended after the characters copied to str.

Notice that gets is quite different from fgets: not only gets uses stdin as source, but it does not include the ending newline character in the resulting string and does not allow to specify a maximum size for str (which can lead to buffer overflows).

So, basically gets() is not only unsafe (can lead to buffer overflows) but also reads what's in the input buffer.

I recommend you to use fgets(), but if you want a quick (and lazy and stupid) solution, just flush the input buffer:

char name[13]; printf("Profile name: "); fflush(stdin); gets(name); printf("\n%s", name);
Improve this answer

3 Comments

The call to fflush(stdin) is undefined behavior in C.
You're right, it's meant to be used with output buffers. However, some compilers implement fflush(stdin) as an extension. It works as expected but it compromises portability.
Yes, however the language standard considers it as undefined behavior. And its use should not be encouraged.
1

Add this below function to your code and enjoy it.

string GetString() { char ch; string Line=""; while(1) { ch=getchar(); if(ch=='\n') break; else Line+=ch; } return Line; }

This function can effect all Spaces and Backspaces too!!!

Improve this answer

Comments

0

You might need to discard the rest of the line and move on to the beginning of the next one. You can do:

int c; while((c=getchar()) != '\n' && c != EOF);

This discards the rest of the current line (which might have been partially read with scanf(), etc.) and moves on to the next one.

Although fflush(stdin) might also discard the input, it is non-standard behaviour and discouraged.

You should never ever use gets(myString), as it has been completely removed from the C standard due to how unsafe it is. It doesn't enforce the length of the string, so you can go past the end of the buffer and overwrite other values in RAM, causing a buffer overflow, which can be a security vulnerability. Use fgets(myString, myStringSize, stdin); instead. To get rid of the newline in the resulting string, use myString[strcspn(myString, "\n")] = 0;.

Improve this answer

Comments

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.