(Hintergrund)
Im großen und ganzen sieht das aus wie das von Corel PhotoPaint 10 (ein Feld für den Farbton und ein zweites Feld für Helligkeit und Sättigung -> HSV-Format)
Der Unterschied zu Corel liegt darin, das mein Tool kleiner ist (startet schneller usw.) und zweitens die Farbwerte direkt in Hex-Werten ausgibt.
Außerdem möchte ich noch eine Alphafunktion und eventuell diverse DirectX Farbformate einbauen...

(Details)
Das ganze Tool ist komplett mit der WinAPI geschrieben, ich benutze also kein MFC oder so. Compiler: VC++ 6.0
Eine Mausbewegung über den Farbwahlfeldern fange ich mit WM_LBUTTONDOWN / WM_MOUSEMOVE ab. Wenn dort (wParam & MK_LBUTTON)==true wird die Cursorposition in eine Farbe umgerechnet und alle dadurch veränderten Farbwahlfelder neugezeichnet.

(Problem)
Eigentlich läuft das alles schon nur habe ich folgenden Bug:
Wenn ich ca. 30 Sekunden lang mit gedrückter Maustaste über dem Farbwahlfeld herumkreise treten plötzlich komische Fehler auf:
1. das Feld welches die gewählte Farbe zeigt wird plötzlich grau (Fensterhintergrundfarbe)
2. Die Schrift im Arbeitsplatz (Explorer) ändert sich plötzlich in die Standartschrift
(die Schrift die man hat wenn man z.B. einen Button mit CreateWindowEx erzeugt und kein WM_SETFONT sendet)
3. diverse Icons verschwinden (z.B. die Kopieren/Einfügen-Schaltflächen im Arbeitsplatz)

(bisherige Lösungsversuche)
Ich hatte zunächst den Verdacht das irgendwie zuviele WM_PAINT-Nachrichten erzeugt werden und dadurch Windows irgendwie lahmgelegt wird. (die Farbfelder werden ja bei einer Mausbewegung andauernt neugezeichnet)
Also hab ich vor jedem RedrawWindow(...) einfach eine Varibale == true gesetzt und bei jedem neuzeichnen (in der WndProc) wieder ==false
Das brachte leider keine Wirkung.

Habt ihr sonst noch Ideen?

THX,
Coolcat

Ich habe jetzt die letzten beiden Stunden damit verbracht na Speicherlecks zu suchen, allerdings ohne Erfolg.

Könnte es daran liegen das ich die Farbwahlfelder zuerst in ein HBitmap zeichne (per SetPixelV) und erst dann mit BitBlt auf den Bildschrim zeichne?
Die HBitmaps sind global und werden am Anfang erzeugt und erst am Ende wieder freigegeben. Der zugehörige HDC war zunächst auch global, aber ich hab das jetzt so geändert, das er jedes mal wenn gezeichnet wird neu erzeugt wird (mit CreateCompatibleDC). Nach dem zeichnen wird er dann brav mit DeleteDC gelöscht.
Naja, das hat auch nix gebracht.

Zumindest weiß ich jetzt in welche Richtung ich suchen muss....aber wenn euch sonst noch etwas hilfreiches einfällt...

Coolcat

HGDIOBJ hOld = SelectObject(hdc, hXYZ);
.
.
.
SelectObject(hdc, hOld);

Coolcat

Bedenke auch, daß Du alles, was Du in einen DC hineinselektierst, auch wieder herausholen mußt!

HGDIOBJ hOld = SelectObject(hdc, hXYZ);
.
.
.
SelectObject(hdc, hOld);

Warum ist das überhaupt nötig? Gibt es vielleicht irgendwo in der MSDN eine Erklärung dafür?

@krendil: keine Ahnung.

Sonst noch Ideen?

Coolcat

Warum ist das überhaupt nötig? Gibt es vielleicht irgendwo in der MSDN eine Erklärung dafür?

