Discussion :

[Syrus]

Bonjour,

Je ne suis pas très familier des "subtilités" du C++ et j'ai un soucis avec la conversion et concaténation de chaînes de caractère. Voilà mon problème :

J'ai deux int indépendants: 11 et 22
je veux obtenir lun char* de la sorte : "11-22"

Et ça, ça ne marche pas :

Code :

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

1
2
3
4
5
6
7
8
 
char* data;
char converter [8];
 
data = itoa(11, converter, 10);
data = strcat(data, "-");	
data = strcat(data, itoa(22, converter, 10));	
printf("Data : %s", data);

Ca m'affiche : Data : 2222.
En fait ça marcherait si j'utilisais une variable (converter) différente dans chaque itoa... Sauf que j'ai un nombre variable d'entier à convertir, ça m'arrangerait de réutiliser le même "buffer" à chaque fois.

PS : Ma chaîne data doit alimenter la fonction "send" d'un socket. Donc je ne peux pas jouer avec des "string" C++ je crois. Ou alors il faut que je cherche comment convertir un char* en string.

[3DArchi]

Premier conseil, passe par std::string (#include <string>) et std::stringstream (#include <sstream>):

Code :

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

1
2
3
4
5
6
7
 
   std::stringstream l_flux_str;
   std::string l_str;
   int i_1(11);
   int i_2(12);
   l_flux_str<<i_1<<"-"<<i_2;
   l_str = l_flux_str.str();

Ensuite, si tu veux tout de même un buffer:

Code :

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

1
2
 
const char *p_str = l_str.c_str();

Tu peux retrouver des infos sur la FAQ ou sur les tutoriels.

[ctxnop]

Salut (ou plutot rebonjour xD)
Pour rester dans un code style C (et oui, c'est du C que tu présente, pas du C++, en C++ on utilise des string comme le dis 3DArchi) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
#define BUF_SIZE  32 // Un petit define, c'est pratique pour les tailles de buffer
 
// On alloue le buffer, car si tu le fait pas c'est pas itoa qui va le faire
char* data = (char*)malloc(sizeof(char) * BUF_SIZE);
 
// On met 0 partout dans la buffer pour être certain que la chaine résultante sera terminée par un 0
SecureZeroMemory(data, BUF_SIZE); // Manquera ptre un cast ici
 
// On convertie le premier nombre
itoa(11, data, 10);
 
// On ajoute le "-"
strcat(data, "-"); // Pas besoin du data = strcat, puisque data sera directement modifié
 
// On calcul la taille de la chaine
int i = strlen(data);
 
// On convertie le seconde nombre
itoa(22, data + i, 10); // les pointeurs c'est super drole, avec ce +i le résultat de la conversion est directement concacténé

Quand tu aura fini de faire mumuse avec ton data, n'oublie pas de le libérer : free(data);

EDIT : Pour le SecureZeroMemory, j'ai utilisé celui la partant du princique que tu es sous Windows avec Visual Studio 2005+, si c'est pas le cas elle sera pas forcément disponible, mais rassures-toi, tu peux toujours utiliser la fonction memset à la place : memset(data, 0, BUF_SIZE);

[ctxnop]

Ah oui, autre chose, si c'est pour faire juste un printf derriere, tu peux faire directement

Code :

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

1
2
 
printf("Data : %i-%i", 10, 22);

La version C++ :

Code :

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

cout << "Data : " << 10 << "-" << 22 << endl;

ET si tu veux, tu as aussi la possibilité de faire un printf dans un buffer plutot que sur l'écran, regardes du coté de sprintf

[3DArchi]

PS : Ma chaîne data doit alimenter la fonction "send" d'un socket. Donc je ne peux pas jouer avec des "string" C++ je crois. Ou alors il faut que je cherche comment convertir un char* en string.

J'avais pas vu la première fois. Mais, rien n'empêche d'utiliser un std::string. Pour aller de char* vers std::string, tu as les constructeurs:

Code :

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

1
2
string ( const char * s, size_t n );
string ( const char * s );

Attention tout de même au '\0' terminal.
Pour ton émission, donc du std::string au char, tu as string::c_str:

Code :

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

1
2
 
sendto(ma_socket,ma_str.c_str(),ma_str.length()...);

Idem, attention au '\0' terminal.

[CedricMocquillon]

une dernière possibilité sans passer par les stringstream:

Code :

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

1
2
3
4
5
 
std::string l_str;
int i_1(11);
int i_2(12);
l_str = boost::lexical_cast<std::string>(i_1) + "-" + boost::lexical_cast<std::string>(i_2);

[ctxnop]

Ca implique d'utiliser Boost, même si c'est une librairie puissante et répandue c'est très loin d'être standard.
Pour un code aussi simple c'est vraiment inutile de s'encombrer d'une librairie aussi grosse.

[CedricMocquillon]

C'était plus pour la culture générale. Je suis d'accord avec toi que pour convertir des entiers en string dans cette méthode, l'intérêt d'utiliser boost est limité...

PS : Il utilise des sockets, ça peut être intéressant d'aller faire un tours du côté de boost (asio).

[Syrus]

Merci beaucoup pour votre aide.

J'ai implémenté une solution à base de flux et de string (et conversion en char* pour mon socket).