Script langstat.sh : demande de conseils et modifs

Salut,

for lettre in A B C D E F G H I J K L M N O P Q R S T U V W X Y Z
do

for lettre in {A..Z} do…

# on affiche le message dans la console
echo " j'ai testé un deuxième paramètre "
# on affiche le message dans le fichier test.txt
echo " j'ai testé un deuxième paramètre" >> test.txt

echo " j'ai testé un deuxième paramètre " | tee -a test.txt

Mieux vaut prendre l'habitude d'entourer les variables d'accolades et de quotes doubles (

echo "${mavar}"

)

Merci pour ta réponse. Je ne connaissais pas la commande tee

Pour le code suivant comptant le nombre de lignes dans le fichiers test.txt

echo " Nombre de lignes dans test.txt : `wc -l test.txt` "

J'ai l'affichage suivant dans la console:

Le nombre de ligne dans test.txt : 3 test.txt

est-ce que c'est possible supprimer test.txt ? et Comment?

Merci pour ton aide

Hello,

En matiére d'amélio, je te proposerai :

- je vérifie l'enssemble des mes paramétres d'entrée et je défini des variables nomées plutot qu'utiliser $1, $2 etc qui pourront vite devenir ilisible sur de gros scripts!

if [ $# -ge 2 ]
then
    var1="${1}"
    var2="${2}"
elif [ $# -ge 1 ]
    var1="${1}"
else
    echo 'Fichier de dictionnaire introuvable'
    exit 1
fi

- Pour ce qui est du prompt utilisateur, soit c'est oui, soit on fait rien? Aussi, on utilisera plutot le case dans le cas présent.

while true; do
    read -p "on supprime test.text ?  oui/non : " reponse
    case $reponse in
        [Oo]* ) echo "on supprime le fichier test.txt" && rm test.txt; break;;
        [Nn]* )  echo "on garde le fichier test.txt"; break;;

• ) echo "Vous devez répondre [Oo]ui ou [Nn]on";; esacdone

- Il n'y a aucune fonction dans ton code ? Imaginons que tu souhaites passer plusieurs fichiers à ton script ? Tu vas copier coller le code autant de fois que de paramétre possible ?

function_name () {
  for lettre in {A..Z}
  do
    printf "%s - %s\n" $(grep  -ic $lettre $1)  $lettre 
  done
}
function_name "${var1}" | sort -rn

- Bon je chipotte mais générallement, je sépare le traitement de l'affichage.

function_name () {
  declare -A ARRAY
  for lettre in {A..Z}
  do
    ARRAY[$lettre]=`grep  -ic $lettre "${var1}"`
  done
}
function_name "${var1}"
for key in "${!ARRAY[@]}"; do
    printf '%s = %s\n' "$key" "${ARRAY[$key]}"
done

A plus :)

Hello,

Je rajouterai que tu ne vérifies pas l'existance du fichier.

En l'occurence, tu vérifies que le premier paramétre est défini. Que celui ci n'est pas vide. Mais tu ne sais pas si le fichier existe..

Tu peux vérifier qu'un fichier avec :

if [[ -f "$var1" ]]; then
    echo "$var1 exist"
fi

Et juste pour le fun :

$ cat test2
function usage() { echo "$1"; echo "langstat.sh <file> [test]"; exit 1; } 

[[ -f "$1" ]] && var1="$1" || usage 'file not exist';

echo "Traitement du fichier ..."

$ bash test2 "aeazezea"
file not exist
langstat.sh <file> [test]

$ bash test2 "/etc/hosts"
Traitement du fichier ...