Verständnisfrage zum automatischen Reference Counting von Qt

[caetydid]

Hi,

soweit ich weiß, bieten alle QObject-Ableitungen ja die Möglichkeit des automatischen reference countings.
D.h. wenn ich ein QObject lösche, werden vorher dessen Kinder alle gelöscht.
Ich vermute mal, das "Hochzählen" geht über den "=" Operator...

Jetzt ist mir dazu folgendes passiert:

Ich habe Klasse A, die von B und C benutzt wird (in verschiedenen Threads, alle drei sind also auch QObjects).
Im Konstruktor von B und C übergebe ich ein A Objekt als "parent".

Code: Alles auswählen

MyApp::MyApp(...) : QApplication(...)
{
A a( this );
B b( &a );
C c( &a );
}

Wenn jetzt MyApp::~MyApp(){}
ausgeführt wird, sollten a, b, c doch sauber gelöscht werden, oder?

Weil ich in B und C auch const-Routinen von A benutzen will, habe ich
mir den Zeiger kopiert:

Code: Alles auswählen

B::B( A* parent ) : QThread( parent )
{
m_A = (const A* )parent;
...
}

Ist sowas überhaupt erlaubt, oder muß ich z.B.

Code: Alles auswählen

dynamic_cast<A*>(this->parent()) 

nutzen um an den Parent zu kommen?

Ich habe nämlich sonst das Problem, daß in Klasse B irgendwann nach dem Aufruf von A::~A() m_A ungültig wird und ich eine Methode damit aufrufe. Zu diesem Zeitpunkt darf b ja schon nicht mehr existieren!

Ich habe das Gefühl, daß Qt einem da sicher gute Hilfmittel mitgibt, ich sie aber nur noch nicht gefunden habe

vielen Dank für eure Hilfe!
caetydid >8^)

[Christian81]

Wenn jetzt MyApp::~MyApp(){}
ausgeführt wird, sollten a, b, c doch sauber gelöscht werden, oder?

Ja, allerdings werden sie in deinem Fall schon nach verlassen des Konstruktors wieder gelöscht

Weil ich in B und C auch const-Routinen von A benutzen will, habe ich
mir den Zeiger kopiert:

Code: Alles auswählen

B::B( A* parent ) : QThread( parent )
{
m_A = (const A* )parent;
...
}

Ist sowas überhaupt erlaubt, oder muß ich z.B.

Code: Alles auswählen

dynamic_cast<A*>(this->parent()) 

nutzen um an den Parent zu kommen?

Beides überflüssig.... m_A = A; und fertig

Ich habe nämlich sonst das Problem, daß in Klasse B irgendwann nach dem Aufruf von A::~A() m_A ungültig wird und ich eine Methode damit aufrufe. Zu diesem Zeitpunkt darf b ja schon nicht mehr existieren!

Das verstehe ich gerade nicht was Du meinst... ausserdem gibt es QPointer

[caetydid]

Hi Christian, danke für die super-schnelle Antwort

Beides überflüssig.... m_A = A; und fertig

Hm klar. Kann er noch so casten...

Ich habe nämlich sonst das Problem, daß in Klasse B irgendwann nach dem Aufruf von A::~A() m_A ungültig wird und ich eine Methode damit aufrufe. Zu diesem Zeitpunkt darf b ja schon nicht mehr existieren!

Das verstehe ich gerade nicht was Du meinst... ausserdem gibt es QPointer

Okay, ich meinte folgendes:

Beim Beenden von MyApp erhalte ich eine Schutzverletzung, und zwar an einer Stelle in b, wo gerade eine Methode von a aufgerufen wird.
Der Zeiger auf a ( m_A ) ist an dieser Stelle bereits ungültig - das dürfte nicht sein.

Ich habe mir überlegt, ob ich eine Kopie des Zeigers des parent in b machen darf.
Wenn das den Zähler erhöht, dann wird a erst gelöscht, wenn b gelöscht wird (weil dann die m_A - Referenz gelöscht wird).
Aber eigentlich sollte das Löschen von a ja die Löschung seiner Kinder anstoßen.

caetydid >8^)

[caetydid]

... ausserdem gibt es QPointer

Kann ich das eine oder andere mal gebrauchen.
Schlimm, muß meine Qt-Programme immer 20x umschreiben, weil ich immer wieder einen Weg finde, wie man irgendwas noch besser machen kann

[Christian81]

Eine Pointer-Kopie erhöht nie irgend einen Counter! Ich würde mal ein bischen debuggen und Breakpoints bei den jeweiligen Destruktoren setzen.

[caetydid]

Eine Pointer-Kopie erhöht nie irgend einen Counter! Ich würde mal ein bischen debuggen und Breakpoints bei den jeweiligen Destruktoren setzen.

Schon gemacht:
Zuerst wird A::~A aufgerufen, dann C::~C.
Und dann knallt's in B::run() (weil m_A ungültig ist).
B::~B wurde bis dahin nicht aufgerufen.

m_A muß doch so lange gültig bleiben, bis A::~A komplett abgearbeitet ist, oder?
Also sieht es für mich so aus, als ob B::~B einfach nicht aufgerufen wird, aber wie kann das sein, wenn das A-Objekt das 'parent' ist?

Sorry, vermutlich stehe ich irgendwie auf dem Schlauch...

caetydid >8^)

[Christian81]

Übergibst Du auch den parent immer richtig an QObject?

[caetydid]

Übergibst Du auch den parent immer richtig an QObject?

An QThread (aber das erbt ja von QObject). Trotzdem könnte es was mit QThread zu tun haben, das QObject funktioniert nämlich.

Code: Alles auswählen

TrackerEngine::TrackerEngine( QObject* parent, unsigned int mtBufSize, unsigned int sdBufSize ) : 
	QObject( parent )
{
    mMotionTracker = new MotionTracker( this, mtBufSize );
    mCompass = new Compass( mMotionTracker );
    mStepDetector = new StepDetector( mMotionTracker, sdBufSize );
    ....
}

StepDetector::StepDetector( MotionTracker* mt, unsigned int bs ) : 
	mBufSize( qMin( bs, mt->bufferSize() - 1 ) ),
	QThread( mt )
{
    ...
    start();
}

Compass::Compass( MotionTracker* mt ) : QObject( mt )
{
    ...
}

Auf MotionTracker greife ich mit parent() zu.
In StepDetector::run() krachts dann...

Danke!
Caetydid >8^)