Vérifier l'extension d'un fichier passé en argument

Version imprimable

Bonjour,

J'ai écrit un programme qui prend en entrée en argument un fichier au format .glv.
Néanmoins j'aimerai que l'exécution du programme se termine si on donne en argument un fichier qui a une mauvaise extension.
Je pensais que ce bout de code remplissait cette tache:
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 int verifieExtension(char s[]) // verifie si on passe en argument un fichier cyber .glv { int j; int cpt =strlen(s); for (j = 0; j < cpt-3; j++) { if ((s[j] = "g") && (s[j+1] = "l") && (s[j+2] = "v")) { return 1; } } return 0; } int main(int argc, char *argv[]) { /** ... **/ // Si fichier donne en argument pas au format .glv: sortie if (verifieExtension(argv[1]) == 1) { printf("format de fichier non correct: .glv accepte seulement\n"); exit(EXIT_FAILURE); } /** ... **/ return 0; }
Mais le compilateur me met en erreur:
"invalid conversion from `const char*' to `char' ". au niveau de la ligne

Code:

if ((s[j] = "g") && (s[j+1] = "l") && (s[j+2] = "v"))

J'ai beau retourner le problème, mais je n'arrive pas à savoir comment le résoudre. Si vous pouviez m'éclairer, merci :)

14/08/2007, 11h10
Beldom

Essaie avec des ' ' plutôt que des " " qui indiquent une chaine quand tu ne testes qu'un caractère.

Ah, et aussi : Pas

Code:

if ((s[j] = "g") && (s[j+1] = "l") && (s[j+2] = "v"))

Mais :

Code:

if ((s[j] == 'g') && (s[j+1] == 'l') && (s[j+2] == 'v'))

== pour un test, = pour une assignation...

Ca devrait aller mieux. (Pas regardé le reste du code par contre, mais je crois que je vais le faire quand même ^^)
14/08/2007, 11h13
Franck.H

Il y a franchement plus simple pour retrouver l'extension ! Tu fait une recherche par exemple du . qui commence l'extension avec la fonction strrchr puis tu compares la chaîne à partir de ce pointeur !

Citation:

Envoyé par willem77

Mais le compilateur me met en erreur:
"invalid conversion from `const char*' to `char' ". au niveau de la ligne

Code:

if ((s[j] = "g") && (s[j+1] = "l") && (s[j+2] = "v"))

J'ai beau retourner le problème, mais je n'arrive pas à savoir comment le résoudre. Si vous pouviez m'éclairer, merci :)

Le mien est nettement plus bavard...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `verifieExtension':
main.c:4: warning: implicit declaration of function `strlen'
main.c:7: warning: assignment makes integer from pointer without a cast
main.c:7: warning: assignment makes integer from pointer without a cast
main.c:7: warning: assignment makes integer from pointer without a cast
main.c: In function `main':
main.c:22: warning: implicit declaration of function `printf'
main.c:23: warning: implicit declaration of function `exit'
main.c:23: error: `EXIT_FAILURE' undeclared (first use in this function)
main.c:23: error: (Each undeclared identifier is reported only once
main.c:23: error: for each function it appears in.)
main.c: At top level:
main.c:15: warning: unused parameter 'argc'
Process terminated with status 1 (0 minutes, 1 seconds)
3 errors, 7 warnings

Tu devrais poster du code complet. On ne sais pas ce que tu as réellement oublié...

Ceci compile et fonctionne (de façon naïve)

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
32
33
34
35
36
 
#include <string.h>
#include <stdio.h>
#include <stdlib.h>
 
int verifieExtension (char s[]) // verifie si on passe en argument un fichier cyber .glv
{
   int j;
   int cpt = strlen (s);
   for (j = 0; j < cpt - 3; j++)
   {
   }
 
   if ((s[j] == 'g') && (s[j + 1] == 'l') && (s[j + 2] == 'v'))
   {
      return 1;
   }
   return 0;
}
 
int main (int argc, char *argv[])
{
   if (argc > 1)
   {
      // Si fichier donne en argument pas au format .glv: sortie
      if (verifieExtension (argv[1]) != 1)
      {
         printf
            ("'%s' : format de fichier non correct: .glv accepte seulement\n",
             argv[1]);
         exit (EXIT_FAILURE);
      }
   }
 
   return 0;
}

