Fenster wirklich neu zeichnen

Th69

Die OnDraw-Methode (bzw. intern die Reaktion auf die PAINT-Message) darf nur reine Zeichenfunktionen enthalten, keine sonstigen Aktualisierungen oder ähnliches.
Diese Aktualisierungen mußt du in deiner UI-Logik durchführen (also was du als "Rücksprung aus den Einstellungen" bezeichnet hast).
Diese Aktualisierungsfunktionen (wie Invalidate(), InvalidateRect(...), RedrawWindow(), etc.) versenden intern einfach die PAINT-Message (und wenn dies nicht intern von der Windows-Messagequeue abgefangen wäre, würde es, wie du es jetzt gemacht hast, zu einer Endlosschleife kommen).

DocShoe

Kann da erst mal nix Merkwürdiges sehen, außer dass beim Zeichnen auf Einstellungsänderungen reagiert wird. Ungewöhnlich, sollte in der Konstellation aber ok sein.

Wo kommt denn brush für pDC->SelectObject( &brush ) her? Wird der Original-Brush am Ende wiederhergestellt?
Kannst du das Zeichnen eurer Zeilen komplett auskommentieren und einfach mal Dummy Elemente zeichnen?

DocShoe

@Th69
Hab ich auch erst gedacht, aber das passiert nur, wenn g_bViewSettingChange true ist. In dem Fall wird`s auf false gesetzt und das Neuzeichnen getriggert. Prinzipiell hast du aber recht, das sollte an anderer Stelle passieren.

elmut19

@Th69
Hier noch de eine Fktn von meinem Vorgänger:

void UpdateCmdUI(BOOL bFromThread)
{
	if(bFromThread) // 20100520	RH
		theApp.m_pFrame->SendMessage(WM_COMMAND,ID_UPDATE_TOOLBAR);

//	else
//		theApp.m_pFrame->m_wndCommandBar.OnUpdateCmdUI(theApp.m_pFrame,TRUE);
}

DocShoe

Wie ich schon geschrieben habe, kommentier zur Fehlersuche mal alles aus und nach- und nach wieder ein. Oder lass alles auskommentiert und zeichne ein paar Elemente selbst um zu gucken, ob das funktioniert.

elmut19

@DocShoe
Die "brushes", etc, sind lokal definiert.
Am Ende der "DrawMainView(..)" erfolgt:

	if(pOldFont)
		pDC->SelectObject(pOldFont);
	delete pCompatibleDC;
	if(pOldBrush)
		pDC->SelectObject(pOldBrush);
	if(CheckSprache(pAnlage,4))
		CreateStdFont(FALSE);

elmut19

@DocShoe
Zwischen meinem ersten Versuch, der funktionierte, habe ich nichts an dem Teil des Quellcodes geändert.
Aber ich versuche auch mal das auskommentieren und auch die alte Version, die funktionierte.

DocShoe

@elmut19
Ja wenn nix geändert worden ist müssen es ja Neutrinoeinschläge in der CPU oder GPU sein.

elmut19

@DocShoe
oder Windows Update ...
... Typische Aussage, ich weiss auch: "Wir haben nix geändert ..."
Klingt schon lustig.

Aber eine ältere Version (leider nicht die, die erste Implementierung dazu hatte) funktioniert auch nicht mehr.

Erhard Henkes

Lass doch mal die KI drüber schauen. In Syntax-Fragen ist sie gut.

Martin Richter

HeapCompact hat nicht im OnDraw zu tun.
OnDraw wird im Kontext von einer WM_PAINT Nachricht versendet.
Entsprechend wird diese Nachricht nur erzeugt wenn:
a. das Fenster/der Auschnitt sichtbar ist
b. der Ausschnitt auch aktualisiert werden muss
In OnDraw hat ein UpdateWindow/RedrawWindow/InvalidateRect nichts zu suchen. Denn dies führt evtl. zu rekursiven aufrufen.
In einem OnDraw musst Du alle Bereiche auch neu zeichnen die betroffen sind.
Nach dem OnDraw / WM_PAINT werden alle Fenster/Bereiche validiert, das heißt sie bekommen das Flag "aktuell"
Durch ein InvalidateRect bekommt ein Bereich/Fenster das Flag "nicht aktuell"
Ein UpdateWidnow/RedrawWindow wird alle Bereiche aktualisieren, die als "nicht aktuell" markiert sind.

elmut19

@Erhard-Henkes
Hatte ich auch schon mal gemacht (anderer Fall)
Hat erkannt, was ich tun wollte. Aber nicht den Fehler gefunden.

Quiche Lorraine

@elmut19 sagte in Fenster wirklich neu zeichnen:

oder Windows Update ...

Bitte bleibt doch mal sachlich. Das Problem ist die WinAPI mit ihren vielen Seiteneffekten. Erst neulich hat Dr. Memory z.B. bei mir einen Fehler im Subclassing entdeckt. Und so wie es aussieht, verursachte dieser Fehler nur in 0,001% aller Fälle einen Absturz.

Ein paar weitere Punkte:

DIe Anweisung delete pCompatibleDC; finde ich etwas merkwürdig und sollte man überprüfen.
Ferner würde ich den folgenden Code prüfen:

if(bFromThread) // 20100520	RH
		theApp.m_pFrame->SendMessage(WM_COMMAND,ID_UPDATE_TOOLBAR);

Wenn der Aufruf hier durch einen Thread erfolgte, so sendest du eine WM_COMMAND Nachricht an m_pFrame. Oder genauer gesagt, fügst du eine WM_COMMAND Nachricht in die Message Queue von m_pFrame ein und wartest bis diese bearbeitet wurde.

-> Was passiert wenn du stattdessen PostMessage()WM_COMMAND` aufrufst?

