erreur de concaténation de deux chaines

Bonjour,
j'ai 3 double que je veux les concaténer, je l'ai donc converti en string avec sprintf et jusqu'à maintenant j'affiche mais string et c'est bon.
Ensuite j'ai concténer ces chaines en utilisant strcat mais j'ai eu une chaîne qui contient une première partie (du n'importe quoi) et une deuxième partie(mes 3 doubles concaténés)
Je développe avec VS2008

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

char pos[50], str_X[20], str_Y[20], str_Z[20];   sprintf(str_X, "%f", x); std::cout<<"x "<<x<<std::endl; sprintf(str_Y, "%f", y); std::cout<<"y "<<y<<std::endl; sprintf(str_Z, "%f", z); std::cout<<"z "<<z<<std::endl;   std::cout<<"pos "<<pos<<std::endl; strcat(pos,str_X); std::cout<<"pos "<<pos<<std::endl; strcat(pos,", "); std::cout<<"pos "<<pos<<std::endl; strcat(pos,str_Y); strcat(pos,", "); std::cout<<"pos "<<pos<<std::endl; strcat(pos,str_Z); drawString(pos); std::cout<<"pos "<<pos<<std::endl;

---

voici un exemple de la chaîneaffichée

Code:

---

ö*^3.333332,5.447821,9.776854

---

Merci pour vos aides

Lai,

Avant tout appel de srtcat(Pos,...), il faut penser à initialiser Pos comme une chaîne vide (Pos[0] = 0).

Ce problème ne se pose pas pour les autres, car les fonctions utilisées se chargent de considérer que la chaîne est vide, ce qui bien entendu n'est pas le cas de strcat, puisqu'elle sert à ajouter quelque chose à la fin de la chaîne. :)

Merci droggo :)

Du C++, des appels à sprintf et strcat ?

Ah, nous ne sommes pas dans le même monde :)

Code:

---

1 2 3 4 5 6

#include <sstream>   std::stringstream s; s << x << "," << y << "," << z; std::cout << "xyz = " << s.str() << std::endl;

---

* std::string
* std::stringstream

sont tes amis, fait leur confiance ! :)

Ils sont plus lents, tout de même.

Citation:

---

Envoyé par oodini

Ils sont plus lents, tout de même.

---

Oui, mais s'ils avaient été utilisés, il n'y aurais pas eu de crash :)

Ensuite, il sont plus lents donc à éviter dans le cas de fonctions critiques, mais quel pourcentage du code d'une application est critique à ce point ? A partir du moment ou on construit une chaine de caractère à partir de données existante, c'est qu'on va envoyer cette chaine (ou une version traitée de celle-ci) sur un périphérique (au sens large : l'écran, une base de données, etc). Les temps de latence des sorties disponibles sont bien souvent plus important que le temps qu'on va perdre à créer la chaine. Du coup, la criticité de l'opération est loin d'être prouvée.

Enfin, plus lent, c'est vrai, mais dans quelle mesure ? Pour faire un strcat, il faut faire une allocation mémoire avant d'appeler strcat. On peut optimiser et faire une seule allocation si on a plusieurs strcat à faire - un cas qu'on peu facilement émuler en créant un std::string d'une taille donnée et en utilisant std::copy par la suite. Il faudrait faire un test en condition réelle, avec les optimisations qui s'imposent (c'est à dire une utilisation un peu plus poussée) avant de déterminer que l'écart de performance entre les deux est véritablement un problème.