Discussion :

[Amnael]

Bonjour/Bonsoir

J'ai un problème simple mais assez embêtant.

Soit un script dont le paramètre positionnel $1 vaut toto

a=1
b=a

Comment dans un script retrouver la valeur du paramètre $1 en partant de b ?

Le mieux que j'ai pu faire c'est:

echo \$$b qui va donner $a mais sans l'évaluer.

[N_BaH]

Bonjour,

en bash ?

Code :

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

1
2
3
4
a=123
b=a
echo ${!b}
123

c'est appelé une indirection.

[Amnael]

Pardon j'ai oublié de préciser; en sh.

Et ok merci pour le terme, dur de trouver des infos dessus sur internet sans savoir comment ça s'appelle

[N_BaH]

je crains qu'en sh tu n'aies pas d'autres choix que d'utiliser eval
c'est une commande très dangereuse, car, imagine que l'argument passé au script soit "$(rm -rf /)", alors même
eval echo "$1" va effacer ton système.

Il serait préférable de nous exposer la raison pour laquelle tu pense avoir besoin de faire ainsi, pour qu'on cherche un contournement.

[Amnael]

Le script est censé extraire les fichiers d'une archive.

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
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
 
#!/bin/sh
 
estDansTar() {
 
if [ $# -ne 2 ]
then
	exit 1;
fi
for i in `tar -tf $1`
	do
	if [ -d $2 ]
	then
		exit 2;
	fi
	if [ `basename $i` = $2 ]
	then
		exit 0;
	fi
done
}
 
 
 
if [ $# -lt 2 ]
then
	echo "Usage $0 archive.tar fichier.txt [...]";	
	exit 1;
fi
k=2;
nbExtrait=0;
for i in `tar -tf $1`
	do
	while [ $k -le $# ] #tant que l'on a pas parcouru tout les paramètres. (je pourrais user un switch mais j'ai besoin de garder mon $1
	do
		if [ -d $... ]  # c'est ici que je dois vérifier si le paramètre $ $k est un fichier ou non idem pour tous les $k qui suivent.
		then
			echo "Usage $0 archive.tar fichier.txt [...]";
			echo "$k n'est pas un fichier accessible;"
			exit 2;
		fi
		if [ estDansTar $1 $k -eq 0 ] 
		then
			nbExtrait=`expr $nbExtrait + 1`;
			tar -xf $1 $k;
		fi
		k=`expr $k +1`;	
	done
done
if [ $nbExtrait -eq 0 ]
then
	echo "Aucun fichier correspondant extrait";
	exit 3;
fi
exit 0;

[N_BaH]

[...]

Code :

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

1
2
3
4
5
6
7
8
9
k=2
nbExtrait=0
premParam="$1"
tar -tf "$premParam" | while read i
do
shift
   for f
   do  
      if [ -d "$f" ]

[...]?

[Amnael]

Je n'avais pas pensé que je pouvais stocké le paramètre dont j'allais me resservir merci

Du coup je peux bien parcourir ma liste de paramètres après avoir fait un switch.


Un autre plantage tout de même

Lorsque je réalise le test

Code :

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

if [ estDansTar $archive $k -eq 0 ]

J'obtiens l'erreur suivante.

./extraireFichierTar.sh: 47: [: estDansTar: unexpected operator
expr: erreur de syntaxe

répété trois fois

Alors que; estDansTar est pourtant censé me renvoyer un nombre avec exit (j'ai mee testé avec return) et pas moyen.


pour simplifier le problème,si je fais une fonction

toujours0() {

exit 0;
}

Et que je fais un test du style if [ toujours0 -eq 0 ] ça devrait pouvoir se faire.

[N_BaH]

j'ai été mon précédent message pour changer la première boucle for en un pipe vers une boucle while.

pour ton test :

Code :

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

if test "$(taFonction "$arg1" "$arg2")" -eq 0; then

[Amnael]

Maintenant en mettant cette syntaxe

Code :

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

if [ "$(estDansTar "$archive" "$k")" -eq 0 ]

J'obtiens illegal number

Un nombre peut être illégal O_o

[N_BaH]

mais... elle affiche quelque chose ta fonction ?

Code :

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

1
2
3
4
5
maFonc() {
   return 1
}
test $(maFonc) && echo ok || echo ko
ko

ou mieux

Code :

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

1
2
3
4
5
maFonc() {
   return 1
}
if maFonc; then echo ok; else echo ko; fi
ko

ou encore

Code :

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

1
2
3
4
5
maFonc() {
   echo 1
}
test $(maFonc) -eq 0 && echo ok || echo ko
ko

[Amnael]

Après avoir fait quelques test; non elle n'affiche rien; et surtout ne retourne rien. Je ne sais pas du tout pourquoi.

Pour rappel la fonction est là

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
19
 
estDansTar() {
 
if [ $# -ne 2 ]
then
	return 1;
fi
for i in `tar -tf $1`
	do
	if [ -d $2 ]
	then
		return 2;
	fi
	if [ `basename $i` = $2 ]
	then
		return 0;
	fi
done
}

[N_BaH]

en shell, return n'affiche pas la valeur (de la même manière que exit), c'est "un exit pour les fonctions".

[Amnael]

en shell, return n'affiche pas la valeur (de la même manière que exit), c'est "un exit pour les fonctions".

Donc au final je dois faire un echo 0 pour que ça marche.

[N_BaH]

pas forcément, tu peux tester le code de retour renvoyé par la fonction avec return comme je l'ai montré dans mon avant dernier message.

[Amnael]

Je vois pas comment j'aurais pu utiliser directement le code de retour; je suis donc passé par un echo pour mon test dans le main.

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
19
20
21
22
23
24
25
deTar() {
 
if [ $# -ne 2 ]
then
	echo 1;
	return 1;
fi
for i in `tar -tf $1`
	do
	if [ -d $2 ]
	then
		echo 2;
		return 2;
	fi
	if [ `basename $i` = $2 ]
	then
 
		tar -xf $1 $2;
		echo 0;
		return 0;
	fi
done
echo 3;
return 3;
}

Au passage j'ai changé el but de la fonction, je trouvais ça stupide de simplement tester si le fichier était présent; alors que je pouvais aussi désarchiver dans la fonction.

Malgré tout, lorsque je rentre dans le cas où un fichier est bien présent, ça plante.

Code :

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

1
2
tar: toto.txt : non trouvé dans l'archive
tar: Arrêt avec code d'échec à cause des erreurs précédentes

En plus je lance le script avec un seul fichier à détarrer et j'ai la même erreur qui boucles 4 fois; avec 3 affichages du 0 en prime.

[N_BaH]

là, ça dépasse le cadre du sujet du fil : revois ton algorithme !

ouvre un nouveau sujet, avec ton script actuel, et les messages d'erreurs, et éventuellement une x-trace d'exécution.