动态指向现有或创建新对象

Dynamically point to existing or create new object

我有一个函数,它使用存储在结构 A 中的信息。这个函数经常被调用,大多数时候它只能依赖存储在 A 中的信息。 在某些(罕见的)情况下,A 的对象之一不能使用。必须创建一个新对象,并且它的寿命不应长于函数(因为它只能使用一次并使用大量存储空间)。 我有一些伪代码可以证明我的问题。目前我真的不喜欢它,因为调用 "new" 但我想不出另一种方法来使用智能指针来实现这一点。

正如我在每本书中读到的,现在你不应该直接使用指针并依赖智能指针,我不确定,急于正确实现我想要的东西。

struct A{
  myData data;
}

void often_called_function(int i, A &structA)
{

  // Pointer which shall dynamically point to existing or newly created object
  myData *current_data;

  // we rarely land here, data can't be used
  if (i == 10) {
    current_data = new myData(special arguments);
    current_data->reinit(i);
  }
  // most of the time we land here, no need to re-create data, just reinit existing data from struct A
  else {
    structA.data.reinit(i);
    current_data = &structA.data;
  }

  // do more stuff with current_data
  current_data->do_something(); 
}

所以基本上我正在寻找一个 "nicer" 和安全的方法来完成这个,有人可以帮助我吗? 提前致谢

您可以创建一个智能指针来拥有新对象并在函数结束时释放它,但除非您需要创建新对象,否则将其设为空。当你创建它时,只需让 current_data 引用智能指针管理的对象:

void often_called_function(int i, A &structA)
{

  // Pointer which shall dynamically point to existing or newly created object
  myData *current_data;
  // smart pointer that will own the dynamically-created object, if needed:
  std::unique_ptr<myData> owner;

  // we rarely land here, data can't be used
  if (i == 10){
    owner = std::make_unique<myData>(special_arguments);
    current_data = owner.get();
    current_data->reinit(i);
  }
  // most of the time we land here, no need to re-create data, just reinit existing data from struct A
  else{
    structA.data.reinit(i);
    current_data = &structA.data;
  }

  // do more stuff with current_data
  current_data->do_something();
}

N.B。看起来您可以通过将 reinit(i) 调用移出条件分支来稍微简化函数:

  // we rarely land here, data can't be used
  if (i == 10){
    owner = std::make_unique<myData>(special_arguments);
    current_data = owner.get();
  }
  // most of the time we land here, no need to re-create data, just reinit existing data from struct A
  else{
    current_data = &structA.data;
  }
  current_data->reinit(i);