switch, break, case c++

[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