为什么我不能创建包含vec3对象的联合?

时间:2018-05-29 06:16:43

标签: c++ unions glm-math

我似乎无法创建一个联合,其中一个成员是或包含一个glm::vec3对象(一个用于表示坐标的对象,在这种情况下包含3个浮点数)。 (source code for glm::vec)

在以下代码中使用:

struct Event {
    enum Type{
        tRaw,
        tAction,
        tCursor,
    } type;
    union {
        SDL_Event raw;
        struct {
            uint16 actionID;
            bool released;
        } action;
        struct {
            glm::vec3 prevPos;
            glm::vec3 pos;
        } cursor; // offending object, compiles if this is removed
    } data;
};

Visual Studio给出了以下智能感知错误。

"Error: the default constructor of "union Event::<unnamed>" cannot be referenced -- it is a deleted function"

如果删除,联合编译没有任何问题。可能导致这个问题的原因是什么,我能做些什么来补救它?

1 个答案:

答案 0 :(得分:7)

一旦你的union中有一个非常重要的类型(那么,一个语言强制执行&#34;正确&#34;初始化,即构造函数调用),你必须明确地编写构造函数/析构函数:

#include <SDL/SDL.h>
#include <glm/vec3.hpp>
#include <stdint.h>
#include <new>
#include <vector>

struct Event {
    enum Type{
        tRaw,
        tAction,
        tCursor,
    } type;
    struct Cursor {
        glm::vec3 prevPos;
        glm::vec3 pos;
    };
    union {
        SDL_Event raw;
        struct {
            uint16_t actionID;
            bool released;
        } action;
        Cursor cursor;
    };
    Event(const SDL_Event &raw) : type(tRaw) {
        new(&this->raw) SDL_Event(raw);
    }
    Event(uint16_t actionID, bool released) : type(tAction) {
        this->action.actionID = actionID;
        this->action.released = released;
    }
    Event(glm::vec3 prevPos, glm::vec3 pos) : type(tCursor) {
        new(&this->cursor) Cursor{prevPos, pos};
    }
    Event(const Event &rhs) : type(rhs.type) {
        switch(type) {
        case tRaw:      new(&this->raw) SDL_Event(raw); break;
        case tAction:   memcpy((void *)&action, (const void *)&rhs.action, sizeof(action)); break;
        case tCursor:   new(&this->cursor) Cursor(rhs.cursor);
        }
    }

    ~Event() {
        if(type == tCursor) {
            this->cursor.~Cursor();
        }
        // in all other cases, no destructor is needed
    }
};

int main() {
    // Construction
    Event ev(1, false);
    SDL_Event foo;
    Event ev2(foo);
    glm::vec3 pos;
    Event ev3(pos, pos);
    // Copy construction & destruction
    std::vector<Event> events;
    events.push_back(ev);
    events.push_back(ev2);
    events.push_back(ev3);
    events.clear();
    return 0;
}

一些注意事项:

  • 我避开data成员,而是选择匿名union;这避免了很多样板,否则我必须在union内编写这些构造函数(因为它是 union构造函数删除后必须明确定义),然后在外面添加转发器;它还大大简化了析构函数的编写(同样,它必须写在union内,但union不知道外部type;你可以解决这个问题,但这很繁琐冗长);
  • 我必须明确命名Cursor联合,否则它在语法上不可能调用它的析构函数(禁止模板技巧);
  • 我没有实现赋值运算符;它并不复杂,但说实话,这很乏味。您可以找到一个基本蓝图(检查是否相同type;如果相同,请执行常规分配;否则,将活动成员和展示位置 - new销毁到新的in the link I posted before in the comments上。

所有这些都说,这些东西已经在C ++ 17中以更通用的方式实现为std::variant,所以,如果你有一个最近的编译器,你可以考虑使用它。