Discussion :

[YuGiOhJCJ]

Bonjour,

J'ai écrit une fonction[1] bash qui :

• affiche le nom de la commande à lancer
• lance la commande
• affiche un message pour dire si la commande a réussi ou échoué

Elle s'utilise comme ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
COMMAND="sleep 1"
execcmd
COMMAND="echo toto  tata   tutu"
execcmd
COMMAND="sudo iwconfig ra0 essid \"Wi-Fi De Chez Moi\""
execcmd

Le problème c'est que quand la commande est complexe, il arrive parfois que la commande ne soit pas correctement exécutée.

Dans mon exemple ci-dessus, je commence par appeler sleep (pour faire une pause de 1 seconde), puis j'appelle echo (pour afficher du texte) et enfin j'appelle iwconfig pour changer l'ESSID de ma carte Wi-Fi.

Les deux premières commandes passent bien car elles sont simples, mais la 3ème pose problème.

Regardez par vous même le résultat dans la console :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
$ ./execcmd.sh 
executing the 'sleep 1' command... Done.
executing the 'echo toto  tata   tutu' command... toto tata tutu
Done.
executing the 'sudo iwconfig ra0 essid "Wi-Fi De Chez Moi"' command... iwconfig: unknown command "De"
Failure.

Donc ma fonction ne marche pas à tous les coups...
Est-ce que vous savez ce que je dois changer dans le code de ma fonction ?

Merci.

[1]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
#!/bin/bash
VERBOSE=1
QUIET=0
function execcmd()
{
echo -n "executing the '${COMMAND}' command... "
if test ${VERBOSE} -eq 0; then
	${COMMAND} 1>/dev/null 2>&1
	else
		${COMMAND}
fi
if test $? -ne 0 -a ${QUIET} -eq 0; then
	echo "Failure."
	exit 1
	else
		echo "Done."
fi
}

[N_BaH]

Bonjour,

}Les deux premières commandes passent bien car elles sont simples, mais la 3ème pose problème.

c'est pourquoi il est recommandé de na pas mettre des commandes complexes dans une variables, mais d'utiliser une fonction.

à part ça :

• function et () sont redondants.
• les variables doivent toujours être mises entre guillemets, a fortiori dans des tests.
• dans le deuxième if, $? va retourné le code d'erreur du premier if, et pas celui de la commande.

[YuGiOhJCJ]

Merci pour les petits conseils à la fin je corrigerai ça après avoir corrigé mon problème principal.

Par contre au niveau du problème principal, je n'ai pas bien compris la solution.
Je dois remplacer ma variable COMMAND par une fonction c'est ça?
Je ne vois pas bien comment une fonction peut remplacer le rôle de ma variable...

Tu peux me montrer un exemple?

[ok.Idriss]

Bonjour.

A priori on dirait que tu essayes de reproduire le comportement de eval...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
[idriss@hp-dv6:~]$ COMMAND="echo \"salut\""
[idriss@hp-dv6:~]$ eval $COMMAND
salut
[idriss@hp-dv6:~]$

Quel est ton véritable besoin ?

Cordialement,
Idriss

[N_BaH]

effectivement, dans ton cas, une fonction n'y fera rien.
utilise un tableau :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
command=( sudo iwconfig ra0 essid "Wi-Fi De Chez Moi" )
printf 'j'\''exécute la commande : %s...' "${command[@]}"
${command[@]}

[matafan]

La meilleure solution à mon avis c'est de passer la commande à exécuter en argument. Quelque chose du genre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
execcmd () {
    echo -n "executing the '$@' command... "
    "$@"
    echo
}
 
execcmd sleep 1
execcmd echo "toto  tata   tutu"
execcmd sudo iwconfig ra0 essid "Wi-Fi De Chez Moi"

Si tu passes par une variable comme tu essaies de le faire, tous les arguments sont mis à plat et tu perds la notion de groupage (i.e. "toto tata tutu" est un seul "mot").