reconnaitre une image de fomat .ppm en C

Version imprimable

27/05/2007, 17h18
romain1986

reconnaitre une image de fomat .ppm en C

Bonjour,

Mon problème est le suivant! Je viens de terminer mon projet maintenat avec toutes les fonctions! J'aimerais quand meme améliorer mon programme encore mieux! :) Mon programme est programmé de telle facon et c'est le but qu'il ne travaille que avec des images format .ppm! Alors pour cela j'aimerais pouvoir controler l'orsque l'utilisateur entre un nom de fichier à ouvrir que ma fonction puisse reconaitre non selement si le fichier existe mais galement si le format correspond car comme ca elle seras surtout utile à la sauvegarde l'orsque l'utilisateur donnera un nouveau nom de fichier dans lequel sous lequel il sera sauvegardé! Et si il se trompe pour écrire .ppm alors il y auaas un message d'erreur!

Voici ce que j'ai à présent mais cela est très faible!

if ((fp = fopen(nom_Fichier_ouverture, "r")) == NULL)
{
printf("Erreur a l'ouverture de l'image ...\n");
exit(-1);
}
else
printf("Image ouverte avec succes ...\n");

Comme vous le voyez dans ce cas si le format serais faux il y aura quand meme un message d'erreur mais pour la sauvegarde ce système ne marche pas car un nouveau fichier doit etre créer et alors le format entré au clavier n'est pas controlé avec cette méthode! :( Sauriez vous m'aidez svp? MERCI
27/05/2007, 19h02
scriptoff

Citation:

Envoyé par specification

Each PPM image consists of the following:

1. A "magic number" for identifying the file type. A ppm image's magic number is the two characters "P6".

tu récupere les deux premiers octets
si il sont différents de P6 erreur
27/05/2007, 19h14
droggo

Fao,

Citation:

Envoyé par scriptoff

tu récupere les deux premiers octets
si il sont différents de P6 erreur

Et s'ils sont égaux à P6, il y a une chance pour que ce soit le bon format. Une chance seulement, pas une certitude. :D

si c'est l'extension du nom que tu veux vérifier, c'est simple :

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
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
 
#include <string.h>
 
.....
 
char *p=NULL ;
char *p0 =NULL ;
 
....
 
/* Cherche si il y a une extension déjà dans le nom */
 
p = strrchr ( nom_Fichier_ouverture, '.' );  
 
 
/* Si il n'y en a pas, on doit allouer de la place pour copier le nom et ajouter l'extension */
 
if ( p == NULL )
  { 
      p0 = malloc ( (strlen(nom_Fichier_ouverture)+4+1) ); /* 4 pour l'extension, 1 pour le caractère de fin de chaine */
      if ( p0 != NULL )
        {
           sprintf ( p0, "%s.ppm", nom_Fichier_ouverture );
 
            /* Met le pointeur comme étant le début du nom */
            p = p0 ;
        }
  }
/* Si l'extension existait, vérifie qu'il n'y ait pas de fautes */
else
  {
      if ( strlen(p) >= 4 ) /* Si il y a au moins 4 caractères */
        {
            if ( strcmp ( p, ".ppm" ) != 0 )  /* Si ce n'est pas la bonne */
                strcpy ( p, ".ppm" );
 
            /* Met le pointeur comme étant le début du nom */
            p = nom_Fichier_ouverture ;
        }
      else
        {
            /* L'extension est trop courte, il faut allouer de nouveau */
 
            p0 = malloc ( (strlen(nom_Fichier_ouverture)+4+1) ); 
            if ( p0 != NULL )
              {
                  sprintf ( p0, "%s.ppm", nom_Fichier_ouverture );
 
 
                 /* Met le pointeur comme étant le début du nom */
                  p = p0 ;
              }
        }
  }
 
/* Maintenant on est prêt, on a un nom de fichier complet, indiqué par p  */
 
if ( p != NULL )
  {
.............
 
     /* On libèrera la mémoire une fois qu'on a fini, si on est passé par une allocation */
 
     if ( p == p0 )
       {
          free(p0);
          p0 = NULL ;
       }
  }

27/05/2007, 19h30
scriptoff

Citation:

Envoyé par droggo

Fao,

Et s'ils sont égaux à P6, il y a une chance pour que ce soit le bon format. Une chance seulement, pas une certitude. :D

Ta remarque est intèressante mais ne correspond pas à ce qu'il demande , enfin je crois .
Ainsi il demande seulement si l'utilisateur en cas de mauvaise saisie vérifie juste si cela pourrait corresponde a un fichier .ppm .
En aucun cas il a demandé de traiter le cas ou le fichier est abimé .

Citation:

Et si il se trompe pour écrire .ppm alors il y auaas un message d'erreur!
27/05/2007, 19h31
scriptoff

500 OOPS
desolé , double poste
27/05/2007, 20h09
romain1986

rebonjour :)

