调试断言失败 c >= -1 && c <= 255

Question

我很难用我的代码理解这个运行时错误。这是我班上的一项作业，我认为这很容易，但有些奇怪的事情正在发生。我在下面发布了我的代码。

这个任务的提示是创建一个程序，要求用户输入一些单词，然后它应该输出有多少单词。我有一个朋友运行该程序并且它可以工作，但是，每当我尝试运行它时，我都会收到此运行时错误。 [http://imgur.com/FcdN3zK]

请记住，我是 C++ 和一般编程的初学者，所以我无法理解非常技术性的响应。提前感谢您的帮助。

#include <string>
#include <iostream>
#include <cstring>

using namespace std;

int wordCounter(char *);

int main()
{
    char *stringArray = nullptr;
    stringArray = new char[120];

    cout << "Please enter a saying or a phrase that has more than one word: " << endl;
    cin.getline(stringArray, 120);

    int words = wordCounter(stringArray);
    cout << "Your statement contains " << words << " words." << endl;

    system("pause");
    return 0;
}

int wordCounter(char *stringTest)
{
    char characterToTest;
    int numberOfWords = 0;
    for (int i=0; i < 120; i++)
    {
        characterToTest = *(stringTest + i);
        if (isspace(characterToTest) && i != 120)
        {
            char characterToTestTemp = *(stringTest + (i + 1));
            if (isalnum(characterToTestTemp))
            {
                numberOfWords++;
            }
        }
    }
    return numberOfWords;
}

Answer 1

您不会在输入字符串的终止处停下来，而是进入输入缓冲区尾部留下的非输入、不确定的缓冲区数据。

这可能是你想要做的：

int wordCounter(const char *str)
{
    int numberOfWords = 0;
    while (*str)
    {
        // skip any leading whitespace
        while (*str && isspace(static_cast<unsigned char>(*str)))
            ++str;

        // if we're still on string data, we have another word
        if (*str)
        {
            // skip everything up to more whitespace
            while (*str && !isspace(static_cast<unsigned char>(*str)))
                ++str;

            // and count the word
            ++numberOfWords;
        }
    }
    return numberOfWords;
}

或类似的东西。可能需要应用额外的处理来考虑标点符号等。一旦到达输入字符串的终止符，这就会停止。

祝你好运。

Answer 2

isspace 和朋友很难正确调用。具体来说，您不能安全地将char 传递给他们，因为char 的值可以是负数，这是不允许的，如错误消息所述。

为了安全起见，您应该转换为 unsigned char：

isspace(static_cast<unsigned char>(characterToTest))

这不是这里的主要问题，您需要先修复溢出，但这也是您收到错误消息的部分原因。

Answer 3

无论实际读取的输入字符串的长度如何，您的 wordcount 函数始终处理 120 个字符。因此，您正在阅读字符串末尾并处理未初始化的内存。

Microsoft 文档显示“如果 c 不是 EOF 或在 0 到 0xFF 范围内（含），则 _isctype 和 _isctype_l 的行为未定义。当使用调试 CRT 库且 c 不是这些值之一时，函数会引发断言。”

这就是断言的原因。

C 字符串以零结尾，您必须在循环中测试该条件。

int wordCounter(char *stringTest)
{
    char characterToTest;
    int numberOfWords = 0;
    for (int i = 0; i < 120; i++)
    {
        characterToTest = *(stringTest + i);
        if (characterToTest == 0)  
            break; // <-- exit the loop at the end of the string
        if (isspace(characterToTest) && i != 120)
        {
            char characterToTestTemp = *(stringTest + (i + 1));
            if (isalnum(characterToTestTemp))
            {
                numberOfWords++;
            }
        }
    }
    return numberOfWords;
}

您的计数函数中可能存在其他逻辑错误，例如不计算第一个单词。

Answer 4

感谢大家的帮助。阅读此内容（我确实意识到我有逻辑错误，如果用户不输入任何内容并提交它，它仍然会打印 1），我能够想出一个稍微不同的方法。

这是修改后的代码。

#include <string>
#include <iostream>
#include <cstring>

using namespace std;

int wordCounter(char *, int);

int main()
{
    cout << "Please enter a saying or a phrase that has more than one word: " << endl;

    string testString;
    getline(cin, testString);

    int stringLength = testString.length();

    char *stringArray = nullptr;
    stringArray = new char[stringLength];

    strcpy(stringArray, testString.c_str());

    int words = wordCounter(stringArray, stringLength);
    cout << "Your statement contains " << words << " words." << endl;

    system("pause");
    return 0;
}

int wordCounter(char *stringTest, int length)
{
    char characterToTest;
    int numberOfWords = 1;
    for (int i=0; i < length; i++)
    {

            characterToTest = *(stringTest + i);

            if (isspace(characterToTest) && (i != length))
            {

                    char characterToTestTemp = *(stringTest + (i + 1));
                    if (isalnum(characterToTestTemp))
                    {
                            numberOfWords++;
                    }
            }
    }
    return numberOfWords;
}