C：程序跳过下一个输入

Question

My program skips the next input after 1 pass through it. I have read the threads on removing the newline character that fgets has, but nothing that was suggested worked . Is there anything that would work with microsoft visual studio? The best suggestion was "words[strcspn(words, "\\r\\n")] = 0;" and this did not remove the new line, unless I am formatting it incorrectly. I am not allowed to use the strtok function.

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define SIZE 50
#define STOP "quit\n"

char *copywords(char *dest, const char *source, size_t n);

int main(void) 
{

    char words[50];
    char newwords[50];
    size_t num;

    for (;;) {

        printf("\nType a word, or type 'quit' to quit: ");
        (fgets(words, SIZE, stdin));


        if (strcmp(words, STOP) == 0) {
            printf("Good bye!\n");
            return 0;
        }

        printf("Type the # of chars to copy: ");
        scanf_s("%d", &num);

        copywords(newwords, words, num);
        printf("The word was %s\n", words);
        printf("and the copied word is %s", newwords);


    }
}

char *copywords(char *dest, const char *source, size_t n) {
    size_t i;
    for (i = 0; i < n && source[i] != '\0'; i++) {
        dest[i] = source[i];
    }
    dest[i] = '\0';
    return dest;
}

Answer 1

The problem is that you leave the \\n on the input when you call scanf. ie the user types number [return]. You read the number. When you loop around and call fgets agains the return is still waiting to be read so thats what fgets gets and it returns immediately.

I would probably just call fgets the second time you want to read input as well and then use sscanf to read from the string. ie

printf("Type the # of chars to copy: ");
fgets(buffer, ...)
sscanf(buffer, "%d", ...)

As an aside I would also say to check return values as it is easy for fgets or *scanf to fail.

Answer 2

My program skips the next input after 1 pass through it.

If I understand you correctly, the problem is that scanf_s (which I assume is like the C standard's scanf) will read the digits into num, but scanf won't remove the following newline from stdin, and so in the next iteration of the loop fgets will see that newline and behave as if it had seen a blank line. I have usually avoided scanf for this reason and instead read a line into a buffer and then parse it. For example:

char buf[50];
...
fgets(buf,sizeof(buf),stdin);
sscanf(buf,"%d",&num);

(I'd also recommend adding a whole lot more error checking throughout.)

Answer 3

Here's a straightforward solution.

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define SIZE 50
#define STOP "quit\n"

char *copywords(char *dest, const char *source, size_t n);

int main(void) 
{

    char words[50];
    char newwords[50];
    size_t num, run = 0;

    for (;;) {

        printf("\nType a word, or type 'quit' to quit: ");
        if(run)
            getchar();
        (fgets(words, SIZE, stdin));

        if (strcmp(words, STOP) == 0) {
            printf("Good bye!\n");
            return 0;
        }

        printf("Type the # of chars to copy: ");
        scanf("%d", &num);

        copywords(newwords, words, num);
        printf("The word was %s\n", words);
        printf("and the copied word is %s", newwords);
        run = 1;
    }
}

char *copywords(char *dest, const char *source, size_t n) {
    size_t i;
    for (i = 0; i < n && source[i] != '\0'; i++) {
        dest[i] = source[i];
    }
    dest[i] = '\0';
    return dest;
}

Since we know there will be an extra '\\n' character left in the stream due to the scanf, just take it out.