GDI-Objekte besitzen einen internen Zähler, der erhöht wird, wenn sie in einen DC eigesetzt werden. Wenn ein Objekt löscht, wird zunächst der Zähler um eins erniedrigt. Da der Zähler nicht gegen 0 geht, wenn das Objekt noch irgendwo eingesetzt ist, wird nichts gelöscht.

Dieser Umstand ist in der Tat total lasch dokumentiert.

Coolcat schrieb:

@King: Nee, das war es leider auch nicht

Wenn Du das vorher nicht hattest, ist das auf jeden Fall ein Teil der Lösung. Mach die Änderung nicht wieder Rückgängig!

Coolcat schrieb:

Sonst noch Ideen?

Ideen braucht es aber keine mehr: Entweder gibst Du nach Gebrauch nicht alles wieder frei oder Du selektierst nicht alles ordentlich zurück oder beides. Punkt.

Noch genauer könnte man etwas sagen, wenn Du einen Auszug Deines Quells zeigen würdest.

Das hier wird bei Maus-Drag andauernt aufgerufen:

WndProc des Farbwahlfeldes:

case WM_LBUTTONDOWN:	
case WM_MOUSEMOVE:
{
   if (wParam & MK_LBUTTON)
   {

       //.....

       hDC = GetDC(hColorSV); //hColorSV ist das Farbwahlfeld
       if (!hDC) return 0;
       HDC hDCCursorSV = CreateCompatibleDC(hDC);
       HGDIOBJ hGdiObj = SelectObject(hDCCursorSV, hCursorSV); 
                            // hCursorSV ist ein HBITMAP das den Cursor enthält

       // alten Cursor löschen
       BitBlt(hDC, xCurPos-4, yCurPos-4, 9, 9, hDCCursorSV, 0, 0, SRCINVERT);
       // neuen Cursor zeichnen
       xCurPos = xCur; // xCur ist die Mausposition
       yCurPos = yCur; // xCurPos ist eine globale Variable 
       BitBlt(hDC, xCurPos-4, yCurPos-4, 9, 9, hDCCursorSV, 0, 0, SRCINVERT);

       SelectObject(hDCCursorSV, hGdiObj);

       DeleteDC(hDCCursorSV);
       ReleaseDC(hColorSV, hDC);

       //....

       // Farbansichtsfeld neuzeichnen
       RedrawWindow(hViewColor, NULL, NULL, RDW_INVALIDATE | RDW_UPDATENOW);
       // siehe WM_PAINT unten!

       //....

       return 0;
   }
}

WndProc des Farbanzeigefeldes:

case WM_PAINT:
{
    PAINTSTRUCT ps;
    hDC = BeginPaint(hViewColor, &ps); // hViewColor ist das Anzeigefeld
    if (!hDC) return 0;  
    RECT rect;
    rect.top    = ps.rcPaint.top;
    rect.right  = ps.rcPaint.right;
    rect.left   = ps.rcPaint.left;
    rect.bottom = ps.rcPaint.bottom;		

    // Draw 5 Unit Border
    HGDIOBJ hOldBrush = SelectObject(hDC,    CreateSolidBrush(GetSysColor(COLOR_BTNFACE)));
    HGDIOBJ hOldPen = SelectObject(hDC, CreatePen(PS_NULL, 0, 0));

    if (rect.left   < 5  ) 
    { 
        Rectangle(hDC, 0, 0, 6, 47);
        rect.left   = 5  ; 
    }
    if (rect.top    < 5  ) 
    { 
        Rectangle(hDC, 5, 0, 82, 6);
        rect.top    = 5  ; 
    }
    if (rect.right  > 82 ) 
    { 
        Rectangle(hDC, 81, 0, 87, 47);
        rect.right  = 82; 
    }
    if (rect.bottom > 42 ) 
    { 
        Rectangle(hDC, 5, 41, 82, 47);
        rect.bottom = 42 ; 
    }	

    // Draw Rect with choosen Color
    SelectObject(hDC, CreateSolidBrush(dwColorHSV));
    Rectangle(hDC, rect.left, rect.top, rect.right, rect.bottom); 

    // Restore old settings
    SelectObject(hDC, hOldPen);
    SelectObject(hDC, hOldBrush);

    EndPaint(hViewColor, &ps);
    return 0;
}

Also ich sehe da jetzt so keine Fehler?!

Coolcat

P.S.: @Kommentare: Ja ich weiß, Deutsch-Englisch-Mix soll man nicht machen...die deutschen Kommentare hab ich nur hier fürs Forum schnell eingefügt...

Naja ich denke mal ihr werdet euch nicht durch meinen kompletten Quellcode (ca. 1000 Zeilen) wühlen....daher die Kurzform:

Sicher nicht, deswegen schrieb ich auch Auszug.

Coolcat schrieb:

// Draw Rect with choosen Color
    SelectObject(hDC, CreateSolidBrush(dwColorHSV));
    Rectangle(hDC, rect.left, rect.top, rect.right, rect.bottom);

Was passiert mit dem Brush, den Du da mit CreateSolidBrush erzeugst? Wie, wann und wo wird der freigegeben? Wo wird der zuvor erzeugte Pen gelöscht? Wo wird der zuvor erzeugte Brush (mit COLOR_BTNFACE) gelöscht?

Coolcat

Ach, die muss ich auch wieder löschen?

Wenn Du sie nicht mehr beötigst: Ja! Aber das hat Dir flenders bereits vor unzähligen Beiträgen gesagt.

BTW: Der zweite in den DC eingesetzte Brush holt den Ersten wieder aus dem DC heraus. Bedenke das!

Ja! Aber das hat Dir flenders bereits vor unzähligen Beiträgen gesagt.

Hm, ich hatte einen HBrush nicht als GDI-Objekt angesehen...

BTW: Der zweite in den DC eingesetzte Brush holt den Ersten wieder aus dem DC heraus. Bedenke das!

Logisch, ich kann ja immer nur mit einer Farbe malen.

HGDIOBJ hGdiObj = SelectObject(hDC, hBrush1);
SelectObject(hDC, hBrush2);
SelectObject(hDC, hGdiObj);

Aber so hab ich doch den alten Brush der vorher drin war wieder eingesetzt, oder?

Coolcat

Ich hab die Brushes und Pens jetzt anschließend wieder mit DeleteObject gelöscht, aber das ist es auch nicht.

Mit Sicherheit nicht alle, und deswegen geht es auch noch immer nicht.

also es lag an folgendem :

1. Folgendes ist falsch:

HGDIOBJ hOldGdiObj = SelectObject(hDC, hBrush);
//mit Brush arbeiten...
SelectObject(hDC, hOldGdiObj);

Es muss so heißen:

HBRUSH hOldBrush = (HBRUSH)SelectObject(hDC, hBrush);
//mit Brush arbeiten...
SelectObject(hDC, hOldBrush);

2. Man darf kein GDI-Objekt löschen während es selektiert ist. Ich hatte einen Brush immer zuerst gelöscht und dann erst den alten wieder selektiert.
Kann mir jemand sagen warum?

THX @ All

Coolcat

Coolcat schrieb:

2. Man darf kein GDI-Objekt löschen während es selektiert ist. Ich hatte einen Brush immer zuerst gelöscht und dann erst den alten wieder selektiert.
Kann mir jemand sagen warum?

das ist eigentlich alles eine Frage des genauen Lesens der Beschreibungen in der MSDN zu den jeweiligen Funktionen. Für DeleteObject findet sich dazu die wichtige Bemerkung:

MSDN schrieb:

**Do not delete a drawing object (pen or brush) while it is still selected into a device context.
**

das ist das ganze Geheimnis, einfach befolgen, was dort steht, und man hat keine (na gut, weniger ) Probleme

MfG

Coolcat