Frage zur zuweisung mit Fragezeichenoperator in c

[Marc (Gast)]

Hallo,
ich habe eine frage. Bei folgender zuweisungwird nicht mehr kompiliert

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

Bekomme die Meldung
error: lvalue required as left operand of assignment
             x = direction == 1 ? x+=2 : x-=2;
Mit einem kleinem Pfeil auf das -=.
schreibe ich anstelle von x-=2 einfach nur einen Wert, wird durchgebaut. 
Warum geht das nicht mit dem -= ? x+=2 ist ja anscheinend auch kein 
Problem.

[Goofz (Gast)]

Setz doch mal Klammern

[(prx) A. K. (prx)]

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

[Yalu X. (yalu) (Moderator)]

?: bindet stärker als -=

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

Da steht also effektiv

x = ((direction == 1 ? x+=2 : x) -= 2);
//   <————— kein L-Value ——————>

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

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

[Marc (Gast)]

A. K. schrieb:
> Erklär lieber mal, was du mit diesem seltsamen Statement erreichen
> willst.

Eigentlich ist es ganz einfach was ich vorhabe.
Wenn
direction==1 ist, soll x um 2 erhöht werden,
ist
direction==0, soll x um 2 verringert werden.

[Goofy (Gast)]

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

so?

[Goofy (Gast)]

Marc schrieb:
> Eigentlich ist es ganz einfach was ich vorhabe.

Dann tuste so:

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

[S. R. (svenska)]

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

Wirkt kaputt.

Besser:

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

Noch besser:

if(direction == 1)
  x = x + 2;
else
  x = x - 2;

[Kalr (Gast)]

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

[Yalu X. (yalu) (Moderator)]

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:

x += forward ? 2 : -2;

[Rolf M. (rmagnus)]

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:

if (direction == Forward)
    x += SPEED;
else
    x -= SPEED;

Das finde ich am besten lesbar.

[mh (Gast)]

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:
>
> x += forward ? 2 : -2;

Ich bin ja in Zeiten von Spectre und Konsorten dafür möglichst jeden 
Branch zu vermeiden. Also:

x += 2 * (2 * forward - 1);

;-)

[Dirk B. (dirkb2)]

@mh nochmal. 0 und -4 passen nicht

[mh (Gast)]

Dirk B. schrieb:
> @mh nochmal. 0 und -4 passen nicht

Welche 0 und welche -4?

[Dirk B. (dirkb2)]

? jetzt kann ich auch nicht mehr Punkt- vor Strichrechnung

[S. R. (svenska)]

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