Ich erstelle in einer Funktion ein Objekt auf dem Heap.

ViewQuerryOption *newView= new ViewQuerryOption();
newView->tableViewOption(&querry, &viewName, &TqmStorage);

Das möchte ich am Ende der Funktion nun wie folgt löschen.

delete newView;

Nur leider bekomme ich an der Stelle, genauer beim Destruktor des Objekts, einen Pointer Fehler [...] "double free or corruption (out)" [...]
Durch googeln nach diesem Fehler finde ich zahlreiche Beispiele wie ich mir noch so den Speicher zerschießen kann, nur nichts was ich adaptieren kann, zumindest seh ich es nicht.

Hier der Code zu "ViewQuerryOption.h"

#ifndef VIEWQUERRYOPTION_H
#define VIEWQUERRYOPTION_H

#include "MainTable.h"
#include "ModulQuery.h"

#include <QWidget>
#include <QtGui>
#include <QMap>

using namespace std;

class ViewQuerryOption : public QDialog
{
    Q_OBJECT
public:
    explicit ViewQuerryOption(QWidget *parent = 0);
    ~ViewQuerryOption();
    void tableViewOption(QString *querry, QString *viewName, QMap <QString, ModulQuery> *TqmStorage);
private:
    MainTable *ptrAufTable;

    QCheckBox plz;
    QCheckBox number;
    QCheckBox city;
    QCheckBox street;

    QCheckBox firstName;
    QCheckBox lastName;
    QCheckBox male;

    QCheckBox roll;
    QCheckBox companyName;

    QCheckBox phone;
    QCheckBox mail;

    QRadioButton  rbUser;
    QRadioButton  rbFirma;
    QRadioButton  rbInstanz;

    QComboBox  whereSelect;
    QComboBox  relation;
    QLineEdit  searcher;
    QLineEdit  nameOfV;

    QVBoxLayout  vBoxLall;
    QVBoxLayout  vRadioL;
    QVBoxLayout  vBoxLaddress;
    QVBoxLayout  vBoxLinstanz;
    QVBoxLayout  vBoxLnext;
    QVBoxLayout  vBoxLcontact;
    QVBoxLayout  vL;
    QVBoxLayout  vButtonNemaL;

    QHBoxLayout  hWhereL;
    QHBoxLayout  hInAnL;
    QHBoxLayout  hNeCoL;
    QHBoxLayout  hL;
    QHBoxLayout  hViewLabelL;
    QHBoxLayout  hFunctionButtonL;

    QPushButton  ok;
    QPushButton  cancel;

    QGroupBox  preSelect;
    QGroupBox  selectViews;
    QGroupBox  selectWhere;
    QGroupBox  nameLabel;

    QLabel  addressData;
    QLabel  instanzData;
    QLabel  nextData;
    QLabel  contectData;
    QLabel  nameOfView;

    QSignalMapper  signalMapper;
    QString  *makingQuery;
    QString  *localViewName;
    QMap <QString, ModulQuery> *localTmpMap;

    QMessageBox msgBox;
signals:

public slots:
    void okPushed();
    void createQuery();
    void addToComboBox(const int &CheckBoxName);

};

#endif // VIEWQUERRYOPTION_H

Hier der Code des Objektes "ViewQuerryOption.cpp"

#include "ViewQuerryOption.h"

