Question

我是编程新手，我拿起了一本C手册来自学。我不想使用数组，因为我试图用getchar（）练习。如果用户输入除数字或字母之外的任何内容，我希望能够输出错误消息。我也在尝试练习C库函数isalpha（）和isdigit（）。这是我到目前为止所写的，但我的输出并不完全正确。

输入1 ：“你好”

预期输出 ：“有效输出”

输入2 ：“hello5”

预期输出 ：“有效输出”

输入3 ：“你好！”

预期输出 ：“输出无效”

但我的程序会为上面的所有三个输入返回“有效输入” 请帮助新手尝试学习。我非常感谢。

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

int main ()
{
    char ch;
    int len;
    int valid;


    printf("Enter a word: ");

    for(length = 0; (ch = getchar()) != '\n'; len++)
    {

        if(isalpha(ch) || isdigit(ch))
        {
            valid = 1;          
        }
        else
        {
            valid = 0;
        }

    }
    printf("Input length: %d\n", len);

    if (valid == 1)
    {
        printf("Valid\n");
    }
    if(valid == 0)
    {
        printf("\n");
    }

    return 0;   
}

Answer 1

这里的错误是在检查每个字符时，通过将有效值切换为0和1，实际上您只查看最后一个字符。一旦找到一个不是数字/字母字符的字符然后退出

，你想要做的就是每个字符

你可以用这样的东西替换你的if / else

vlaid = 1; //always assume its valid 
for(int length = 0; (ch = getchar()) != '\n'; length++)
{

    if(!isalnum(ch)) //if not a valid character 
    {
        valid = 0; // string is not valid anymore
        break; // we can exit the loop 
    }

}

Answer 2

你快到了。几个陷阱：

首先，您的变量名称为“length”而不是“len”。

第二个正如Mitchel0022所说，你的程序将始终显示“有效”，前提是输入的最后一个字符是有效的，因为你在每次迭代时为变量“valid”重新分配一个新值。但你不必使用'break语句，因为你需要循环才能获得你的长度，所以坚持你的旗帜。

现在你的程序运行正常。复制并粘贴以下代码：

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

int main ()
{
    char ch;
    int len;
    //flag is set true
    int valid = 1;

    printf("Enter a word: \n");

    for(len = 0; (ch = getchar()) != '\n'; len++)
    {
        //change flag if character is not number nor letter
        if(!isalpha(ch) && !isdigit(ch))
        {
            valid = 0;
        }
    }
    printf("Input length: %d\n", len);

    if (valid == 1)
    {
        printf("Valid\n");
    }
    else
    {
        printf("Invalid\n");
    }

    return 0;   
}

如何使用getchar（）循环访问用户输入并检查字符是字母还是数字

2 个答案: