c - scanf ("%[^\n]d", x) 或 scanf ("%[0-9]d", x) 将 1 变为 49 并将 2 变为 50

Question

问题:

我在使用 scanf() 时遇到问题。我从阅读论坛之类的地方知道 scanf() 在 C 中有很多问题，但我只是还在学习基础知识，所以我不知道所有的细节。

我想解决的代码片段。

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

struct Biodata {
    char name[21],
    address[65],
    date[11],
    phone[17];
};

int main() {
    struct Biodata bio[10];
    int input_max = 0,
    index_struct = 0;
    while (printf("Input the amount of data! ") && scanf("%[0-9]*d", &input_max) < 1) {
        printf("Error! Please, try again!\n\n");
       fflush(stdin);
    }
    for (index_struct = 1; index_struct <= input_max; index_struct++) {
        printf("Your input data count is %d.\n", input_max);
        printf("Data %d.\n", index_struct);
        fflush(stdin);
        while (printf("Name\t: ") && scanf("%[A-Z a-z]s", &bio[index_struct].name) < 1) {
            printf("Error! Please, try again!\n\n");
            fflush(stdin);
        }
        fflush(stdin);
        while (printf("Address\t: ") && scanf("%[^\n]s", &bio[index_struct].address) < 1) {
            printf("Error! Please, try again!\n\n");
            fflush(stdin);
        }
        fflush(stdin);
        while (printf("Birthdate\t: (YYYY/MM/DD)\n") && scanf("%[^\n A-Z a-z]s", &bio[index_struct].date) < 1) {
            printf("Error! Please, try again!\n\n");
            fflush(stdin);
        }
        fflush(stdin);
        while (printf("Phone Number\t: ") && scanf("%[^\n A-Z a-z]s", &bio[index_struct].phone) < 1) {
            printf("Error! Please, try again!\n\n");
            fflush(stdin);
        }
        fflush(stdin);
        printf("\n");
    }
    printf("Input the index number you'd like the data to be pulled from! ");
    scanf("%d", &index_struct);
    printf("%-10s\n%-64s\n%-10s\n%-16s",
           bio[index_struct].name, bio[index_struct].address,
           bio[index_struct].date, bio[index_struct].phone);
    return 0;
}

当输入是空格时，我试图让每个输入都能够输出错误。 [^\n] 或 [A-Z] 或 [0-9] 的扫描集通常在更简单的情况下对我有帮助。但是，在这一个中，当我在 input_max for

中输入任何数字作为输入时

while (printf("Input the amount of data! ") && scanf("%[0-9]*d", &input_max) < 1) {
    printf("Error! Please, try again!\n\n");
    fflush(stdin);
} 

input_max 给出的数字与给定的数字不同。这里发生了什么？我该怎么做才能避开它？

我也不完全理解这段代码是如何作为错误输出工作的，因为我在网上的某个地方发现了这一点。

编辑:正如@JonathanLeffler 所建议的，scanf() 将我的输入作为 ASCII、ISO 8859-x 或 Unicode 或所有格式的代码点他们中的。但是，当我删除扫描集并将其转换为 scanf(%d, &input_max) 时，输入保持原样。但是，我确实需要扫描集，这样我就可以输入一个空格，并在 scanf() 中输入一个空格时弹出我设置的错误消息。

Answer 1

