Yalu X. schrieb:
> Da direction nur die Werte 0 und 1 annehmen kann, also eine boolesche
> Größe ist, würde ich stattdessen eine Variable vom Typ bool namens
> forward oder rightward o.ä. verwenden und den Ausdruck so schreiben:
Ich würde den Namen beibehalten und einen enum mit den Werten "forward"
und "backward" oder wie immer sie heißen sollen verwenden. Am besten die
2 noch durch sowas wie "speed" ersetzen, so dass diese magische Zahl
auch noch verschwindet. Ich würd's wohl auch ausschreiben und den
?:-Operator hier gar nicht verwenden. Der ist ja eher dafür gedacht,
zwischen zwei Werten auszuwählen. Hier wird aber eher zwischen zwei
Oeratoren ausgewählt.
Würde also bei mir wohl in etwa so aussehen:
1 | if (direction == Forward)
|
2 | x += SPEED;
|
3 | else
|
4 | x -= SPEED;
|
Das finde ich am besten lesbar.