我的输出的第一个字符在 C 中被省略了

Question

我似乎无法弄清楚我的输出发生了什么。我正在读取多行用户输入并输出超出下限的相应输入。由于某种原因，当我输出时，输出的字符串省略了字符串的第一个字符。谁能告诉我为什么会这样？

#include <stdio.h>

typedef struct{
   char  name[4];
   int   population;
} state;

enum { MAX_STATES = 10 };

int main()
{
    state myStates[MAX_STATES];

    int c;
    int i = 0;

    while ((c = getchar())!= EOF)
    {
        scanf("%s %d\n", myStates[i].name, &myStates[i].population);
        i++; 
    }

    // printf("Last character is [%d]\n", c);
    printf("");

    if (c <= 0)
    {
        for(int j = 0; j <= MAX_STATES; j++)
        {
            if(myStates[j].population >= 10)
                printf("%s %d\n", myStates[j].name, myStates[j].population);
            else
                break;
        }
    }

    return 0;
}

输入：

TX 23
CA 45

输出：

X 23
A 45

更新代码：

#include <stdio.h>

typedef struct{
   char  name[4];
   int   population;
} State;

enum { MAX_STATES = 10 };

int main()
{
    State myStates[MAX_STATES];

    int i, j;

    // Function to read in multiple lines (up to 10) of user input; loop 
    // controls in place, detects format problems, prevents string buffer 
    // overflows.
    for (i = 0; i < MAX_STATES; i++)
    {
        if (scanf("%2s %d\n", myStates[i].name, &myStates[i].population) != 2)
            break;
    }

    // Function to output (stdout) array of State structs that exceed 10 
    // population. 
    for(j = 0; j < i; j++)
        {
            if(myStates[j].population >= 10)
                printf("%s %d\n", myStates[j].name, myStates[j].population);
            else
                break;
        }

    return 0;
}

发布的输出只会持续到输入小于 10 并跳出循环。当我没有那个 break 语句时，我在最后一行得到了垃圾输出。有什么改进输出的建议吗？

Answer 1

替换：

int i = 0;

while ((c = getchar()) != EOF)
{
    scanf("%s %d\n", myStates[i].name, &myStates[i].population);
    i++; 
}

与：

int i;

for (i = 0; i < MAX_STATES; i++)
{
    if (scanf("%3s %d", myStates[i].name, &myStates[i].population) != 2)
        break;
}

这可以防止您进入太多状态，使用for 循环来放置循环控件，检测格式问题，防止字符串缓冲区溢出，并将第一个字符读入名称中。此外，如果输入是以交互方式输入的，则格式字符串中的尾随空格（例如空格或换行符）在scanf() 格式字符串中是一个非常糟糕的主意。如果输入来自文件，则不那么严重，但大多数时候仍然没有必要。 （有关更多信息，请参阅Trailing blank in scanf() format。）

保持一个while循环

如果你真的坚持需要while 循环，那么你可以使用：

int i = 0;

while (i < MAX_STATES && (c = getchar()) != EOF)
{
    ungetc(c, stdin);
    if (scanf("%3s %d", myStates[i].name, &myStates[i].population) != 2)
        break;
    i++; 
}

或：

int i = 0;

while (i < MAX_STATES && (c = getchar()) != EOF)
{
    myStates[i].name[0] = c;
    if (scanf("%2s %d", &myStates[i].name[1], &myStates[i].population) != 2)
        break;
    i++; 
}

请注意，这些while 循环仍然保持大量溢出保护——溢出主数组和溢出名称字段。请注意，两个scanf() 语句之一使用%3s，另一个使用%2s；你应该能够解释为什么。 （是的，scanf() 不计算空字节，因此您必须在转换规范中使用“减一”长度。）

毫无疑问，还可以使用其他技术。不过，我想您会发现for 循环更接近于惯用的 C。

通常明智的一种替代方法是使用fgets()（或POSIX getline()，如果可用）读取整行，然后使用sscanf() 解析行。这通常会导致更有弹性的程序和更好的错误报告。它还可以阻止那些试图将所有 50 个州的信息放在一行中，或者将每个数据放在单独的一行中并在它们之间用空行隔开的人逃脱格式错误的数据。你可以悄悄地坚持​​两个字段（如果你小心的话，只有两个字段）就行了。

输出代码呢？

我可以询问有关正确显示输出的建议吗？

你有：

printf("");

if (c <= 0)
{
    for(int j = 0; j <= MAX_STATES; j++)
    {
        if(myStates[j].population >= 10)
            printf("%s %d\n", myStates[j].name, myStates[j].population);
        else
            break;
    }
}

第一个printf() 什么都不做；它应该去。 if (c <= 0) 条件有点可疑。可以输入一个空字节（通常是 Control-@ 或 Control-Shift-2），但要打破原始循环有点困难. for 循环应该更像for (int j = 0; j < MAX_STATES; j++) — 这是 C 中安全 for 循环的模板。您最常使用 for (int i = 0; i < MAX; i++)。但是，您只想打印已读取的状态，因此您需要使用i 作为限制，而不是使用MAX_STATES。如果您真的只想打印前 9 个州（加利福尼亚州、德克萨斯州、佛罗里达州、纽约州、伊利诺伊州、宾夕法尼亚州、俄亥俄州、乔治亚州、北卡罗来纳州 - 请参阅 Wikipedia；它说密歇根州只差 10M），那么 @987654352 @条件很好。

因此，您可以使用（注意输入循环将i 设置为成功读取的状态数）：

for (int j = 0; j < i; j++)
    printf("State: %.2s, Pop'n: %dM\n", myStates[j].name, myStates[j].population);

当然，您可以调整格式以满足您的要求。如果没有读取任何状态或读取的状态数，这将不打印任何内容。如果您真的想将条件应用于人口，那么您可以使用：

for (int j = 0; j < i; j++)
{
    if (myStates[i].population >= 10)
        printf("State: %.2s, Pop'n: %dM\n", myStates[j].name, myStates[j].population);
}

Answer 2

另一种选择是：

int i = 0;
char temp[100];
for(i=0; i<MAX_STATES; i++){
    fgets(temp, 100, stdin);
    if(strcmp(temp, "\n") == 0)
        break;
    sscanf(temp, "%s %d\n", myStates[i].name, &myStates[i].population);
}

Answer 3

您可以尝试在 scanf 中的 %s 之前添加一个空格，或者指定严格的字符数。

scanf(" %3s",

甚至可以像Beej's guide:一样使用

// read all whitespace, then store all characters up to a newline 

scanf(" %[^\n]", s);

Answer 4

您可以尝试在 scanf 中的 %s 之前添加一个空格，或者指定严格的字符数。

或者甚至使用这个：

//读取所有空格，然后将所有字符存储到换行符

scanf(" %[^\n]", s);