Question

一个不受限制的union类的示例，它包含一个int的映射或一个int的向量：

#include <iostream>
#include <string>
#include <vector>
#include <map>

typedef std::vector<int> int_vec; 
typedef std::map<std::string, int> int_map; 

struct Vec {

  bool is_map;

  union {
    int_vec int_vec_val;
    int_map int_map_val;
  };

  // MOVE CONSTRUCTORS
  Vec(int_vec&& val)  : is_map(false), int_vec_val(std::move(val)) {
    std::cout << "move vec" << std::endl;
  }

  Vec(int_map&& val)  : is_map(true),  int_map_val(std::move(val)) {
    std::cout << "move map" << std::endl;
  }

  Vec(Vec&& rhs) : is_map(rhs.is_map) {
    if (is_map) {
      int_map_val = std::move(rhs.int_map_val);
    } else {
      std::cout << "Crashing now ... " << std::endl;
      int_vec_val = std::move(rhs.int_vec_val); // <--- how to do this correctly?
    }
  }

  // DESTRUCTOR
  ~Vec() noexcept {
    if (is_map) {
      int_map_val.~int_map(); 
    } else {
      int_vec_val.~int_vec();      
    }
  }
};

Vec gen_Vec() {
  Vec v1(int_vec({1, 2, 3}));
  return Vec(std::move(v1));
}

int main() {
  Vec v2  = gen_Vec();
}

此代码与*** Error in ./tmp2: munmap_chunk(): invalid pointer: 0x0000000000400f6d ***移动构造函数中的Vec(Vec&&)崩溃：

g++ -std=c++11 -O2 tmp2.cpp -lm -o tmp2 && ./tmp2
move vec
Crashing now ... 
*** Error in `./tmp2': munmap_chunk(): invalid pointer: 0x0000000000400f6d ***
======= Backtrace: =========
/lib/x86_64-linux-gnu/libc.so.6(+0x777e5)[0x7fba03ab47e5]
/lib/x86_64-linux-gnu/libc.so.6(cfree+0x1a8)[0x7fba03ac0ae8]
./tmp2[0x400d0d]
./tmp2[0x400a7c]
/lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf0)[0x7fba03a5d830]
./tmp2[0x400af9]
======= Memory map: ========
00400000-00402000 r-xp 00000000 08:11 13240011                           /store/Dropbox/dev/jamr/src/tmp2
00601000-00602000 r--p 00001000 08:11 13240011                           /store/Dropbox/dev/jamr/src/tmp2
00602000-00603000 rw-p 00002000 08:11 13240011                           /store/Dropbox/dev/jamr/src/tmp2
006ce000-00700000 rw-p 00000000 00:00 0                                  [heap]
7fba03734000-7fba0383c000 r-xp 00000000 08:01 1065106                    /lib/x86_64-linux-gnu/libm-2.23.so
7fba0383c000-7fba03a3b000 ---p 00108000 08:01 1065106                    /lib/x86_64-linux-gnu/libm-2.23.so
7fba03a3b000-7fba03a3c000 r--p 00107000 08:01 1065106                    /lib/x86_64-linux-gnu/libm-2.23.so
7fba03a3c000-7fba03a3d000 rw-p 00108000 08:01 1065106                    /lib/x86_64-linux-gnu/libm-2.23.so

我的问题是：

移动这些矢量和地图的正确方法是什么？
有没有办法在mem初始化列表中完全定义Vec(Vec&& rhs)移动构造函数？

非常感谢。

Answer 1

作业int_vec_val = std::move(rhs.int_vec_val);表示int_vec_val表示类型为int_vec的现有活动对象，您希望通过分配来更改其值。情况并非如此：由于匿名联盟的成员没有默认成员初始化程序，也没有相应的 mem-initializer ，因此不会对它们执行初始化，所以他们的生命不会开始。

这意味着当输入构造函数块时，int_map_val和int_vec_val都不存在。您必须使用placement new：

在正确的位置实际构建一个正确类型的新对象

Vec(Vec&& rhs) : is_map(rhs.is_map) {
   if (is_map) {
      std::cout << "move contained map\n";
      new(&int_map_val) int_map(std::move(rhs.int_map_val));
   } else {
      std::cout << "move contained vec\n";
      new(&int_vec_val) int_vec(std::move(rhs.int_vec_val));
   }
}

第二个问题的答案是：不，不是真的，因为rhs.is_map通常只在运行时知道，并且 mem-initializer-list 的内容需要知道在编译时。但是，在这种情况下，在构造函数块中进行初始化没有问题（并且通过在 ctor-initializer 中进行初始化没有真正的好处），因为默认情况下，不对联合成员执行初始化。

使用invalid_pointer移动无限制联合崩溃的构造函数

1 个答案: