使用 fgets 和 sscanf 的意外重复

Question

这是我的代码的一部分。 gets 和sscanf 的目的是扫描三个由一个空格分隔的变量。如果通过，则再次输出指令。否则，输出错误并退出程序。

我想使用 7 长度的 char 数组来限制行中的数量，只获得像 'g 3 3' 这样的格式。但我的代码似乎有问题。

#include <stdio.h> 

int main (void) {
    char line[7];
    char command;
    int x, y;

    while(1){
        /* problem: g  4 4 or g 4  4 can also pass */
        fgets(line, 7, stdin);
        nargs = sscanf(line, "\n%c %d %d", &command, &x, &y);

        if(nargs != 3){
          printf("error\n");
          return 0;
        }

        printf("%c %d %d\n", command, x, y);
    }
}

意外：

g  4 4
g 4 4
error

预期：

g 4 4
g 4 4
// I can continue type

谁能告诉我为什么它仍然会重复指令？

Answer 1

根据C11 standard, 7.21.6.2p5：

由空白字符组成的指令通过读取输入直到第一个非空白字符（仍然未读取）或直到无法读取更多字符来执行。

这将\n 指令和两个空格字符描述为功能相同：它们将尽可能多地匹配输入中的连续空白（空格、制表符、换行符等）。

如果你想匹配一个空格（并且只有一个空格），我建议使用%*1[ ] 而不是空白指令。您可以使用 %*1[\n] 类似地丢弃换行符。例如，由于换行符出现在行尾：

nargs = sscanf(line, "%c%*1[ ]%d%*1[ ]%d%*1[\n]", &command, &x, &y);

不幸的是，这并不能完全解决您的问题，就像the %d format specifier is also defined to discard white-space characters：

跳过输入的空白​​字符（由isspace 函数指定），除非规范包含[、c 或n 说明符

通过一些巧妙的技巧，您也许可以继续使用sscanf（或者更好的是，没有中间缓冲区的scanf），但是在比较了可维护性成本方面的替代方案之后，我们不妨只使用getchar，因此，如果您正在寻找解决问题的方法而不是您提出的问题的答案，我建议您使用 gsamaras answer。

Answer 2

你在那里的东西不会起作用，因为如果用户输入一两个空格，sscanf() 不会被打扰。

您可以通过利用short circuiting 和使用getchar() 以一种简单的方式解决此问题，如下所示：

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

#define SIZE 100

int main(void) {
    int c, i = 0;
    char line[SIZE] = {0};
    while ((c = getchar()) != EOF) {
        // is the first char an actual character?
        if(i == 0 && !isalpha(c)) {
                printf("error\n");
                return -1;
        // do I have two whitespaces in 2nd and 4th position?
        } else if((i == 1 || i == 3) && c != ' ') {
                printf("error\n");
                return -1;
        // do I have digits in 3rd and 5th position?
        } else if((i == 2 || i == 4) && !isdigit(c)) {
                printf("error\n");
                return -1;
        // I expect that the user hits enter after inputing his command
        } else if(i == 5 && c != '\n') {
                printf("error\n");
                return -1;
        // everything went fine, I am done with the input, print it
        } else if(i == 5) {
                printf("%s\n", line);
        }
        line[i++] = c;
        if(i == 6)
                i = 0;
    }
    return 0;
}

输出：

gsamaras@gsamaras:~$ gcc -Wall px.c
gsamaras@gsamaras:~$ ./a.out 
g 4 4
g 4 4
g  4 4
error

Answer 3

谁能告诉我为什么它仍然会重复指令？

棘手的部分是"%d" 消耗前导空白，因此代码需要先检测前导空白。

" " 消耗 0 个或更多空白并且永远不会失败。

所以"\n%c %d %d" 不能很好地检测到中间空格的数量。

---

如果ints 可以超过 1 个字符，请使用这个，否则请参见下面的简化。

使用"%n检测sscanf()进度在缓冲区中的位置。

它使用sscanf() 完成工作，这显然是必需的。

// No need for a tiny buffer
char line[80];
if (fgets(line, sizeof line, stdin) == NULL) Handle_EOF();

int n[6];
n[5] = 0;
#define SPACE1 "%n%*1[ ] %n"
#define EOL1   "%n%*1[\n] %n"

// Return value not checked as following `if()` is sufficient to detect scan completion.
// See below comments for details
sscanf(line, "%c" SPACE1 "%d" SPACE1 "%d" EOL1, 
  &command, &n[0], &n[1],
  &x,       &n[2], &n[3],
  &y,       &n[4], &n[5]);

// If scan completed to the end with no extra
if (n[5] && line[n[5]] == '\0') {
  // Only 1 character between?
  if ((n[1] - n[0]) == 1 && (n[3] - n[2]) == 1 && (n[5] - n[4]) == 1) {
    Success(command, x, y);
  }
}

也许添加测试以确保command 不是空格，但我认为在命令处理中无论如何都会发生这种情况。

---

如果ints 只能是 1 位数字并且使用将 @Seb 答案与上述内容结合的 mod，则可以进行简化。这是可行的，因为每个字段的长度在可接受的答案中是固定的。

// Scan 1 and only 1 space
#define SPACE1 "%*1[ ]"

int n = 0;
// Return value not checked as following `if()` is sufficient to detect scan completion.
sscanf(line, "%c" SPACE1 "%d" SPACE1 "%d" "%n", &command, &x, &y, &n);

// Adjust this to accept a final \n or not as desired.
if ((n == 5 && (line[n] == '\n' || line[n] == '\0')) {
  Success(command, x, y);
}

---

@Seb 和我深入研究了检查sscanf() 的返回值的需要。尽管cnt == 3 测试是多余的，因为n == 5 仅在扫描整行并且sscanf() 返回3 时才为真，但许多代码检查器可能会引发一个标志，指出sscanf() 的结果未被检查。在使用保存的变量之前不限定sscanf() 的结果不是健壮的代码。这种方法使用n == 5 的简单而充分的检查。由于许多代码问题源于未进行任何限定，因此缺乏对sscanf() 的检查可能会在代码检查器中引发误报。添加冗余检查很容易。

// sscanf(line, "%c" SPACE1 "%d" SPACE1 "%d" "%n", &command, &x, &y, &n);
// if (n == 5 && (line[n] == '\n' || line[n] == '\0')) {
int cnt = sscanf(line, "%c" SPACE1 "%d" SPACE1 "%d" "%n", &command, &x, &y, &n);
if (cnt == 3 && n == 5 && (line[n] == '\n' || line[n] == '\0')) {

Answer 4

你的程序有问题吗？ gdb 是你最好的朋友 =)

gcc -g yourProgram.c
gdb ./a.out
break fgets
run
finish
g 4  4

然后单步执行语句，无论何时遇到 scanf 或 printf 只需键入完成，您会看到程序成功完成了此迭代，但程序没有等待输入，只是打印错误消息？为什么 ？井型：

man fgets

fgets 最多读取 ONE LESS 小于 size，因此在您的情况下， fgets 只允许读取 6 个字符，但您给了它 7 个字符！是的，换行符就像空格一样是一个字符，那么第 7 个会发生什么？它将被缓冲，这意味着您的程序不会从键盘读取，而是会看到缓冲区中有字符并将使用它们（本例中为一个字符）。 编辑：这是使您的程序正常运行的方法
你可以忽略空行，如果（ strccmp(line, "\n") == 0 ）然后跳转到下一个迭代，如果你不允许使用 strcmp 一个解决方法是比较 line[0]=='\ n'。