Try and catch with while loop：仅当第一个输入不正确时，即使在输入正确后也会跳转到catch块-C

Question

我试图检查用户是否输入了他们的名字，有或没有空格，即乔·布洛格斯。他们的名字中不能有特殊的字符或数字，否则会弹出一条错误信息，即Jo3_Bl0ggs。我想做的是，如果他们以错误的格式输入他们的名字，一条错误信息将被提醒，程序将要求用户再次输入他们的名字，直到他们输入正确。

我使用的是当循环这样做，如果它是正确的，我改变了标志的值，并打破了循环，如果不是，我会重新运行setName（）函数，要求他们的名字。

然而，我遇到的问题是，如果他们第一次输入不正确，它会要求他们再次输入自己的名字，如果第二次输入正确，则会显示一条消息“Welcome Joe Bloggs”，但循环将继续，并要求他们再次输入自己的名字。

我能避免这个问题的唯一方法是，如果第一个输入是正确的，但这样做会破坏try-and-catch块的整个要点。

下面是我关心的两个函数。如果有人能给我指出正确的方向，那就太好了。我是c语言的新手，这就是为什么我对此有点困惑的原因。

inputclass::inputclass(){//empty constructor}

void inputclass::validateName(string name){
    int flag = 1;
    while ( flag == 1){
        try{

            for (int i=0;  i < name.length(); i++){

                if (name[i] != ' '){

                    if (!isalpha(name[i])){

                        cout << "You have entered incorrectly. Please try again. "  << endl;
                        cin.clear();   //I'm trying to clear input buffer but still doesn't work
                        cin.ignore(256, '\n');
                        setName();
                        throw 0;  //do i need this?

                        break; //do i need breaks?

                    }else if(i == name.length()-1){

                        cout << "Welcome to the program " << name << endl;
                        flag = 2;   //break out of while loop
                        break;  //not sure if this does anything
                    }
                }
            }
        }catch (int e){
            cout << "There's been an error" << endl;
            setName();   //until input is correct, this will ask the user for their name.
        }
    }
}
void inputclass::setName(){

    string name;
    cout << "Please enter your name: " << endl;
    getline(cin, name);
    validateName(name);


}

Answer 1

我诚实地认为你的算法逻辑本身是错误的。验证应该作为输入的结果来完成。输入本身应该是验证失败时回收的决策点

1、编写一个简单的验证函数

这比你想象的要容易，尤其是因为标准库提供了一些简洁的算法来为你做很多迭代工作。

2.在setName（）中专门集成验证功能；而不是相反。

你的输入机制应该接受所说的输入，验证，如果无效，循环输入。验证器应该只返回yes或nay。将输入机制留给输入函数，不要从验证器触发它们。验证器就是为了这个；验证，而不是循环输入循环。

第一个非常简单，因为标准库为您提供了所需的大多数算法。下面的示例使用了一个函子来实现这一点，但C 11可以用lambda轻松实现这一点：

struct is_not_space_or_alpha
{
    bool operator()(char c) const
    {
        return c != ' ' && !std::isalpha(static_cast<unsigned char>(c));
    }
};

static bool is_valid_name(const std::string& name)
{
    return !name.empty() &&
        name.end() != std::find_if(name.begin(), name.end(), is_not_space_or_alpha());
}

这样一来，你的setName（）就成为了验证输入结果的位置；验证不会在内部触发更多输入。这由调用方（setName（））决定：

std::string name;
while (true)
{
    std::cout << "Please enter your name: ";
    std::flush(std::cout);

    if (std::getline(std::cin, name) && is_valid_name(name))
    {
        std::cout << "Welcome to the program " << name << "!\n";
        break;
    }

    std::cout << "You have entered incorrectly. Please try again. \n";
    std::cin.clear();
}

样本输出

Please enter your name: Monkey777
You have entered incorrectly. Please try again. 
Please enter your name: Wiley Coyote
Welcome to the program Wiley Coyote!

底线是：不要触发来自验证的输入。相反，从输入触发验证并相应地采取行动。