在这种情况下，qobject_cast是否应该默默失败？

Question

我正在尝试用Qt编写一个应用程序，该应用程序运行得比较好，直到遇到这个愚蠢的问题。 我有一个Node类，它可以具有一个指向Workplace类实例的指针。 我还为该指针定义了getter / setter方法。 像这样：

class Node : public QGraphicsItem
{
    Q_DECLARE_TR_FUNCTIONS(Node)

public:
    Node(const QString &type, Project* nodeProject);
    ~Node();

    void setWorkplace(Workplace* workplace);
    void removeWorkplace();
    Workplace* usedWorkplace();

    (...)
private:    
    Workplace* myWorkplace;
}

因此，现在在我的代码中，我必须检查某个Node是否具有指向定义的Workplace的指针，以便对该信息进行一些操作。 这就是我的做法：

Workplace* currentWorkplacePointer = qobject_cast<Workplace*>(node->usedWorkplace());
if(!currentWorkplacePointer){
     // no workplace, do some stuff
}else{
     // workplace exists, do other stuff
}

这工作得很好，但是现在由于某种原因它会随机断断续续。 我随便说qobject_cast ，因为有时，如果我尝试清理并重建项目，则它会正常工作-即， qobject_cast静默失败并按照我的预期进行； 其他时候，即使我清理/重建，它也会再次出现段错误。

调试器控制台始终指向上面我的代码中的对象强制转换线。 据我所知，如果不存在强制转换的对象，强制转换是否应该仅默默地失败？

请注意，我对Qt / C ++还是陌生的，因此，感谢您提供任何帮助和意见-请保持温柔！ ：） 提前致谢！

Answer 1

正如塞巴斯蒂安·雷德尔（Sebastian Redl）所说的那样，将指针转换为自身类型似乎是胡说八道。 不过，您的代码可能会有所改进，以便将来使用强制转换。 我的第一点来自qobject_cast的文档，该文档要求在类标头中调用Q_OBJECT宏才能使用。

其次，由于您的Node继承了QGraphicsItem ，因此我建议改用qgraphicsitem_cast ，它在处理或过滤场景中存在的项目时以不同的方式更快。 从文档改编

 class Node : public QGraphicsItem
 {
    ...
    enum { Type = UserType + 1 };

    int type() const
    {
        // Enable the use of qgraphicsitem_cast with this item.
        return Type;
    }
    ...
 }

//in your application code
Node *n = qgraphicsitem_cast<Node*>(expected_pointer_on_node);
if (n) {
//then n points a Node
}

Answer 2

这实际上比我想象的要简单得多，而我的问题实际上与以下事实有关： node->usedWorkplace()方法无法返回任何内容，因为在尝试获取Workplace*之前未对其进行初始化。 笨蛋，我知道...

这样，我只是将指针初始化为0 （谢谢@ratchetfreak），然后从那里开始工作。 现在一切正常。

因此，当我创建 Node的实例时，我将指针初始化为0 ：

nodeToPlace->setWorkplace(0);

然后，当我要检查Node实例是否存在Workplace* ：

Workplace* tempWorkplace = node->usedWorkplace();
if(!tempWorkplace){
     // do stuff
}else{
     // other stuff
}

因此，我实际上根本不需要qobject_cast 。

注意：这最终与我有关qobject_cast问题无声无息地联系qobject_cast /没有回答。 我最终发现是， cast将返回0或者如果它失败， 或者如果您尝试投0到的东西。 我认为我的初始代码正在发生。

来自IBM Linux编译器文档 （根据Qt的文档，C ++的dynamic_cast报告大致相同）：

表达式dynamic_cast（v）将表达式v转换为类型T。类型T必须是指向完整类类型的指针或引用，或指向void的指针。 如果T是指针，并且dynamic_cast运算符失败，则该运算符返回类型T的空指针。 如果T是引用，并且dynamic_cast运算符失败，则该运算符将引发异常std :: bad_cast。

因此，我得出的结论是，我将垃圾称为强制转换的arg，这使我的应用程序出现段错误（如注释中提到的@hyde，因此也对他+1）。

如果有人有不同的意见和/或解释，请告诉我-我想详细了解我的错误/错误。