"无法转换为' A'到' B&'

时间:2016-01-25 05:54:18

标签: c++ templates c++11 null-object-pattern

我正在使用模板元编程构建实体 - 组件系统。我不断收到Cannot convert from [base type] to [type user requested]&Cannot convert NullComponent to [type user requested]&错误:

class Entity {
public:
    Entity() = default;
    ~Entity() = default;

    template<typename C, typename... Args>
    void AddComponent(Args&&... args);

    template<typename C>
    C& GetComponent();

protected:
private:
    //...add/get helper methods here...

    unsigned int _id;
    std::vector<std::unique_ptr<IComponent>> _components;
};

template<typename C>
C& Entity::GetComponent() {
    for(auto c : _components) {
        if(std::is_base_of<a2de::IComponent&, C&>().value && std::is_same<decltype(c), C&>().value) {
            return *c; //<-- error here
        }
    }
    return NullComponent(); //<-- and here
}

修改

这些选项目前似乎有效。

template<typename C>
const C& Entity::GetComponent() const {
    for(auto& uc : _components) {
        auto* c = dynamic_cast<C*>(uc.get());
        if(c && std::is_base_of<a2de::IComponent&, C&>().value && std::is_same<decltype(c), C&>().value) {
            return *c;
        }
    }
    throw std::runtime_error(std::string("Component not available."));
}

OR

class Entity {
public:
    //same as before...
protected:
private:
    //same as before...
    a2de::NullComponent _null_component;
};

template<typename C>
const C& Entity::GetComponent() const {
    for(auto& uc : _components) {
        auto* c = dynamic_cast<C*>(uc.get());
        if(c && std::is_base_of<a2de::IComponent&, C&>().value && std::is_same<decltype(c), C&>().value) {
            return *c;
        }
    }
    return _null_component;
}

2 个答案:

答案 0 :(得分:2)

至少有三件事:

  • GetComponent()中,您对unique_ptr个元素进行迭代,并将其类型(始终为std::unique_ptr<IComponent>)与std::is_same中的其他内容进行比较。你可能不希望这样。
  • 您似乎在最终回复中返回对临时的引用。
  • return *c需要dynamic_cast,除非C == IComponent。

修改

此外:

  • std::is_base_of对引用毫无意义。即使使用class NullComponent : IComponent {};,您仍然会获得std::is_base_of<IComponent&, NullComponent&>::value == false
  • 并且您不检查nullptr

最后,在我看来,你应该用

替换你的for循环
for(auto& component : _components) {
  auto* c = dynamic_cast<C*>(component.get());
  if (c)
  {
    return *c;
  }
}

答案 1 :(得分:0)

在高层次上,根据我的判断,返回类型不能用于定义模板类型。参数列表可用于定义模板类型。

因此,例如,这可能有用 -

template<typename C>
void Entity::GetComponent(C *obj) {
    for(auto c : _components) {
        if(std::is_base_of<a2de::IComponent&, C&>().value && std::is_same<decltype(c), C&>().value) {
            obj = c; //<-- error here
            return;
        }
    }
    obj = NULL;
    return; //<-- and here
}

希望这有帮助。