ViewQuerryOption::ViewQuerryOption(QWidget *parent) : QDialog(parent)
{
    //Erzeugung aller grafischen bestandteile des fensteres und verschachtelung in Layouts

    //Hinzufügen der Objete, deren Signale gebündelt werden
    signalMapper.setMapping(&plz,1);
    signalMapper.setMapping(&number,2);
    signalMapper.setMapping(&city,3);
    signalMapper.setMapping(&street,4);
    signalMapper.setMapping(&firstName,5);
    signalMapper.setMapping(&lastName,6);
    signalMapper.setMapping(&male,7);
    signalMapper.setMapping(&roll,8);
    signalMapper.setMapping(&companyName,9);
    signalMapper.setMapping(&phone,10);
    signalMapper.setMapping(&mail,11);

    // Beschriftung der RadioButton
    rbInstanz.setText("Instanz / Person");
    rbFirma.setText("Firma");
    rbUser.setText("D-Manager Nutzer");

    // vertikales Layout für die Radiobutton in der oberen GroupBox
    vRadioL.addWidget(&rbInstanz);
    vRadioL.addWidget(&rbFirma);
    vRadioL.addWidget(&rbUser);
    //Fügt der GroupBox das Layout mit den 3 RadioButton hinzu
    preSelect.setLayout(&vRadioL);

    // Beschriftung der Labels
    addressData.setText("Adressdaten");
    instanzData.setText("Instanz / Personendaten");
    nextData.setText("weitere Informationen");
    contectData.setText("Kontaktdaten");
    nameOfView.setText("Name der View:");

    // Beschriftung der CheckBoxen
    street.setText("Strasse");
    number.setText("Hausnummer");
    plz.setText("Postleitzahl");
    city.setText("Stadt");
    male.setText("Anrede");
    lastName.setText("Nachname");
    firstName.setText("Vorname");
    companyName.setText("Firmenname");
    roll.setText("Instanzname");
    phone.setText("Telefon");
    mail.setText("E-Mail");

    // vertikales Layout für die Checkboxen zur Adresse
    vBoxLaddress.addWidget(&addressData);
    vBoxLaddress.addWidget(&street);
    vBoxLaddress.addWidget(&number);
    vBoxLaddress.addWidget(&plz);
    vBoxLaddress.addWidget(&city);

    // vertikales Layout für die Checkboxen zur Instanz
    vBoxLinstanz.addWidget(&instanzData);
    vBoxLinstanz.addWidget(&male);
    vBoxLinstanz.addWidget(&lastName);
    vBoxLinstanz.addWidget(&firstName);

    // Horizontales Layout für die vertikalen Layouts der Checkboxen zur Adresse und Instanz
    hInAnL.addLayout(&vBoxLinstanz);
    hInAnL.addLayout(&vBoxLaddress);

    // vertikales Layout für die Checkboxen zur weiteren Daten
    vBoxLnext.addWidget(&nextData);
    vBoxLnext.addWidget(&companyName);
    vBoxLnext.addWidget(&roll);

    // vertikales Layout für die Checkboxen zur Kontaktdaten
    vBoxLcontact.addWidget(&contectData);
    vBoxLcontact.addWidget(&phone);
    vBoxLcontact.addWidget(&mail);

    // vertikales Layout für die Radiobutton in der mittleren GroupBox
    hNeCoL.addLayout(&vBoxLnext);
    hNeCoL.addLayout(&vBoxLcontact);

    // Fügt die 2 horizontalen layouts nochmal i ein vertikales zusammen
    vBoxLall.addLayout(&hNeCoL);
    vBoxLall.addLayout(&hInAnL);

    //Fügt der mittleren GroupBox das Layout mit den CheckBoxen hinzu
    selectViews.setLayout(&vBoxLall);

    //Fügt den Comboboxen die Standard Auswahlmöglichkeiten hinzu
    whereSelect.addItem("Suche verfeinern");
    relation.addItem("=");
    relation.addItem("<");
    relation.addItem(">");

    // vertikales Layout für die untere Filtefunktion
    hWhereL.addWidget(&whereSelect);
    hWhereL.addWidget(&relation);
    hWhereL.addWidget(&searcher);

    // fügt layout GroupBox hinzu
    selectWhere.setLayout(&hWhereL);

    // Button vorbereiten
    ok.setText("OK");
    cancel.setText("Abbrechen");
    //Namenslabel und Funktionsbutten zu einem Layout formen
    hViewLabelL.addWidget(&nameOfView);
    hViewLabelL.addWidget(&nameOfV);
    hFunctionButtonL.addWidget(&ok);
    hFunctionButtonL.addWidget(&cancel);
    vButtonNemaL.addLayout(&hViewLabelL);
    vButtonNemaL.addLayout(&hFunctionButtonL);

    nameLabel.setLayout(&vButtonNemaL);

    // Fügt die 3 Groupboxen eine horizontalen Layout hinzu
    hL.addWidget(&preSelect);
    hL.addWidget(&selectViews);
    vL.addLayout(&hL);
    vL.addWidget(&selectWhere);
    vL.addWidget(&nameLabel);

    // Setzt das gesammte Layout
    setLayout(&vL);

    QObject::connect(&ok,SIGNAL(clicked()),this,SLOT(okPushed()));
    QObject::connect(&cancel,SIGNAL(clicked()),this,SLOT(close()));

    //Immer wenn eine Checkbox angeklickt wurde und sich damit ihr Status geänder hatte wird diese CheckBox
    //in den Mappere geschrieben
    QObject::connect( &plz, SIGNAL(clicked()), &signalMapper, SLOT(map()) );
    QObject::connect( &number, SIGNAL(clicked()), &signalMapper, SLOT(map()) );
    QObject::connect( &city, SIGNAL(clicked()), &signalMapper, SLOT(map()) );
    QObject::connect( &street, SIGNAL(clicked()), &signalMapper, SLOT(map()) );
    QObject::connect( &firstName, SIGNAL(clicked()), &signalMapper, SLOT(map()) );
    QObject::connect( &lastName, SIGNAL(clicked()), &signalMapper, SLOT(map()) );
    QObject::connect( &male, SIGNAL(clicked()), &signalMapper, SLOT(map()) );
    QObject::connect( &roll, SIGNAL(clicked()), &signalMapper, SLOT(map()) );
    QObject::connect( &companyName, SIGNAL(clicked()), &signalMapper, SLOT(map()) );
    QObject::connect( &phone, SIGNAL(clicked()), &signalMapper, SLOT(map()) );
    QObject::connect( &mail, SIGNAL(clicked()), &signalMapper, SLOT(map()) );

    QObject::connect(&signalMapper, SIGNAL(mapped(const int&)), this, SLOT(addToComboBox(const int &)) );
}

