problème avec malloc

Version imprimable

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:

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); 
      }
   }

03/04/2008, 15h25
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
03/04/2008, 15h30
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'" ...
:?
03/04/2008, 15h35
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.
03/04/2008, 15h39
oeil de lynx

Ah oui en effet, ca parait plus logique.
Par contre j'ai toujours une erreur : "invalid conversion from 'void*' to 'char*'
03/04/2008, 15h43
Melem

Citation:

Envoyé par oeil de lynx

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.
03/04/2008, 15h52
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 !
03/04/2008, 15h53
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.
03/04/2008, 15h55
Melem

Citation:

Envoyé par 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 !

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++).
03/04/2008, 15h57
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...
03/04/2008, 16h01
oeil de lynx

Merci pour les erreurs trouvées c'est corrigé !! :)
03/04/2008, 16h01
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?
03/04/2008, 16h03
oeil de lynx

La console de Devc++, c'est à dire quand je clique sur 'run' ! Je m'exprime pas tres bien, dslee...
03/04/2008, 16h07
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.
03/04/2008, 16h13
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]);
}
03/04/2008, 16h23
Médinoc

Oui, mais il faut mettre un zéro à la place du O.
03/04/2008, 16h24
oeil de lynx

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