Discussion :

[oeil de lynx]

Bonjour tout le monde,

Je suis en train d'écrire un programme qui ouvre un fichier qui contient du texte.
Ensuite je met chaque caractere du texte dans un tableau pour apres recuperer les valeurs hexadecimales de chaque caractere car j'en ai besoin pour faire des opérations apres.

On m'a conseillé d'utiliser malloc() pour allouer la bonne taille. Le probleme est que cette ligne me cause un "Project.exe a renconreé un probleme et doit fermer."

Je vous donne mon code et donnez moi votre avis, svp!

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
 
unsigned long n ;
 char *p;
 
 FILE *fp = fopen ("plaintext.txt", "r");
   if (fp != NULL)
   {
      fseek (fp, 0, SEEK_END);
      {
         n = ftell (fp);
         rewind (fp);
         printf("n\%d",(size_t)-1);
 
         if (n < (size_t) - 1)
         { 
            *p = (char) malloc (n);
 
            if (p != NULL)
            {
               fread (p, n, 1, fp);
 
               {
                  size_t i;
                  for (i = 0; i < n; i++)
                  {
                     putchar (p[i]);
                  }
               }
               free (p);             
             }
         }
         fclose (fp); 
      }
   }

[contremaitre]

Ton malloc va te renvoyer un pointeur vers une zone mémoire contenant des char.
Donc c'est (char*) et pas (char)
Et il vaut mieux ne rien mettre du tout de toute façon devant le malloc

[oeil de lynx]

Si je met rien devant malloc il me dit : "invalid conversion from 'void*' to 'char'".
et si je met (char *) il me dit :" invalid conversion from 'char *' to 'char'" ...

[contremaitre]

oui pardon j'avais pas vu, c'est p = malloc ...
et pas *p

p permet d'accéder à l'adresse, et *p à la valeur
Or le malloc t'alloue une adresse.

[oeil de lynx]

Ah oui en effet, ca parait plus logique.
Par contre j'ai toujours une erreur : "invalid conversion from 'void*' to 'char*'

[Melem]

Par contre j'ai toujours une erreur : "invalid conversion from 'void*' to 'char*'

Ca veut dire que l'extension de ton fichier est .cpp (C++) ou que tu compiles en C++. L'extension d'un fichier C est .c.

[oeil de lynx]

Ohlala exact.... j'avais pas fait attention
du coup, apres avoir fait un nouveau projet en C (et non en c++ comme j'avais du le faire), une autre erreur m'est indiquée:
iostream:no such file or directory, c'est aussi du C++?je suis perdue !

[Melem]

D'autre part, ftell retourne long et non unsigned long et le printf après le rewind est mal écrit. \n et non n\ et %d attend un int et non un size_t (pour afficher un size_t, C99 uniquement, c'est %z si je me souviens bien). Si tu veux convertir en non signé, unsigned int ou unsigned long par exemple, c'est respectivement %u et %lu qu'il faut utiliser.

[Melem]

Ohlala exact.... j'avais pas fait attention
du coup, apres avoir fait un nouveau projet en C (et non en c++ comme j'avais du le faire), une autre erreur m'est indiquée:
iostream:no such file or directory, c'est aussi du C++?je suis perdue !

Oui iostream c'est du C++ et même en C++ tu n'en as pas besoin ici. Le fichier d'en-tête qui déclare printf, fopen, fread, etc. est stdio.h (C), pas iostream (C++).

[oeil de lynx]

non en effet, ce n'est pas du C
donc mes bibliotheques sont :
#include<stdio.h>
#include <stdlib.h>

Le probleme maintenant est que la fenetre du compilateur s'ouvre et se ferme directement...

[oeil de lynx]

Merci pour les erreurs trouvées c'est corrigé !!

[Médinoc]

Ce ne sont pas des bibliothèques mais des fichiers d'en-tête.

Que veux-tu dire par "fenêtre du compilateur" ? La boîte de dialogue de Dev-C++ ou la fenêtre console du programme?

[oeil de lynx]

La console de Devc++, c'est à dire quand je clique sur 'run' ! Je m'exprime pas tres bien, dslee...

[Médinoc]

C'est dû à un bug de Dev-C++ qui n'empêche pas la console de se fermer naturellement quand on clique sur Run.

Durant la phase de développement, tu peux contourner le bug en ajoutant system("pause"); à la fin du programme. Il sera sans doute approprié de retirer cette ligne de la version définitive.

[oeil de lynx]

Oui si je met des system("PAUSE"); la du coup ca ouvre la console. merci

J'ai encore une question ! Par rapport a mon code, je rentre donc les caracteres dans p[i] et imaginons que je veuille afficher ce que contient p[i], est ce que je peux faire :

printf("resultat :\n");
for(i=0;i<n;i++)
{
printf("%O2x ",p[i]);
}

[Médinoc]

Oui, mais il faut mettre un zéro à la place du O.

[oeil de lynx]

C'est bon, tout marche ! Je vous remercie !!