switch, break, case c++

[EWeiss]

Interesse halber hätte ich gern gewusst wie break in Switchs richtig angewendet werden.

Der break nach FullScreen soll falsch sein nur warum ?
Ist genauso wie bei der case Anweisung darüber.

Bei meiner Frage geht es ausschließlich um die Anordnung Zuweisung von switch, break, case der Code ist nebensächlich.

Code:

LRESULT CALLBACK WndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
{
    WPARAM wpItem;
    LRESULT IntI = 0;

    switch (message)
    {
    case WM_GETMINMAXINFO:
    {
        MINMAXINFO* pInfo = (MINMAXINFO*)lParam;
        pInfo->ptMinTrackSize.x = XMIN_SIZE;
        pInfo->ptMinTrackSize.y = YMIN_SIZE;
    }
    break;

    case WM_COMMAND:
    {
        if (HIWORD(wParam) == CBN_SELCHANGE)
        {
            switch LOWORD(wParam)
            {
            case ID_CBVRENDERER:
            {
                WCHAR * Buffer[5];

                IntI = SendMessage(HW(ID_CBVRENDERER), CB_GETCURSEL, IntI, 0);
                if (IntI != CB_ERR)
                    SendMessage(HW(ID_CBVRENDERER), CB_GETLBTEXT, IntI, (WPARAM)(&Buffer));

                KVideo_SetVideoRenderer((TVRenderer)IntI);

                if (KVideo_GetPlayerState() != psNotReady)
                    Confirm(L"Designated video renderer will be applied since next opening media file.");
            }
            break;

            case ID_CBPLAYBACKRATE:
            {
                WCHAR * Buffer[5];

                IntI = SendMessage(HW(ID_CBPLAYBACKRATE), CB_GETCURSEL, IntI, 0);
                if (IntI != CB_ERR)
                    SendMessage(HW(ID_CBPLAYBACKRATE), CB_GETLBTEXT, IntI, (WPARAM)(&Buffer));

                WCHAR * fIn = (WCHAR*)(&Buffer);
                float fOut = (float)wcstod(fIn, NULL);

                KVideo_SetPlaybackRate(fOut);
            }
            break;
            }
        }

        wpItem = LOWORD(wParam);
        switch (wpItem)
        {
        case ID_OPEN:
        {
            if (KVideo_GetPlayerState() != psNotReady)
            {
                KillTimer(gP.MovieHandle, MOVIE_TIMER);
                KVideo_CloseFile();
                // Einstellungen zurück setzen
                gP.MenuAudioChecked = 0;
                gP.MenuVideoChecked = 0;
                SetWindowText(HW(LB_STREAMDURATION), L"00:00:00\\");
                SetWindowText(HW(LB_STREAMELAPSED), L"00:00:00");
                SetWindowText(HW(LB_VIDEOINFO), L"0 x 0, 0 FPS");
                SetWindowText(HW(LB_FILENAME), L"_");
                SetWindowText(HW(LB_AUDIOPROPERTY), L"0 Channel, 0 SPS, 0 BPS");
                SetScrollPos(gP.MovieHandle, SBS_HORZ, 0, TRUE);
                InvalidateRect(hWnd, NULL, FALSE);
                UpdateWindow(hWnd);
            }

            // OpenFileDialog Initialisieren           
            CoInitialize(NULL);

            const WCHAR *lpstrFilter = L"Media Files\0*.aac;*.asf;*.avi;*.m4a;*.mp3;*.mp4;*.m4v;*.wav;*.wma;*.wmv;*.3gp;*.3g2;*.mpeg;*.mpg;*.mov;*.qt;*.mkv;*.flv;*.vob\0"
                L"All files\0*.*\0";

            HANDLE hf;
            OPENFILENAME OpenMediaFile;
            ClearMemory(&OpenMediaFile, sizeof(OpenMediaFile));

            WCHAR szFile[MAX_PATH];
            szFile[0] = L'\0';

            OpenMediaFile.lStructSize = sizeof(OpenMediaFile);
            OpenMediaFile.hwndOwner = NULL;
            OpenMediaFile.lpstrFile = szFile;
            OpenMediaFile.nMaxFile = sizeof(szFile);
            OpenMediaFile.lpstrFilter = lpstrFilter;
            OpenMediaFile.nFilterIndex = 1;
            OpenMediaFile.lpstrFileTitle = NULL;
            OpenMediaFile.nMaxFileTitle = 0;
            OpenMediaFile.lpstrInitialDir = NULL;
            OpenMediaFile.Flags = OFN_PATHMUSTEXIST | OFN_FILEMUSTEXIST;

            // Video starten
            if (GetOpenFileName(&OpenMediaFile))
            {
                hf = CreateFile(OpenMediaFile.lpstrFile, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, 0);

                if (hf > 0)
                {
                    CloseHandle(hf);
                    LRESULT IntI = SendMessage(HW(ID_CBVRENDERER), CB_GETCURSEL, 0, 0);
                    KVideo_SetVideoRenderer((TVRenderer)IntI);

                    // Mediafile starten
                    RunMediaFile(szFile, FALSE);
                }
            }
            CoUninitialize();
        }
        break;
        case ID_PLAY:
        {
            if (KVideo_GetPlayerState() == psPlaying)
                return 0;
            if (KVideo_GetPlayerState() == psPaused || KVideo_GetPlayerState() == psStopped)
                if (KVideo_Play())
                    SetTimer(gP.MovieHandle, MOVIE_TIMER, 200, NULL);
        }
        break;
        case ID_PAUSE:
        {
            if (KVideo_GetPlayerState() == psPlaying)
                if (KVideo_Pause())
                    KillTimer(gP.MovieHandle, MOVIE_TIMER);
        }
        break;
        case ID_STOP:
        {
            if (KVideo_GetPlayerState() == psPlaying || KVideo_GetPlayerState() == psPaused)
                if (KVideo_Stop())
                    KillTimer(gP.MovieHandle, MOVIE_TIMER);

            SetWindowText(HW(LB_STREAMDURATION), L"00:00:00\\");
            SetWindowText(HW(LB_STREAMELAPSED), L"00:00:00");
            SetScrollPos(gP.MovieHandle, SBS_HORZ, 0, TRUE);
            InvalidateRect(hWnd, NULL, FALSE);
            UpdateWindow(hWnd);
        }
        break;
        case ID_FULLSCREEN:
        {
            gP.bFullScreen = !gP.bFullScreen;
            SetForegroundWindow(hWnd);
            if (gP.bFullScreen)
            {
                RECT rect;
                rect.right = GetSystemMetrics(SM_CXSCREEN);
                rect.bottom = GetSystemMetrics(SM_CYSCREEN);
                SetWindowLongPtr(hWnd, GWL_STYLE, WS_POPUP | WS_VISIBLE);
                AdjustWindowRect(&rect, WS_POPUP, FALSE);
                MoveWindow(hWnd, 0, 0, rect.right, rect.bottom, FALSE);
            }
            else
            {
                RECT rect;
                rect.right = dpi(XMIN_SIZE);
                rect.bottom = dpi(YMIN_SIZE);
                rect.left = (GetSystemMetrics(SM_CXSCREEN) - rect.right) / 2;
                rect.top = (GetSystemMetrics(SM_CYSCREEN) - rect.bottom) / 2;
                SetWindowLongPtr(hWnd, GWL_STYLE, WS_OVERLAPPEDWINDOW | WS_VISIBLE);
                AdjustWindowRect(&rect, WS_OVERLAPPEDWINDOW, FALSE);
                MoveWindow(hWnd, rect.left, rect.top, dpi(XMIN_SIZE), dpi(YMIN_SIZE), FALSE);
            }
        }
        break;
        case ID_CAPTURE:
        {
            TPlayerState ps = KVideo_GetPlayerState();
            if (ps == psStopped || ps == psNotReady)
            {
                Confirm(L"It's not playing state.");
                return 0;
            }
            else
            {
                HBITMAP hbmp = 0;
                if (KVideo_GetCapturedImage(&hbmp))
                {
                    WCHAR Image[MAX_PATH];
                    ZeroMemory(Image, MAX_PATH * sizeof(WCHAR));

                    WCHAR * Path = { 0 };
                    Path = EXEpath();

                    _snwprintf_s(Image, MAX_PATH, 259, L"%s%d%s", Path, gP.PicCount, L"_Capture.jpg");

                    SaveBitmapToFile(Image, hbmp, 0);
                    gP.PicCount++;
                }
                DeleteObject(hbmp);
            }
        }
        break;
        case ID_CLOSE:
            DestroyWindow(hWnd);
            break;
            // Menüauswahl bearbeiten:
        case ID_CKB_SEARCH:
        {
            if (KVideo_GetPlayerState() != psNotReady)
            {
                BOOL checked = IsDlgButtonChecked(hWnd, ID_CKB_SEARCH);
                if (checked) {
                    CheckDlgButton(hWnd, ID_CKB_SEARCH, BST_UNCHECKED);
                }
                else
                {
                    CheckDlgButton(hWnd, ID_CKB_SEARCH, BST_CHECKED);
                }
                KVideo_SetSearchMode(EnableWindow(HW(ID_CBPLAYBACKRATE), !checked));
                break;
            }
        }
        break;
        case IDM_ABOUT:
            DialogBox(GetModuleHandle(0), MAKEINTRESOURCE(IDD_ABOUTBOX), hWnd, About);
            break;
        case IDM_EXIT:
            DestroyWindow(hWnd);
            break;
        default:
            return DefWindowProc(hWnd, message, wParam, lParam);
        }
    }
    break;
    case WM_HSCROLL:
    {
        switch LOWORD(wParam)
        {
        case TB_ENDTRACK:
            gP.bVolume = FALSE;
            break;
        case TB_THUMBTRACK:  // ziehen des "Sliders"
        case TB_TOP:         // Pos1
        case TB_BOTTOM:      // Ende
        case TB_LINEDOWN:    // Pfeiltasten oben/unten
        case TB_PAGEDOWN:    // Bild runter & in die Leiste geklickt
        case TB_PAGEUP:      // Bild auf & in die Leiste geklickt
        {
            gP.bVolume = TRUE;
            gP.Position = (int)SendMessage(HW(ID_TBVOLUME), TBM_GETPOS, 0, 0);
            if (gP.Position == 0)
                KVideo_SetVolume(-10000);
            else
                KVideo_SetVolume(LONG(5000 * log10(gP.Position)) - 10000);
        }
        break;
        }
    }
    break;
    case WM_SIZE:
    {
        EnumChildWindows(hWnd, &AnchorEnum, 0);
        RedrawWindow(hWnd, NULL, NULL, RDW_ERASE | RDW_INVALIDATE | RDW_ERASENOW | RDW_UPDATENOW | RDW_ALLCHILDREN | RDW_FRAME);
    }
    break;
    case WM_CTLCOLORSTATIC:
    {
        HDC hdcStatic = (HDC)wParam;
        SetBkMode(hdcStatic, TRANSPARENT);
        return (INT_PTR)GetStockObject(WHITE_BRUSH);
    }
    break;
    case WM_PAINT:
    {
        PAINTSTRUCT ps;
        HDC hdc = BeginPaint(hWnd, &ps);
        EndPaint(hWnd, &ps);
    }
    break;
    case WM_DESTROY:
        PostQuitMessage(0);
        break;
    default:
        return DefWindowProc(hWnd, message, wParam, lParam);
    }
    return 0;
}

Auch mit den Klammern komme ich nicht klar.
Ab wann bzw.. wie vielen Zeilen nach case wird die Klammer benötigt?


gruss

[zeras]

Soweit ich C kenne, wird jeder Case Zweig durchlaufen, außer du machst am Ende jedes Zweiges das break. Das ist ja der große Unterschied zu Delphi.
Das break muss dann natürlich innerhalb der Klammer sein.

[DeddyH]

Und soweit ich es noch weiß, werden Klammern für einen Switch-Case gar nicht benötigt, im Gegensatz zu Delphi.

[EWeiss]

Zitat:

Zitat von zeras (Beitrag 1379559)

Soweit ich C kenne, wird jeder Case Zweig durchlaufen, außer du machst am Ende jedes Zweiges das break. Das ist ja der große Unterschied zu Delphi.
Das break muss dann natürlich innerhalb der Klammer sein.

Ich muss das mal Code mäßig sehen. :)

Manche machen es so.

Code:

case x:
{
  // Anweisung
}
break;

andere machen es so

Code:

case x:
{
  // Anweisung
  break;
}

dann wieder so.

Code:

case x:
  // Anweisung
  break;

da blickt keiner mehr durch.

gruss

[zeras]

Zitat:

Zitat von EWeiss (Beitrag 1379562)

andere machen es so

Code:

case x:
{
  // Anweisung
  break;
}

Ich meine, dass es so sein sollte.

[EWeiss]

Zitat:

Zitat von zeras (Beitrag 1379564)

Zitat:

Zitat von EWeiss (Beitrag 1379562)

andere machen es so

Code:

case x:
{
  // Anweisung
  break;
}

Ich meine, dass es so sein sollte.

Mein Problem ist... Annahmen nutzen mir nichts denn es würde auch so gehen.

Code:

case x:
   // Anweisung
   break;

Warum einmal so dann wieder so.

Gibt es da keine ganz klare Definition an der man sich halten kann (muss) ?
Letztendlich wird man für blöd gehalten wenn man nicht mal in der Lage ist ein break; korrekt zu setzen.

gruss

[DeddyH]

Ohne Gewähr:

Code:

switch (Irgendwas){
  case Dings:
    MachWas();
    break;
  case Bums:
    MachWasAnderes();
    break;
  case Wuppdi:
    if (Bedingung){
      TollerCode();
    }
    break;
  default:
    WasWoandersNichtPasst();
}

BTW: https://msdn.microsoft.com/de-de/library/k0t5wee3.aspx

[EWeiss]

Genau hier.

Code:

   case 'a' :  
       {  
       // Declaration of szChEntered OK. Local scope.  
       char szChEntered[] = "Character entered was: ";  
       cout << szChEntered << "a\n";  
       }  
       break;  
  
   case 'b' :  
       // Value of szChEntered undefined.  
       cout << szChEntered << "b\n";  
       break;  
  
   default:  
       // Value of szChEntered undefined.  
       cout << szChEntered << "neither a nor b\n";  
       break;  
   }

einmal klammer nach Case einmal keine..
welcher sinn steckt dahinter.

Code:

  case Dings:
     MachWas();
     break;
  case Dings:
     MachWas();
     {
         MachWasViel?();
         Vielezeilen
         break; << break hier
     }
     break; << oder break hier

gruss

[DeddyH]

Ganz ehrlich: ich hab keine Ahnung, dafür reicht mein bescheidenes C/C++ nicht aus.

[EWeiss]

Zitat:

Zitat von DeddyH (Beitrag 1379571)

Ganz ehrlich: ich hab keine Ahnung, dafür reicht mein bescheidenes C/C++ nicht aus.

Deshalb stelle ich ja die Frage hier ;)
Man steht da wie ein blöder weil niemand einem sagen kann wie es nun korrekt ist.

Wie gesagt Annahmen ohne konkreten beleg nutzen mir nichts.

gruss

[DeddyH]

Ich persönlich schreibe im Zweifel den Code einfach so, wie ich denke, dass es stimmen müsste, und verfolge den Ablauf dann schrittweise im Debugger. Dann bin ich auch nicht von Annahmen anderer abhängig ;)

[EWeiss]

Zitat:

Zitat von DeddyH (Beitrag 1379575)

Ich persönlich schreibe im Zweifel den Code einfach so, wie ich denke, dass es stimmen müsste, und verfolge den Ablauf dann schrittweise im Debugger. Dann bin ich auch nicht von Annahmen anderer abhängig ;)

In Delphi kannst du so etwas auch nicht machen.

Delphi-Quellcode:

WM_COMMAND:
  bla;
  bla, bla;
  bla, bla, bla;

das hingegen schon.

Delphi-Quellcode:

WM_COMMAND:
  begin
    bla;
    bla, bla;
    bla, bla, bla;
  end;

Der eine macht es so der andere wieder SO..
Wenn man es dann nicht SO macht wie der andere es macht ist man der Dumme.
Es muss doch ganz klare Richtlinien geben was nun korrekt ist.

Mit so etwas wird man dann darauf hingewiesen..

Zitat:

Also your SWITCH CASE looks very strange to me because of the extra braces.

Nur warum es seltsam aussieht kann niemand erklären weil jeder macht was er will.

EDIT:
Nochmal

Code:

   case 'a' : 
        {  
        // Declaration of szChEntered OK. Local scope. 
        char szChEntered[] = "Character entered was: "; 
        cout << szChEntered << "a\n"; 
        }  
        break; 
   
    case 'b' : 
        // Value of szChEntered undefined. 
        cout << szChEntered << "b\n"; 
        break; 
   
    default: 
        // Value of szChEntered undefined. 
        cout << szChEntered << "neither a nor b\n"; 
        break; 
    }

bei case a..
nach Case a Klammer (warum? )
nach Case a abschließende klammer dann break.. so wie ich es mache aber angeblich nicht richtig. (warum? )
Case b keine Klammer (warum? )

was für ein mist!
Im Moment ohne sinn und zweck.. das ganze.

wieder andere machen es so. Berechtigte Frage! Was soll der Quatsch..

Code:

 case 'a' : 
        {  
        // Declaration of szChEntered OK. Local scope. 
        char szChEntered[] = "Character entered was: "; 
        cout << szChEntered << "a\n"; 
        break;
        }

Das einzige was mir einleuchtet ist das wenn ich eine Klammer setze das diese auch wieder geschlossen werden muss.
Also was ist in meiner WinProc bzg. Switch, break, case falsch? Bitte keine Annahmen sondern begründet!

gruss

[himitsu]

Im Grunde gehört es wohl eher in einen der CASE/SWITCH-Blöcke, bzw. an deren Ende.
Aber da in C++ ein CASE/SWITCH nur eine Liste von "Einspungspunkten" ist, würde demnach auch alles verarbeitet, was zwischen diesen CASE-Blöcken liegt, wenn man vorher nicht raus springt.
Ich hätte nur erwartet, dass die Syntax von C++ eine Deklaration zwischen den Blöcken verbietet. :stupid:

Delphi hat dagegen ein implizites BRAKE an jedem CASE-Block und man springt automatisch wieder raus, nach Abarbeitung des einen angspungenen Blocks.


Quatsch:
Damit der Highlighter dir das ENDE des Blocks verrät.
Stell dir SWITCH wie ganz viele GOTO vor. Da brachst du auch kein BEGIN/END dazischen.

[EWeiss]

Danke..
Aber das beantwortet nicht meine Frage bzg. das letztendlich jeder machen kann was er will.
Und wenn man es tut einem angebliche Fehler vorgeworfen werden.

Was ist denn nun bei meinem Case, Break bei FullScreen falsch.
Wenn ich es doch am ende genauso mache wie bei "Case a"

Code:

        case ID_FULLSCREEN:
         {
             gP.bFullScreen = !gP.bFullScreen;
             SetForegroundWindow(hWnd);
             if (gP.bFullScreen)
             {
                 RECT rect;
                 rect.right = GetSystemMetrics(SM_CXSCREEN);
                 rect.bottom = GetSystemMetrics(SM_CYSCREEN);
                 SetWindowLongPtr(hWnd, GWL_STYLE, WS_POPUP | WS_VISIBLE);
                 AdjustWindowRect(&rect, WS_POPUP, FALSE);
                 MoveWindow(hWnd, 0, 0, rect.right, rect.bottom, FALSE);
             }
             else
             {
                 RECT rect;
                 rect.right = dpi(XMIN_SIZE);
                 rect.bottom = dpi(YMIN_SIZE);
                 rect.left = (GetSystemMetrics(SM_CXSCREEN) - rect.right) / 2;
                 rect.top = (GetSystemMetrics(SM_CYSCREEN) - rect.bottom) / 2;
                 SetWindowLongPtr(hWnd, GWL_STYLE, WS_OVERLAPPEDWINDOW | WS_VISIBLE);
                 AdjustWindowRect(&rect, WS_OVERLAPPEDWINDOW, FALSE);
                 MoveWindow(hWnd, rect.left, rect.top, dpi(XMIN_SIZE), dpi(YMIN_SIZE), FALSE);
             }
         }
         break;

Da blickt niemand wirklich durch.

gruss

[Motzi]

Ob das break innerhalb der Klammern steht oder nicht macht keinen Unterschied. Die Klammern bilden einen Scope, der die Gültigkeit der darin definierten Variablen beschränkt, genauso wie zb die Klammern bei deinem if-else, wo du wieder zwei Scopes hast in denen jeweils eine "rect" Variable definiert ist.

Warum das break nach ID_FULLSCREEN falsch sein soll kann ich dir auch nicht sagen, aber es gibt durchaus Fälle wo man das break absichtlich weglässt um den Code des nächsten case-Zweiges ebenfalls auszuführen. In deinem Fall wäre das dann eben der case-Zweig von ID_CAPTURE - ob das allerdings Sinn macht kann ich dir so auch nicht sagen...

[Glados]

http://en.cppreference.com/w/cpp/language/switch

Das break muss über die letzte schließende Klammer, nicht darunter.
Das wurde aber auch schon erwähnt.

[nahpets]

Ohne jetzt was verbindliches zu sagen:

Ich stelle mit { und } immer als Ersatz für begin und end vor.

Bei einigen Sachen in Delphi benötige ich begin und end zwingend, kann sie aber auch zur Strukturierung an Stellen nutzen, an denen sie nicht zwingend erforderlich sind.

Analog ist es auch mit den {}.

Delphi-Quellcode:

case i of
  1 : Machwas;  // Begin und end nicht erforderlich.
  2 : begin // Begin und end nicht erforderlich, schadet aber nicht.
        machwas;
      end;
  3 : begin // Begin und end erforderlich.
        Machwas1;
        Machwas2;
      end;
  else
    AltenativeMachen;
    AltenativeMachen;
    AltenativeMachen;
    AltenativeMachen;
    AltenativeMachen;
  end; 
end;

Bei der Else ginge es auch so:

Delphi-Quellcode:

  else
    begin
      AltenativeMachen;
      AltenativeMachen;
      AltenativeMachen;
      AltenativeMachen;
      AltenativeMachen;
    end;
  end;

Kann man so auf C++ übersetzen, nur dass man das break nicht vergessen darf.

Oder irre ich mich da jetzt?

