Condition pour tester l'existence d'un fichier ne marche pas

Version imprimable

Bonjour à tous !

Je suis débutant et j'ai un souci avec avec la condition dans mon script en batch.
J'aimerai mettre au point pour mes cours un script qui contrôle les fichiers qui ont le droit Suid sur na machine.

Voici ce que j'aimerai mettre en place :

Lors du premier lancement du script, un fichier est créé contenant tous les fichiers qui ont le droit Suid.Ensuite, lors des prochains lancements du scripts, il va détecter que mon fichier existe et en créera un autre temporaire dans lequel il mettra la nouvelle liste des fichiers en Suid. Enfin il fera un diff des 2 fichiers pour les comparer.

Le problème , c'est que ma condition ne marche pas...

Voici mon code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 #!/bin/bash original=/root/Bureau/scripts/Setuid.txt temporaire=/root/Bureau/scripts/Setuidtemp.txt base=/ if [ -e $orignal] then touch $temporaire recherche=`find $base -type f -perm -u=s -print` for fichiers in $recherche do detail=`ls -l $fichiers` echo $detail >> $temporaire done diff -a $orignal $temporaire rm $temporaire else touch $orignal recherche=`find $base -type f -perm -u=s -print` for fichiers in $recherche do detail=`ls -l $fichiers` echo $detail >> $orignal done fi exit
En gros au début je teste si mon fichier existe. Si il existe je crée un fichier temporaire et je fais le diff des 2.
Si il n'existe pas je le crée (le ELSE).

Le problème c'est qu'il ne passe pas au ELSE alors que le fichier n'existe pas... il crée le fichier temporaire et forcéement le diff affiche une erreur. La condition ne marche pas... Au secours !!! Je suis pomé là...

Bonne journée et merci d'avance !

25/03/2011, 18h28
fransoo
Ce qui me saute aux yeux yeux, c'est

Code:

if [ -e $orignal]

alors qu'il faudrait une espace avant le crochet fermant :

Code:

if [ -e $orignal ]

Maintenant, il serait bien de
1. utiliser les "codes tags" (bouton #)
2. indenter le code
pour rendre cela plus lisible.
D'autre part, il me semble qu'il y aune redondance de code qui suggère que l'utilisation d'une fonction serait bienvenue;)

Salut Fransoo !
Merci pour ta réponse :)

Autant pour moi pour l'espace mais j'avais essayé sans, faute de frappe. Et pour l'indentation j'ai fait une copie d'un fichier txt sur ce forum ça n'avait pas pris les alinéas... :?
Par contre tu as raison j'ai commenté mon code :)

Par contre je ne comprends pas où tu voudrais utiliser une fonction ? Par ce que normalement la condition suffit à dire quelle partie du code exécuter.

En gros ce que je voudrais obtenir c'est :

Si le fichier original existe
Création d'un fichier temporaire qui répertorie les fichiers Suid
diff du fichier original et du temporaire
Sinon
Création du fichier original répertoriant les fichiers Suid

C'est tout bête mais il ne prend pas la condition... Il ne passe pas au "sinon" alors que mon fichier n'existe pas...
:cry:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
#!/bin/bash
 
#Variables
 
original=/root/Bureau/scripts/Setuid.txt                    # variable contenant le chemin de mon fichier original
temporaire=/root/Bureau/scripts/Setuidtemp.txt         #contient le chemin de mon fichier temporaire
base=/                                                               #base de la recherche
 
#corps de mon code
 
if [ -e $orignal ]                                                   # si le fichier original fichier existe
       then                                                           #alors
              touch $temporaire                                #création du fichier temporaire
              recherche=`find $base -type f -perm -u=s -print`               # recherche des fichiers Suid
              for fichiers in $recherche                                                 # pour chaque fichiers trouvés
                      do                                                                       #faire
                               detail=`ls -l $fichiers`                                    #ls -l du fichier
                               echo $detail >> $temporaire                            # écrire le résultat dans mon fichier temporaire
                      done
              diff -a $orignal $temporaire                                               # différence ds 2 fichiers pour voir si un fichier Suid à été rajouté
              rm $temporaire                                                               # Supression du fichier temporaire
     else                                                                                     #Sinon
               touch $orignal                                                              #Je créé le fichier original
               recherche=`find $base -type f -perm -u=s -print`               #Je recherche les fichiers Suid
               for fichiers in $recherche                                                 #Pour chaque fichiers trouvés
                      do                                                                       #faire
                                detail=`ls -l $fichiers`                                   #ls-l du fichier
                                echo $detail >> $orignal                                 #écrire le résultat dans mon fichier temporaire
                      done
       fi
exit

Je comprends pas pourquoi il ne passe pas au "sinon" alors que la condition est fausse.. 8O.

Merci d'avance !

26/03/2011, 12h11
shinobitom

Oups ! J'arrive pas à mettre des espaces et des alinéas dans mon post pour indenter mon code... LOL Désolé (alalala il est vraiment nul ce débutant en batch...)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
#!/bin/bash
 
#Variables
 
original=/root/Bureau/scripts/Setuid.txt # variable contenant le chemin de mon fichier original
temporaire=/root/Bureau/scripts/Setuidtemp.txt #contient le chemin de mon fichier temporaire
base=/ #base de la recherche
 
#corps de mon code
 
if [ -f $original ] # si le fichier original fichier existe
then #alors
   touch $temporaire #création du fichier temporaire
   recherche=`find $base -type f -perm -u=s -print` # recherche des fichiers Suid
   for fichiers in $recherche # pour chaque fichiers trouvés
   do #faire
      detail=`ls -l $fichiers` #ls -l du fichier
      echo $detail >> $temporaire # écrire le résultat dans mon fichier temporaire
   done
   diff -a $original $temporaire # différence ds 2 fichiers pour voir si un fichier Suid à été rajouté
   rm $temporaire # Supression du fichier temporaire
else #Sinon
   touch $original #Je créé le fichier original
   recherche=`find $base -type f -perm -u=s -print` #Je recherche les fichiers Suid
   for fichiers in $recherche #Pour chaque fichiers trouvés
   do #faire
      detail=`ls -l $fichiers` #ls-l du fichier
      echo $detail >> $original #écrire le résultat dans mon fichier temporaire
   done
fi
exit

Bonjour,

oulala... tout ça, tout ça ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#!/bin/bash
 
#Variables
# variable contenant le chemin de mon fichier original
orignal=/root/Bureau/scripts/Setuid.txt
# contient le chemin de mon fichier temporaire
temporaire=/root/Bureau/scripts/Setuidtemp.txt
# base de la recherche
base=/
 
if [ -e "$orignal" ]
then find $base -type f -perm -u=s -print -ls | diff -a $orignal -
else find $base -type f -perm -u=s -print -ls > $orignal
fi

c'est tout ?

26/03/2011, 19h16
bhamp0

[HS]Mais pourquoi tout le monde veut tester l'existence de l'élan ?[/HS]
26/03/2011, 20h39
N_BaH

:mouarf:
27/03/2011, 14h12
shinobitom

LOL

Merci merci à tous ça marche ! :D
Ouai c'est vrai que mon code était un peu long pour ce que je devais faire ! :lol:
Mais bon pour comprendre j'aime bien tout décomposer au début.

EN fait grace à Nbah j'ai compris que ma condition ne marchait pas car il me manquait les doubles cotes " autour de ma variable dans la condition. C'est pour ça qu'elle ne passait pas !

je mettais if [ -e $original ] à la place de if [ -e "$original" ] :mrgreen:

VOUALA alors bien le merci à tous je reviendrais ;)