MERCI beaucoup à tous! Alors ce que vous avez fait souviron34 est fabuleux! Il y a juste un problème encore! :S Je sais je suis un sacré cas mais nous en cours on a pas encore vu les malloc et je ne comprends pas ce système! N0existe t'il pas une autre méthode peut etre?

Car cela: p0 = malloc ( (strlen(nom_Fichier_ouverture)+4+1) );

je ne comprends pas du tout! :(

Ou si vous ne voyez pas d'autresméthodes sauriez vous me donner des explications suivantes?

Et que veut dire strrchr?

Est -e que signifie que l'on cherce un string?

[-mod- merci de ne pas ecorcher les pseudos... Corrigé]
27/05/2007, 20h11
romain1986

rebonjour

MERCI également à scriptoff mais le problème est que un fichier .ppm contient également très souvent P3 et pour d'abord pouvoir controler si on a P3 dans le fichier il faudrait vérifier si l'utilisateur ne s'est pas trompé en entrant le nom de fichier! :)

[-mod- merci de ne pas ecorcher les pseudos... Un peu plus d'attention STP. Corrigé]
27/05/2007, 20h14
droggo

Luo,

Citation:

Envoyé par scriptoff

Ta remarque est intèressante mais ne correspond pas à ce qu'il demande , enfin je crois .
Ainsi il demande seulement si l'utilisateur en cas de mauvaise saisie vérifie juste si cela pourrait corresponde a un fichier .ppm .
En aucun cas il a demandé de traiter le cas ou le fichier est abimé .

Je n'ai pas parlé de fichier abimé ou pas.

Rien n'interdit qu'un fichier n'ayant rien à voir avec le format désiré commence par les 2 caractères P6, alors que ton message prenait cela comme une preuve de validité. :D
27/05/2007, 20h21
romain1986

hehe ;)

hehe ben alors je m'excuse Mr! Mais c'est un truc o?u j'ai un peu de mal en ce qui concerne pour reconnaitre le format! Je cherche depuis quelques temps et je n'abouit à rien! Le reste j'ai tout

bonjour souviron34

Je viens de tout compiler et l'erreur d'avant c'était une petite betise! :) En tout cas maintenat si jemets un nom correct avec le bon format il me mrque format correct mais si fais exprès une erreur dans le format ou j'oublie le point mon programme débugge! :(

Sauriez vous m'aidez?

Voici le 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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
#include <string.h>
 
main ()
{
   char *p=NULL ;
   char *p0 =NULL ;
   char nom_Fichier_ouverture[20];
 
   printf("Entrez un nom de fichier: ");
   scanf("%s",nom_Fichier_ouverture);
   /* Cherche si il y a une extension déjà dans le nom */
   p = strrchr ( nom_Fichier_ouverture, '.' );  
 
   /* Si il n'y en a pas, on doit allouer de la place pour copier le nom et ajouter l'extension */
   if ( p == NULL )
   { 
      *p0 = malloc ( (strlen(nom_Fichier_ouverture)+4+1) ); /* 4 pour l'extension, 1 pour le caractère de fin de chaine */
      if ( p0 != NULL )
      {
         sprintf ( p0, "%s.ppm", nom_Fichier_ouverture );
         /* Met le pointeur comme étant le début du nom */
         p = p0 ;
      }
   }
   /* Si l'extension existait, vérifie qu'il n'y ait pas de fautes */
   else
   {
      if ( strlen(p) >= 4 ) /* Si il y a au moins 4 caractères */
      {
         if ( strcmp ( p, ".ppm" ) != 0 )  /* Si ce n'est pas la bonne */
            strcpy ( p, ".ppm" );
 
         /* Met le pointeur comme étant le début du nom */
         p = nom_Fichier_ouverture ;
         printf("format correct");
      }
      else
      {
         /* L'extension est trop courte, il faut allouer de nouveau */
 
         *p0 = malloc ( (strlen(nom_Fichier_ouverture)+4+1) ); 
         if ( p0 != NULL )
         {
            sprintf ( p0, "%s.ppm", nom_Fichier_ouverture );
            printf("format incorrect");
 
            /* Met le pointeur comme étant le début du nom */
            p = p0 ;
         }
      }
   }
 
   /* Maintenant on est prêt, on a un nom de fichier complet, indiqué par p  */
   if ( p != NULL )
   {
      /* On libèrera la mémoire une fois qu'on a fini, si on est passé par une allocation */
      if ( p == p0 )
      {
         free(p0);
         p0 = NULL ;
      }
   }
}

27/05/2007, 20h59
scriptoff

Citation:

Envoyé par droggo

Luo,

Je n'ai pas parlé de fichier abimé ou pas.

Rien n'interdit qu'un fichier n'ayant rien à voir avec le format désiré commence par les 2 caractères P6, alors que ton message prenait cela comme une preuve de validité. :D

Comme dirais mon prof de math :
"c'est bon , je ferme ma gueule ".

Oui quans tu alloues p0 tu as mis une étoile en trop dans :
Code:

1 2 3 4 5 6 *p0 = malloc ( (strlen(nom_Fichier_ouverture)+4+1) ); /* 4 pour l'extension, 1 pour le caractère de fin de chaine */ .... *p0 = malloc ( (strlen(nom_Fichier_ouverture)+4+1) ); ...
c'est :
Code:

1 2 3 4 5 6 p0 = malloc ( (strlen(nom_Fichier_ouverture)+4+1) ); /* 4 pour l'extension, 1 pour le caractère de fin de chaine */ .... p0 = malloc ( (strlen(nom_Fichier_ouverture)+4+1) ); ...
qu'il faut mettre....

Ensute, quand tu dis :

Citation:

Envoyé par romain1986

Et que veut dire strrchr?

En fait la fonction strchr va chercher la première occurence du caractère spécifié. La fonction strrchr va chercher la dernière occurence du caractère...

Voir les man pages :

http://www.linux-kheops.com/doc/man/...strrchr.3.html

Ensuite, quand tu dis :

Citation:

Envoyé par romain1986

Car cela: p0 = malloc ( (strlen(nom_Fichier_ouverture)+4+1) );

je ne comprends pas du tout!

strlen() est une fonction qui calcule la longueur d'une chaîne de caractères
(voir http://www.linux-kheops.com/doc/man/.../strlen.3.html).

malloc() est une fonction qui alloue un certain nombre d'octets en mémoire (voir http://www.linux-kheops.com/doc/man/.../malloc.3.html). Tu lui passe un nombre d'octes à allouer, et elle va chercher dans la mémoire du système à quel emplacement elle peut trouver tant d'octets consécutifs. Quand elle a trouvé, elle te les réserve, et te renvoie l'adresse de début.

Donc ce que je faisais dans cette ligne c'était demander un bloc qui fasse (la longueur du nom + 4 caractères (pour .ppm) + 1 (pour le caractère '\0' de fin de chaine). Et je stockais l'adresse de retour dans p0.

Maintenant, si tu n'as pas encore vu ces fonctions, mais que tu as vu les pointeurs, tu peux t'en sortir comme ceci :
a) pour la fonction strrchr :

tu peux remplacer par une boucle :
Code:

1 2 3 4 for ( i = 0 ; i < strlen(nom_Fichier_Ouverture) ; i++ ) if ( nom_Fichier_Ouverture[i] == '.' ) p = &(nom_Fichier_Ouverture[i]) ;
b) Pour la fonction malloc :

si tu as dimensionné ton nom à 20 comme dans ton programme, tu peux dimensionner une chaîne à 25.
Code:

1 2 3 4 5 6 7 8 char chaine[25] ; .... sprintf ( chaine, "%s.ppm", nom_Fichier_ouverture ); printf("format incorrect"); p0 = &chaine ;
Mais si tu n'as pas encore vu les pointeurs, ni strlen, ni strchr, ni malloc, tu peux t'en tirer comme ceci : :

A la place des p et p0, tu mettras un entier :
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 int depart = -1 ; char NouveauNom[25] ; for ( i = 0 ; i < 20 ; i++ ) { if ( nom_Fichier_Ouverture[i] == '/0' ) /* Fin de chaine */ break ; if ( nom_Fichier_Ouverture[i] == '.' ) depart = i ; } if ( depart < 0 ) /* On n'a pas trouvé d'extension */ { sprintf ( NouveauNom, "%s.ppm", nom_Fichier_Ouverture ); } else { /* On recopie le nom dans le nouveau nom */ sprintf ( NouveauNom, nom_Fichier_Ouverture ); /* Là on force à tous les coups l'extension pour être la bonne */ sprintf ( &(NouveauNom[depart+1], "%s", "ppm" ); }
NOTE : mon pseudo est souviron34 et non souvenir34 ;)

rebonjour souviron34 :-)

