Eine (warscheinlich) dumme frage

[dot]

Das Problem ist offenbar, das du deine Variable pdis an einer Stelle in einem switch definiert hast, die potentiell übersprungen werden kann; check mal ab, ob du auch überall richtig deine {} gemacht hast...

switch (x)
{
  case 1:
    int bla;
    break;

  case 2:
    break;  // Problem: wenn x == 2 ist, wird die Definition von bla übersprungen
}

[hustbaer]

Da fehlt ein {

LPDRAWITEMSTRUCT pdis = (LPDRAWITEMSTRUCT) lParam; 
  switch (pdis->CtlID) /* und zwar hier */
    case IDC_BUTTON1:{

Tip:
Bei folgendem Einrückungsstil/Klammerstil ist es viel einfacher zu erkennen wo { oder } fehlen:

void Fun()
{
    switch (x)
    {
    case 1:
        {
            SomeType var = SomeFunction();
            // ...
        }
        break;
    }

    if (y)
    {
        for (int i = 0; i < 100; i++)
        {
            if (z)
                 SingleLineWithoutBlockIsOkToo();
        }
    }
}

[Aaron3219]

Mmh okay das hat alles funktioniert.
Allerdings gibt es nun ein neues Problem.

Normalerweise, invertiere ich die Button Farbe wenn man draufklickt. Das ist jetzt nicht mehr der Fall.
Woran liegt das?

case WM_DRAWITEM: {
  LPDRAWITEMSTRUCT pdis = (LPDRAWITEMSTRUCT) lParam;
  switch (pdis->CtlID){
     case IDC_BUTTON1:{
          HDC hDC = GetDC(hButton);
          HDC hComDC = CreateCompatibleDC(hDC);
          SelectObject(hComDC, hBitmap);
          BitBlt(hDC, 0, 0, bmpbut.bmWidth, bmpbut.bmHeight, hComDC, 0, 0, SRCCOPY);
          DeleteObject(hComDC);
          ReleaseDC(hButton, hDC);}
          break;

          case IDC_BUTTON2:{
          HDC hDC = GetDC(hButton1);
          HDC hComDC = CreateCompatibleDC(hDC);
          SelectObject(hComDC, hBitmap1);
          BitBlt(hDC, 0, 0, bmpbut.bmWidth, bmpbut.bmHeight, hComDC, 0, 0, SRCCOPY);
          DeleteObject(hComDC);
          ReleaseDC(hButton1, hDC);}
          break;

    if(pdis->itemState & ODS_SELECTED)
       InvertRect (pdis->hDC, &pdis->rcItem); // wenn Button gedrückt invertieren
  }}

[hustbaer]

Der selbe Code, nur anders eingerückt:

case WM_DRAWITEM:
		{ 
			LPDRAWITEMSTRUCT pdis = (LPDRAWITEMSTRUCT) lParam; 
			switch (pdis->CtlID)
			{ 
			case IDC_BUTTON1:
				{ 
					HDC hDC = GetDC(hButton); 
					HDC hComDC = CreateCompatibleDC(hDC); 
					SelectObject(hComDC, hBitmap); 
					BitBlt(hDC, 0, 0, bmpbut.bmWidth, bmpbut.bmHeight, hComDC, 0, 0, SRCCOPY); 
					DeleteObject(hComDC); 
					ReleaseDC(hButton, hDC);
				} 
				break; 

			case IDC_BUTTON2:
				{ 
					HDC hDC = GetDC(hButton1); 
					HDC hComDC = CreateCompatibleDC(hDC); 
					SelectObject(hComDC, hBitmap1); 
					BitBlt(hDC, 0, 0, bmpbut.bmWidth, bmpbut.bmHeight, hComDC, 0, 0, SRCCOPY); 
					DeleteObject(hComDC); 
					ReleaseDC(hButton1, hDC);
				} 
				break; 

				if (pdis->itemState & ODS_SELECTED) 
					InvertRect(pdis->hDC, &pdis->rcItem); // wenn Button gedrückt invertieren 
			}
		}

Siehst du es jetzt?

Was ich über Einrückungsstil/Klammerstil geschrieben habe hat schon seinen Grund.

ps: Der Compiler müsste dir hier auch eine Warning geben. Irgendwas über "unreachable code". Warnings ignorieren ist auch keine gute Idee.

[Aaron3219]

mmh ne hat auch nicht geholfen

[hustbaer]

Beschreib mal wie du meinst dass die Ausführung die Zeile if (pdis->itemState & ODS_SELECTED) erreichen soll.

[Aaron3219]

Tja gute frage.

Aber wenn ich nur einen button habe funktioniert ja alles. Warum nicht wenn ich 2 habe

[Aaron3219]

Okay ich habe das Problem gelöst. Waren nur 2 kleine Wörter.

break;
// und
break;

[hustbaer]

Nein, die break s sind da schon OK.

Ich glaube dir fehlen einfach noch viel zu viele Grundlagen.

Eine "korrekte" Lösung wäre:

case WM_DRAWITEM:
        { 
            LPDRAWITEMSTRUCT pdis = (LPDRAWITEMSTRUCT) lParam; 
            bool invertIfSelected = false;
            switch (pdis->CtlID)
            { 
            case IDC_BUTTON1:
                invertIfSelected = true;
                { 
                    HDC hDC = GetDC(hButton); 
                    HDC hComDC = CreateCompatibleDC(hDC); 
                    SelectObject(hComDC, hBitmap); 
                    BitBlt(hDC, 0, 0, bmpbut.bmWidth, bmpbut.bmHeight, hComDC, 0, 0, SRCCOPY); 
                    DeleteObject(hComDC); 
                    ReleaseDC(hButton, hDC);
                } 
                break; 

            case IDC_BUTTON2:
                invertIfSelected = true;
                { 
                    HDC hDC = GetDC(hButton1); 
                    HDC hComDC = CreateCompatibleDC(hDC); 
                    SelectObject(hComDC, hBitmap1); 
                    BitBlt(hDC, 0, 0, bmpbut.bmWidth, bmpbut.bmHeight, hComDC, 0, 0, SRCCOPY); 
                    DeleteObject(hComDC); 
                    ReleaseDC(hButton1, hDC);
                } 
                break; 
            }

            if (invertIfSelected && (pdis->itemState & ODS_SELECTED))
                InvertRect(pdis->hDC, &pdis->rcItem); // wenn Button gedrückt invertieren 
        }

[Aaron3219]

Mmh nachdem ich aber die breaks; weggenommen habe, hat alles funktioniert

[hustbaer]

Ja, das ist mir schon klar.
Trotzdem ist es Mist ohne die break s.

Ist dir denn klar was ohne die break s passiert wenn das ganze mit pdis->CtlID == IDC_BUTTON1 ausgeführt wird?

[Aaron3219]

ne warum?

[hustbaer]

Verstehst du überhaupt irgendwas davon was das Programm macht?

[merano]

hustbaer schrieb:

Verstehst du überhaupt irgendwas davon was das Programm macht?

Die erste Frage war eigentlich schon Beweis genug, das dies nicht der Fall ist:

https://www.c-plusplus.net/forum/331622-10

Weitere Buttons waren als Kommentar schon vorgesehen ...