Und ein Fehler ist nur das, was zu einem anderen Programmablauf führt, als beabsichtigt.

Blocks zur besseren Strukturierung einzufügen, egal ob in Delphi mit begin und end oder in C++ durch { und } ist kein Fehler, allenfalls eine Geschmacksache oder ein Programmierstil, aber kein Fehler.

[EWeiss]

Zitat:

Zitat von Glados (Beitrag 1379588)

http://en.cppreference.com/w/cpp/language/switch

Das break muss über die letzte schließende Klammer, nicht darunter.
Das wurde aber auch schon erwähnt.

Sorry dann sage das MS denn in deren Beispiel ist es nicht So.
Und das wurde auch schon erwähnt.
Siehe case a eines der letzten Beispiele.

Code:

   case 'a' : 
         {  
         // Declaration of szChEntered OK. Local scope. 
         char szChEntered[] = "Character entered was: "; 
         cout << szChEntered << "a\n"; 
         }  
         break;

Ich kann dir auch Zig Links nennen wo es jedes mal unterschiedlich ist.
Dann müsste nach deiner Annahme das Beispiel in der MSDN falsch sein.

Zitat:

Ich stelle mit { und } immer als Ersatz für begin und end vor.

Habe ich auch bin davon aber sehr schnell von abgekommen weil es anscheinend keine vom Compiler vorgeschriebene Definition für Case Anweisungen gibt
mal abgesehen davon das Klammern geschlossen werden müssen.
Nun wer definiert nun was richtig und was falsch ist.

MSDN, diverse C++ Tutorials die Wiki oder was auch immer.

Zitat:

Fälle wo man das break absichtlich weglässt um den Code des nächsten case-Zweiges ebenfalls auszuführen

Das ist korrekt :)


gruss

[himitsu]

Zitat:

Zitat von EWeiss (Beitrag 1379586)

Danke..
Aber das beantwortet nicht meine Frage bzg. das letztendlich jeder machen kann was er will.

Zitat:

Zitat von himitsu (Beitrag 1379583)

Quatsch:
Damit der Highlighter dir das ENDE des Blocks verrät.
Stell dir SWITCH wie ganz viele GOTO vor. Da brachst du auch kein BEGIN/END dazischen.

Es ist wie mit roten Ampeln.
Jeder macht was er will.

[EWeiss]

Zitat:

Zitat von himitsu (Beitrag 1379596)

Zitat:

Zitat von EWeiss (Beitrag 1379586)

Danke..
Aber das beantwortet nicht meine Frage bzg. das letztendlich jeder machen kann was er will.

Zitat:

Zitat von himitsu (Beitrag 1379583)

Quatsch:
Damit der Highlighter dir das ENDE des Blocks verrät.
Stell dir SWITCH wie ganz viele GOTO vor. Da brachst du auch kein BEGIN/END dazischen.

Es ist wie mit roten Ampeln.
Jeder macht was er will.

Bitte was möchtest du damit jetzt sagen ?
Ernsthaft hilft mir das? Oder jemand anderem der vor dem gleichen Dilemma steht ?

gruss

[mensch72]

Code:

   case 'a' : 
       {  
       // Declaration of szChEntered OK. Local scope. 
       char szChEntered[] = "Character entered was: "; 


       cout << szChEntered << "a\n"; 
       }  
       break; 
  
   case 'b' : 
       // Value of szChEntered undefined. 
       cout << szChEntered << "b\n"; 
       break; 
  
   default: 
       // Value of szChEntered undefined. 
       cout << szChEntered << "neither a nor b\n"; 
       break; 
   }

"einmal klammer nach Case einmal keine..
welcher sinn steckt dahinter."...


=> Der Sinn von Klammern ist also, das es innerhalb derer möglich ist, neue lokal gültige Variablen zu definieren
=> wenn keine Variablen definiert werden, braucht es beim "Case" oder "Default" keine geschweiften Klammern
=> das "Break" ist kein "End" im Sinne einer schließenden Klammer... man kann auch mehrere Breaks innerhalb eines "Case" oder "Default" verwenden
-> beim "Break" vor oder nach der letzten schließenden geschweiften Klammer gibt es nur einen Unterschied. Hält man per Debugger da an, sind innerhalb der Klammern die lokalen Variablen noch gültig und prüfbar, nach der schließenden Klammer sind die davor innerhalb der Klammern angelegten Variablen ungültig und nicht mehr zugreifbar...
- dem "switch" ist die Position eines beliebigen "breaks" innerhalb der "switch"-Klammern egal, das "break" ist nix weiter wie ein "goto EndOfSwitch;" :)


Die Syntax ist durchaus eindeutig...
- im "case" MUSS man bei Verwendung von lokalen Variablen Klammern verwenden
- für den "switch" ist die Position der "breaks" völlig egal... das erste was zur Ausführung kommt macht ein "GotoEndOfSwich" (ohne "break" geht es also auch, denn so schreibt man in "C" das was man in Delphi mit Komma getrennen Werten macht:

Code:

switch(x)
{
  case 0:
       x=-1;
       break;

  case 1:
  case 2:
  case 3:
       y=x*3;
       break;

  case 4:
  case 5:
  case 6:
       y=x*5;
       break;

  default:
       y=-1;
       break;
}

[Glados]

Irgendwie widersprichst du dir hier selber:

Zitat:

Was ist denn nun bei meinem Case, Break bei FullScreen falsch.

Zitat:

Sorry dann sage das MS denn in deren Beispiel ist es nicht So.
Und das wurde auch schon erwähnt.
Siehe case a eines der letzten Beispiele.
--
Ich kann dir auch Zig Links nennen wo es jedes mal unterschiedlich ist.

Du hast dir deine Antwort also schon selber gegeben.
=> scheinbar ist nichts falsch.

[EWeiss]

Das ist eine gute Erklärung bin jetzt schon mal etwas weiter. Danke! ;)

Zitat:

- dem "switch" ist die Position eines beliebigen "breaks" innerhalb der "switch"-Klammern egal, das "break" ist nix weiter wie ein "goto EndOfSwitch;"

Was ich hier aber festgestellt habe..
Das beim Debuggen wenn ich den Breakpoint auf break setze (FullScreen) er innerhalb des Switch auf das nächste break springt.
Ist das korrekt?

Denn das widerspräche ja deiner Aussage.
Durch den break sollte ich doch eigentlich aus der Winproc rausspringen ohne Rückgabe von DefWindowProc.

gruss

[EWeiss]

Zitat:

=> scheinbar ist nichts falsch.

Du hast aber meinen Einwand bzw.. die Beschwerde eines anderen gelesen ? ;)

Zitat:

Also your SWITCH CASE looks very strange to me because of the extra braces.

Ich möchte nicht gerne als ein Dummerschen hingestellt werden nur weil ich angeblich(<>) ein Break falsch setze.
Deshalb ja der Thread hier!

Also wäre auch das korrekt. ?