Pose des questions si tu ne comprends pas (et cesse de confondre ' et ").

Ce code a un comportement indéfini si le nom du fichier fait moins de 3 caractères. Il faut considérablement renforcer l'algorithme.

Il faut aussi être plus clair avec la définition.

Ceci est-il acceptable : fichier.glv.txt ?

bref, le vrai boulot commence...

14/08/2007, 11h19
pada51

il me semble que cela aussi devrait passer :

Code:

if (strcmp(&s[j], ".glv") == 0)

vous confirmez ?

ps : pour strlen il faut inclure <string.h>, pour printf inclure <stdio.h>, pour exit inclure <stdlib.h> et cela devrait compiler sans erreur avec cette ligne :

Code:

if ((s[j] == 'g') && (s[j+1] == 'l') && (s[j+2] == 'v'))

Je confirme que oui normalement ^^

Un truc possible, sans chercher à vraiment optimiser ou quoi que ce soit, si on a inclut ce qu'il faut :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
int verifieExtension(char s[]) 
{
    int j;
 
    if (strlen(s) > 4) //Vérifie si on a au moins 5 caractères dans le nom du fichier...
    {
       for (j = 0; s[j + 3]; j++)
       {
          if (!strcmp(&s[j], ".gls"))
          {
             return 1;
          }
       }
    }
   return 0;     
}

14/08/2007, 11h26
Franck.H

Ma solution proposée plus haut ne serait-elle pas mieux !?!?

Je ne propose cependant pas le code car vu qu'il débute, il serait bon qu'il tente au moins de faire le travail lui même et de nous le proposer ici pour qu'on vois !
14/08/2007, 11h36
Beldom

Citation:

Envoyé par Franck.H

Ma solution proposée plus haut ne serait-elle pas mieux !?!?

Si en effet ^^ Mais je l'avais zappée, et je voulais reprendre le principe qu'il avait écrit au départ... Mais finalement, j'ai pas pu m'empecher de corriger plus que prévu :mouarf:

Pour le code, je sais pas, il y avait un code à la base, et pas mal de choses à redire, donc j'ai proposé quelque chose... J'aurais peut-être du m'abstenir. Ca sera pour la prochaine fois. willem77, néanmoins, je te conseille de coder ce qu'a proposer Franck.H et de le tester. Ca sera plus propre, et ca tiens en 4 ou 5 lignes...

Merci de toutes vos réponses :)

effectivement ça ne marchait pas juste à cause des double quotes... j'ai pas été malin sur ce coup là...
Sinon je n'avais pas oublié d'inclure les différents headers, après c'est sûr que le compilateur de devcpp n'est pas très verbeux...

Donc je retiens la solution de Franck.H qui est plus esthétique que celle que j'ai utilisée à la base ;)
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 #include <stdlib.h> #include <stdio.h> #include <string.h> #include <time.h> int verifieExtension(char s[]) // verifie si on passe en argument un fichier cyber .glv { char *extension; extension = strrchr(s,'.'); if ((extension[0] == '.') && (extension[1] == 'g') && (extension[2] == 'l') && (extension[3] == 'v')) { return 1; } return 0; } //------------------ MAIN --------------------- int main(int argc, char *argv[]) { // Si fichier donne en argument pas au format glv: sortie if (!(verifieExtension(argv[1]))) { printf("format de fichier non correct: .glv accepte seulement\n"); Sleep(5000); exit(EXIT_FAILURE); } return 0; }
J'incluerai par la suite les différentes optimisations que vous avez proposées et auxquelles je n'ai pas pensé!

Encore merci à tous :)

14/08/2007, 12h23
Beldom

Code:

if ((extension[0] == '.') && (extension[1] == 'g') && (extension[2] == 'l') && (extension[3] == 'v'))

Après relecture, penser à tester extension en plus.
Euh... strcmp ? Enfin, là, c'est comme tu veux, mais ca réduit la ligne à :

Code:

if (extension != NULL && !strcmp(extension, ".gls"))

Mais bon, ca change rien, juste que vala, y'a des fonctions qui le font toutes seules... Et comme ca, si tu dois changer d'extension, une macro ou une petite modif et ca marche tout seul ;)

Bon bin voici ma fonction, je ne sais pas si tu l'as fait mais la voici:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
static int verifieExtension (const char * filename, const char * extension)
{
   int ret = 0;
 
   if (filename != NULL && extension != NULL)
   {
      char * p = strrchr (filename, '.');
 
      if (p != NULL)
      {
         if (strcmp (++p, extension) == 0)
         {
            ret = 1;
         }
      }
   }
 
   return ret;
}

L'extension est ici à fournir sans le . (point) !

14/08/2007, 13h27
Melem

Code Windows : _splitpath (stdlib.h)