Discussion :

[ChPr]

Bonjour à toutes et à tous,

Je cherche à convertir un String en char* avec la méthode suivante, ça compile, mais apparemment, cela ne fonctionne pas. Quel peut être le problème ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
char* strToChar(String s) {
  char buf[32];
  int n = s.length();
  for (int i = 0; i < n; i++)
    buf[i] = s[i];
  s[n] = '\0';
  return buf;
}

NOTA : développant pour un système Alduino, je n'ai pas accès à la méthode : string.c_str()

Merci de votre aide.

Pierre

[Ehonn]

Bonjour

buf est créée dans la fonction sur la pile et est donc supprimée à la fin de la fonction. Le pointeur (char *) retourné pointe donc sur un espace mémoire invalide. L'accès à cette adresse provoque un comportement indéterminé (le plus souvent une erreur de segmentation).

Comme tu n'a pas accès aux méthodes c_str ou data (il devrait y avoir un équivalent...), tu peux :
- passer ton tableau buf en paramètres de la fonction
- faire une allocation dynamique dans la fonction
- écrire ta propre classe string et jetter celle que tu utilises qui à l'air toute pourrie (ce que je recommande le plus)

Actuellement ta fonction est buggée :
- que ce passe-t-il si la taille de la chaîne de caractère est supérieure ou égale à 32 ?

Un petit lien utile autour de l'allocation dynamique : FAQ C++ - Gestion dynamique de la mémoire

Edit : Si tu as les méthodes begin et end, tu peux recopier ton string en vector<char> avec le constructeur et récupérer un char * const depuis le vector (attention à \0).

[Médinoc]

Tu veux dire Arduino? Convertir une String Arduino en tableau de char?

Je n'ai jamais essayé cette plate-forme, mais en voyant la méthode toCharArray(), j'essaierais ceci:

Code C++ :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
char* strToChar(String s) {
	unsigned int bufSize = s.length() + 1; //String length + null terminator
	char* ret = new char[bufSize];
	s.toCharArray(ret, bufSize);
	return ret;
}

[ChPr]

Je vous remercie pour ces réponses. Pour autant, je me pose des questions.

La méthode proposée par Medinoc fonctionne. Or, Ehonn a dit :

... buf est créée dans la fonction sur la pile et est donc supprimée à la fin de la fonction. Le pointeur (char *) retourné pointe donc sur un espace mémoire invalide. L'accès à cette adresse provoque un comportement indéterminé (le plus souvent une erreur de segmentation). ...

Cela veut donc dire que, dans ma méthode proposée par Medinoc :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

char* ret = new char[bufSize];

n'est pas supprimé quand la fonction se termine.

Alors, si j'appelle 10000 fois la fonction, autant de fois est créée la variable "ret". Ce qui risque, à la longue, de me saturer la mémoire ???

Cordialement.

Pierre

[Ehonn]

En passant : «Oui».
Grosso modo, un new, un delete.

Avec ce genre de fonction c'est la fonction qui fait le new, et l'utilisateur qui fait le delete :s
C'est assez sale en C++. On préfère : «celui qui fait le new, fait le delete». Tu peux te renseigner sur le RAII.

[Médinoc]

Oui, une fois que tu n'as plus besoin du buffer, il faut le supprimer avec delete[].

Un new, un delete, et un new[], un delete[].

..Sinon, il doit y avoir l'option de faire la même chose avec un std::vector<char> au lieu d'un tableau dynamique; le nettoyage sera automatique.

Probablement un truc de ce genre (non-testé!)

Code C++ :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
vector<char> strToChar(String s) {
	unsigned int bufSize = s.length() + 1; //String length + null terminator
	vector<char> ret(bufSize);
	s.toCharArray(&ret[0], bufSize);
	return ret;
}

[ChPr]

Oui, une fois que tu n'as plus besoin du buffer, il faut le supprimer avec delete[].

Un new, un delete, et un new[], un delete[]. ..

Oui, mais si j'ai fait cette remarque, c'est parce que je ne vois pas comment faire un delete sur la variable "ret" de l'extérieur de la fonction. Et si je le fais de l'intérieur, je retombe sur la remarque de Ehonn ??

Cordialement.

Pierre