Code:

        case ID_FULLSCREEN:
             gP.bFullScreen = !gP.bFullScreen;
             SetForegroundWindow(hWnd);
             if (gP.bFullScreen)
             { // << hier Klammer weil eine globale Variable definiert wird in dem Fall RECT
                 RECT rect;
                 rect.right = GetSystemMetrics(SM_CXSCREEN);
                 rect.bottom = GetSystemMetrics(SM_CYSCREEN);
                 SetWindowLongPtr(hWnd, GWL_STYLE, WS_POPUP | WS_VISIBLE);
                 AdjustWindowRect(&rect, WS_POPUP, FALSE);
                 MoveWindow(hWnd, 0, 0, rect.right, rect.bottom, FALSE);
             }
             else
             { // << das selbe hier
                 RECT rect;
                 rect.right = dpi(XMIN_SIZE);
                 rect.bottom = dpi(YMIN_SIZE);
                 rect.left = (GetSystemMetrics(SM_CXSCREEN) - rect.right) / 2;
                 rect.top = (GetSystemMetrics(SM_CYSCREEN) - rect.bottom) / 2;
                 SetWindowLongPtr(hWnd, GWL_STYLE, WS_OVERLAPPEDWINDOW | WS_VISIBLE);
                 AdjustWindowRect(&rect, WS_OVERLAPPEDWINDOW, FALSE);
                 MoveWindow(hWnd, rect.left, rect.top, dpi(XMIN_SIZE), dpi(YMIN_SIZE), FALSE);
             }
         break;

oder?

Code:

        case ID_FULLSCREEN:
        {
            RECT rect;
            gP.bFullScreen = !gP.bFullScreen;
            SetForegroundWindow(hWnd);
            if (gP.bFullScreen)
            {
                rect.right = GetSystemMetrics(SM_CXSCREEN);
                rect.bottom = GetSystemMetrics(SM_CYSCREEN);
                SetWindowLongPtr(hWnd, GWL_STYLE, WS_POPUP | WS_VISIBLE);
                AdjustWindowRect(&rect, WS_POPUP, FALSE);
                MoveWindow(hWnd, 0, 0, rect.right, rect.bottom, FALSE);
            }
            else
            {
                rect.right = dpi(XMIN_SIZE);
                rect.bottom = dpi(YMIN_SIZE);
                rect.left = (GetSystemMetrics(SM_CXSCREEN) - rect.right) / 2;
                rect.top = (GetSystemMetrics(SM_CYSCREEN) - rect.bottom) / 2;
                SetWindowLongPtr(hWnd, GWL_STYLE, WS_OVERLAPPEDWINDOW | WS_VISIBLE);
                AdjustWindowRect(&rect, WS_OVERLAPPEDWINDOW, FALSE);
                MoveWindow(hWnd, rect.left, rect.top, dpi(XMIN_SIZE), dpi(YMIN_SIZE), FALSE);
            }
        }
        break;

gruss

[mensch72]

Ein Grundsatzhinweis bezgl. der Frage was stimmt beim break vom FullScreen nicht...

- bitte rücke IMMER nach der öffnenden Klammer hinter dem "switch" alle "case" und das "default" um eine Standardbreite ein!
- so wie aktuell geschrieben zerstörst du deine sonst ja sehr einheitliche optische Struktur, so dass es mir gerade "optisch" ohne Editor nicht möglich ist, "einfach" den Punkt zu finden, wo du dich in der Klammerebene vertan hat, dass es dann (wohl etwas später) zum Problem bei "break vom FullScreen" kommt
- das ist nur ein netter Hinweis der dir und anderen die Arbeit beim lesen deines Source wesentlich erleichtert(du bist deshalb kein Dummerchen)

[EWeiss]

Zitat:

Ein Grundsatzhinweis bezgl. der Frage was stimmt beim break vom FullScreen nicht...

Sollte das nicht der Formatierer tun?

Manchmal rückt er das break ein aber manchmal auch nicht.
Hier würde er ihn einrücken.

Code:

   case 'b' : 
        // Value of szChEntered undefined. 
        cout << szChEntered << "b\n"; 
        break;

hier aber nicht.

Code:

   case 'b' :
   { 
        // Value of szChEntered undefined. 
        cout << szChEntered << "b\n"
   } 
   break;

EDIT:
Habe es mal getestet..

Zitat:

=> Der Sinn von Klammern ist also, das es innerhalb derer möglich ist, neue lokal gültige Variablen zu definieren
=> wenn keine Variablen definiert werden, braucht es beim "Case" oder "Default" keine geschweiften Klammern

In beiden fällen kompiliert es Fehler frei.

Code:

        case ID_FULLSCREEN:
       // {
            RECT rect;
            gP.bFullScreen = !gP.bFullScreen;
            SetForegroundWindow(hWnd);
            if (gP.bFullScreen)
            {
                rect.right = GetSystemMetrics(SM_CXSCREEN);
                rect.bottom = GetSystemMetrics(SM_CYSCREEN);
                SetWindowLongPtr(hWnd, GWL_STYLE, WS_POPUP | WS_VISIBLE);
                AdjustWindowRect(&rect, WS_POPUP, FALSE);
                MoveWindow(hWnd, 0, 0, rect.right, rect.bottom, FALSE);
            }
            else
            {
                rect.right = dpi(XMIN_SIZE);
                rect.bottom = dpi(YMIN_SIZE);
                rect.left = (GetSystemMetrics(SM_CXSCREEN) - rect.right) / 2;
                rect.top = (GetSystemMetrics(SM_CYSCREEN) - rect.bottom) / 2;
                SetWindowLongPtr(hWnd, GWL_STYLE, WS_OVERLAPPEDWINDOW | WS_VISIBLE);
                AdjustWindowRect(&rect, WS_OVERLAPPEDWINDOW, FALSE);
                MoveWindow(hWnd, rect.left, rect.top, dpi(XMIN_SIZE), dpi(YMIN_SIZE), FALSE);
            }
            break;
       // }

Code:

        case ID_FULLSCREEN:
        {
            RECT rect;
            gP.bFullScreen = !gP.bFullScreen;
            SetForegroundWindow(hWnd);
            if (gP.bFullScreen)
            {
                rect.right = GetSystemMetrics(SM_CXSCREEN);
                rect.bottom = GetSystemMetrics(SM_CYSCREEN);
                SetWindowLongPtr(hWnd, GWL_STYLE, WS_POPUP | WS_VISIBLE);
                AdjustWindowRect(&rect, WS_POPUP, FALSE);
                MoveWindow(hWnd, 0, 0, rect.right, rect.bottom, FALSE);
            }
            else
            {
                rect.right = dpi(XMIN_SIZE);
                rect.bottom = dpi(YMIN_SIZE);
                rect.left = (GetSystemMetrics(SM_CXSCREEN) - rect.right) / 2;
                rect.top = (GetSystemMetrics(SM_CYSCREEN) - rect.bottom) / 2;
                SetWindowLongPtr(hWnd, GWL_STYLE, WS_OVERLAPPEDWINDOW | WS_VISIBLE);
                AdjustWindowRect(&rect, WS_OVERLAPPEDWINDOW, FALSE);
                MoveWindow(hWnd, rect.left, rect.top, dpi(XMIN_SIZE), dpi(YMIN_SIZE), FALSE);
            }
        }
        break;

Seltsam.. Es scheint wohl keine Richtlinien zu geben.

gruss

[mensch72]

Die Regel, das es für "lokale Variablen" Klammern braucht gilt für die Komaptibilität mit der C-Syntax, in C++ kann man überall innherhalb eines Klammer-Blocks neue Variablen anlegen.
Wenn also nach C++ Syntax eine Variable nach einem "case" ohne Klammern angelegt wird, wird die einfach dem "swich" Klammerblock als Scope zugeordnet. (ich bemühe mich in so OldScool-Quellen wie deiner da da auch so OldScool und kompatibel wie möglich im C-Syle zu schreiben. (So wie du da jetzt habe ich früüüüher ganze große Programme unter Win31 und Win95 16Bit geschrieben, seit dem weiß ich zwar noch wie es geht, mache es aber "ohne Zang" nicht mehr:) )

ähm... was für ein Formatierer ?
Nur zur Sicherheit die Rückfrage: meckert der Compiler das es sich nicht übersetzen lässt, oder meckert nur der "Editor"/"Formatierer"? das es struckturell nicht passt?

So sieht dein Code nach etwas "Formatierung" bei mir im RadStudio mit aktiviertem CN-Pack aus :)

Code:

LRESULT CALLBACK WndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
{
    WPARAM wpItem;
    LRESULT IntI = 0;

    switch (message)
    {
        case WM_GETMINMAXINFO:
        {
            MINMAXINFO* pInfo = (MINMAXINFO*)lParam;
            pInfo->ptMinTrackSize.x = XMIN_SIZE;
            pInfo->ptMinTrackSize.y = YMIN_SIZE;
        }
        break;

        case WM_COMMAND:
        {
            if (HIWORD(wParam) == CBN_SELCHANGE)
            {
                switch LOWORD(wParam)
                {
                    case ID_CBVRENDERER:
                    {
                        WCHAR * Buffer[5];

                        IntI = SendMessage(HW(ID_CBVRENDERER), CB_GETCURSEL, IntI, 0);
                        if (IntI != CB_ERR)
                            SendMessage(HW(ID_CBVRENDERER), CB_GETLBTEXT, IntI, (WPARAM)(&Buffer));

                        KVideo_SetVideoRenderer((TVRenderer)IntI);

                        if (KVideo_GetPlayerState() != psNotReady)
                            Confirm(L"Designated video renderer will be applied since next opening media file.");
                    }
                    break;

                    case ID_CBPLAYBACKRATE:
                    {
                        WCHAR * Buffer[5];

                        IntI = SendMessage(HW(ID_CBPLAYBACKRATE), CB_GETCURSEL, IntI, 0);
                        if (IntI != CB_ERR)
                            SendMessage(HW(ID_CBPLAYBACKRATE), CB_GETLBTEXT, IntI, (WPARAM)(&Buffer));

                        WCHAR * fIn = (WCHAR*)(&Buffer);
                        float fOut = (float)wcstod(fIn, NULL);

                        KVideo_SetPlaybackRate(fOut);
                    }
                    break;
                }
            }

            wpItem = LOWORD(wParam);
            switch (wpItem)
            {
                case ID_OPEN:
                {
                    if (KVideo_GetPlayerState() != psNotReady)
                    {
                        KillTimer(gP.MovieHandle, MOVIE_TIMER);
                        KVideo_CloseFile();
                        // Einstellungen zurück setzen
                        gP.MenuAudioChecked = 0;
                        gP.MenuVideoChecked = 0;
                        SetWindowText(HW(LB_STREAMDURATION), L"00:00:00\\");
                        SetWindowText(HW(LB_STREAMELAPSED), L"00:00:00");
                        SetWindowText(HW(LB_VIDEOINFO), L"0 x 0, 0 FPS");
                        SetWindowText(HW(LB_FILENAME), L"_");
                        SetWindowText(HW(LB_AUDIOPROPERTY), L"0 Channel, 0 SPS, 0 BPS");
                        SetScrollPos(gP.MovieHandle, SBS_HORZ, 0, TRUE);
                        InvalidateRect(hWnd, NULL, FALSE);
                        UpdateWindow(hWnd);
                    }

                    // OpenFileDialog Initialisieren
                    CoInitialize(NULL);

                    const WCHAR *lpstrFilter = L"Media Files\0*.aac;*.asf;*.avi;*.m4a;*.mp3;*.mp4;*.m4v;*.wav;*.wma;*.wmv;*.3gp;*.3g2;*.mpeg;*.mpg;*.mov;*.qt;*.mkv;*.flv;*.vob\0"
                        L"All files\0*.*\0";

                    HANDLE hf;
                    OPENFILENAME OpenMediaFile;
                    ClearMemory(&OpenMediaFile, sizeof(OpenMediaFile));

                    WCHAR szFile[MAX_PATH];
                    szFile[0] = L'\0';

                    OpenMediaFile.lStructSize = sizeof(OpenMediaFile);
                    OpenMediaFile.hwndOwner = NULL;
                    OpenMediaFile.lpstrFile = szFile;
                    OpenMediaFile.nMaxFile = sizeof(szFile);
                    OpenMediaFile.lpstrFilter = lpstrFilter;
                    OpenMediaFile.nFilterIndex = 1;
                    OpenMediaFile.lpstrFileTitle = NULL;
                    OpenMediaFile.nMaxFileTitle = 0;
                    OpenMediaFile.lpstrInitialDir = NULL;
                    OpenMediaFile.Flags = OFN_PATHMUSTEXIST | OFN_FILEMUSTEXIST;

                    // Video starten
                    if (GetOpenFileName(&OpenMediaFile))
                    {
                        hf = CreateFile(OpenMediaFile.lpstrFile, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, 0);

                        if (hf > 0)
                        {
                            CloseHandle(hf);
                            LRESULT IntI = SendMessage(HW(ID_CBVRENDERER), CB_GETCURSEL, 0, 0);
                            KVideo_SetVideoRenderer((TVRenderer)IntI);

                            // Mediafile starten
                            RunMediaFile(szFile, FALSE);
                        }
                    }
                    CoUninitialize();
                }
                break;

                case ID_PLAY:
                {
                    if (KVideo_GetPlayerState() == psPlaying)
                        return 0;
                    if (KVideo_GetPlayerState() == psPaused || KVideo_GetPlayerState() == psStopped)
                        if (KVideo_Play())
                            SetTimer(gP.MovieHandle, MOVIE_TIMER, 200, NULL);
                }
                break;

                case ID_PAUSE:
                {
                    if (KVideo_GetPlayerState() == psPlaying)
                        if (KVideo_Pause())
                            KillTimer(gP.MovieHandle, MOVIE_TIMER);
                }
                break;

                case ID_STOP:
                {
                    if (KVideo_GetPlayerState() == psPlaying || KVideo_GetPlayerState() == psPaused)
                        if (KVideo_Stop())
                            KillTimer(gP.MovieHandle, MOVIE_TIMER);

                    SetWindowText(HW(LB_STREAMDURATION), L"00:00:00\\");
                    SetWindowText(HW(LB_STREAMELAPSED), L"00:00:00");
                    SetScrollPos(gP.MovieHandle, SBS_HORZ, 0, TRUE);
                    InvalidateRect(hWnd, NULL, FALSE);
                    UpdateWindow(hWnd);
                }
                break;

                case ID_FULLSCREEN:
                {
                    gP.bFullScreen = !gP.bFullScreen;
                    SetForegroundWindow(hWnd);
                    if (gP.bFullScreen)
                    {
                        RECT rect;
                        rect.right = GetSystemMetrics(SM_CXSCREEN);
                        rect.bottom = GetSystemMetrics(SM_CYSCREEN);
                        SetWindowLongPtr(hWnd, GWL_STYLE, WS_POPUP | WS_VISIBLE);
                        AdjustWindowRect(&rect, WS_POPUP, FALSE);
                        MoveWindow(hWnd, 0, 0, rect.right, rect.bottom, FALSE);
                    }
                    else
                    {
                        RECT rect;
                        rect.right = dpi(XMIN_SIZE);
                        rect.bottom = dpi(YMIN_SIZE);
                        rect.left = (GetSystemMetrics(SM_CXSCREEN) - rect.right) / 2;
                        rect.top = (GetSystemMetrics(SM_CYSCREEN) - rect.bottom) / 2;
                        SetWindowLongPtr(hWnd, GWL_STYLE, WS_OVERLAPPEDWINDOW | WS_VISIBLE);
                        AdjustWindowRect(&rect, WS_OVERLAPPEDWINDOW, FALSE);
                        MoveWindow(hWnd, rect.left, rect.top, dpi(XMIN_SIZE), dpi(YMIN_SIZE), FALSE);
                    }
                }
                break;

                case ID_CAPTURE:
                {
                    TPlayerState ps = KVideo_GetPlayerState();
                    if (ps == psStopped || ps == psNotReady)
                    {
                        Confirm(L"It's not playing state.");
                        return 0;
                    }
                    else
                    {
                        HBITMAP hbmp = 0;
                        if (KVideo_GetCapturedImage(&hbmp))
                        {
                            WCHAR Image[MAX_PATH];
                            ZeroMemory(Image, MAX_PATH * sizeof(WCHAR));

                            WCHAR * Path = { 0 };
                            Path = EXEpath();

                            _snwprintf_s(Image, MAX_PATH, 259, L"%s%d%s", Path, gP.PicCount, L"_Capture.jpg");

                            SaveBitmapToFile(Image, hbmp, 0);
                            gP.PicCount++;
                        }
                        DeleteObject(hbmp);
                    }
                }
                break;

                case ID_CLOSE:
                    DestroyWindow(hWnd);
                break;

                // Menüauswahl bearbeiten:
                case ID_CKB_SEARCH:
                {
                    if (KVideo_GetPlayerState() != psNotReady)
                    {
                        BOOL checked = IsDlgButtonChecked(hWnd, ID_CKB_SEARCH);
                        if (checked) {
                            CheckDlgButton(hWnd, ID_CKB_SEARCH, BST_UNCHECKED);
                        }
                        else
                        {
                            CheckDlgButton(hWnd, ID_CKB_SEARCH, BST_CHECKED);
                        }
                        KVideo_SetSearchMode(EnableWindow(HW(ID_CBPLAYBACKRATE), !checked));
                        break;
                    }
                }
                break;

                case IDM_ABOUT:
                    DialogBox(GetModuleHandle(0), MAKEINTRESOURCE(IDD_ABOUTBOX), hWnd, About);
                break;

                case IDM_EXIT:
                    DestroyWindow(hWnd);
                break;

                default:
                    return DefWindowProc(hWnd, message, wParam, lParam);
            }
        }
        break;

        case WM_HSCROLL:
        {
            switch LOWORD(wParam)
            {
                case TB_ENDTRACK:
                    gP.bVolume = FALSE;
                break;

                case TB_THUMBTRACK: // ziehen des "Sliders"
                case TB_TOP:        // Pos1
                case TB_BOTTOM:     // Ende
                case TB_LINEDOWN:   // Pfeiltasten oben/unten
                case TB_PAGEDOWN:   // Bild runter & in die Leiste geklickt
                case TB_PAGEUP:     // Bild auf & in die Leiste geklickt
                {
                    gP.bVolume = TRUE;
                    gP.Position = (int)SendMessage(HW(ID_TBVOLUME), TBM_GETPOS, 0, 0);
                    if (gP.Position == 0)
                        KVideo_SetVolume(-10000);
                    else
                        KVideo_SetVolume(LONG(5000 * log10(gP.Position)) - 10000);
                }
                break;
            }
        }
        break;

        case WM_SIZE:
        {
            EnumChildWindows(hWnd, &AnchorEnum, 0);
            RedrawWindow(hWnd, NULL, NULL, RDW_ERASE | RDW_INVALIDATE | RDW_ERASENOW | RDW_UPDATENOW | RDW_ALLCHILDREN | RDW_FRAME);
        }
        break;

        case WM_CTLCOLORSTATIC:
        {
            HDC hdcStatic = (HDC)wParam;
            SetBkMode(hdcStatic, TRANSPARENT);
            return (INT_PTR)GetStockObject(WHITE_BRUSH);
        }
        break;

        case WM_PAINT:
        {
            PAINTSTRUCT ps;
            HDC hdc = BeginPaint(hWnd, &ps);
            EndPaint(hWnd, &ps);
        }
        break;

        case WM_DESTROY:
            PostQuitMessage(0);
        break;

        default:
            return DefWindowProc(hWnd, message, wParam, lParam);
    }
    return 0;
}

[EWeiss]

Zitat:

bitte rücke IMMER nach der öffnenden Klammer hinter dem "switch" alle "case" und das "default" um eine Standardbreite ein!

Der Quelltext Formatierer?
Sieht in Delphi und VC anders aus bzw.. er Formatiert anders.

Delphi

Code:

case ID_CLOSE:
    DestroyWindow(hWnd);
break;

VSC

Code:

case ID_CLOSE:
    DestroyWindow(hWnd);
    break;

Tab ist auf 4 eingestellt.

gruss

[mensch72]

dein Code war/ist schlicht uneinheitlich, mal ist das "break" vom "case" eingerückt und mal nicht !?
Kein Formatierer wird das so machen, aber wenn du schon "if,for,switch" so klammerst und einrückst, dann ist dies die resultierende logische Einrückung für das "break" zum "case" BEI DIR.

=> Wenn du mit AutoFormat arbeiten möchtest, dann setze bei Klammerverwendung das "break" in die Klammern, nur dann ist die Struktur optisch und syntaktisch auch bzgl. Scope einheitlich.

Code:

switch(x)
{   
    case 1:
        y=x+1;
        break;

    case 2:
    {
        y=x+2;
        break;
    }

    default:
        y=-1;
}

[EWeiss]

Zitat:

dein Code war/ist schlicht uneinheitlich, mal ist das "break" vom "case" eingerückt und mal nicht !?

JO.. Und das wollte ich nur sagen weil du mich darauf hingewiesen hast das ich es tun sollte.
Nur ich kann den Formatierer nicht dazwischen funken wenn er es nicht macht.

Ich versuche immer den Code Formatiert zu halten. ;)

gruss

[mensch72]

Achtung Syntaxfehler:

"WCHAR * Buffer[5];" soll/muss zu 99,9% "WCHAR Buffer[5]" heißen!

Du willst ja sicher 4Zeichen + abschließende "NULL" darin speichern(LB_GETTEXT)... dazu brauchst du ein eigentlich "ArrayOfWChar" und kein "ArrayOfWCharPointers"... das es funktioniert liegt daran, 5 Pointer zufällig mehr Speicher reservieren wie du für 5Zeichen brauchst und du dann irgendwie auch noch passend einen "cast" machst.

Denke mal drüber nach und versuche zu verstehen, warum es obwohl falsch doch in diesem Fall so funktioniert... :)

[EWeiss]

Zitat:

Denke mal drüber nach und versuche zu verstehen, warum es obwohl falsch doch in diesem Fall so funktioniert...

Kann ich dir nicht sagen warum das trotzdem funktioniert.

aber da wir schon dabei sind.
Ist es überhaupt nötig ein Default: Case im switch (wpItem) bei WM_COMMAND zu verwenden?

sollte eigentlich am ende der WinProc reichen.

gruss

[mensch72]

ja das "default" im "wpItem" ist notwendig, weil das andere "default" von da aus nicht mehr erreicht werden kann.

Lass bei "*BUFF[5]" bitte trotzdem mal das Sternchen weg und ändere das auf "BUFF[5]".

Als allgemeiner guter Tip für C Anfänger bei über 95% der "normalen" Variablen Definition:
- (Char,...) Arrayvariablen, also TYPE x[size] werden fast immer ohne * definiert ! (also immer zuerst ohne versuchen und möglichst verstehen warum es (nicht) geht)
- mit Sternchen "TYPE* x[size]" oder als "PTYPE x[size]" was dann ein ArrayOfTypePointers entspricht arbeiten auch Pointerfanatiker in C/C++ nur wirklich selten und es kommt z.B. bei WINAPI kaum vor... (hier ist es im aktuellen Fall schwerer zu verstehen warum es obwohl falsch doch wegen genug Speicherreservierung doch geht)

[EWeiss]

Zitat:

ja das "default" im "wpItem" ist notwendig, weil das andere "default" von da aus nicht mehr erreicht werden kann.
Lass bei "*BUFF[5]" bitte trotzdem mal das Sternchen weg und ändere das auf "BUFF[5]".

Danke für deine Hilfe :)
Habe es gemacht und kompiliert ohne Probleme.

gruss

[mensch72]

keine Ursache:)

im Gegensatz zu dir beschäftige mich mit grafischer oder hier solcher Detailarbeit eigentlich im Alltag garnicht mehr. Ich bin da selbst schon echt zu faul dafür geworden.
Obwohl für mein erstes großes C-Style WinApi 16Bit-Programm was damit jetzt sogar 64Bit fähig würde... da würde ich am liebsten mal wirklich die alten 16Bit WinApi(v3.11) Quelltexte von 1994..1998 hervorsuchen und das "1:1 mal fix" von 16Bit hoch auf 64Bit portieren... einzig bin ich nicht mehr bei der damaligen Firma und nur für NiceToHave ist mir meine Zeit eben dann wohl doch zu schade:(

Aber ev. kann auch von dir was noch lernen... wenn du magst schicke mir ne PM(du hast die PM-Empfangsfunktion selbst ja deaktivert)...
"Meinen Quelltext der DLL SK_AERO gibt es NUR auf persönlicher Anfrage. Das Teil ist nicht geheim, aber habe einiges an Zeit dafür investiert"

Egal ob Delphi,C,API,SDK oder was auch immer, ich lese gerne in fremden Quelltexten und meckere wenn dann nur sachlich begründet einzelne mir aufgefallene Details "nett" an:)

[EWeiss]

Zitat:

"Meinen Quelltext der DLL SK_AERO gibt es NUR auf persönlicher Anfrage. Das Teil ist nicht geheim, aber habe einiges an Zeit dafür investiert"

JO den könnte ich dir schicken.
Aber dieser ist in Delphi geschrieben nicht in C++

Das C++ Project habe ich nur für die KVideoPlayer_x64.dll geschrieben.. weil diese MadVR und LAVFilter unterstützt.
Vielleicht braucht das ja mal jemand.

Habe dich zu meinen Kontakten addiert.
Schick mir ne Mail..

Habe jetzt noch den Vollbildmodus umgeschrieben.

Code:

        
case ID_FULLSCREEN:
    if (KVideo_GetPlayerState() != psNotReady)
        ToggleFullScreen();
    break;

Code:

void ToggleFullScreen()
{
    static DWORD screenWidth, screenHeight;
    static RECT rcSaved;
    long nTop = 0;
    

    gP.bFullScreen = !gP.bFullScreen;

    if (gP.bFullScreen)
    { 
        // Wechseln in den Vollbildmodus.
        GetWindowRect(gP.hMain, &rcSaved);

        rcSaved.right = GetSystemMetrics(SM_CXSCREEN);
        rcSaved.bottom = GetSystemMetrics(SM_CYSCREEN);
        SetWindowLongPtr(gP.hMain, GWL_STYLE, WS_POPUP);
        AdjustWindowRect(&rcSaved, WS_POPUP, FALSE);

        SetWindowPos(gP.hMain, HWND_TOPMOST, 0, 0, 0, 0, SWP_NOMOVE | SWP_NOSIZE | SWP_FRAMECHANGED | SWP_SHOWWINDOW);

        HMONITOR hMonitor = MonitorFromWindow(gP.hMain, MONITOR_DEFAULTTONEAREST);

        MONITORINFO tmi;
        ClearMemory(&tmi, sizeof(tmi));
        tmi.cbSize = sizeof(tmi);

        RECT rw; 
        GetWindowRect(gP.hMain, &rw);
        GetMonitorInfo(hMonitor, &tmi);

        screenWidth = abs(Width(tmi.rcMonitor));
        screenHeight = abs(Height(tmi.rcMonitor));

        long ctrlpanelHeigh = 97;
        nTop = GetSystemMetrics(SM_CYMENU);
        rw.left = tmi.rcMonitor.left;
        rw.top = tmi.rcMonitor.top - nTop;
        rw.right = screenWidth; 
        rw.bottom = screenHeight + nTop + ctrlpanelHeigh;
        MoveWindow(gP.hMain, rw.left, rw.top, rw.right, rw.bottom, TRUE);
        KVideo_Resize();
    }
    else
    { 
        // Zurück zum Fenstermodus.
        SetWindowLongPtr(gP.hMain, GWL_STYLE, WS_OVERLAPPEDWINDOW | WS_VISIBLE);
        MoveWindow(gP.hMain, rcSaved.left, rcSaved.top, dpi(XMIN_SIZE), dpi(YMIN_SIZE), FALSE);
        KVideo_Resize();
    }
}

gruss

[Rollo62]

Mensch72 hat das schon ausführlich und richtig erklärt.

Bin mir nicht sicher ob das erwähnt wurde (habe mit jetzt nicht Alles durchgelesen):

Zitat:

Soweit ich C kenne, wird jeder Case Zweig durchlaufen,

Delphi-Quellcode:

switch(x)
{   
    case 1:
        Code1;
        break;  // --> Springt raus

    case 2:
        Code2;
        break; // --> Springt raus

    case 3:
        Code3;  // --> Läuft weiter zur nächsten Anweiseung (4)  

    case 4:
        Code4;
        break; // --> Springt raus

    default:
        Code5;
}

// Bei 3 --> Code3 ... Code4
// Bei 4 --> Code4
//
// Das entspräche in etwa in Pascal (leider nicht ganz so):

  
case (expression) of
   1 : Code1;
   2 : Code2;
   3, 4: Code3_4;  // So verhält sich das in Pascal, leider nicht ganz so wie in C
   else
         Code5;
end; 

// Bei 3 --> Code3_4
// Bei 4 --> Code3_4
//
//--> Da muss ich in Code3_4 nochmal checken ob es jetzt aus 3 oder 4 kommt

Rollo

[EWeiss]

Na ja mich interessiert weniger wie es unter Delphi abläuft das ist ja bekannt.
Mein Unverständnis liegt/lag darin das es kein festgelegte Definition gibt wie ein Break gesetzt werden muss.

Ist halt nicht das gleiche wie

Delphi-Quellcode:

X:
begin
  result := 0;
  exit; // nicht gleich break;
end;

gruss

[Motzi]

Du kannst das break überall im Code eines case-Zweigs setzten - du kannst theoretisch auch mehrere breaks an unterschiedlichen Stellen haben. Ob dabei jetzt irgendwelche Klammern (Scopes) verwendet werden oder nicht macht fürs break keinen Unterschied!
Du kannst dir das so ähnlich vorstellen wie ein Exit in Delphi - das kannst du überall innerhalb einer Funktion haben, es spring immer ans Ende der Funktion (finally Blöcke ignorieren wir mal). Ein break in einem case spring immer ans Ende des switch. Wenn dir der Debugger dazwischen noch bei irgendwelchen anderen break Zeilen stehen bleibt kannst du das ignorieren - das hängt mit dem generierten Code zusammen, macht aber keinen semantischen Unterschied.

[EWeiss]

Zitat:

Wenn dir der Debugger dazwischen noch bei irgendwelchen anderen break Zeilen stehen bleibt kannst du das ignorieren

Ok ;)
Danke.

gruss