C++受保护成员的运行时错误

Question

我正在尝试做一个家庭作业，我们使用链接堆栈（因此使用 struct 和 typedef）将一个字符串插入到指定点的字符串中。无论如何，当我尝试访问 InsertAfter 方法内的 StringModifier 类中的 stringLength 时，出现运行时错误，并且无法找出问题所在。我应该能够访问和修改该变量，因为它是受保护的并且派生类是公开继承的。

struct StringRec
{
    char theCh;
    StringRec* nextCh;
};

typedef StringRec* StringPointer;

class String
{
    public:
        String();
        ~String();
        void SetString();
        void OutputString();
        int GetLength() const;
    protected:
        StringPointer head;
        int stringLength;
};

class StringModifier : public String
{
    public:
        StringModifier();
        ~StringModifier();
        void InsertAfter( StringModifier& subString, int insertAt );
};

void StringModifier::InsertAfter( StringModifier& subString, int insertAt )
{
// RUN TIME ERROR HERE
    stringLength += subString.stringLength;
}

在 MAIN 中

StringModifier test;
StringModifier test2;

cout << "First string" << endl;
test.SetString();
test.OutputString();
cout << endl << test.GetLength();
cout << endl << "Second string" << endl;
test2.SetString();
test2.OutputString();
cout << endl << test2.GetLength();
cout << endl << "Add Second to First" << endl;
test.InsertAfter( test2, 2 );
test.OutputString();
cout << endl << test.GetLength();

//String Class

String::String()
{
    head = NULL;
    stringLength = 0;
}

String::~String()
{
// Add this later
}

void String::SetString()
{
    StringPointer p;
    char tempCh;

    int i = 0;
    cout << "Enter a string: ";
    cin.get( tempCh );
// Gets input and sets it to a stack
    while( tempCh != '\n' )
    {
        i++;
        p = new StringRec;
        p->theCh = tempCh;
        p->nextCh = head;
        head = p;
        cin.get( tempCh );
    }

    stringLength = i;
}

void String::OutputString()
{
    int i = stringLength;
    int chCounter;
    StringPointer temp;
// Outputs the string bottom to top, instead of top to bottom so it makes sense when read
    while( head != NULL && i > 0 )
    {
        temp = head;
        chCounter = 0;
        while( temp != NULL && chCounter < (i-1) )
        {
            temp = temp->nextCh;
            chCounter++;
        }
        cout << temp->theCh;
        i--;
    }
}

int String::GetLength() const
{
    return stringLength;
}

StringModifier 类有空的构造函数和析构函数。

Answer 1

只是提示：C++ 中的运行时错误与公共/受保护/私有访问完全无关。编译器在编译代码时已经检查是否遵循所有类成员访问规则。

运行时错误意味着您的程序中存在错误，很可能是某种内存损坏。

Answer 2

您确定运行时错误位置吗？你是怎么检查的？我看到一个非常可疑的地方：

while( temp != NULL && chCounter < (i-1) )
{
        temp = temp->nextCh;
        chCounter++;
}
cout << temp->theCh; // temp can be NULL here?

在循环头中 temp != NULL 始终为 true，或者在某些情况下您确实有 NULL 指针取消引用，这本身就是一个运行时错误。

Answer 3

您可以尝试在 Valgrind（免费）或 Purify（可能不是免费）下运行您的程序，以尽早检测内存错误。错误消息也应该更加清晰。

另外，只需在调试器下运行程序，当程序崩溃时，检查其状态。是你所期望的吗？

Answer 4

您确定运行时错误确实发生在 InsertAfter 函数中吗？在我看来，当您修改 stringLength 时，您应该在 OutputString 中遇到访问冲突，因为您实际上尚未添加字符。在 while 循环中添加 (temp != NULL) 子句几乎可以避免这种情况 - 但是看看如果由于 temp 变为 NULL 而实际上离开循环会发生什么...

响应您的评论：我'恐怕我仍然对这个运行时错误发生在哪里有点怀疑！如果代码确实如问题中给出的那样，并且假设您没有混乱的构建，那么在 InsertAfter 中拥有 AV 或其他内容几乎是不可能的（好吧，在 C++ 中说这是一件危险的事情，但是嘿 - 您只是更改在堆栈上分配的对象的成员的值）。请注意，您无法判断错误发生在 InsertAfter 方法中，只是因为如果您不调用它，错误就会消失 - 实际上，OutputString 中的错误仅通过调用 InsertAfter 才会暴露，因此它应该如果删除 InsertAfter 调用，/em> 就会消失。要进行检查，请使用调试器，或在可疑语句之前和之后向 InsertAfter 添加一些日志记录。

Answer 5

感谢大家的帮助。事实证明，我一开始就在列表中添加了错误的内容。我没有将其设置为堆栈，而是将其设置为队列，一切都运行良好！我听取了你们提供的建议，并在其他地方寻找问题。谢谢你！

void String::SetString()
{
    StringPointer p, last;
    char tempCh;
    last = head;
    int i = 0;
    cout << "Enter a string: ";
    cin.get( tempCh );

    while( tempCh != '\n' )
    {
        i++;
        if( i == 1 )
        {
            p = new StringRec;
            p->theCh = tempCh;
            head = p;
            last = p;
        }
        else
        {
            p = new StringRec;
            p->theCh = tempCh;
            p->nextCh = last;
            last->nextCh = p;
            last = p;
        }
        cin.get( tempCh );
    }

    last->nextCh = NULL;

    stringLength = i;
}