ViewQuerryOption::~ViewQuerryOption()
{

}

//fügt die angehakten spalten zur ComboBox hinzu um diese weiter zu nutzen.
void ViewQuerryOption::addToComboBox(const int &CheckBoxName)
{

    switch (CheckBoxName)
    {
    case 1 :
        if (plz.isChecked())
        {
            whereSelect.addItem("PLZ");
        }else
        {
            whereSelect.removeItem(whereSelect.findText("PLZ"));
        }
    break;
    case 2 :
        if (number.isChecked())
        {
            whereSelect.addItem("Hausnummer");
        }else
        {
            whereSelect.removeItem(whereSelect.findText("Hausnummer"));
        }
    break;
    case 3 :
        if (city.isChecked())
        {
            whereSelect.addItem("Stadt");
        }else
        {
            whereSelect.removeItem(whereSelect.findText("Stadt"));
        }
    break;
    case 4 :
        if (street.isChecked())
        {
            whereSelect.addItem("Strasse");
        }else
        {
            whereSelect.removeItem(whereSelect.findText("Strasse"));
        }
    break;
    case 5 :
        if (firstName.isChecked())
        {
            whereSelect.addItem("Vorname");
        }else
        {
            whereSelect.removeItem(whereSelect.findText("Vorname"));
        }
    break;
    case 6 :
        if (lastName.isChecked())
        {
            whereSelect.addItem("Nachname");
        }else
        {
            whereSelect.removeItem(whereSelect.findText("Nachname"));
        }
    break;
    case 7 :
        if (male.isChecked())
        {
            whereSelect.addItem("Anrede");
        }else
        {
            whereSelect.removeItem(whereSelect.findText("Anrede"));
        }
    break;
    case 8 :
        if (roll.isChecked())
        {
            whereSelect.addItem("Rolle");
        }else
        {
            whereSelect.removeItem(whereSelect.findText("Rolle"));
        }
    break;
    case 9 :
        if (companyName.isChecked())
        {
            whereSelect.addItem("Firma");
        }else
        {
            whereSelect.removeItem(whereSelect.findText("Firma"));
        }
    break;
    case 10 :
        if (phone.isChecked())
        {
            whereSelect.addItem("Telefon");
        }else
        {
            whereSelect.removeItem(whereSelect.findText("Telefon"));
        }
    break;
    case 11 :
        if (mail.isChecked())
        {
            whereSelect.addItem("E-Mail");
        }else
        {
            whereSelect.removeItem(whereSelect.findText("E-Mail"));
        }
    break;

    }
}

void ViewQuerryOption::okPushed()
{
    if (nameOfV.text()=="")
    {
        const wchar_t* QboxText = L"Bitte geben Sie einen Namen für die neue Ansicht ein!\n";
        msgBox.setWindowTitle("Name fehlt");
        msgBox.setIcon(QMessageBox::Warning);
        msgBox.setText(QString(QString::fromWCharArray(QboxText)));
        msgBox.exec();
    }else if(localTmpMap->contains(nameOfV.text()))
    {
        /*ModulQuery foo;
        foo = localTmpMap->value(nameOfV.text());
        cout <<"Querry: " <<foo.getQuery().toStdString() <<endl;
        */
        const wchar_t* QboxText = L"Dieser Name existiert bereits! Bitte wählen Sie einen anderen.\n";
        msgBox.setWindowTitle("Name existiert");
        msgBox.setIcon(QMessageBox::Warning);
        msgBox.setText(QString(QString::fromWCharArray(QboxText)));
        msgBox.exec();
    }else
    {
        this->createQuery();
        this->close();
    }

}
/*
 * Übernimmt die Zeiger auf den Querry und den Tabnamen auf Klasseenvariablen
 * und startet die View.
*/
void ViewQuerryOption::tableViewOption(QString *query, QString *viewName, QMap <QString, ModulQuery> *TqmStorage)
{
    makingQuery=query;
    localViewName=viewName;
    localTmpMap = TqmStorage;
    this->exec();
}

