“无法从 'A' 转换为 'B&'
"Cannot convert from 'A' to 'B&'
我正在使用模板元编程构建实体组件系统。我不断收到 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."));
}
或
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;
}
在高层次上,据我所知,return 类型不能用于定义模板类型。参数列表可用于定义模板类型。
因此,例如,这可能有效 -
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
}
希望对您有所帮助。
至少三件事:
- 在
GetComponent()
中迭代 unique_ptr
元素并将它们的类型(总是 std::unique_ptr<IComponent>
)与 std::is_same
中的其他元素进行比较。你可能不想要那个。
- 你似乎在 return 引用了最后一个 return 中的临时文件。
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;
}
}
我正在使用模板元编程构建实体组件系统。我不断收到 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."));
}
或
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;
}
在高层次上,据我所知,return 类型不能用于定义模板类型。参数列表可用于定义模板类型。
因此,例如,这可能有效 -
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
}
希望对您有所帮助。
至少三件事:
- 在
GetComponent()
中迭代unique_ptr
元素并将它们的类型(总是std::unique_ptr<IComponent>
)与std::is_same
中的其他元素进行比较。你可能不想要那个。 - 你似乎在 return 引用了最后一个 return 中的临时文件。
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;
}
}