3. for Schleife verkürzen

for Schleife verkürzen

  • ?

    lass dsn schwachsinn

    0
  • X

    Ok, aber die kürzung bis jetzt ist doch noch lesbar, oder etwa nicht?
    Selbst ich, als quasi-anfänger kann das lesen

    0
  • S

    aber du als quasi profi in 10 jahren wirst es nicht merh so ohne weiteres verstehen

    wahrscheinlich macht der compiler sowieso das selbe ergebnis draus...
    also lohnt sich das nicht

    wenn die gefahr der endlosschleife besteht dann mach so viele abfragen wei es braucht um sie auszuschliessen

    bringt nix wenn du 1 zeile code (statt 10) hast aber alle 3 oder 4 durchläufe kommt eine endlose zustande...

    0
  • N

    Dieses Kürzen hat nicht im Entferntesten mit Code-Optimierung zu tun.

    • Du sparst dir damit einige Zeilen. Gut, aber was bringt das?
    • In solchen Code schleichen sich Fehler viel schneller ein.
    • Man sieht auch nicht auf den ersten Blick, was getan wird.
    • Wie gesagt wirst du in Zukunft nur Mühe damit haben.
    • Du gewinnst dadurch nicht einmal Performance.
    • Im Übrigen gilt es nicht als "gut", "krass" oder "1337", wenn du alles in eine Zeilen packen kannst.

    Bilanz: Nichts gewonnen, aber die Übersichtlichkeit und Fehleranfälligkeit haben sich zum Schlechten verlagert.

    0
  • X

    @all:
    Ok, ich werde eure Hinweise beherzigen. Aber um hier nicht das letzte Wort zu haben: Beantwortet ist die Frage ja nun noch nicht wirklich 😉

    0
  • A

    XHansWurstX schrieb:

    Ich wollte folgende Schleife kürzen:
    ...
    geschafft habe ich es bis hier:
    ...
    Ok, ich werde eure Hinweise beherzigen. Aber um hier nicht das letzte Wort zu haben: Beantwortet ist die Frage ja nun noch nicht wirklich...

    Dir ist hoffentlich bewusst, das die zweite Variante mit der ersten nichts mehr zu tun hat? In der ersteren beendest du die Schleife wenn entweder i>=NUM oder player[i].is_beg erfüllt ist. In letzterer beendest du die schleife nur noch bei der Bedingung player[i].is_beg.

    Wenn schon abkürzen dann ohne Logikänderung

    [edit: muss selbst meine Logik nochmal anpassen ;)]

    0
  • X

    Thx, genau sowas in der Art habe ich gesucht.

    asc schrieb:

    In letzterer beendest du die schleife nur noch bei der Bedingung player[i].is_beg.

    Jup, war mir bewusst. Hatte ich aber erwähnt...

    0
  • A

    XHansWurstX schrieb:

    Thx, genau sowas in der Art habe ich gesucht.

    Leider war ich auch schon zu müde zum schreiben 😉

    0
  • X

    asc schrieb:

    [edit: muss selbst meine Logik nochmal anpassen ;)]

    hehe, netter Versuch. Musste 2x hingucken 😉

    0
  • Administrator

    XHansWurstX schrieb:

    Wie kriege ich die Return-Anweisung in die Schleife unter,

    Wenn man diese Anweisung befolgt, dann so:

    for(int i=0;;++i){if(i>=NUM){return 0}else if(player[i].is_beg){return i}}

    Aber davon ist wirklich dringend abzuraten 😮

    Grüssli

    0
  • X

    Jup...

    dann lieber gleich alles so lassen...

    0
Anmelden zum Antworten