如果用户输入超过 16 个字符，为什么我的 C 程序会打印两次消息？ [关闭]

Question

我正在尝试制作一个小型帐户注册程序。它提示用户输入userID 和password。我正在处理userID 部分，但它似乎不对。如果我输入< 8 字符userID，它工作正常，但是当我输入超过16 字符的userID 时，要求用户输入不同userID 的消息会打印两次。这是为什么呢？

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

int main(void)
{
    // Setting up variables
    char userID[18];
    char passcode[51];

    // Firstly promt the user for user ID
    do // This loop is for asking the user to type another userID if his chosen one isn't approprite
    {
        printf("Please choose your user ID! (it must be between 6 - 16 charaters long)\n");
        for (int i = 0; i < 17; i++) // This loop store the userID into the character array
        {
            userID[i] = getchar(); // Store userID into array character by         character
            if (userID[i] == '\n')
            {
                userID[i+1] = '\0';
                break;
            }

        }
    }
    while (strlen(userID) < 7  ||  (strlen(userID) > 16  &&  userID[16] != '\n'));

    // Printing info on the screen.
    printf("Your acount info are:\n\tuserID  : %s\tpasscode: ", userID);

Answer 1

如果我输入 < 5 字符 userID 效果很好，但是当我输入超过 16 字符的 userID 时，要求用户输入不同 userID 的消息会打印两次.这是为什么呢？

• 当你输入超过16字符时，17字符被扫描到字符串userID中，因为你没有输入'\n'，所以不会在字符串末尾放置空字符userID.

• 然后，当您使用 strlen(userID) 时，这会调用 未定义的行为，因为末尾没有终止 \0 字符。尝试这样做：

• 当您重新开始迭代时，剩余的额外整数将被纳入。

---

解决方案：

为避免上述问题：

1. 在每次迭代中在index + 1 位置放置一个空字符。

2. flushstdin 在每次迭代结束时。

   do
   {
       printf("Please choose your user ID! (it must be between 6 - 16 charaters long)\n");
   
       for (int i = 0; i < 17; i++)
       {
           userID[i] = getchar();
           userID[i+1] = '\0'; //placing null character at the position index+1
   
           if(userID[i] == '\n')
           {
               userID[i] = '\0';
               break;
           }    
   
       }
   
       fflush(stdin); //flushing the extra input at the end of each iteration
   
   } while (strlen(userID) < 7  ||  (strlen(userID) > 16  &&  userID[16] != '\n'));
   

示例输入：

more than 16 123456789
1123
123456789

样本输出：

Please choose your user ID! (it must be between 6 - 16 charaters long)
more than 16 123456789
Please choose your user ID! (it must be between 6 - 16 charaters long)
1123
Please choose your user ID! (it must be between 6 - 16 charaters long)
123456789

Your acount info are:
    userID  : 123456789
    passcode: 

---

注意：

避免使用fflussh(stdin)，因为它存在comments 中@JonathanLeffler 所建议的可移植性问题。还有一个解释性帖子为什么不使用fflush(stdin)：click

这是避免使用fflush(stdin)的代码实现

do
{
    int i;

    printf("Please choose your user ID! (it must be between 6 - 16 charaters long)\n");

    for (i = 0; (userID[i] = getchar()); i++)
    {

        userID[i+1] = '\0'; //placing null character at the position index+1

        if(userID[i] == '\n')
        {
            userID[i] = '\0';
            break;
        }

        //if user enters more than 16 characters then,
        //scan in 17th character for `strlen(userID)` purpose and
        //consume all other characters using an integer `c`
        if( i > 17)
        {
            int c;

            while( ((c = getchar())!= '\n') && (c!=EOF) );
            break;
        }
    }

} while ( (strlen(userID) < 7)  ||  (strlen(userID) > 16 ) );