你误会了 [成为 %s 的修饰符和 %d -例如。 %3d - 它不是。 %[本身是一个转换说明符，工作方式类似于 %s .

因此，正如@user3629249 在评论中指出的那样，s和 d在 %[ 的末尾说明符(例如在 %[^\n A-Z a-z]s 中)是无关紧要的。还有 %[ 中的空格事情。所以%[A-z a-z]不同于%[A-Za-z]

让我们看看在打开格式警告的情况下进行编译时遇到的问题。 (-Wformat，如果您使用的是 gcc 或 clang)，您将得到如下内容:

foo.c:19:68: warning: format specifies type 'char *' but the argument has type 'int *' [-Wformat]
  while (printf("Input the amount of data! ") && scanf("%[0-9]*d", &input_max)<1) {
                                                        ~~~~~      ^~~~~~~~~~
                                                        %d
foo.c:29:55: warning: format specifies type 'char *' but the argument has type 'char (*)[21]' [-Wformat]
    while (printf("Name\t: ") && scanf("%[A-Z a-z]s", &bio[index_struct].name)<1) {
                                        ~~~~~~~~~     ^~~~~~~~~~~~~~~~~~~~~~~
foo.c:34:54: warning: format specifies type 'char *' but the argument has type 'char (*)[65]' [-Wformat]
    while (printf("Address\t: ") && scanf("%[^\n]s", &bio[index_struct].address)<1) {
                                           ~~~~      ^~~~~~~~~~~~~~~~~~~~~~~~~~
foo.c:39:78: warning: format specifies type 'char *' but the argument has type 'char (*)[11]' [-Wformat]
    while (printf("Birthdate\t: (YYYY/MM/DD)\n") && scanf("%[^\n A-Z a-z]s", &bio[index_struct].date)<1) {
                                                           ~~~~~~~~~~~~~     ^~~~~~~~~~~~~~~~~~~~~~~
foo.c:44:67: warning: format specifies type 'char *' but the argument has type 'char (*)[17]' [-Wformat]
    while (printf("Phone Number\t: ") && scanf("%[^\n A-Z a-z]s", &bio[index_struct].phone)<1) {
                                                ~~~~~~~~~~~~~     ^~~~~~~~~~~~~~~~~~~~~~~~

您的代码还有其他问题:

1. 您将生物数据从 1 索引到 10(查看 index_struct for 循环)但是它被声明为大小为 10 的数组 Biodata bio[10];

在 C 中数组是 0基于，所以它们来自 0至 size-1并且你的for-loop会遇到段错误，因为bio[10]将是未定义的。

1. 您要求的是 input_max 在您的 for 循环中，但您需要它用于 for 循环。

2. 如果 input_max 大于 bio 的声明大小会发生什么数组？

其他一些值得考虑的好事:

1. printf 是用于报告错误的糟糕函数，错误应该转到 stderr，而不是 stdout，因此最好使用 fprintf 并指定 stderr。

2. 既然您有兴趣确保正确解析输入，为什么不创建自己的解析器而不是使用 scanf？

3. 你强制重新提示错误，让我们把它分解成它自己的函数。

让我们一起做吧。

关于我在下面使用的 C 语言风格和命名约定的注意事项

我的 C 风格与你的有点不同，我有我的理由 :-)，既然这只是个人意见，那我们就按照我的来吧。

带有一些关于我们想要的内容的注释的结构

struct biodata {
    char name[21];        /* format: FirstName LastName */
    char address[65];     /* format: Free-form upto 65 chars */
    char birthday[11];    /* format: YYYY/MM/DD */
    char phone[17];       /* format: up to digits or a spaces */
};

一些匹配器

下面是一组匹配函数，它接收输入行并告诉我们该行是否完全符合我们的预期。如果是，则返回 true，否则返回 false。您将需要 #include <stdbool.h>为此。

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <stdint.h>
#include <stdbool.h>
#include <time.h>

struct biodata {
        char name[21];        /* format: FirstName LastName */
        char address[65];     /* format: Free-form upto 65 chars */
        char birthday[11];    /* format: YYYY/MM/DD */
        char phone[17];       /* format: up to digits or a spaces */
};


bool match_name(const char *line)
{
    char tmp[128];
        return line!=NULL
        && sscanf(line, "%128[A-Za-z]%128[ ]%128[A-Za-z]", tmp, tmp, tmp) == 3
        && strlen(tmp) < 21;
}

bool match_address(const char *line)
{
    return line != NULL
        && strlen(line) > 5
        && strlen(line) < 65; /* no format check here, maybe later */
}

bool match_telephone(const char *line) 
{
    char tmp[128];
    return line  /* notice the alternate form of checking line!=NULL */
        && sscanf(line, "%128[0-9 ]", tmp)==1
        && strlen(tmp) < 17;
}

/* here we'll use strptime to see if our line is a valid date */
bool match_birthday(const char *line) 
{
     struct tm tm; /* time structure */
     if(!line)
          return false;

     return strptime(line, "%Y/%m/%d", &tm) != NULL;
}

char * ask(const char *prompt, char *line, size_t maxlen) 
{
    printf("%-30s:", prompt);
    fflush(stdout);
    fgets(line, maxlen, stdin);
    return line; /* we return the pointer for ease of use */
}

/* a test function */
void test_matchers() {

    char line[256];
    /* remember ask returns our line so we are able to pass it to our matchers */
    while(!match_name(ask("Name (first last)", line, sizeof(line))))
        ;
    while(!match_address(ask("Address (at least 5 chars)", line, sizeof(line))))
        ;
    while(!match_birthday(ask("Birthday (YYYY/MM/DD)", line, sizeof(line))))
        ;
    while(!match_telephone(ask("Telephone (max 16 digits)", line, sizeof(line))))
        ;
}


int main() 
{
     test_matchers();
     return 0;
}

测试一下。

$ ./bar

Name (first last)             :Ahmed Masud
Address (at least 5 chars)    :1999 Somewhere Out there, Bingo, Bongo, 10002, USA
Birthday (YYYY/MM/DD)         :1970/01/10
Telephone (max 16 digits)     :1-201-555-1212

现在让我们以合理的方式将东西复制到我们的结构中

生物数据打印功能

/* add a function to print a biodata */

void print_bio(const struct biodata *bio)
{
    printf("***** bio data *****\n"
       "Name: %-10s\nAddress: %-64s\nBirthday: %-10s\nPhone: %-16s\n",
           bio->name, bio->address,
           bio->birthday, bio->phone);
}

新的主要功能

注意大部分像test_matches .除了我们添加了复制行到适当的字段

int main()
{

    char line[256];
    struct biodata bio;

    while(!match_name(ask("Name (first last)", line, sizeof(line))))
        ;
    strncpy(bio.name, line, sizeof(bio.name));

    while(!match_address(ask("Address (at least 5 chars)", line, sizeof(line))))
        ;
    strncpy(bio.address, line, sizeof(bio.address));

    while(!match_birthday(ask("Birthday (YYYY/MM/DD)", line, sizeof(line))))
        ;
    strncpy(bio.birthday, line, sizeof(bio.birthday));

    while(!match_telephone(ask("Telephone (max 16 digits)", line, sizeof(line))))
        ;
    strncpy(bio.phone, line, sizeof(bio.phone));

    print_bio(&bio);
    return 0;
}

好吧，我们可以提示用户并将内容放入我们的结构中，但是在 main 中执行它很笨重，所以让我们将其放入自己的函数中。

int get_bio(struct biodata *bio)
{

    char line[256];

    while(!match_name(ask("Name (first last)", line, sizeof(line))))
        ;
    strncpy(bio->name, line, sizeof(bio->name));

    while(!match_address(ask("Address (at least 5 chars)", line, sizeof(line))))
        ;
    strncpy(bio->address, line, sizeof(bio->address));

    while(!match_birthday(ask("Birthday (YYYY/MM/DD)", line, sizeof(line))))
        ;
    strncpy(bio->birthday, line, sizeof(bio->birthday));

    while(!match_telephone(ask("Telephone (max 16 digits)", line, sizeof(line))))
        ;
    strncpy(bio->phone, line, sizeof(bio->phone));

    return 0;
}


int main()
{
    struct biodata bio[3]; /* let's get 3 records */
        int i;

        /* bio is made up of a bunch of struct biodata's so we divide its size by sizeof the struct biodata to get how many (in our case 3) */

        for(i = 0; i < sizeof(bio)/sizeof(struct biodata); i++)
        {
            printf("\n\nEnter record number: %d\n", 1+i); /* why 1+i here ? :) */
            get_bio(&bio[i]);
        }

        for(i=0; i < sizeof(bio)/sizeof(struct biodata); i++)
        {
                print_bio(&bio[i]);

        }
    return 0;
}

练习

我将把其余的功能留作练习。

与此同时，我希望您考虑一下我们开发它的方式。从最里面的功能开始，慢慢向外移动。

像拼乐高积木一样分解问题，首先处理内部零件，测试它们是否完全按照您的要求进行操作，然后围绕它们慢慢构建。

显然，匹配器应该在开发 ask 之前单独开发和测试。我留给你来分解它。