Clearing von Layouts in Qt

Bin ich erstellen, eine Anwendung in Qt, die es Benutzern ermöglicht zu schleppen, um verschiedene "Module" in ein QGraphicsView. Wenn eines dieser Module ausgewählt ist, sendet er ein signal um, welches dann abgeholt von einem "ConfigurationWidget", die eine Seite-panel, welches anzeigen soll, können alle relevanten Parameter des ausgewählten Moduls:

class ConfigurationWidget : public QWidget
{
  Q_OBJECT

  public:
    ConfigurationWidget(QWidget *parent) : QWidget(parent) {}  

  public slots:
    void moduleSelected(Module* m)
    {
      if(layout())
      { 
        while (itsLayout->count()>0) 
        { 
          delete itsLayout->takeAt(0); 
        }
      }
      delete layout();

      itsLayout = new QFormLayout(this);
      itsLayout->addRow(QString(tr("Type:")),     new QLabel(m->name()));
      itsLayout->addRow(QString(tr("Instance:")), new QLabel(m->instanceID()));
      //... Display a whole bunch of other fields that depends on the module
    }
};

Das problem ist, dass die ConfigurationWidget nie wird tatsächlich gelöscht, wenn ein Modul ausgewählt ist. Die neuen Felder sind nur gezeichnet, über die alten. Ich habe versucht, verschiedene Kombinationen von hide() und show(), invalidate(), update(), etc. ohne Erfolg.

Was ist der richtige Weg, um ein widget, das können Sie ändern Sie die Felder wie dieses on-the-fly?

Sie sollten eine QStackedWidget

InformationsquelleAutor rcv | 2011-01-31

  1. 26

    Den code der Schleife, die ich verwendet habe, vor, ist wie folgt:

    void clearLayout(QLayout *layout) 
