Problème de if / fi

Bonjour,

j'ai le code suivant mais j'ai une erreur : "problème de fi"...
Je ne me souviens pas exactement de l'intitulé mais ça concernant le "fi" juste avant le "else"...

Je ne m'y connais pas trop mais je ne vois pas où pourrait se trouver l'erreur :/

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

#!/bin/ksh # script TEST.sh   numsoc=1 numrun=2 # Is the file existing and not empty ? if [-f $fichier/$MSV01.$numsoc.$numrun.xml] then nbrmot= $(wc -l $fichier/MSV01.$numsoc.$numrun.xml)   if [$nbrmot > 1] then cp $fichier/MSV01.$numsoc.$numrun.xml $tmp if [$? != 0] then echo "copying problem of the file" fi fi else echo "$fichier/$MSV01.$numsoc.$numrun.xml not present" fi

---

Merci;)

A priori, je dirais ça:

Please note the spacing inside the [ and ] brackets! Without the spaces, it won't work!

http://www.linuxconfig.org/Bash_scri...-fi-statements

bonjour,

Code:

---

1 2	if [ expr ]; then ...

---

de plus, les évaluations arithmétiques s'effectuent dans...
des évaluations arithmétiques :

Code:

---

1 2	if (( expr )); then ...

---

merci à vous :ccool:

mais qu'entendez-vous par évaluation arithmétique ?

des comparaisons de nombres :

Code:

---

1 2 3 4 5 6 7 8 9 10

var=12 var1=2 if (( var > var1 )) then echo "\$var est supérieur à \$var1" elif (( var < var1 )) then echo "\$var est inférieur à \$var1" else # c'est à dire (( var == var1 )) echo "\$var est égal à \$var1" fi

---

comme tu le vois, le signe dollar n'est pas nécessaire entre les double-parenthèses (( ))(sauf pour les éléments d'un tableau, qui doivent conserver leur forme: ${array[n]})

PS: n'oublie de mettre des espaces entre les crochets droits [ ] et l'expression testée.
ces espaces indispensables entre crochets droits ne le sont pas entre double parenthèses.

Merci, ça marche maintenant :) mais j'ai cette erreur :

Code:

---

1 2 3

./TEST.sh[8]: 60: not found NBRMOT: ./TEST.sh[10]: > 1 : syntax error

---

Problème pour considérer le nombre de wc (=60) dans le fichier visé ?


sauf pour

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

#!/bin/ksh # script TEST.sh   numsoc=1 numrun=28781 # Is the file existing and not empty ? if [ -f $fichier/"MSV01".$numsoc.$numrun.xml ];then nbrmot= $(wc -l $fichier/"MSV01".$numsoc.$numrun.xml) echo "NBRMOT: $nbrmot" if (( $nbrmot > 1 ));then cp $fichier/MSV01.$numsoc.$numrun.xml $tmp/PPI if (( $? != 0 ));then echo "copying problem of the file" fi fi else echo "$fichier/"MSV01".$numsoc.$numrun.xml not present" fi

---

il ne devrait pas y avoir d'espaces autour du = lors de l'assignation d'une variable :

Code:

---

1 2 3

nb2lignes=_$(wc -l $fichier/"MSV01".$numsoc.$numrun.xml) #non nb2lignes=$(wc -l $fichier/"MSV01".$numsoc.$numrun.xml) #oui

---

Tu peux encore simplifier :

Code:

---

if (( $? != 0 )) ; then

---

est équivalent à

Code:

---

if (( $? )) ; then

---

Une expression numérique non nulle est considérée comme vraie.

voire même :

Code:

---

cp $fichier/MSV01.$numsoc.$numrun.xml $tmp/PPI || echo "copying problem of the file"

---

simplement :)

Merci à tous pour votre aide, cela m'a bien aidé à mettre tout cela un peu en ordre :zoubi: