C++ "Getter"方法在尝试返回同一类中的私有(private)变量时抛出访问冲突

Question

我遇到的问题是，每当我的代码中的“getter”方法试图返回它为其设计的变量时，它就会抛出访问冲突异常。我在下面包含了构造函数，因为最后一行(cout)不会导致访问冲突。 LinkedNode 是一个头文件。

#ifndef _LinkedNodeClass_
#define _LinkedNodeClass_

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

class LinkedNode {
public:

LinkedNode(string first, string last, int ID, LinkedNode* rightNode, LinkedNode* leftNode)
{
    Fname = first;
    Lname = last;
    EID = ID;
    right = rightNode;
    left = leftNode;

    cout<<"LNODE EID: "<<EID<<endl;  //Does not cause access violation
}

int getEID()
{
    return EID;  //Does cause access violation
}

private:

string Fname;
string Lname;
int EID;
LinkedNode *right;
LinkedNode *left;

};

#endif

LinkedNode 对象由一个名为 head 的 LinkedNode 指针指向。这是使用 LinkedNode 的类中的代码:

#include <iostream>
#include <fstream>
#include <string>
#include "LinkedNode.h"

using namespace std;

class MJCTree
{
private:
LinkedNode* head;
int size;

public:
MJCTree()
{
            head = 0;  //not sure how to null.
    size = 0;
}

void insert(string first, string last, int bEID)
{
    if(size == 0)
    {
        head = new LinkedNode(first, last, bEID, 0, 0);
        ++size;
    }
}

int getFirst() //This method leads to the Access Violation
{
    return head->getEID();
}
};

和异常:

First-chance exception at 0x013821f6 in Project 3.exe: 0xC0000005: Access violation reading location 0xcccccd0c.
Unhandled exception at 0x013821f6 in Project 3.exe: 0xC0000005: Access violation reading location 0xcccccd0c.

Answer 1

您的 LinkedNode 类看起来不错，但您的 MJCT 类很危险。getFirst 方法使用 head，但不能保证 head 已经被初始化。 MJCTree 构造函数不会初始化它，如果您还没有尚未调用插入，您可能会遇到访问冲突。

类构造函数应该初始化一个对象并使其处于可以安全使用的状态。您的 MJCTree 构造函数不会这样做，因为对 getFirst 函数的调用将使用空(或以前未初始化的)指针。您可以修改构造函数，以便它分配一个 head 节点供 getFirst 使用。或者您可以修改 getFirst，以便通过添加对 size>0 或 head!=0 的检查，即使 head 为 null 也可以安全调用。如果该检查失败，则不能使用 head 并且需要返回错误或抛出异常。

我们需要查看创建 MJCTree 实例并调用 getFirst 的代码，以便准确判断出了什么问题，但设计一个可以安全/正确使用的类更为重要。