(erledigt) bash case quoting, doppelt quoten

[ren22]

Hallo,

warum bekomme ich "xdotool: Unknown command: 1" ausgeworfen, wenn ich den code aus dem erste "case" nehme?

Code: Alles auswählen

case "$1" in
bla)
xdotool mousemove --sync "$gm1_xy" click 1
;;
esac

und warum muss man doppelt quoten das kein Fehler ausgegeben wird?

Code: Alles auswählen

case "$1" in
bla)
xdotool mousemove --sync ""$gm1_xy"" click 1
;;
esac

[tobo]

Du quotest nicht zweimal, sondern gar nicht (Quote auf/zu Variable quote auf/zu). Also einfach alle weglassen. Verlangt werden wohl 2 Parameter, x und y. Wenn Du quotest übergibst du nur einen.

[ren22]

achso, ja na klar, richtig, Denkfehler, dachte es genügt, wenn in der Variable

Code: Alles auswählen

gm1_xy="100 100"

steht ...

dann werde ich das mal zerglegen in variablen ... ala
xdotool mousemove --sync "$1gm" "$x" "$y" click 1

Danke

[Meillo]

Bitte noch den Thread auf erledigt setzen.

[tobo]

dann werde ich das mal zerglegen in variablen ... ala
xdotool mousemove --sync "$1gm" "$x" "$y" click 1

Kannste natürlich machen. Ich dachte jetzt aber eher an Variante 2, nur vor und hinter der Variablen alle " einfach weglassen. Und die Zuweisung gequotet belassen.

[ren22]

du meinst so? : @tobo

Code: Alles auswählen

xdotool mousemove --sync "$gm_x" "$gm_y" click 1 

[tobo]

Ne, so:

Code: Alles auswählen

gm1_xy="100 100"
...
xdotool mousemove --sync $gm1_xy click 1

Wobei, wenn das ungequotet da rumsteht, fragt man sich später irgendwann, wieso das nicht gequotet ist!? Ich nehme alles zurück: 2 Variablen und beide (unnötig) quoten ist besser/lesbarer.

Code: Alles auswählen

x=100
y=100
...
xdotool mousemove --sync "$x" "$y" click 1

[owl102]

Wobei, wenn das ungequotet da rumsteht, fragt man sich später irgendwann, wieso das nicht gequotet ist!?

Dann schreibt man eben

Code: Alles auswählen

# shellcheck disable=SC2086

darüber, dann weiß man, daß das so gewollt ist.

[breakthewall]

Alternativ kannst auch ein Array verwenden, was hinsichtlich mehrerer (auch positionaler) Parameter recht praktisch wäre. Dann kannst ebenfalls deine Doublequotes setzen, und umgehst somit das Problem einer regulären Variable in diesem Anwendungsfall.

Code: Alles auswählen

gm1_xy=(100 100)
xdotool mousemove --sync "${gm1_xy[@]}" click 1