再帰関数を使った回文判定のプログラム

Question

再帰関数を使って回文判定をするプログラムを書いています。
下のコードで、カンマは正しく処理しますがスペースを受け取るとそこで入力が別々になってしまいます。
どうすればスペースを入力しても、正しく出力できますか？

#include <iostream>
#include <cctype>
using namespace std;

bool palindrome(char str[], int a, int b);

int main() {
    char str[100];
    while (true) {
        cout << "Enter a string: ";
        cin >> str;
        if (strcmp(str, "quit") == 0) {
            break;
        }
        else if (isAPalindrome(str, 0, strlen(str) - 1) == true) {
            cout << str << " is a palindrome" << endl;
        } else {
            cout << str << " is not a palindrome" << endl;;
        }
    }
}

bool palindrome(char str[], int a, int b) {
    if (a > b) {
        return true;
    } else if (ispunct(str[a]) == true || isspace(str[a]) == true) {
        return a++;
    } else if (ispunct(str[b]) == true || isspace(str[b]) == true) {
        return b--;
    } else if (toupper(str[a]) == toupper(str[b])) {
        return palindrome(str, a + 1, b - 1);
    } else {
        return false;
    }
}

/*
出力結果
Enter a string: abA
abA is a palindrome
Enter a string: ab,a
ab,a is a palindrome
Enter a string: ab a
ab is not a palindrome //ここでabとaに分かれる
Enter a string: a is a palindrome //ここは自動で出力される
Enter a string: quit
*/
/*　理想
Enter a string: ab a
ab a is a palindrome
*/

Comments

[kazuma-s]

質問のコードはコンパイルエラーになり、実行できません。
コメントに書かれている出力結果を得られるコードを提示してください。

Answer 1

私が読み違いしているかもしれないけれど、ispunct, isspace 判定にかかった時の return はそれぞれ対象のパラメータを動かした palindrome 関数なのではないでしょうか？
return a++ または b-- だと、そこで再帰がとまってしまうような。返り値 bool でもないし。

Comments

[fana]

こんな感じで，貼られたコードに質問内容（本件で言えば入力の方法である）とは関係ない間違いがある場合，
その話だけを回答として書くべきかどうか？　って迷いますよね．

[WoodenHamlet]

書いた後でitagagakiさんの解答見て「あ、俺質問に答えてねえ！」ってなりました。これは単純に私のミスですねｗ

[fana]

「まず，質問への回答はこれ．　で，それはそれとして……」という形にして，プラスアルファ的に含めてしまう形態の回答が既に成されてしまっていると，
同じ手が使えない（「回答」の部分が同じになっちゃう）から，困るんすよね．

[WoodenHamlet]

そういう場合は、私なんかだと例えば「質問の件に関しては●●さんが回答されている通りだと思います。他に質問者さんのソースを読んでいると、……が気になります」とか回答しますかねえ

Answer 2

cinからの入力はタブやスペースごとに区切られます。
入力にはgetlineを使うと良いと思います。

あと、ispunctとisspaceの戻り値はboolではなくintで0または0以外の何らかの値ですので、boolにキャストせずtrueと比較するのは正しくないです。boolにキャストするか、もしくは0かどうかで判定しましょう。