如何将scanf中带有空格的3个字符输入到字符数组中？

Question

我以前是一名 java 程序员，但我现在正在大学学习 C 课程（计算机科学专业）。

我需要用户能够输入 3 个字符，前 2 个是数字，最后 1 个是 'v' 或 'h'。 例如“1 2 v”。

我需要用户能够在每个字符之间输入空格。

这是我当前的代码：

void manageInput(char box[][width]){
    char move[4];
    char input[16]; 
    while(1){

            scanf("%s", input);
            int i = 0;
            while(input[i] != 0){

                    if(input[i] != ' ' && input[i] != "\n"){
                            move[i] = input[i];
                    }
                    i++;
            }
            printf("%s\n", move);

            makeMove(box, move);
            printBox(box, height, width);
            // TODO
            if(move[0] == 'x'){
                    exit(0);
            }
    }

}

但是，如果我运行它，当我输入没有空格的字符（如“12v”）时它可以正常工作，但如果我输入“1 2 v”，它将打印出“1”，调用 printBox，然后打印出“ 2"，然后再次打印出框，以此类推。

如果有人能解释我在这里做错了什么，我将不胜感激。

Answer 1

如果有人能解释我在这里做错了什么，我将不胜感激。

简短的故事是：您的代码不符合您的要求。它根本不会做你想做的事。

您的要求是：

• 所有字段都必须是一个字符。您的代码未满足此要求。您的代码将错误地接受每个字段的多个字符。
• 字段之间必须有一个空格（正好是一个空格？）。您的代码未满足此要求。字段之间可能有多个空格，您的代码会错误地接受。

事实上，您的代码通过越界访问 move 数组来调用未定义的行为。 考虑到上述情况之一的结果，i 可能会变为高于 3 的某个值. 这段代码可能会发生什么：move[i] = input[i];?

您的代码也太复杂了。您的所有功能都可以由scanf 单独执行。这是一个非常强大的功能，当您知道如何正确使用它时...建议您有机会时多阅读和理解the manual。你会学到很多！

我注意到您在所展示的逻辑中忽略了一些内容：预计第一个字段也可能是 'x'，它对应于 exit 用例。这是一个糟糕的设计；来电者没有机会清理......但我会带着它跑。你真的应该使用return（并返回一个int值或其他东西，对应于错误/成功）。

让我们抛开最后一段，因为我们可以简单地将'x' 视为无效输入（并因此退出），我不想更改您的函数的合同；我会把它留给你。到目前为止描述的表达式似乎是int x = scanf("%1[0123456789]%*1[ ]%1[0123456789]%*1[ ]%1[vh]", a, b, c);。

请注意，预计a、b 和c 将有足够的空间来存储一个字节长度的字符串。也就是说，他们的声明应该是这样的：char a[2], b[2], c[2];。

确保检查返回值（在示例中为x）！ 如果x 为3，则可以安全地假设三个变量a、b 和c 可以安全使用。如果x 为2，则可以安全地假设a 和b 可以安全使用，依此类推...如果x 是EOF 或0，则它们都不能安全使用.

通过检查返回值，您可以拒绝与该精确模式不匹配的输入，即：

• 宽度不完全为一个字节的字段将被拒绝。
• 太多或太少的空格都会被拒绝。

弹出了一些您忽略提及的内容，并且它也出现在您的代码中： Chux 提到您可能希望输入以 '\n'（换行符）字符终止。这也可以使用scanf以多种方式实现：

• scanf("%1*[\n]"); 将尝试准确地读取并丢弃一个 '\n' 字符，但无法确保成功。 getchar 更适合此目的；类似if (getchar() != '\n') { exit(EXIT_FAILURE); } 的东西可能是有道理的，如果你想确保输入的行是完美的，并且当它们不是时被轰掉...#define BOMB_OUT?
• scanf("%*[^\n]"); scanf("%*c"); 更有意义；如果您有兴趣阅读每行一个项目，那么丢弃该行中剩余的所有内容以及换行符本身是有意义的。请注意，您的程序应该在丢弃或截断输入时始终告诉用户。您也可以为此使用 getchar。

---

void manageInput(char box[][width]){
    for (;;) {
        char a[2], b[2], c[2];
        int x = scanf("%1[0123456789]%*1[ ]%1[0123456789]%*1[ ]%1[vh]", a, b, c);
        if (x != 3) {
            /* INVALID INPUT should cause an error value to be returned!
             * However, this function has no return value (which makes it
             * poorly designed)... Calling `exit` gives no opportunity for
             * calling code to clean up :(
             */
            exit(EXIT_FAILURE);
        }

        if (getchar() != '\n') {
#           ifdef BOMB_OUT
            exit(EXIT_FAILURE);
#           else
            scanf("%*[^\n]");
            getchar();
            puts("NOTE: Excess input has been discarded.");
#           endif
        }

        char move[4] = { a[0], b[0], c[0] };
        printf("%s\n", move);
        makeMove(box, move);
        printBox(box, height, width);
        // TODO
        if(move[0] == 'x'){
            exit(0);
        }
    }
}

Answer 2

这看起来像两个简单的错误。

你需要为 move[] 和 input[] 使用单独的索引

    int i = 0;
    while(input[i] != 0){

            if(input[i] != ' ' && input[i] != "\n"){
                    move[i] = input[i];
            }
            i++;
    }

想象一下 1 2 v 的输入

input[0] != 0，所以我们进入循环

也不是 ' ' 或 '\n'，所以我们将 input[0] 复制到 move[0]

到目前为止一切顺利

你增加 i，然后发现 input[1] == ' '

但是你又增加了 i

您发现您对 input[2] (2) 感兴趣 - 因此您将其复制到 move[2]，而不是 move[1]。糟糕！

然后更糟糕的是，您永远不会在 move[] 的最后一个有效字符之后放置字符串结束字符。

Answer 3

%s 使用 scanf 读取一个以空格分隔的字符串，所以如果这不是您想要的，那就不要使用它。 %c 读取单个字符，但不跳过空格，因此您可能还希望格式中的 （空格）跳过空格：

char input[3];

scanf(" %c %c %c", intput, input+1, input+2);

将读取 3 个非空白字符并跳过它们之前或之间的任何空白。您还应该检查 scanf 的返回值以确保它为 3 - 如果不是，则在到达文件结尾之前输入中的字符少于 3 个。

Answer 4

由于潜在的缓冲区溢出，通过scanf 读取字符串通常是个坏主意。考虑使用fscanf 或更好的fgets，如

fgets(input, 15, stdin);

注意 '\0' 的额外字节。

---

此外，您在此处将 char 与字符串进行比较：input[i] != "\n"。它应该是input[i] != '\n'。

---

顺便说一句，你可以使用类似的东西

int x, y;
char d;
scanf("%d%d%c", &x, &y, &d);