Question

我正在编写一个程序，以便用户可以创建一个文本文件，存储他们的梦想车辆，车库库存，任何车辆列表的列表。我使用fgets和sscanf向用户询问make，然后是year，然后是model，然后是value，然后是color。但由于某种原因，它跳过获取模型的用户输入阶段并跳转到要求值，但仍然在printf上方显示“输入汽车模型：”的printf以输入值。下面是我的代码中包含我的fgets / sscanf函数的摘录，下面将是输出的示例，这样你就可以看到问题是什么（ha，if）我的描述不是很清楚。

if (!listappend){
    printf("File nonexistent/inaccessible.");
    return 1;
}
    memset(colorin, 0, sizeof(colorin));
    memset(makein, 0, sizeof(makein));
    memset(modelin, 0, sizeof(modelin));
    memset(yearin, 0, sizeof(yearin));
    memset(valuein, 0, sizeof(valuein));


    printf("\nEnter car make: ");
    fgets(makein, sizeof(makein), stdin);
    sscanf(makein, "%[^\n]",makein);
    fprintf(listappend, "\n%s", makein);
    printf("\nEnter car manufacture year: ");
    fgets(yearin, sizeof(yearin), stdin);
    sscanf(yearin, "%d", &yearinf);
    printf("\nEnter car model: ");
    fgets(modelin, sizeof(modelin), stdin);
    sscanf(modelin, "%[^\n]",modelin);
    fprintf(listappend, "\n%s", modelin);
    printf("\nEnter approximate car value: $");
    fgets(valuein, sizeof(valuein),stdin);
    sscanf(valuein, "\n%f", &valueinf);
    fprintf(listappend, "\n%d %'.2f", yearinf, valueinf);
    printf("\nEnter car color: ");
    fgets(colorin, sizeof(colorin), stdin);
    sscanf(colorin, "%[^\n]",colorin);
    fprintf(listappend, "\n%s", colorin);

    fclose(listappend);

示例输出：

|-|-|-|-|-|-|-|-|-|-|Car-lection List|-|-|-|-|-|-|-|-|-|-|
----------------------------------------------------------
[A]dd New Car                   [V]iew List
[M]enu                          [D]eveloper Info
[C]lear List      [Q]uit

Option?
A

Enter car make: Honda

Enter car manufacture year: 2000

Enter car model:
Enter approximate car value: $5000

Enter car color: Blue

Option?
V
    Entry   Year    Color   Make    Model   Color   Approx. Value
    1| 2000 Blue Honda, approximate value: $5,000.00

如您所见，用户应该能够输入汽车模型，但程序会立即询问近似汽车价值，跳过输入汽车型号信息的机会。这里是car_list.txt的cat，我保存要读取的字符串和值的文本文件：

    cat car_list.txt

    Honda


    2000 5,000.00
    Blue

我是C编程的新手，所以如果这是一个愚蠢的错误，我为浪费你的时间而道歉，但它一直让我发疯。提前感谢您的帮助！

Answer 1

对sscanf()的来电不正确：

sscanf(makein, "%[^\n]", makein);

这会导致未定义的行为。 sscanf()的正式POSIX（和C99，C11）规范是：

int sscanf(const char *restrict s, const char *restrict format, ...);

restrict表示对sscanf()的调用中可能没有任何其他参数是s参数的别名（或与其重叠）。在您的代码中，您有makein作为源字符串和目标字符串，这违反了restrict条件，因此会导致未定义的行为。

代码中的重复会因使用函数而哭泣。

您没有向我们展示您定义的字符串的大小。您还从未检查fgets()是否成功，以及sscanf()是否成功。我的怀疑是你的一个字符串太短，所以fgets()实际上并没有读取整行，而'skipped'读取前一行的最后一部分。您可以通过打印输入数据来诊断它。例如，在每个（成功）fgets()之后，您可以打印：

printf("<<%s>>\n", makein);

这将确保程序已经看到了您希望它看到的输入 - 这是最基本的调试技术之一。

所以你说我需要为每个sscanf设置一个新变量，重复第一个参数作为第三个参数？我想说的是：sscanf(makein, "%[^\n]", &makeins)而不是sscanf(makein, "%[^\n]", makein)？

或多或少，是的。

有两个问题需要解决。一个是你对sscanf()的原始调用产生了未定义的行为 - 这很容易。另一部分是你假设每一行的数据都适合变量 - 而且由于你没有显示大小，很难知道这是否合理。还不清楚所有领域是否大小相同;我的推测是它们的大小不同。

有几种方法可以解决这个问题。我可能会写一个函数：

int read_value(const char *prompt, char *buffer, size_t buflen)
{
    char line[4096];   // Big!

    buffer[0] = '\0';  // Null terminate output in case of EOF.
    printf("%s: ", prompt);
    fflush(stdout);    // Optional
    if (fgets(line, sizeof(line), stdin) == 0)
        return EOF;
    size_t len = strlen(line);
    if (line[len-1] == '\n')
        line[--len] = '\0';
    if (len > buflen)
        len = buflen - 1;
    memmove(buffer, line, len);
    buffer[len] = '\0';
    return len;
}

这假定截断数据是正常的，并且不需要跳过前导空格，也不需要追踪除换行符之外的空格。删除前导空间是微不足道的：strspn()是理想的。删除尾随空间不是那么简单;你必须自己编写一个递减循环，注意正确处理一个全空行。

该函数打印提示字符串，并在末尾添加冒号和空白;如果您不想这样做，请仅使用printf("%s", prompt);或fputs(prompt, stdout); - 请勿使用printf(prompt);，因为如果提示符包含百分比符号"Enter discount (%)"，则可能会崩溃，因为例。通常不需要fflush()（系统通常会为您执行此操作），但它会确保提示可见。

fgets()读入大行缓冲区;出于所有实际目的，它确保您阅读全部内容。对换行的测试可以给出一个处理截断行的else子句 - 例如JSON数据（例如书签文件）可以是单一的数据 - 你可以报告错误，或者吞噬到最后线或EOF，或任何其他适合您的机制。

如果数据不适合传递给函数的缓冲区，代码也会截断读取的数据 - 再次，如果您愿意，可以报告错误，或使用更复杂的截断算法而不是简单地覆盖数据最大长度（例如，搜索先前的分词）。

memmove()加号赋值null在复制后终止数据。返回字符串的长度（或出错时为EOF）。这是fgets()的更好版本会自动执行的（并且POSIX getline()函数已经完成了。）

使用此功能，您的代码为：

printf("\nEnter car make: ");
fgets(makein, sizeof(makein), stdin);
sscanf(makein, "%[^\n]",makein);

会变成：

if (read_value("\nEnter car make", makein, sizeof(makein)) == EOF)
    …handle error…

您现有的代码忽略了EOF，因此这是一项改进。我还注意到你的memset()电话不是必需的;返回的字符串将以null结尾。

程序使用fgets和sscanf跳过用户输入的机会

1 个答案: