Discussion :

[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 :

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

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 :

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

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 :

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

#!/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 :

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

#!/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]

Code :

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

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

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

[ok.Idriss]

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 :

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

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 :

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

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 :

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

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]

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

[zipe31]

Code :

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

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

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

Code :

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

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.