BTW: EIn Klassiker beim Threading sind Synchronisationsfehler. Welche Thread Safety benötigst du? Und wie Thread Safe ist dein Code?

DocShoe

Lass da generell mal cppcheck drüberlaufen.

elmut19

@Martin-Richter
Tja, wo kann ich das "Invalidate.."-Zeugs dann hintun?
Meine "DrawMainView(..)" ist ja praktisch Teil der "OnDraw()".
Und dort wird eigentlich alles neu gezeichnet.
Aber nicht nur das Neue, sondern auch das Alte. Scheint zumindest so.
Andererseits werden in dem View zyklisch Texte neu ausgegeben, die sich ändern.
Diese werden nicht übereinander geschrieben.

Das mit "rekursive Aufrufe" habe ich ja erfolgreich unterbunden.
Was mein Vorgänger mit seinem "Heap"-Zeugs vorhatte, weiss ich nicht.
Es hat seit 20? Jahren nicht gestört.

Martin Richter

Invalidate brauchst Du gar nicht. Außer Du willst das neu Zeichnen erzwingen.
Wenn Dinge übereinander gezeichnet werden, dann wird evlt. der Hintergrund nicht neu gezeichnet. Also das alte Zeugs überschrieben.
Du musst nichts "unterbinden", sondern es unterlassen.
Wenn es nicht stört heißt es nicht dass es richtig ist.

DocShoe

@elmut19 sagte in Fenster wirklich neu zeichnen:

@Martin-Richter
Tja, wo kann ich das "Invalidate.."-Zeugs dann hintun?

An die Stelle, an der die Konfiguration geändert worden ist. Am Besten entkoppelst du das auch noch und löst das per Observer Pattern:

(Pseudocode)
void SomeClass::set_configuration( MyConfigurationData const& cfg )
{
   // Beim Setzen einer neuen Konfiguration wird ein Ereignis ausgelöst, das an alle angemeldeten Subscriber abgesetzt wird.
   Configuration_ = cfg;
   OnConfigurationChanged.fire( );
}

CViewTitanCe.cpp
CViewTitanCe::CViewTitanCe()
{
   // Subscriber anmelden
   instance_of_some_class().OnConfigurationChanged.subscribe( OnConfigurationChanged );
}

CViewTitanCe::~CViewTitanCe()
{
   // Subscriber abmelden
   instance_of_some_class().OnConfigurationChanged.unsubscribe( OnConfigurationChanged );
}

void CViewTitanCe::OnConfigurationChanged()
{
   // Callback für Konfigurationsänderung: Neuzeichnen anstossen
   Invalidate();
}

Leider gibt's im Standard C++ keine Publisher/Subscriber Unterstützung, da musst du dir was eigenes basteln. Für den Anfang reicht da vielleicht auch schon ein Funktionszeiger oder std::function.

elmut19

@Martin-Richter
Ja, ich wollte ja das Neuzeichnen erzwingen.
Aber was läuft dann schief?

Ich muss noch Einiges Überdenken.

Danke jedenfalls erstmal Euch allen

Th69

Wie schon geschrieben, mußt du das Neuzeichnen außerhalb des Zeichenvorgangs anstoßen.

Für das Löschen des Hintergrunds ist normalerweise die Nachricht WM_ERASEBKGND zuständig. Kannst ja mal schauen, ob dies in eurem Projekt verwendet wird (falls nicht, sollte es automatisch so funktionieren) - evtl. auch mal den Parameter fErase (wie im Link beschrieben) überprüfen.

elmut19

@Martin-Richter @Th69 @DocShoe
Danke für Eure Hilfe und für den Beispielcode.
Habe wieder was dazugelernt.

Ich habe jetzt jedenfalls verstanden und akzeptiert, dass das "Invalidate.."-Zeug nicht in diese "Draw()" oder "OnDraw()" Funktionen gehört.
Vor drei Wochen, als ich das implementiert habe, war zumindest mein Eindruck, dass das "Invalidate" eine Aktion ausgelöst hatte.
Nun scheint es das nichtmehr zu tun. Wie auch immer ... vielleicht nicht alle Fälle getestet ....

Inzwischen habe ich auch herausgefunden, dass, auch ohne "Invalidate", etc., ein Neuzeichnen stattfindet.
Somit lag der Fehler in meinem Algorithmus, der in einem bestimmten Fall die erste Zeile falsch berechnete.

Also vielen Dank nochmals an alle.