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

geschafft habe ich es bis hier:

int i=0;
for(i=0; !player[i].is_beg; i<NUM ? i++ : i=0 ); 

return i;

Wie kriege ich die Return-Anweisung in die Schleife unter, sodass ich mir das erste int i=0; sparen kann? Das wäre mir auch deshalb lieber, da die Gefahr einer Endlosschleife besteht, sollte player[i].is_beg immer falsch sein...

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

• 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.

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 ;)]

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

Thx, genau sowas in der Art habe ich gesucht.

Leider war ich auch schon zu müde zum schreiben

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

hehe, netter Versuch. Musste 2x hingucken

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

dann lieber gleich alles so lassen...