0

假设我有一个以 RAII 方式管理某些资源的类:

class C
{
   HANDLE hResource_;

   // prevent sharing the ownership over the resource among multiple instances of C
   C(const C&);
   C& operator=(const C&);

public:
   C() : hResource_(INVALID_HANDLE){}

   C(int arg1, const std::string& arg2,...)
   {
      ...
      allocResource(arg1, arg2, ...);
      ...
   }

   ~C
   {
      ...
      FreeResource(hResource_);
      hResource_ = INVALID_HANDLE;
      ...
   }

   void allocResource(int arg1, const std::string& arg2, ...)
   {
      if(hResource_ == INVALID_HANDLE)
      {
          hResource_ = AllocateResource(arg1, arg2,...);
      }
   }

   HANDLE handle() {return hResource_;}
};

它的构造函数需要一些资源分配所需的参数,我可以创建它的一个实例,使用它并让它存在于某个范围内:

// some global function 
void goo()
{
   C c(123, "test");
   UseResource(c.handle(),...);
   ... 
}

假设我现在想要一个实例C成为某个类的成员,并且想要延迟在C的 c-tor 中发生的资源分配。这需要C' 默认 c-tor 和 someC执行资源分配的成员函数(例如allocResource()调用AllocateResource())。

class A
{
   C c_;

public:
   void foo1()
   {
      ...
      c_.allocResource(123, "test"); 
      UseResource(c_.handle(),...);
      ...
   }   

   void foo2()
   {
      ...         
      UseResource(c_.handle(),...);
      ...
   }   
};

通过使用专用函数,我们C以某种我不喜欢的方式暴露了 的内部结构。

我的问题是:这种方法是启用延迟初始化的常用方法吗?有没有其他选择?


编辑:这是关于以下(MSalters')建议的可能类设计:

class C
{
   HANDLE hResource_;

   // prevent sharing the ownership over the resource 
   // among multiple instances of C
   C(const C&);
   C& operator=(const C&);

public:      

   // prevent object creation if resource cannot be acquired
   C(int arg1, const std::string& arg2,...)
   {          
      hResource_ = AllocateResource(arg1, arg2,...);

      // assumption: AllocateResource() returns 
      // INVALID_HANDLE in case of failure
      if(hResource_ == INVALID_HANDLE)
         throw resource_acquisition_exception();
   }

   ~C
   {
      ...
      FreeResource(hResource_);
      hResource_ = INVALID_HANDLE;
      ...
   }

   HANDLE handle() {return hResource_;}
};

class A
{
   std::unique_ptr<C> c_;

public:
   void foo1()
   {
      try
      {
         ...
         c_ = std::unique_ptr<C>(new C(123, "test"));
         UseResource(c_->handle(),...);
         ...
      }
      catch(const resource_acquisition_exception& exc)
      {
         ...
      }
      catch(...)
      {
         ...
      }
   }   

   void foo2()
   {
      ...         
      UseResource(c_->handle(),...);
      ...
   }   
};
4

2 回答 2

4

不,这不是执行 RAII 的常用方法。事实上,它根本不是 RAII。如果您无法为 a 分配必要的资源C,请不要创建C.

于 2012-04-17T11:24:16.633 回答
1

问题确实是您暴露了 C 的内部结构,但是您已经使用 handle() 函数这样做了,这已经限制了进行延迟实例化的机会。

如果实际上调用 C 来做某事而不是仅仅获取处理程序会更容易。但是,由于 handle() 是一个 getter,并且您已经可以在构造函数中传递所需的参数(无需实例化,而是通过存储参数),您可以在 handle() 中检查 hResource_ 是否有效,如果不是,则分配资源(如果分配失败则抛出异常)。

于 2012-04-17T11:28:13.513 回答