QLayoutItem *Element; 
while((Element = layout->takeAt(0))) { 
if (Eintrag->layout()) { 
clearLayout(Einzelteil->layout()); 
Element löschen->layout(); 
} 
if (Eintrag->widget()) { 
Element löschen->widget(); 
} 
Element löschen; 
} 
}

    Hoffentlich wird hilfreich sein für Sie!

    Ich möchte hinzufügen if(QSpacerItem *SpacerItem = Item->spacerItem()){ delete SpacerItem; } als gut 🙂
    Weitere Untersuchungen zeigen auch, dass delete item ist unnötig, da die Qt-Griffe, die für uns (verwenden Sie Qlayout::count() zu überprüfen)
    Sie brauchen nicht die if (item->widget()). Löschen NULL ist ein nop.
    das führt zu einem Fehler: item->spacerItem() im wesentlichen return dynamic_cast<QSpacerItem*>(item). I. e. Sie würde das doppelte Befreiung, block im Speicher. Hinweis: ich kenne diese nur empirisch.
    Dies ist die einzige gültige code von allen die Antworten. Es gibt jedoch eine effizientere Methode, die nicht erforderlich ist die Rekursion. Zuerst löschen Sie alle widgets, die direkte Kinder, dann löschen Sie layout-Elemente, ohne die Rekursion.

    InformationsquelleAutor Wes Hardaker

  2. 8

    Wenn Sie übertragen Sie das layout zu einem anderen widget auf dem Stapel reserviert, die widgets in diesem layout werden die Kinder der neuen widget. Wenn das temporäre Objekt außerhalb des Gültigkeitsbereichs zerstört es automatisch das layout und alle widgets.

    void moduleSelected(Module* m)
{
    if(layout())
        QWidget().setLayout(layout());

    itsLayout = new QFormLayout(this);
    itsLayout->addRow(QString(tr("Type:")),     new QLabel(m->name()));
    itsLayout->addRow(QString(tr("Instance:")), new QLabel(m->instanceID()));
    //... Display a whole bunch of other fields that depends on the module
}
    Das ist wahnsinnig hässlich und wahnsinnig clever zugleich. 😉 Vielen Dank für die gemeinsame Nutzung dieser hack. Ich wünschte, es war QLayout::clear()...

    InformationsquelleAutor alexisdm

  3. 4

    Sieht es aus wie der beste Weg, dies zu tun, ist die Verwendung einer QStackedLayout, wie angedeutet durch armonge:

    void ConfigurationWidget::moduleSelectedSlot(Module* m)
{
  QStackedLayout *stackedLayout = qobject_cast<QStackedLayout*>(layout());

  QWidget *layoutWidget = new QWidget(this);
  QFormLayout *formLayout = new QFormLayout(layoutWidget);
  formLayout->addRow(QString(tr("Type:")),     new QLabel(m->name()));
  formLayout->addRow(QString(tr("Instance:")), new QLabel(m->instanceID()));
  //... Display a whole bunch of other fields that depends on the module

  delete stackedLayout->currentWidget();
  stackedLayout->addWidget(layoutWidget);
  stackedLayout->setCurrentWidget(layoutWidget);
}
    Bei der Verwendung von QObject abgeleitete Klasse, die Sie verwenden sollten qobject_cast statt dynamic_cast: doc.qt.nokia.com/latest/qobject.html#qobject_cast
    Behoben.

    InformationsquelleAutor rcv

  4. 1

    Ich vor kurzem laufen in das gleiche problem mit einem QFormLayout.
    Was für mich gearbeitet (und ist auch in der Qt-Dokumentation: http://qt-project.org/doc/qt-5/layout.html) war die takeAt(int) - Methode.

    void clearLayout(QLayout *layout)
{
     QLayoutItem *item;
     while ((item = layout->takeAt(0)))
         delete item;
}
    Seien Sie vorsichtig : dies wird nicht löschen Sie die widgets, nur entfernen Sie Sie aus dem layout als layout-Elemente, die keine eigenen widgets. Das Programm kann also am Ende Essen alle der Speicher, wenn man neu das ui viel mal mit diesem code. Dies ist nicht ein echter memory leak, weil die child-widgets werden gelöscht mit dem Eltern-widget, aber das kann den gleichen Effekt haben.

    InformationsquelleAutor Kádi

  5. 1

    Mit while((item = layout->takeAt(0)))verursachen Warnmeldung "Invalid index nehmen Sie bei 0".
    Ich benutze count() statt

    void clearLayout(QLayout *layout) 
{
    if (layout) {
        while(layout->count() > 0){
            QLayoutItem *item = layout->takeAt(0);
            QWidget* widget = item->widget();
            if(widget)
                delete widget;
            delete item;
        }
    }
}
    Eine gültige layout-Element kann null-Zeiger mit dem Artikel->widget(), zum Beispiel mit dem Kind layouts oder Abstandshalter. Dieser code wird den Absturz in diesen Fällen.
    Kommt drauf an, aber ja, im Allgemeinen Fall ist es nicht klug, rufen Sie löschen auf nulltpr. stackoverflow.com/questions/6731331/...
    Anstelle von "löschen widget', ich würde vorschlagen, mit widget->deleteLater(). In den meisten Fällen, es wäre schwer, den Unterschied bemerken, aber es kann einer werden. Als Dokumentation über QObject sagt: "Das Objekt wird gelöscht, wenn die Kontrolle wieder an die event loop". Dies ist der wichtigste Punkt. Ist es möglich das widget sendet irgendein signal, und nach dem senden das signal einige andere Anrufe in das gleiche widget-Instanz. Wenn Sie löschen würde-widget in einem "raw-Weg", dann würden Sie nicht mehr gültig sind, und in diesem Fall kann man nicht behandeln, alles nach dem signal gesendet wird.

    InformationsquelleAutor Dorian

  6. 0

    Reparent die Elemente auf NULL, wenn Sie wollen, dass Sie verschwindet vom Fenster. Ich hatte ähnliche Probleme vor kurzem und übernahme gelöst.

    Herzlich willkommen auf Stack Overflow. Diese Frage hat bereits eine Handvoll von Antworten sowie eine, die angenommen wurde. Erklären Sie, wie Ihre Antwort ist anders oder besser als die Lösungen bereits zur Verfügung gestellt.

    InformationsquelleAutor nyu

Schreibe einen Kommentar