c ++另一个类的私有对象：初始化并使用它来调用该类中的函数

Question

我有两个类 A 和 B，其中类 B 看起来像

B.h

class B
{
 public:
    B();
    virtual ~B();
    void eval(int a, int b);
 private:
    A* APointer;
};

相应地我有

B.cpp

B::B():APointer(NULL){}
B::~B(){}

void B::eval(int a, int b)
{
   if a == b
      {
      APointer->run(); // run() is a public method defined in class A
      }
}

那么A类就是这样的：

A.h

#include "LuaBridge.h"
extern "C" {
#include "lua.h"
#include "lualib.h"
#include "lauxlib.h"
}
using namespace luabridge;

class LuaParser
{
  public:
    LuaParser(lua_State* L);
    virtual ~LuaParser();
    void run();

  private:
    LuaRef mRun;
    LuaRef mStop;
    lua_State* mL;
};

还有

A.cpp

LuaParser::LuaParser(lua_State* L) :mRun(L),mStop(L),mL(L) {}
LuaParser::~LuaParser(){}

void LuaParser::run(){
std::cout<<"This project runs!"<<std::endl;
}

系统很复杂，实际上我在类C成员函数中调用了这个eval函数。在那个类中，我通过B* BPointer 定义了一个私有成员BPointer，在构造函数中我做了C(B* BPointer = NULL)，然后我只是在类C 成员函数中使用BPointer->eval(a,b)。

在我的 main 代码中，我在 B 类中定义了一个指针，如 B* BPointer，我使用该指针调用方法 B::eval by

BPointer -> eval(a, b);

但是，当我在 Visual Studio 中逐步运行它时，在命令行APointer->run(); 我注意到this 指针类似于：

Value: 0xcdcdcdcd{APointer=???}

当我尝试运行此命令时，出现错误：

Access violation reading location 0xCDCDCDD1.

嗯...我不知道如何解决这个问题，我想知道的是：

1. 整个想法（即在此类的方法中使用私有对象调用另一个类的函数）是否可行？

2. 如果这个想法可行，那我应该如何修改我的代码呢？

欢迎提出任何建议或想法！

Answer 1

是整个想法（即使用私有对象调用函数 这个类的方法中的另一个类）是否可行？

是的，这很常见。

如果这个想法可行，那我应该如何修改我的代码呢？

您需要了解对象和指针之间的区别。指针是内存区域的地址，甚至可能未分配。

这里有两个指针不指向任何分配的内存，因此没有可以调用其方法的对象。

BPointer 是未初始化的，它包含随机内存区域的地址（或调试版本中的幻数 0xcdcdcdcd），其中可能包含任何属于或不属于你的东西。取消引用它是undefined behavior。为避免这种情况，您应该创建B 对象并将其指针分配给BPointer：

B *BPointer = new B;

在你使用完BPointer之后，你应该释放内存并通过调用来调用B的析构函数

delete BPointer;

APointer 用NULL 指针初始化，这意味着它不指向任何内存区域。取消引用 NULL 也是未定义的行为。和BPointer类似，应该用有效的对象指针初始化，稍后释放：

B::B() : APointer(new A(/* lua state pointer */ NULL)) {}
B::~B() { delete APointer; }

---

如果您不一定需要指针，我建议对B 使用堆栈分配并将A 存储为普通成员（而不是指针）。这样您就无需为指针初始化和删除而烦恼。

B.h

class B
{
 public:
    B();
    void eval(int a, int b);
 private:
    A AObject;
};

B.cpp

B::B() : AObject(/* lua state pointer */ NULL) {}

void B::eval(int a, int b)
{
    if (a == b) 
    {
        AObject.run();
    }
}

main.cpp

// ...
B b;
b.eval()
// ...

Answer 2

0xcdcdcdcd

是已分配但尚未初始化的内存的状态。您是否以适当的方式实例化了您的BPointer？例如

BPointer = new B(); 

除了你的 APointer 是 NULL 所以你不能调用任何方法

APointer->run();

在构造 A 类型的对象并将其也分配给 APointer 之前。

Answer 3

您必须在 B 类的构造函数中使用一些真实对象初始化 APointer 数据成员。它应该如下所示，

B::B()
{
   this->APointer = new A();
}

B::~B()
{
  delete this->APointer;
  this->APointer = NULL;
}
...
...

int main()    
{
   B* BPointer = new B();

   int x = 5;
   int y = 5;

   BPointer->eval(x, y);
}

在您上面提到的代码中，APointer 被初始化为NULL。我认为当B::eval()被执行时，会导致一些未定义的行为。

如果你也能贴出A类的代码就好了。