/*
 * Erstellt aus den ANgaben des Nutuzers einen gültigen SQL Query
*/
void ViewQuerryOption::createQuery()
{

    *makingQuery="SELECT";

    if (plz.isChecked())
    {*makingQuery=*makingQuery+" plz,";}

    if (number.isChecked())
    {*makingQuery=*makingQuery+" hausnummer,";}

    if (city.isChecked())
    {*makingQuery=*makingQuery+" ort,";}

    if (street.isChecked())
    {*makingQuery=*makingQuery+" strasse,";}

    if (firstName.isChecked())
    {*makingQuery=*makingQuery+" vorname,";}

    if (lastName.isChecked())
    {*makingQuery=*makingQuery+" nachname,";}

    if (male.isChecked())
    {*makingQuery=*makingQuery+" anrede,";}

    if (roll.isChecked())
    {*makingQuery=*makingQuery+" rollenname,";}

    if (companyName.isChecked())
    {*makingQuery=*makingQuery+" firmenname,";}

    if (phone.isChecked())
    {*makingQuery=*makingQuery+" telefonnummer, beschreibung,";}

    if (mail.isChecked())
    {*makingQuery=*makingQuery+" mail,";}

    //löscht das letzte Komma bevor FROM geschrieben wird. SQL Fehler vermeiden!
     makingQuery->remove((makingQuery->length()-1),1);
    *makingQuery=*makingQuery+" FROM";

    if (plz.isChecked() || number.isChecked() || city.isChecked() || street.isChecked())
    {*makingQuery=*makingQuery+" dm_anschrift,";}

    if (firstName.isChecked() || lastName.isChecked() || male.isChecked())
    {*makingQuery=*makingQuery+" dm_person,";}

    if (phone.isChecked())
    {*makingQuery=*makingQuery+" dm_phone,";}

    if (mail.isChecked())
    {*makingQuery=*makingQuery+" dm_mail,";}

    //löscht das letzte Komma vor WHERE geschrieben wird. SQL Fehler vermeiden!
    makingQuery->remove((makingQuery->length()-1),1);

    if (whereSelect.currentText()=="Suche verfeinern")
    {
    }else
    {
        *makingQuery=*makingQuery+" WHERE ";
        *makingQuery=*makingQuery+whereSelect.currentText();
        *makingQuery=*makingQuery+relation.currentText();
        *makingQuery=*makingQuery+"'";
        *makingQuery=*makingQuery+searcher.text();
        *makingQuery=*makingQuery+"'";

    }
    *localViewName=nameOfV.text();
}

BTW.: Ich arbeite hier mit der QT Bibliothek da es sich aber um Speicherfehler handelt dachte ich ein C++ Forum ist geeigneter. Ich weis es ist viel Code, das Meiste sind Form Objekte die in Layouts geschachtelt werden.

Ist die Funktion ein gängiger Workaround oder liegt da schon der Fehler begraben?

void ViewQuerryOption::tableViewOption(QString *query, QString *viewName, QMap <QString, ModulQuery> *TqmStorage)
{
    makingQuery=query;
    localViewName=viewName;
    localTmpMap = TqmStorage;
    this->exec();
}

Vielen Dank schonmal für eure Hilfe!

Gruß Meho

Geraten: Qt löscht deine View schon, dein delete ist dann eins zu viel

Ich führe

this->close();

in "void ViewQuerryOption::okPushed()" aus und erbe von QDialog also passt
"If the widget has the Qt::WA_DeleteOnClose flag, the widget is also deleted."

Vielen Dank für eure schnelle Hilfe!

BTW: Kann ich irgenwie nachvolziehen ob das Objekt gelöscht wurde im Debugger habe ich nichts gefunden, da sieht man nur den Stack.

Gruß Meho

BTW: Kann ich irgenwie nachvolziehen ob das Objekt gelöscht wurde im Debugger habe ich nichts gefunden, da sieht man nur den Stack.

Haltepunkt oder Ausgabe im Destruktor

Auf der anderen Seite sucht, googelt und wühlt man an einem Fehler, stunden lang, den andere in nicht mal 5 min beantwortet haben!

Oft ist es leider andersrum, viele fragen einfach in Foren nach, ohne auch nur 2 min selbst nachzulesen oder google zu fragen...

Also auf der einen Seite bin ich über eure schnelle und kompetente Hilfe sehr beeindruckt und sehr dankbar!
Auf der anderen Seite sucht, googelt und wühlt man an einem Fehler, stunden lang, den andere in nicht mal 5 min beantwortet haben!

Das nennt man Erfahrung

Erfahrung gewinnt man (auch), indem man Fehler macht!