switch, break, case c++

[DeddyH]

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

[EWeiss]

Zitat von DeddyH:

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 von 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

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.

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.

[EWeiss]

Zitat von 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.

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

[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;
}