Discussion :

[vinzzzz]

Hello,
je souhaite réaliser un script bash (je débute...) qui cherche simplement l'occurence d'un mot dans certains fichiers et qui ensuite déplace ces fichiers.
Mon pb est le suivant: je souhaite stocker le résultat d'une commande dans une variable ms je ne trouve pas comment faire. Pour le moment j'ai le petit script suivant, qui ne fonctionne pas:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
#!/usr/local/bin/bash -norc
for i in `ls | grep .gen`
do
       res=`grep bact $i` # pb
 
       if test $res = bact
       then
                mv $i bact_f/
       fi
done

La variable res semble ne pas stocker le résultat... Si quelqu'un peut me mettre sur la voie pr stocker le résultat d'une commande...

Merci d'avance.

[2Eurocents]

...

Code :

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

1
2
3
4
5
6
7
8
9
10
#!/usr/local/bin/bash -norc
for i in `ls | grep .gen`
do
       res=`grep bact $i` # pb
 
       if test $res = bact
       then
                mv $i bact_f/
       fi
done

La variable res semble ne pas stocker le résultat... Si quelqu'un peut me mettre sur la voie pr stocker le résultat d'une commande...

En es tu vraiment sûr ? As tu tenté un "echo $res" juste après l'affectation ?
Les back-quotes sont pourtant l'opérateur "old-school" qui permet de faire ce que tu souhaite.
La nouvelle syntaxe POSIX, à privilégier, est cependant $( ) plutôt que ``.

On devrait donc pouvoir écrire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
#!/usr/local/bin/bash -norc
for i in $(ls | grep .gen)
do
       res=$(grep bact $i) # pb
echo "res vaut [$res]"
 
       if test $res = bact
       then
                mv $i bact_f/
       fi
done

Par ailleurs, ton test, dans le if, ne me paraît pas forcément pertinent.
Pourquoi ne pas te baser simplement sur le code de retour du grep ? Il vaut 0 en cas de succès du grep, 1 en cas d'échec (et 2 en cas d'erreur).


Bon courage,

[vinzzzz]

Par ailleurs, ton test, dans le if, ne me paraît pas forcément pertinent.
Pourquoi ne pas te baser simplement sur le code de retour du grep ? Il vaut 0 en cas de succès du grep, 1 en cas d'échec (et 2 en cas d'erreur).

Oui il n'est pas vraiment pertinent, même si dans l'exercice que j'ai a faire il fonctionne... Je sais pas comment tester si la commande a marché ou si la chaine est nulle.
J'ai regardé un petit peu sur internet et j'ai trouvé la structure suivante:

Code :

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

1
2
3
4
5
 
if [-n "$res"]  # Si la chaine est non nulle
then
	mv $i virus_file/
fi

Mais ca semble ne pas passer: le shell me renvoie "[ -n commande introuvable"...
Comment récupérer le code de retour du grep?

MErci a toi de ta réponse en tout cas

[vinzzzz]

Ok c'est bon, j'ai trouvé, dans l'exemple qu'on me donnait sur internet ils avaient oublié le mot test, et les crochets semble également être de trop.
Le test donne ceci finalement:

Code :

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

1
2
3
4
5
 
if test -n "$res"  # Si la chaine est non nulle
then
	mv $i virus_file/
fi

Merci a toi en tout cas. Juste, pr finir, j'ai testé si la chaine était nulle ou non, est-ce ce que tu voulais me faire faire? Si non, peut tu me dire comment récupérer le code de retour de grep?

[2Eurocents]

Ok c'est bon, j'ai trouvé, dans l'exemple qu'on me donnait sur internet ils avaient oublié le mot test, et les crochets semble également être de trop.

test et crochets sont synonymes.
Par contre, les crochets doivent être séparés de tout autre texte par un blanc. Il faut donc écrire

Code :

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

[ -n "$res" ]

Et comme je l'ai dit, pourquoi ne pas se baser tout simplement sur le code de retour de grep ?

Code :

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

1
2
3
if grep montexte monfichier ; then
   echo "J'ai trouvé ..."
fi

Sinon, après l'exécution de chaque commande, chaque code de retour se trouve dans la variable $?

Bon courage.

[vinzzzz]

Code :

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

1
2
3
 
test et crochets sont synonymes.
Par contre, les crochets doivent être séparés de tout autre texte par un blanc. Il faut donc écrire

Ok je pense que j'avais pas mis les espaces...
Pr le retour du grep, je me serai basé dessus si j'avais su comment faire
Maintenant c'est le cas.

Juste pr vérifier:

Code :

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

1
2
3
4
 
if grep montexte monfichier ; then
   echo "J'ai trouvé ..."
fi

est donc équivalent a:

Code :

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

1
2
3
4
5
 
if $? -eq 1 
then
   echo "J'ai trouvé ..."
fi