【问题标题】:transform array of objects into array of pointers to unique objects将对象数组转换为指向唯一对象的指针数组
【发布时间】:2018-01-22 19:36:39
【问题描述】:

我正在尝试将对象数组转换为对象指针数组,其中指针指向包含第一个数组的所有唯一对象的数组元素。

我使用的对象复制起来并不便宜,因为它们涉及缓冲区分配和缓冲区复制。但是,它们移动起来很便宜。

示例: 数组

[G,F,E,G,E,G]

应该转化为唯一的对象数组
U = [E,F,G] 和一个指针数组
P = [&U[2], &U[1], &U[0], &U[2], &U[0], &U[2]]

我目前正在使用以下代码来实现这一点:

int N; // 50 Millions and more
std::vector<MyObj> objarray; // N elements
std::vector<MyObj*> ptrarray; // N elements
...
std::vector<MyObj> tmp(objarray.begin(), objarray.end());

std::sort(objarray.begin(), objarray.end());
auto unique_end = std::unique(objarray.begin(), objarray.end());

// now, [objarray.begin(), unique_end) contains all unique objects

std::map<MyObj, int> indexmap;

// save index for each unique object
int index = 0;
for(auto it = objarray.begin(); it != uniqueend; it++){
    indexmap[*it] = index;
    index++;
}

//for each object in original array, look up index in unique object array and save the pointer
for(int i = 0; i < N; i++)
    ptrarray[i] = &objarray[indexmap[tmp[i]]];

有没有更有效的方法来实现这一点,可能不需要创建原始数组的副本,因为对象副本很昂贵?

【问题讨论】:

  • 它们移动便宜吗?而且该代码看起来是错误的,因为它在跟踪它们的来源之前进行了排序和消除。另外,在消除重复后,它无法修剪数组的末尾。
  • 是的,搬家很便宜。我将此信息添加到帖子中。修剪未显示在代码中。它发生在填充 ptrarray 之前。
  • 另一个问题是您要存储指向向量中元素的指针。如果以任何方式调整该向量的大小,则这些指针可能会失效。例如:ptrarray[i] = &amp;objarray[indexmap[tmp[i]]];
  • @Abator Abetor,使用每个对象的向量索引(位置)作为标识符让我担心..
  • @PaulMcKenzie 我知道这一点。设置指针后 objarray 不会改变。 ptrarray 和 objarray 的生命周期也是相同的。

标签: c++ arrays algorithm duplicates


【解决方案1】:
struct r {
  std::vector<MyObj> objects;
  std::vector<MyObj*> ptrs;
};

r func( std::vector<MyObj> objarray ) {

  // makes a vector containing {0, 1, 2, 3, ..., N-1}
  auto make_index_buffer = [&]{
    std::vector<std::size_t> r;
    r.reserve(objarray.size());
    for (std::size_t i = 0; i < objarray.size(); ++i)
      r.push_back( i );
    return r;
  };

  // build a buffer of unique element indexes:
  auto uniques = make_index_buffer();

  // compares indexes by their object: 
  auto index_less = [&](auto lhs, auto rhs) { return objarray[lhs]<objarray[rhs]; };
  auto index_equal = [&](auto lhs, auto rhs) { return objarray[lhs]==objarray[rhs]; };

  std::sort( uniques.begin(), uniques.end(), index_less );
  uniques.erase( std::unique( uniques.begin(), uniques.end(), index_equal ), uniques.end() );

  // build table of index to unique index:
  std::map<std::size_t, std::size_t, index_less> table;
  for (std::size_t& i : uniques)
    table[i] = &i-uniques.data();

  // list of index to unique index for each element:
  auto indexes = make_index_buffer();

  // make indexes unique:
  for (std::size_t& i:indexes)
    i = table[i];

  // after this, table will be invalidated.  Clear it first:
  table = {};

  // build unique object list:
  std::vector<MyObj> objects;
  objects.reserve( uniques.size() );
  for (std::size_t i : uniques)
    objects.push_back( std::move(objarray[i]) );

  // build pointer objects:
  std::vector<MyObj*> ptrarray; // N elements
  ptrarray.reserve( indexes.size() );
  for (std::size_t i : indexes)
    ptrarray.push_back( std::addressof( objects[i] ) );

  return {std::move(objects), std::move(ptrarray)};
}

这恰好是MyObj 的 N 次移动,其中 N 是原始向量中唯一 MyObj 的数量。

你做了 M lg M 次 MyObj 的移动,以及 N 个副本,其中 M 是对象的数量,N 是唯一对象的数量。

我做了一些分配(size_ts),你可能可以清理,但这会使其不太清楚。

【讨论】:

  • 我猜你仍然在指针向量上构建你的代码以最小化更改为清晰起见.. 引入一个结构体和 int 表示对象绝对位置以及它的对象指针?
  • 您的代码中还有一个小错误。 std::unique 的 lambda 应该测试等效性,不少于。尽管如此,伟大的工作。谢谢。
  • @AbatorAbetor 已修复。我为不编译和运行付出的代价。
  • @MohammadKanan 也许吧。这将取决于用途是什么以及如何仔细测量寿命。指向向量和偏移量的指针与指向元素的指针具有不同的失效问题。您甚至可以使用共享指针的别名 ctor 并将元素向量存储在共享 ptr 中。
猜你喜欢
  • 1970-01-01
  • 2012-05-03
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2016-12-23
  • 2012-03-09
  • 2017-12-26
  • 1970-01-01
相关资源
最近更新 更多