Question

我在设计回文检查器时遇到问题。我对单个单词（“ noon”，“ 2002”等）没有任何问题，但是每次当我输入包含多个带有空格的单词的短语（例如“ laminate pet animal”）时，我的程序都会失去理智并进入无限循环。也许与检查有关（请确保字符串不为NULL或大于80个字符），然后放入其中？我一直在一步步调试，没有成功。我认为这与字符串在内存中的存储方式有关，但我无法将其准确放置。

    //Preprocessor directives
    #include <iostream>
    #include <cstdlib>
    #include <string>
    #include <iterator>
    #include <vector>
    #include <stdlib.h>

    using namespace std;

    //Function declarations
    string input();
    bool check(string a);

    int main()
    {
        //Repeater variable
        bool rep = 1;
        bool result = 0;

    //Declares string to be checked
    string palin;
    while (rep == 1)
    {
        //Creates string and calls input function
        palin = input();

        //Close function if stopped midway
        if (palin == "2")
        return 0;

        result = check(palin);

        //Displays the results
        if (result == 1)
            cout << palin << " is a palindrome." << endl;
        else
            cout << palin << " is not a palindrome." << endl;

        //Ask if the user wants to enter another Palindrome
        cout << "Continue? (1 for yes, 0 for no): ";
        cin >> rep;
    }

    cout << "Closing program..." << endl;
    system("pause");
    return 0;
}

string input()
{
    //Asks for and receives input string
    string temp;
    cout << "Please enter a string (type zero or 0 to quit): ";
    cin >> temp;

    //Close program if user enters 0 or zero
    if (temp == "0" || temp == "zero")
    {
        cout << "Exiting program..." << endl;
        system("pause");
        return "2";
    }

    //Check if string is null, then ask for input again
    if (temp.empty())
    {
        cout << "There is nothing entered. Please enter a string: ";
        cin >> temp;
    }

    //Check if string is too long, ask for input again
    if (temp.length() >= 80)
    {
        while (temp.length() > 80)
        {
            cout << "The string is too long. Please enter a smaller string: ";
            cin >> temp;
        }
    }
    return temp;
}

bool check(string a)
{
    //Creates 2 iterators that traverse the string
    string::iterator test1;
    string::reverse_iterator test2;

    test1 = a.begin();
    test2 = a.rbegin();

    //Continue until the end of either side of the string
    while (test1 != a.end() && test2 != a.rend())
    {
        //Check if the current symbol is alphanumeric
        while (test2 != a.rend() && !isalnum(*test2))
            ++test2;
        while (test1 != a.end() && !isalnum(*test1))
            ++test1;
        //Break loop when you hit the end
        if (test1 == a.end() || test2 == a.rend())
            break;
        //If they're not the same it's not a palindrome, exit function
        if (tolower(*test1) != tolower(*test2))
            return 0;

        ++test1;
        ++test2;
    }
    return 1;
}

Answer 1

std::cin的{{1}}运算符只能读取下一个空格字符。如果要阅读整行，请使用>>。

std::getline()

发生的事情是，输入“ race car”后的第一个读取操作将读取“ race”，然后将“ car”留在流中，然后下一个读取操作将读取“ car”，从而导致意外的行为，因为您的代码预期为1或0。

这与您的问题无关，但it's usually good form to not use using namespace std。原因多种多样，但是最基本的原因是，如果编写自己的名为cin >> temp; //reads until the next whitespace getline(cin, temp); //reads until the next newline character的函数，则会遇到问题。

Answer 2

用cin >> temp;替换getline(cin, temp);以获得空格分隔的字符串，并在cin >> rep;之后添加cin.ignore();刷新换行符。

Answer 3

我觉得您已经使该代码复杂化了：我将在几行代码中为您展示一个简化的版本。该代码简洁，简洁，易读且表达能力强；它会完全按照它说的去做。然后，我将在使用合适的工具或算法完成工作的同时，对实现的描述进行最后的解释，以说明您出了错：

#include <algorithm>
#include <iostream>
#include <string>

void runPalindrome();

int main() {
    runPalindrome();
    return 0;
}

void runPalindrome() {
    std::string quit;
    do {
        std::cout << "Please enter text to test if it is a palindrome:\n";
        std::string input;
        std::getline(std::cin, input, '\n');

        std::string checker(input);
        std::reverse(checker.begin(), checker.end());

        if (input == checker)
            std::cout << input << " is a palindrome!\n";
        else 
            std::cout << input << " is not a palindrome!\n";

        std::cout << "Press Q or q to (Q)uit or any other character to continue...\n";
        std::getline(std::cin, quit, '\n');

    } while ( quit != std::string( "Q" ) && quit != std::string( "q" ) );

    std::cout << "Exiting the program!\n";
}

您遇到的问题是您使用的是std::cin >> variable，它将带入直到看到的第一个white space character的文本。该行上的其余文本仍在缓冲区中，但没有存储到变量中。在这里，您需要使用std::getline()，它至少需要两个参数，第一个是输入的来源，例如std::cin或std::ifstream或std::istringstream等。第二个参数是您要存储信息的变量。

第三个参数是可选的，在这种情况下，我们确实要使用它。第三个参数寻找定界符，在这种情况下，我们要寻找第一个换行符'\n'。我们在这里使用它的原因是它将从iostream缓冲区中检索它，但不会将其存储到您的字符串中。当我们检查它是否是回文检查时，这非常有用。

获得用户输入的文本字符串后，我们将创建一个名为std::string的{{1}}变量，并使用原始输入对其进行初始化。我们需要直接复制，因为在checker标头中找到了一种称为algorithm的算法，对于我们来说，这是完美的！

但是我们需要该副本，因为std::reverse将执行适当的操作。我们不想丢失原始字符串。因此，现在我们的检查器将与原始输入相反。然后，只需进行一次比较就可以很简单地看到两个字符串是否相等，它们是否显示了适当的消息，以及是否显示相同的消息。最后，我们打印一条消息，询问用户是否要退出。

编辑：-注意：-我已经忘记或忽略了有关该程序的简单内容，上面的回文是std::reverse {{1 }}，我们可以做三件事之一，首先可以将其保留在期望case的地方。我们可以通过将所有字母转换为sensitive或'A' != 'a'来解决此问题，以消除所有区分大小写的问题，但是这些函数仅对单个字符起作用，而不是对整个字符串起作用，因此我们必须编写一个函数以使所有字符字符串中的字符全部大写或全部小写，或者由stl调用另一个不错的算法，即::toupper。再次在::tolower库中找到std::transform(...)。最后但并非最不重要的一点是，我们可以为用户提供两个版本之间的选择。我将把这部分留给您作为练习。

-示例-std::transform()

Algorithm

回文检查器代码陷入无限循环

3 个答案: