Frage zur zuweisung mit Fragezeichenoperator in c

Erklär lieber mal, was du mit diesem seltsamen Statement erreichen 
willst. Also mal vorausgesetzt, du hast nicht Samstag mit Freitag 
verwechselt.

?: bindet stärker als -=

  https://en.cppreference.com/w/c/language/operator_precedence

Da steht also effektiv

1

x = ((direction == 1 ? x+=2 : x) -= 2);

2

//   <————— kein L-Value ——————>

Aber wahrscheinlich möchtest du etwas ganz anders, nämlich

1

x = direction == 1 ? x+2 : x-2;

Goofy schrieb:
> x = (direction == 1) ? x+=2 : x-=2;

Wirkt kaputt.

Besser:

1

x = (direction == 1) ? x+2 : x-2;

Noch besser:

1

if(direction == 1)

2

  x = x + 2;

3

else

4

  x = x - 2;

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:

1

x += forward ? 2 : -2;

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.

@mh nochmal. 0 und -4 passen nicht

? jetzt kann ich auch nicht mehr Punkt- vor Strichrechnung

Oder man lässt den ganzen "direction"-Quatsch und definiert sich "speed" 
mit positiven und negativen Zahlen. Dann reicht "x += speed".