Discussion :

[wabit]

Bonjour a tous.
Voila j'essaye de creer un programme qui servira a nettoyer un serveur de fichier de toute les cochonneries que peuvent y mettre mes utilisateurs. Mais voila j'ai un probleme au niveau de la condition de mon "if" que je n'arrive pas à résoudre.

Code :

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

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
32
33
34
35
36
37
38
39
40
#include <stdio.h>
#include <stdlib.h>
#include <dirent.h>
#include <string.h>
 
main()
{
   int i=0;
   char exe[]=".exe",avi[]=".avi",png[]=".png",bmp[]=".bmp",mp3[]=".mp3";
   char dll[]=".dll",wav[]=".wav",wma[]=".wma",jpg[]=".jpg",gif[]=".gif";
   char tif[]=".tif",asx[]=".asx";
 
   char *ext;
   char tt;
 
   struct dirent *lecture;
   DIR *rep;
   rep = opendir("E:\\");
   while ((lecture = readdir(rep)))
   {     
 
 
         ext=strrchr(lecture->d_name, '.');
         tt=*ext;
         if ((strcmp(tt,exe)==0)||(strcmp(tt,avi)==0)||(strcmp(tt,png)==0)||(strcmp(tt,bmp)==0)||(strcmp(tt,mp3)==0)||
(strcmp(tt,dll)==0)||(strcmp(tt,wav)==0)||(strcmp(tt,wma)==0)||(strcmp(tt,jpg)==0)||(strcmp(tt,gif)==0)||
(strcmp(tt,tif)==0)||(strcmp(tt,asx)==0))
            {     
                  remove(lecture->d_name);
                  i++;
            }
 
   }
   closedir(rep);
 
   printf("Il y a eu %d fichiers supprimer",i);
 
   system ("pause");
   return 0;
}

[Médinoc]

Code :

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

char const * tt;

et

Code :

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

tt = ext+1; // au lieu de *ext

Rappel: Vérifier que ext n'est pas NULL.

[wabit]

Merci, maintenant le programme se compile bien.
J'ai vérifier la valeur de ext et elle prend la valeur NULL que lorsque je rencontre un dossier ce qui fait d'ailleurs planter le programme lors de son éxécution ce qui est tres embétant ...

[Skyounet]

Y'a un truc que je comprends pas tt est un char.

Ensuite tu essai de comparer ton char avec les différentes extensions.

Pourquoi ne pas tout simplement faire

Code :

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

1
2
3
4
 
ext=strrchr(lecture->d_name, '.');
if((strcmp(tt,exe)==0) || ..)
[...]

Meme encore mieux pourquoi tu ne fais pas un tableau contenant toutes les extensions. Et tu compares via une boucle au lieu de faire un big if.

[fearyourself]

Merci, maintenant le programme se compile bien.
J'ai vérifier la valeur de ext et elle prend la valeur NULL que lorsque je rencontre un dossier ce qui fait d'ailleurs planter le programme lors de son éxécution ce qui est tres embétant ...

Quand on dit vérifier, on est bien d'accord que tu as fait quelque chose du genre :

Code :

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

1
2
3
4
5
6
 
ext=strrchr(lecture->d_name, '.');
if(ext!=NULL)
  { 
  tt=*ext;
  }

Mais de toute façon le code est faux parce qu'après tu fais ceci :

Code :

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

1
2
3
4
5
6
7
8
9
 
if ((strcmp(tt,exe)==0)||(strcmp(tt,avi)==0)||(strcmp(tt,png)==0)||
(strcmp(tt,bmp)==0)||(strcmp(tt,mp3)==0)||(strcmp(tt,dll)==0)||
(strcmp(tt,wav)==0)||(strcmp(tt,wma)==0)||(strcmp(tt,jpg)==0)||
(strcmp(tt,gif)==0)||(strcmp(tt,tif)==0)||(strcmp(tt,asx)==0))
            {     
                  remove(lecture->d_name);
                  i++;
            }

Or il y a un problème de typage. strcmp prend deux pointeurs...

Le code correct serait pas plutôt :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
ext=strrchr(lecture->d_name, '.');
if(ext!=NULL)
  { 
         if ((strcmp(ext,exe)==0)||(strcmp(ext,avi)==0)||(strcmp(ext,png)==0)||
(strcmp(ext,bmp)==0)||(strcmp(ext,mp3)==0)||(strcmp(ext,dll)==0)||
(strcmp(ext,wav)==0)||(strcmp(ext,wma)==0)||(strcmp(ext,jpg)==0)||
(strcmp(ext,gif)==0)||(strcmp(ext,tif)==0)||(strcmp(ext,asx)==0))
            {     
                  remove(lecture->d_name);
                  i++;
            }
}

Jc

[wabit]

Merci les gars maintenant le code fonctionne nikel !!
Vous êtes trop fort