Discussion :

[Moi302]

Bonjour
J'ai un soucis avec mon code shell
J'ai un fichier qui contient des codes

Fichier1
OHE
COZ
CEIUBC
CBZ

Je voudrais parcourir toutes les lignes du fichier et compter le nombre de caractères. Si le nombre de caractère est égal à 3 je continue le traitement sinon je sort en erreur
Voici mon code

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
while read line
do
 
nbr=$(echo "$line" | wc -l)
if [ $nbr -eq 3 ]
   then
      echo  " Format du fichier Param Valide"
else 
      echo  " Format du fichier Param InValide"
  fi
  done
fi

Je compte sur votre aide

[al1_24]

Pourquoi faire si compliqué ?

Code :

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

1
2
3
4
5
6
if ! grep -vq '^...$' Fichier1
then
  echo  " Format du fichier Param Valide"
else 
  echo  " Format du fichier Param InValide"
fi

Par ailleurs, c'est wc -c pour compter le nombre de caractères .
wc -l compte les lignes

[Moi302]

Merci pour ta réponse mais wc -c compte le nombre de caractére dans tout le fichier alors que dans mon cas j'ai juste besoin de compter le nb caractéres de chaque ligne.
Sinon je viens de tester avec ta solution

Code :

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

1
2
3
4
5
6
if [ ! grep -vq '^...$' $fichier2 ]
then
echo " Format du fichier Param Valide"
else
echo " Format du fichier Param InValide"
fi

Comme erreur, j'ai eu ceci

-vq: unknown test operator

Je crois qu'il n'a pas aimé le syntaxe de ma condition if

[jack-ft]

Merci pour ta réponse mais wc -c compte le nombre de caractére dans tout le fichier alors que dans mon cas j'ai juste besoin de compter le nb caractéres de chaque ligne.

Oops! Que de confusions!

wc -c compte le nombre de caractères dans ce qu'il reçoit sur son entrée standard, c'est-à-dire, dans le cas présent, dans une ligne.
Alors que wc -l compte le nombre de caractères dans ce qu'il reçoit sur son entrée standard, c'est-à-dire, dans le cas présent, dans une ligne. Du coup, s'il retourne autre chose que 1, je veux bien manger mon chapeau (en effet, chaque ligne contient une ligne (logique, non?)).

Sinon je viens de tester avec ta solution

Code :

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

1
2
3
4
5
6
if [ ! grep -vq '^...$' $fichier2 ]
then
echo " Format du fichier Param Valide"
else
echo " Format du fichier Param InValide"
fi

Comme erreur, j'ai eu ceci

Code :

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

-vq: unknown test operator

Je crois qu'il n'a pas aimé le syntaxe de ma condition if

Ben oui!
Tu n'as as pas testé avec la solution d'al1_24, mais avec une adaptation personnelle... qui ne fonctionne pas très bien!
En fait, tu as ajouté des crochets là où il n'en faut pas!
Du coup, c'est équivalent à un 'test'... et 'test' n'aime pas du tout le '-vq' qui ne ressemble à aucun opérateur connu de 'test', d'où le message!
Essaie plutôt avec le vrai code fourni par al1_24.

Sinon, juste pour corriger ton code initial:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
while read line
do
 
nbr=$(echo "$line" | wc -l)
if [ $nbr -eq 3 ]
   then
      echo  " Format du fichier Param Valide"
else 
      echo  " Format du fichier Param InValide"
  fi
  done # PAS BON!
fi

Il y a un 'fi' de trop!

Si c'est le premier (pour respecter le test), dans aucun langage de programmation digne de ce nom (même FORTRAN, je crois), tu ne peux intriquer de cette manière deux structures de contrôle!
D'ailleurs, l'éditeur que tu utilises devrait te le dire et rouspéter sur l'indentation!
Pour faire bref, il faut fermer le 'fi' avant le 'done':

Code :

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

1
2
3
4
5
6
7
8
9
while read line ; do
    nbr=$(echo "$line" | wc -c) # Pour compter le nombre de caractères de la ligne
    if [ $nbr -eq 3 ]
    then echo  " Format du fichier Param Valide pour la ligne ${line}"
    else 
        echo  " Format du fichier Param InValide (${nbr} caractères dans la ligne '${line}')"
        break # Pour sortir du while
    fi
done

Attention! Je donne ce code corrigé juste pour que tu comprennes ce qui ne fonctionnait pas, mais je ne le recommande pas!
D'autres solutions bien meilleures ont été proposées.

[ok.Idriss]

Bonsoir.

Cette syntaxe est valide en bash et ksh et probablement dans d'autres shell :

Code :

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

1
2
3
4
5
6
7
8
while read ligne; do 
    if [ ${#ligne} -eq 3 ]; then
        echo "Ligne valide ! Je continue ..."
    else
        echo "Ligne invalide ! Je sort en erreur"
        exit 1 # Pour sortir en erreur
    fi
done < fichier.txt

L'avantage ici, c'est qu'on a pas besoin de faire appel à des commandes externes : grep, wc, etc.

Sinon dans ton code de départ, hormis toutes les erreurs qui ont été relevée, je constate qu'il manque également ton fichier en entrée

Cordialement,
Idriss

[Moi302]

Merci pour vos réponses
J'ai testé avec ceci

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
while read ligne
 do 
    if [ ${#ligne} -ne 3 ] 
     	then
	    	echo " Format du Centre "$ligne" invalide "
			return 1 
    else
        echo " Format des centre du fichier "$fichier2" Correcte "
 
    fi
done < $fichier2
 
  fi

Cette méthode marche très bien, par contre comme je suis dans une boucle, quand j'ai 10 lignes correcte dans mon fichier, c'est à dire 10 lignes qui respectent le format, il m'affiches 10 fois le message " Format des centre du fichier "$fichier2" Correcte "
Je veux l'afficher qu'une seule fois sur la console. Est ce possible avec la commande echo ???
Merci d'avance

[ok.Idriss]

Bonjour.

Ou est la difficulté ?

Il suffit d'un peu de réflexion algorithmique de base, pas besoin d'aller chercher systématiquement une option magique dans une commande :

Code :

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

1
2
3
4
5
6
7
8
9
while read ligne; do 
    if [ ${#ligne} -ne 3 ]; then
        echo "Format du centre $ligne invalide !"
        exit 1 # Je préfère au return quand on est pas dans une fonction
        # (ça quittera le script en erreur, sinon si ça n'est pas le comportement voulu, il faudra rajouter une condition dans le while, c'est le plus propre)
    fi
done < $fichier2
 
echo "Format des centre du fichier $fichier2 correct."

Sinon merci de faire attention à indenter ton code mieux que ça

Cordialement,
Idriss

[Moi302]

Merciiii ça marche, mon probleme est résolu
Et désolé pour l'indentation du code, j'y penserai la prochaine fois
Encore Merci