Malloc et affectation de mémoire

Version imprimable

Bonjour,

J'ai un petit soucis quant à l'allocation de mémoire pour un char* avec l'utilisation de malloc.
Voici mon code (bien allégé) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 char *fichier; int taille =1504; if(fichier = (char*)malloc(taille)) { len = strlen( fichier ); printf("Taille = %i \n", len); ... //mon code }
Lorsque je fais mon printf, la taille affichée est 1520 (alors que cela devraot etre 1504, non?).
De plus, lorsque j'affiche ma chaine "fichier" non remplie, j'obtiens en fin de chaine " ²²²²½½½½½½½½¯■¯■ " qui correspondent à ces 16 car en trop...

Comment rectifier le problème et faire en sorte que fichier ne contienne que 1504 char sans les indésirables de la fin ?

Merci:)

04/04/2007, 15h01
ep31

La variable fichier n'est pas initialisée après le malloc donc le strlen(fichier) n'a aucun sens.
Il faudrait affecter une valeur à la variable fichier pour que le calcul de sa taille ait un sens
04/04/2007, 15h02
lun4t1k

char fichier[1504]?
04/04/2007, 15h03
Laurent Gomila

Tu es sûr que tu fais du C++ ? Tout ça c'est du pur C.
04/04/2007, 15h17
Chop_chop
arghh,
oui, c'est vrai que je jongle entre c et c++, désolé.
Mais dans le cas ou j'utilise :
Code:

1 2 char *fichier = new char[1504]; // 1504, ca pourrait etre un autre nombre, c'est juste que je veux mettre bcp de char...
au lieu de malloc, ca me fait le meme probleme...
04/04/2007, 15h20
Chop_chop

Citation:

Envoyé par ep31

La variable fichier n'est pas initialisée après le malloc donc le strlen(fichier) n'a aucun sens.
Il faudrait affecter une valeur à la variable fichier pour que le calcul de sa taille ait un sens

Soit, mais vu que l'espace mémoire est allouée, cela devrait suffire. Dans mon cas ca m'affiche bien, à peu de chose pret (et c'est ce "peu" qui me dérange!!!), ma taille de 1504 (enfin, cela affiche 1520 mais c'est 1504 que je veux :oops:).
04/04/2007, 15h22
lun4t1k

pourquoi pas simplement char fichier[1504] ? (meme si ce n'est pas le pb!)
sinon bzero(&fichier,sizeof(fichier)); m'a évité des trucs... bizarres!
04/04/2007, 15h23
Laurent Gomila

Citation:

Soit, mais vu que l'espace mémoire est allouée, cela devrait suffire

strlen ne fait que parcourir la chaîne jusqu'à rencontrer un 0. Vu que ta chaîne contient n'importe quoi, le résultat est lui aussi n'importe quoi.

Si ta question est "comment savoir quelle taille est allouée au bout d'un pointeur" alors la réponse est que ça n'est pas possible directement.

Si tu fais vraiment du C++ alors tu devrais utiliser std::string ou std::vector.

Citation:

pourquoi pas simplement char fichier[1504] ?

Parce que

Citation:

// 1504, ca pourrait etre un autre nombre, c'est juste que je veux mettre bcp de char...
04/04/2007, 15h24
Ulmo

Il y a n'importe quoi dans ton tableau, et strlen cherche le premier '\0' dedans. Si tu n'as pas de bol, il n'y en a aucun dans tout ton tableau.

Tu confondrais pas sizeof et strlen ?
04/04/2007, 15h24
Chop_chop

Citation:

Envoyé par lun4t1k

pourquoi pas simplement char fichier[1504] ?

:oops: C'est simple et j'y avais pas songé... je vais essayé cela ! Merci de m'avoir un peu raffraichi la mémoire lol

Edit : Finalement, comme la di Laurent Gomila, faut mieux pas alors....
04/04/2007, 15h29
Chop_chop

Citation:

Envoyé par Ulmo

Il y a n'importe quoi dans ton tableau, et strlen cherche le premier '\0' dedans. Si tu n'as pas de bol, il n'y en a aucun dans tout ton tableau.

Tu confondrais pas sizeof et strlen ?

HEu non, je ne pense pas confondre, strlen permet d'obtenir la taille d'un string donc c'est bien ca...
04/04/2007, 15h33
Laurent Gomila

Tu as l'air un peu perdu... tu veux faire quoi au juste avec ce tableau ?
04/04/2007, 15h35
Chop_chop
Bon, je vais m'obstiner une peu (désolé :( )en changeant légèrement mon code :
Code:

1 2 3 4 int taille2 = 1500; char *fichier2 = new char[taille2]; cout<<"taille :"<< strlen( fichier2 );
J'ai donc d'affiché "taille :1512" ... Je me demande toujours si le résultat est normal...
04/04/2007, 15h38
Ulmo
Tu n'as rien changé, le contenu de ton tableau n'est toujours pas initialisé.
Essaye comme ça pour voir :
Code:

1 2 3 4 int taille2 = 1500; char *fichier2 = new char[taille2]; fichier2[0] = '\0'; cout<<"taille :"<< strlen( fichier2 );
04/04/2007, 15h38
Laurent Gomila

Citation:

J'ai donc d'affiché "taille :1512" ... Je me demande toujours si le résultat est normal...

Qu'est-ce qui est incompréhensible dans "strlen cherche un 0, or ta chaîne contient n'importe quoi" ?
04/04/2007, 15h39
Chop_chop

Citation:

Envoyé par Laurent Gomila

Tu as l'air un peu perdu... tu veux faire quoi au juste avec ce tableau ?

Et bien, je vais stoquer un certain nombre de char en provenance d'un fichier.
pour cela il me faut une variable pour les stoquer doù mon char* fichier.
Mais comme , je ne vais pas demander toujours le meme nombre de char(parfois 1500, parfois 150...), il faut que cela soit legerement flexible.

Enfin, puisque , à la fin je veux recuperer les chars stoquer dans ma variable pour les afficher. Et si , il y en a 1512 (au lieu de 1500 du a une mauvaise allocation), et bien cela m'affichera 12 char de trop...

J'espere que je suis pas trop désordonné dans mes explications et que tu arrives a me suivre
04/04/2007, 15h40
Ulmo

Citation:

Envoyé par Chop_chop

HEu non, je ne pense pas confondre, strlen permet d'obtenir la taille d'un string donc c'est bien ca...

Et où as-tu vu un string ? Il n'y en a pas dans ton code.
04/04/2007, 15h40
Chop_chop

Citation:

Envoyé par Laurent Gomila

Qu'est-ce qui est incompréhensible dans "strlen cherche un 0, or ta chaîne contient n'importe quoi" ?

Oyui, mais elle ne devrait pas contenir plus de 1500 char, non ? et elle me di que cela fait 1512... (désolé d'etre autant borné, je ne comprend pas trop)
04/04/2007, 15h42
Chop_chop

Citation:

Envoyé par Ulmo

Et où as-tu vu un string ? Il n'y en a pas dans ton code.

heu char* peut etre considéré comme un string pour certaines fonctions, non?
Désolé si j'affirme des choses qui sont fausses si c'est le cas.
04/04/2007, 15h47
Ulmo
Citation:

Envoyé par Chop_chop

heu char* peut etre considéré comme un string pour certaines fonctions, non?
1. En C++ les chaines de caractères sont des std::string et pas des char *
2. Tant qu'il n'y a pas de '\0' terminal, ce n'est pas une chaine valide
04/04/2007, 15h48
Laurent Gomila

Citation:

Et bien, je vais stoquer un certain nombre de char en provenance d'un fichier.

Dans ce cas comme je te l'ai dit, utilise std::vector ou std::string (selon l'utilisation). Il y a des exemples dans la FAQ.

Citation:

heu char* peut etre considéré comme un string pour certaines fonctions, non?

Un tableau de char terminé par un '\0' est considéré comme une chaîne en C.
En C++, c'est std::string.

Citation:

Oyui, mais elle ne devrait pas contenir plus de 1500 char, non ? et elle me di que cela fait 1512... (désolé d'etre autant borné, je ne comprend pas trop)

new t'a alloué 1500 caractères, mais si strlen ne trouve pas de 0 dans ces 1500 caractères il va continuer (et lire une zone mémoire non définie, d'où un comportement indéfini).
04/04/2007, 15h53
Chop_chop

Merci Laurent,
Je vais utilisé std::string et revenir pour dire si cela fonctionne bien.
Désolé d'avoir pris la tête a tt le monde...:mur: mais avec tes explications, j'ai compris ce qui clochait dans mon code. merci bien

Code:

1
2
3
4
5
6
7
 
int rc=read (Fichier, &buf, sizeof(buf));
  if (rc == -1) 
    {
      perror("read(rc)");
      exit(1);
    }

plus simple et met le contenu de fichier dans buf (assez grand)

Par exemple,

Code:

1
2
std::ifstream ifs("fichier.txt");
std::string s((std::istream_iterator<char>(ifs), std::istream_iterator<char>()));

Ou plutôt

Code:

1
2
std::ifstream ifs("fichier.txt", std::ios_base::binary);
std::string s((std::istreambuf_iterator<char>(ifs), std::istreambuf_iterator<char>()));

Code:

1
2
std::ifstream ifs("fichier.txt", std::ios_base::binary);
std::string s((std::istreambuf_iterator<char>(ifs)), std::istreambuf_iterator<char>());

Qui dit mieux?

05/04/2007, 14h38
lun4t1k

Ca devient crade, ca joue avec les parenthèses... :aie:
05/04/2007, 14h42
Jean-Marc.Bourguet

Citation:

Envoyé par lun4t1k

Ca devient crade, ca joue avec les parenthèses... :aie:

Elles sont necessaires -- la bonne vieille difference entre declaration et definition...

Ou alors...

Code:

1
2
3
4
std::ifstream ifs("fichier.txt", std::ios_base::binary);
std::istreambuf_iterator<char> begin(ifs);
std::istreambuf_iterator<char> end;
std::string s(begin, end);

06/04/2007, 10h07
Chop_chop

Ca m'en fait des choix ;)
06/04/2007, 10h38
smashy

essaie :

fichier2[666] = 0;

le resultat DEVRAIT etre 666 (il peut etre inferieur si ton buffer contient un 0 avant celui que l on vient d ajouter)
06/04/2007, 10h39
smashy

oups, pardon , j'avais pas vu qu'il y avait 3 pages sur le forum ...

Code:

1 2 3 4 5 6 7 8 9 10 11 ifstream fichier("monFichier.txt"); if (!fichier.good()){ // oups :) } vector<string> lignes; string ligne; while ( getline( fichier, ligne ) ){ lignes.push_back(ligne); }
<edit> c'est mochard , il ya un moyen de préciser le type de code ?
<edit2>
fucking guillemet :) (merci au passage)

06/04/2007, 11h05
poukill

t'as oublié un guillemet :D

Fuseau horaire GMT +2. Il est actuellement 19h49.