Discussion :

[mamour]

Bonjour,

Je coince sur ceci question syntaxe dans un script
Ma condition ne fonctionne pas ....
Un petit coup de pouce ?

PS : je ne veux pas utiliser "grep" pour des raisons didactiques. ...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
#!/bin/bash
 
nb=0
mot=echo
while read ligne
do
   if [ expr index $ligne $mot ]; then
      let "nb=nb+1"
   fi
done < compare
echo $nb

Merci d'avance et excellente journée à tous !

[Flodelarab]

Bonjour

Si c'est pour apprendre, alors apprends à mettre des double-quotes autour des variables à moins de savoir pourquoi tu ne le fais pas.

Ensuite, apprends que [ ] est un raccourci pour la commande test. Donc écrire test expr n'a pas de sens puisque tu écris 2 commandes à la suite.
Pour avoir des précisions sur la commande test, tape help test ou man test.

Après if, il y a effectivement une commande. Pas besoin de signe de ponctuation comme en C++ ou java ou python ou cætera

Je ne connaissais même pas cette expression avec let. On peut utiliser aussi:

Code :

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

nb=$((nb+1))

Quand au fait de compter les occurrences ... il y a toujours le problème du chevauchement. Combien y a-t-il de fois "toto" dans "tototo" ?

[disedorgue]

Bonjour,

Ici, il y a une incompréhension entre l'intitulé de la discussion et le besoin vis à vis de ton script car celui-ci ne comptera au plus qu'une occurrence par ligne...

[mamour]

Merci pour vos remarques constructives.

Elles m'ont au moins permis de découvrir et comprendre par moi même (test, [[ ]], "$var", ...) sans "copier/coller" idiot ....

J'en arrive donc à ceci qui fonctionne

Code :

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

1
2
3
4
5
6
7
8
9
nb=0
mot=echo
while read ligne
do
    if [[ "$ligne"`=~ "$mot" ]]; then
        let "nb=nb+1"
    fi 
done < compare
echo "$nb"

Juste une petite chose :

Après if, il y a effectivement une commande. Pas besoin de signe de ponctuation comme en C++ ou java ou python ou cætera

Si je les omet, il me donne une erreur de syntaxe près du symbole inattendu "then" ... une info ?

Excellente journée

[disedorgue]

Voici une autre syntaxe bash :

Code :

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

1
2
3
4
5
6
7
mot="echo"
nb=0
while read ligne
do 
   [[ "$ligne" =~ "$mot" ]] && ((nb++))
done <bobo.txt
echo $nb

[Flodelarab]

Développons:

On peut mettre n'importe quelle commande derrière if. Cette commande renvoie vrai (0) ou faux (1->255, code erreur). Et ceci est interprété par if.
Sans if, on affiche le résultat de la dernière commande par:

Code :

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

echo $?

Une expression numérique n'est pas une commande. Voilà pourquoi il te renvoie une erreur. Tu dois mettre test ou des crochets, simples ou doubles.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
$ if grep -q sucre *.txt; then echo OK;else echo KO;fi
OK
$ if grep -q toto *.txt; then echo OK;else echo KO;fi
KO
$ if gimp; then echo OK;else echo KO;fi
OK
$ if test 5 -eq 0; then echo OK;else echo KO;fi
KO
$ if test $((5-2-3)) -eq 0; then echo OK;else echo KO;fi
OK
$

Dans mon dossier, j'ai des fichiers textes comme des recettes de cuisine (contenant "sucre") mais ils ne contiennent pas "toto".
Le logiciel de retouche photo gimp s'est terminé sans erreur.
5 n'est pas égal à 0 mais 0 est égal à 5-2-3.

[mamour]

Un tout grand merci à tous pour votre patience !!!!!

[frp31]

Bonjour,

Je coince sur ceci question syntaxe dans un script
Ma condition ne fonctionne pas ....
Un petit coup de pouce ?

PS : je ne veux pas utiliser "grep" pour des raisons didactiques. ...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
#!/bin/bash
 
nb=0
mot=echo
while read ligne
do
   if [ expr index $ligne $mot ]; then
      let "nb=nb+1"
   fi
done < compare
echo $nb

Merci d'avance et excellente journée à tous !

si je me base sur ta demande

Code :

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

nb_lignes_contenant_mot=$(sed '/mot/ !d' fichier_a_analyser | wc -l)

évitera les boucles inutiles


y'a bien 3 lignes qui matchent au moins 1 fois chacune le mot titi :

Code :

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

1
2
3
4
5
6
7
8
9
francois@baal:~$ mot=titi
francois@baal:~$ echo toto toto toto toto titi toto toto toto titi toto >> a
francois@baal:~$ echo toto toto toto toto titi toto toto toto titi toto >> a
francois@baal:~$ echo toto toto toto toto titi toto toto toto titi toto >> a
francois@baal:~$ echo >> a
francois@baal:~$ nb=$(sed '/'${mot}'/ !d' a | wc -l) ; echo $nb
3
francois@baal:~$ 
francois@baal:~$ rm a