Question

我有一个简短的程序，用于计算字符串中辅音的数量，方法是首先测试数组中的字符是否为字母字符（跳过任何空格或标点符号）。我的＃34; if（isalpha（strChar））＆＃34;我一直在调试断言失败代码行。

＆＃34; strChar＆＃34;是一个在for循环中赋予char值的变量

很抱歉，如果这是一个补救问题，但我不确定我哪里出错了。在此先感谢您的帮助！

#include <iostream>
#include <cctype>
using namespace std;
int ConsCount(char *string, const int size);


int main()
{
    const int SIZE = 81; //Size of array
    char iString[SIZE]; //variable to store the user inputted string

    cout << "Please enter a string of no more than " << SIZE - 1 << " characters:" << endl;
    cin.getline(iString, SIZE);

    cout << "The total number of consonants is " << ConsCount(iString, SIZE) << endl;
}

int ConsCount(char *string, const int size)
{
    int count;
    int totalCons = 0;
    char strChar;

    for (count = 0; count < size; count++)
    {
        strChar = string[count];
        if (isalpha(strChar))
            if (toupper(strChar) != 'A' || toupper(strChar) != 'E' || toupper(strChar) != 'I' || toupper(strChar) != 'O' || toupper(strChar) != 'U')
            totalCons++;
    }
    return totalCons;
}

Answer 1

我猜问题是你总是循环播放81个字符，即使输入的字数较少。这会导致一些随机数据输入isalpha()。

无论如何，我会更改代码以使用std::string代替char iString[SIZE]来获取输入文本的实际长度。

Answer 2

函数ConsCount(char* string, const int size)应该是这样的：

int ConsCount(char *string, const int size)
{
    int consCount = 0;

    char* begin = string;
    for (char* itr = begin; *itr != '\0'; ++itr)
    {
        if (isalpha(*itr)) {
            char ch = toupper(*itr);

            switch(ch) {
                case 'A':
                case 'E':
                case 'I':
                case 'O':
                case 'U':
                    break; // If ch is any of 'A', 'E', 'I', 'O', 'U'
                default:
                    ++consCount;
            }
        }
    }

    return consCount;
}

正如您所看到的，我使用switch替换了if语句以获得更好的可读性，并使用char*作为迭代器来迭代字符串。您可以删除代码中未使用的参数int size。

我还建议您使用std::string作为安全代码。它还为您提供iterator类来迭代std::string。

Answer 3

int ConsCount(char *string, const int size)
{
    int consCount = 0;

    char* begin = string;
    for (char* itr = begin; *itr != '\0'; ++itr)
    {
        if (isalpha(*itr)) {
            char ch = toupper(*itr);

            switch(ch) {
                case 'A':
                case 'E':
                case 'I':
                case 'O':
                case 'U':
                    break; // If ch is any of 'A', 'E', 'I', 'O', 'U'
                default:
                    ++consCount;
           }
        }
    }

    return consCount;

试试这个

}

isalpha导致＆＃34; Debug Assertion失败＆＃34;在c ++中

3 个答案: