Problème avec un if [Résolu]

[Socomfr]

Bonjour ( ou bonsoir).

Je débute dans la création de petit shell.
Je voudrais faire un petit shell qui supprime des fichiers (temporaire) qui se trouve dans un dossier.
Mais je me confronte a un problème.

Code :

1
2
3
4
5
6
7
8
9
10
#!/system/bin/sh
 
dossier=$(ls /temp | wc -l)  #correspond au nombre de dossier et fichier dans le dossier temp
 
if test $dossier -gt 0 ;
then
	echo "Dossier non vide"
else
	echo "Dossier vide"
fi

Voici se que j'ai dans la console

Code :

shell.sh: 10: Syntax error: "fi" unexpected (expecting "then")

Je pense que le problème est dans la condition du if, mais je ne sait pas comment le résoudre.

Merci pour votre aide futur.

[bredelet]

Code :

#!/system/bin/sh

C'est quoi comme shell ça?

Parce que ton script comme tu l'as écrit, il marche bien avec bash.

[Socomfr]

zut j'ai fait une erreur lorsque j'ai copier le code.

Code :

#!/bin/sh

Mais le problème n'est pas la.

[zipe31]

Salut,

Quel système ?
Quelle distrib ?
Quel shell ?
sh est-il un lien symbolique ( ls -l /bin/sh pour le savoir) ?

[ok.Idriss]

Bonjour.

C'est vraiment étrange ... que ce soit en bash, en sh, en dash, ou même en ksh ça devrait fonctionner

T'es sûr que t'aurais pas fait une fausse manip du genre déplacer ton script durant l'édition de celui-ci ?

Cordialement,
Idriss

[JeitEmgie]

Envoyé par Socomfr

Code :

un ; qui n'a rien à faire là.

[ok.Idriss]

Envoyé par JeitEmgie

un ; qui n'a rien à faire là.

ça ne devrait pas interférer ...

En effet ça peut s'écrire comme ceci en une ligne :

Code :

if test $dossier -gt 0; then echo "Dossier non vide"; else echo "Dossier vide"; fi

Bon pour un script c'est moche mais c'est pas pour ça que ça ne marche pas ...

Cordialement,
Idriss

[jouana]

Bonjour,

Code :

1
2
3
4
5
6
7
 
if [$dossier -gt 0]
 then 
echo "Dossier non vide"
 else 
echo "Dossier vide"
fi

essaye ça on ne sait jamais

[ok.Idriss]

@ jouana : Là pour contre faut pas oublier les espaces :

Code :

1
2
3
4
5
6
if [ $dossier -gt 0 ]
then 
    echo "Dossier non vide"
else 
    echo "Dossier vide"
fi

Edit : @ zipe31 : des espaces oui, des double quotes je ne pense pas que ce soit nécessaire ici ... au contraire (on évalue des nombres pas des chaînes avec ces opérateurs là).

Cordialement,
Idriss

[zipe31]

Envoyé par jouana

Code :

Faut des espaces autour des crochets et des quotes doubles avec des simples crochets

Code :

if [ "${dossier}" -gt "0" ]

[jouana]

au temps pour moi, désolé pour les erreurs de syntaxe.
J'ai posté ce message a la vas vite juste histoire de donner une idée .
Je ferais plus attention par la suite.

Sinon Socomfr tu as fais un man sh et un man [ histoire de voir ou est ton erreur.

[zipe31]

Envoyé par ok.Idriss

Edit : @ zipe31 : des espaces oui, des double quotes je ne pense pas que ce soit nécessaire ici ... au contraire (on évalue des nombres pas des chaînes avec ces opérateurs là).

Cordialement,
Idriss

C'est clair, mais bon c'est des habitudes prises (ou à prendre) qui ne font pas de mal et évitent bien des tracas bien souvent

[FRUiT]

Sauf qu'avec des guillemets tu évalues des chaines de caractères et non des nombres...

Là autant écrire

Code :

[ "$dossier" = "0" ]

Un test plus cohérent serait :

Code :

1
2
dossier=/tmp
if [ "$(ls -A "$dossier")" ]; then...

[Socomfr]

Merci de vos réponses.
Le problème est réglé en utilisant [ "$dossier" = "0" ]