Tu peux utiliser pgrep à la place (surtout que tu utilise pkill juste derrière).
AMHA il y a une coquille tu devrais utiliser $PLAYER à la place de mpv.
for i in ${sta[@]};doecho$i;done
Il y a une astuce super sympa pour ça :
printf"%s\n""${sta[@]}";done
Pour le paramétrage comme tu utilise bash j'aurais plutôt utilisé un tableau associatif :
# Revue de code
Posté par barmic . En réponse au journal Écouter la radio. Évalué à 10.
Tu peux utiliser
pgrepà la place (surtout que tu utilise pkill juste derrière).AMHA il y a une coquille tu devrais utiliser
$PLAYERà la place de mpv.Il y a une astuce super sympa pour ça :
Pour le paramétrage comme tu utilise bash j'aurais plutôt utilisé un tableau associatif :
Ça simplifie pas mal ton code en remplaçant :
Par :
Tous les contenus que j'écris ici sont sous licence CC0 (j'abandonne autant que possible mes droits d'auteur sur mes écrits)