c++ - 从函数返回时对象被销毁

Question

我正在做 qt 项目，发现我的应用程序崩溃了。
在调试时我发现这是因为没有创建/初始化 qpushbutton 对象。

现在我不明白为什么它没有被初始化？

我在命名空间中编写了函数，它为所有 Qpushbutton 调用构造函数，并且我通过引用该函数来传递对象所以它不应该在函数返回后保留它的值吗？但就我而言，函数一返回就被销毁了？跟命名空间有关系吗？？？

任何帮助或批评都会有帮助吗？

   MY_Utility.h
  class MY_Utility
  {

private:
    QPushButton* add_cal_button_; 
    QPushButton*  sub_cal_button_; 
    QPushButton*  mul_button_; 

  }
My_Utility.cpp

namespace
{
   void create_push_button_for_cal( QPushButton *button_cal, QString button_name, QGridLayout *grid, int grid_row, int grid_col )
    {
        button_cal = new QPushButton( button_name );

        if(!button_cal )
    {
        msg.setInformativeText("The code does not come here so object is created");
        QMessageBox msg;
        msg.setText("Messsage");
        msg.setInformativeText("OBject not initialised in create_push button  ");
        msg.exec();;
    }
        button_cal->setFixedSize( 200 , 40 );
        button_cal->setVisible( false );
        grid->addWidget(  button_cal, grid_row, grid_col );
    }


    // function where we call all button created
void create_main_view( MY_Utility* main_p, QTreeWidget* tree_p, QTableWidget* table_p, QPushButton* add_cal_button_,  
                                                                                                   QPushButton* sub_cal_cal_button_, 
                                                                                                   QPushButton* mul_cal_button_,

{
    QWidget*        center_p = new QWidget( main_p );
    QHBoxLayout*    layout_p = new QHBoxLayout( center_p );
    QGridLayout*    grid     = new QGridLayout( center_p );
    grid->setSpacing( 1 );

    create_push_button_for_cal( add_cal_button_, "addition_calculate" , grid, 1, 2 );
    if( !add_cal_button_ )
    {
        QMessageBox msg;
        msg.setText("Messsage");
        msg.setInformativeText("OBject not initialised in first part of message  why not ?? ");
        msg.exec();;
    }
    create_push_button_for_cal( sub_cal_cal_button_ ,"sub_Calculate",           grid, 2, 2 );
    create_push_button_for_cal( mul_cal_button_ ,    "multplication_Calculate", grid, 3, 2 );
    // Make the QWidget the central widget so we can resize within it
    main_p->setCentralWidget( center_p );
   bla .. 
   bla ..



}

}

My_Utility::set_all_cal_button_visible(){
 add_cal_button_->setVisible(true) ; // it crashes here 

}

Answer 1

你的问题是C++问题，不是Qt问题。

void create_push_button_for_cal(QPushButton *button_cal, QString button_name)
{
    button_cal = new QPushButton(button_name);
    Q_ASSERT(button_cal);
}

指针(地址)按值传递。您的代码丢弃这个值，并用指向新实例的指针覆盖它。

检查 null button_cal 是无用的，因为 new 有一个很好的不变性:如果执行它后面的代码，则意味着它已成功分配内存。它让生活变得非常简单:如果 new 返回一个值，那很好。它不会返回无效值或空值。

如果 new 失败，断言将永远不会执行，事实上，new 甚至不会返回值。它会抛出一个异常，当该异常被抛出时，您几乎无能为力，因为几乎任何事情都需要更多内存，而我们刚刚用完了:(

您需要更改函数的签名以返回指向新创建的实例的指针。您还应该通过 const 引用而不是值来传递 Qt 类。如果函数中的其他代码可能会抛出异常，请使用智能指针来保护自己免于泄漏按钮实例。所以:

QPushButton * create_push_button_for_cal(const QString & button_name) {
    QScopedPointer<QPushButton> btn(new QPushButton(button_name));
    ...
    // If any code here throws an exception, the scoped pointer will delete
    // the button instance, so that it won't leak.
    ...
    return btn.take();
}