Tout d'abord mille excuses pour l'incompréhension dans votre nom :( Et MERCI pour tout franchement! Tout fonctionne comme je peux le constater avec le dernier système que vous m'avez proposé sans malloc et sans strlen, strchr et str!

Et bien il y a juste une chose que j'aimerais que vous me dites alors encore! :) Si ce n'est pas trop demandé! Moi j'ai vu les pointeurs mais pas les mallocs et le reste que vous avez noté! Est ce qu'il vous serait possible de me donner un code entier avec les pointeurs pour vérifier le format! Je sais que l'on vient just de voir les fonctionalités strcmp et strcpy mais pas plus!

Car cela est bien mais moi le strlen je ne connais pas Et de plus je me demandais s'il n'y avait pas plus dans le code pour vérifier le format du fichier que ces 3 lignes?
Code:

1 2 3 4 for ( i = 0 ; i < strlen(nom_Fichier_Ouverture) ; i++ ) if ( nom_Fichier_Ouverture[i] == '.' ) p = &(nom_Fichier_Ouverture[i]) ;
Et ici j'aurais aussi également des questions! :S Je dois vous déranger! :(
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 int depart = -1 ; char NouveauNom[25] ; for ( i = 0 ; i < 20 ; i++ ) { if ( nom_Fichier_Ouverture[i] == '/0' ) /* Fin de chaine */ break ; if ( nom_Fichier_Ouverture[i] == '.' ) depart = i ; } if ( depart < 0 ) /* On n'a pas trouvé d'extension */ { sprintf ( NouveauNom, "%s.ppm", nom_Fichier_Ouverture ); } else { /* On recopie le nom dans le nouveau nom */ sprintf ( NouveauNom, nom_Fichier_Ouverture ); /* Là on force à tous les coups l'extension pour être la bonne */ sprintf ( &(NouveauNom[depart+1], "%s", "ppm" ); }
Mais vous notez toujours sprintf je vois! Que signifie cela! ET je voulais également demandé s'il était possible de me dire pourquoi quand je mais au lien de sprintf le printf que j'ai de droles de symboles comme réponse dans la fenêtre DOS! Est ce que je pourais noter au lieu de
Code:

1 2 3 4 5 if ( depart < 0 ) /* On n'a pas trouvé d'extension */ { sprintf ( NouveauNom, "%s.ppm", nom_Fichier_Ouverture ); }
comme ceci pour avoir un message à l'écran au moins que me dise si le format est correct ou pas
Code:

1 2 3 4 5 6 if ( depart < 0 ) /* On n'a pas trouvé d'extension */ { sprintf ( NouveauNom, "%s.ppm", nom_Fichier_Ouverture ); printf("pas d'extension alors on a ajouté .ppm"); }
et donc ici sous le dernier sprintf je pourrais ajouter
Code:

1 2 3 4 5 6 /* Là on force à tous les coups l'extension pour être la bonne */ sprintf ( &(NouveauNom[depart+1], "%s", "ppm" ); printf("format correct"); }
Et encore une dernière chose si vous pourriez m'expliquez encore un peu plus en détail comment fonctionne votre programme! Il est vraiment tres bien et je comprends presque tout mais juste un peu de problème pour comprendre ces plusieures lignes là! Car je vais devoir présenter le projet devant une classe à l'unif et oui je tiens à bien comprendre ce qui se passe étape par étape! Car le truc du sprintf me travaille un peu! Car si j'enlève le s devant alors il m'affiche 2 caractères bizarres à l'écran et oui ce qui est noté derrière je ne suis pas certain d'avoir compris à 100% pour cela si vous auriez encore un tout petit peu de temps à consacrer pour moi je serais trop content :lol:

MERCI déjà d'avance pour tout ce que vous avez fait

[-mod- Attention à utiliser correctement les balises de code. C'était l'embrouille, j'ai tout remis droit...]

28/05/2007, 13h57
Emmanuel Delahaye

Citation:

Envoyé par souviron34

NOTE : mon pseudo est souviron34 et non souvenir34 ;)

J'avais vu ça. J'ai fait les corrections.
28/05/2007, 14h23
crocodilex
Citation:
Envoyé par souviron34

tu peux remplacer par une boucle :

Code:

1 2 3 4 for ( i = 0 ; i < strlen(nom_Fichier_Ouverture) ; i++ ) if ( nom_Fichier_Ouverture[i] == '.' ) p = &(nom_Fichier_Ouverture[i]) ;
Je ne sais pas à quel point l’optimiseur du compilateur est intelligent, mais je déconseille fortement de mettre l’appel d’une fonction dans la condition de sortie de la boucle for.
Il serait plus judicieux de passer par une variable intermédiaire :
Code:

1 2 3 4 5 6 int sz = strlen(nom_Fichier_Ouverture); for ( i=0; i<sz; i++ ) { }
28/05/2007, 15h00
souviron34
Citation:
Envoyé par crocodilex

Je ne sais pas à quel point l’optimiseur du compilateur est intelligent, mais je déconseille fortement de mettre l’appel d’une fonction dans la condition de sortie de la boucle for.
Il serait plus judicieux de passer par une variable intermédiaire :

Code:

1 2 3 4 5 6 int sz = strlen(nom_Fichier_Ouverture); for ( i=0; i<sz; i++ ) { }
c'est vrai, mais vu le niveau, on n'en est pas à parler d'optimisation..... ;)

Vu que le PO n'a même pas encore vu strlen de toutes façons... :aie:
- Citation:
  
  Envoyé par romain1986
  
  ...
  Car cela est bien mais moi le strlen je ne connais pas Et de plus je me demandais s'il n'y avait pas plus dans le code pour vérifier le format du fichier que ces 3 lignes?
  
  Code:
  
  1 2 3 4 for ( i = 0 ; i < strlen(nom_Fichier_Ouverture) ; i++ ) if ( nom_Fichier_Ouverture[i] == '.' ) p = &(nom_Fichier_Ouverture[i]) ;
  Ceci ne vérfiie pas le format du fichier.. Cela fait juste stocker (si cela existe) l'adresse du caractère contenant un point (si il y en a plusieurs cela stockera le dernier).
  
  Le reste du code donné en premier restait inchangé, sauf à remplacer les malloc par les lignes indiquées dans le même paragraphe.
  
  La fonction strlen renvoie la longueur de la chaine. Si tu ne l'a pas vu, cela revient à faire ce que je faisais dans la dernière boucle :
  Code:
  
  1 2 3 4 5 6 7 8 9 int longueur=0 ; for ( i = 0 ; i < 20 ; i++ ) if ( nom_Fichier_Ouverture[i] == '\0' ) { longueur = i ; break ; }
- la fonction sprintf est exactement la même que printf, exepté qu'elle écrit dans une chaine de caractères.... Et fprintf est exactement la même chose, excepté qu'elle écrit dans un fichier.
- Pour vérifier tu peux effectivement mettre les printf tels que tu les as mis, mais je rajouterais à la fin de la chaine écrite un '\n' (retour à la ligne), qui force le printf à être écrit (cette instruction est bufferise et l'impression à l'écran n'est pas forcément synchrone avec le passage sur l'instruction. Donc un retour à la ligne "force" (à priori) l'écriture à l'écran. Donc quelque chose comme :
  
  Code:
  
  printf("pas d'extension alors on a ajouté .ppm\n");
- tu as oublié des parenthèses par rapport à ce que j'avais mis :
  
  Code:
  
  sprintf ( &(NouveauNom[depart+1]), "%s", "ppm" );
- Le principe général est le même dans les 3 cas exposés :
  - On cherche si il y a un point dans la chaine , et si oui on cherche le dernier
  - 2 cas peuvent se présenter :
    
    Il n'y a pas de point : l'utilisateur n'a pas rentré d'extension
    Il y a une extension.
  - Si il n'y a pas d'extension :
    
    Si on est avec des pointeurs, on alloue assez de place en mémoire pour stocker le nom ET l'extension. Si l'allocation a réussi on stocke le nom donné par l'utilisateur et on y ajoute l'extension.
    Si on n'est pas avec des pointeurs, on recopie le nom entré dans une chaine plus grande, et on lui ajoute l'extension.
  - Si l'extension existe, il faut vérifier si c'est la bonne, ou s'il y a une faute :
    
    Si on est avec des pointeurs, on vérifie d'abord si c'est la bonne. Si il y aune faute, on vérifie si elle est assez longue pour qu'on puisse corriger sans dépasser la longueur. Si oui, simplement on remplace par la bonne extension. Sinon il faut demander plus de mémoire, copier le nom, et remplacer l'extenson.
    Si on est pas avec des pointeurs, simplement on recopie le nom dans une chaine plus grande et on force l'extension à ce qu'on veut.
  - Ensuite on se sert du nom ainsi généré.
  - Si on est avec des pointeurs ET qu'on a alloué de la place mémoire, on libère cette place quand on ne se sert plus de la chaine.