Passage de pointeur en paramètre d'une fonction et réallocation de mémoire.

Bonjour,

Je souhaite:

- Passer un pointeur sur une chaîne de caractère à une fonction,
- Modifier cette chaîne de caractère en en augmentant éventuellement le
nombre de caractère (donc réallocation de mémoire),
- Puis modifier le pointeur de façon à ce qu'il pointe bien sur cette nouvelle
chaîne.

J'utilise ceci comme en-tête de fonction:

Citation:

---

int Fonction(char * InputData);

---

Mon problème est que je parviens bien à modifier InputData à l'intérieur de ma
fonction, mais que le pointeur qui a été passé en paramètre de cette fonction
pointe toujours sur l'ancienne adresse. Donc quand j'essaie de l'utiliser à
l'extérieur de ma fonction, je me retrouve avec les anciennes données.

Donc ma question comment modifier la valeur d'un pointeur passé en
paramètre d'une fonction?

Il suffit de passer ton pointeur par référence et non pas par valeur :

Code:

---

int Fonction(char *& InputData);

---

(Plus d'explications dans la FAQ)

Utilise donc std::string

Code:

---

int Fonction(std::string& InputData);

---

stillman:

Quand j'utilise cette solution j'ai l'impression que le compilateur considère la
zone pointée par InputData comme libre. Il l'utilise pour écrire d'autres
données alors que j'ai encore besoin de celles présentes (avant même que je
ne fasse delete InputData). J'ai pû le vérifier au débuggeur après avoir
constaté un segmentation fault.


Laurent:

J'essaie de créer une fonction la plus générique possible, qui puisse prendre
n'importe quel type de donnée en entrée. C'est pour ça que j'utilise le char *,
il me semble qu'il s'agit du format le plus libre.

Peux-tu poster le code de ta fonction ? Ce sera probablement plus facile de te conseiller.

Citation:

---

Envoyé par stillman

Peux-tu poster le code de ta fonction ? Ce sera probablement plus facile de te conseiller.

---

Voici ma fonction, expurgée des éléments inutiles (UnicodeString est un type propriétaire):

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

int Function(char * & InputData, unsigned long int & DataSize) { // Extract input data   UnicodeString data(InputData, DataSize); int length = data.SearchSubString("!"); UnicodeString filepath; filepath = data.SubString(0, length); UnicodeString position; position = data.SubString(length + 1, DataSize - (length + 1)); unsigned long int pos; position.ConvertToNum(&pos, "%u");   //**     /***    Modification de data et de pos. ***/     // Create output data   position.BuildFromNum(pos, "%u");   data.Remove(length + 1, DataSize - (length + 1)); data.Append(position); DataSize = data.GetLengthInByte();   delete InputData;     InputData = (char *) data.ConvertToChar();   //**   return 0; }

---

InputData est composé de "Filepath" concaténé à "position", séparés par le caractère '!'.

Code:

---

InputData = (char *) data.ConvertToChar();

---

data étant une variable locale, il y a de grandes chances que InputData pointe sur n'importe quoi dès la sortie de la fonction.

Citation:

---

Envoyé par Laurent Gomila

Code:

---

InputData = (char *) data.ConvertToChar();

---

data étant une variable locale, il y a de grandes chances que InputData pointe sur n'importe quoi dès la sortie de la fonction.

---

Au contraire, la valeur du pointeur retourné est la même que celle à l'intérieur
de la fonction. Mon problème est que, si après avoir affecté InputData par:

Code:

---

InputData = (char *) data.ConvertToChar();

---

j'effectue une fonction d'affichage comme:

Code:

---

TRACE_C("toto");

---

je me retrouve avec tout ou partie de "toto" dans la chaîne pointée par
InputData, comme si le compilateur l'avait utilisé comme registre tampon.

Problème qui n'apparaît pas si je ne passe pas InputData par référence (mais
alors la valeur du pointeur n'est pas modifiée au retour de la fonction).

Citation:

---

Au contraire, la valeur du pointeur retourné est la même que celle à l'intérieur de la fonction

---

Ca ça ne fait aucun doute. Je dis juste qu'il y a de fortes chances que les octets qui se trouvent à cet emplacement mémoire aient été détruits (par le destructeur de UnicodeString).

Et si je remplace :

Code:

---

1 2 3

delete InputData;   InputData = (char *) data.ConvertToChar();

---

par:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

char * tempInputData = (char *) realloc(InputData, DataSize * sizeof(char));   if ( tempInputData == NULL ) { free(InputData); return 1; } else { InputData = tempInputData; }   for (int i = length + 1; i < DataSize; i++) { InputData[i] = data.ConvertToChar()[i]; }

---

Est-ce que cela résoudrait le problème?

J'ai pas regardé les détails mais l'idée est là, oui. Il faut que InputData ait son propre espace mémoire.

Citation:

---

Envoyé par Laurent Gomila

J'ai pas regardé les détails mais l'idée est là, oui. Il faut que InputData ait son propre espace mémoire.

---

C'est étrange, le programme plante à l'exécution de realloc au niveau de ce test-ci :? :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

int __cdecl __sbh_resize_block (PHEADER pHeader, void * pvAlloc, int intNew) {   /***/   // test if new size is larger than the current one if (sizeNew > sizeEntry) {   /***/   }   /***/   }

---

Voilà l'état de la stack:

Citation:

---

> msvcr80.dll!__sbh_resize_block(tagHeader * pHeader=0x02c994c8, void * pvAlloc=0x02df1630, int intNew=111) Line 1096 + 0x6 bytes C
> msvcr80.dll!realloc(void * pBlock=0x02df1630, unsigned int newsize=111) Line 102 + 0x8 bytes C

---

Tu ne peux pas plutôt utiliser std::vector<char> ?

Citation:

---

The realloc function changes the size of an allocated memory block. The memblock argument points to the beginning of the memory block. If memblock is NULL, realloc behaves the same way as malloc and allocates a new block of size bytes. If memblock is not NULL, it should be a pointer returned by a previous call to calloc, malloc, or realloc.

---

Ton realloc plante ? ton inputdata a-t-il été alloué par malloc et autres ?

Une impression perso :
Pas trés lisible de mélanger du malloc/free et du new/delete dans une même fonction ...

Citation:

---

Laurent:

J'essaie de créer une fonction la plus générique possible, qui puisse prendre
n'importe quel type de donnée en entrée. C'est pour ça que j'utilise le char *,
il me semble qu'il s'agit du format le plus libre.

---

vu ce que tu as à faire, je ne saurais te conseiller comment Laurent : utiliser des std::string ...

Citation:

---

Envoyé par olive_le_malin

ton inputdata a-t-il été alloué par malloc et autres ?

---

Je ne sais pas, il faudrait que je regarde dans le code de UnicodeString.

Citation:

---

Envoyé par olive_le_malin

Une impression perso :
Pas trés lisible de mélanger du malloc/free et du new/delete dans une même fonction ...?

---

C'est pour ça que j'ai remplacé new/delete par malloc/free, donc pas de conflit. ;)

Citation:

---

Envoyé par olive_le_malin

vu ce que tu as à faire, je ne saurais te conseiller comment Laurent : utiliser des std::string ...

---

J'ai fini par suivre vos conseil et par remplacer char* par UnicodeString dans l'en-tête de la fonction. Il faudra que j'adapte mon code en conséquence.


Merci de votre aide.