concatenation de 2 chaines

Bonjour,
voici mon probleme:
j'ai 2 chaines de caracteres, une chaine qui a un nombre de caracteres fixes, et une chaine de caractere non fixe , et jaimerai les concatener pour que ma fonction sorte les 2.
code :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

void send() { TCHAR * pcCommande="BIBLI="; TCHAR * pcBibli=NULL; TCHAR * pcChaineConcate=NULL;   printf("quelle est votre bibli? %s", pcCommande); scanf(%s, pcBibli); chaineConcat=strcat(puByte,pcCommande); printf(chaineConcat);   }

---

est ce quil faut utilise malloc?

Eh bah... il faut connaitre la taille max dans tous les cas avec scanf....

Ou alors tu utilises "fread" sur "stdin", et tu reconstruis la chaîne au fur et à mesure avec strcat !

PS : pense à la balise CODE ! (le petit # dans l'interface)

Citation:

---

Envoyé par rigole

Code:

---

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

void send() { TCHAR * pcCommande="BIBLI="; TCHAR * pcBibli=NULL; TCHAR * pcChaineConcate=NULL;   printf("quelle est votre bibli? %s", pcCommande); scanf(%s, pcBibli); chaineConcat=strcat(puByte,pcCommande); printf(chaineConcat);   }

---

---

Bonjour,

Juste comme ça, j'aurais quelques questions sur ton code?
1*/ Tu initialises pcBibli à NULL mais tu lui enregistres une valeur avec le scanf (donc tu enregistres une valeur dans rien?), c'est normal?
2*/ Ou est l'initialisation de chaineContact et puByte?

Enfin je demande ça mais si tu as autre chose d'écrit et que tu l'as pas noté, je veux bien voir ;)

Merci

Ton code est un mélange hybride de char et de TCHAR. Pourquoi avoid déclaré tes chaînes en TCHAR? Tu ne les utilises qu'avec des fonctions basées sur char...

PS: Comme le dit Metalman, tu fais ton scanf() vers NULL, très mauvais.
PPS: N'utilise pas le préfixe pc pour des pointeurs non-constants. Si tu me dis que c'est pour des pointeurs de char, c'est mauvais, d'une part on tombe dans la "systems hungarian notation", d'autre part tu si tu veux vraiment faire ça tu devrais utiliser psz à la place.
PPPS: Pourquoi cette discussion est-elle marquée comme Résolue?

Citation:

---

Envoyé par Médinoc

c'est mauvais, d'une part on tombe dans la "systems hungarian notation"

---

Tu n'aimes pas la notation hongroise ? ^^

Citation:

---

Envoyé par Médinoc

Comme le dit Metalman, tu fais ton scanf() vers NULL, très mauvais.

---

Je dirai même plus, c'est très mauvais.

Citation:

---

Envoyé par rigole

j'ai 2 chaines de caracteres, une chaine qui a un nombre de caracteres fixes, et une chaine de caractere non fixe , et jaimerai les concatener pour que ma fonction sorte les 2.

---

Non fixe mais forcément bornée ! Il y a une taille maximale que tu autorises pour cette chaine. Cette taille se donne normalement en allouant un espace avec malloc() mais toi, tu as mis NULL... Une fois cette borne (ou alors la taille utile, cad jusqu'à \0, ça dépend de ton utilisation) et la taille de la chaine fixe connues, il te suffit d'allouer (et oui, encore malloc()*) une nouvelle chaine de la bonne taille.

Tu peux utiliser sprintf() si tu es en C99 pour connaitre la taille de la chaine résultante : http://stackoverflow.com/questions/3...s-the-standard

Code:

---

1 2 3

int size = snprintf(NULL, 0, "%d", 132); char * a = malloc(size + 1); sprintf(a, "%d", 132);

---

* : en fait, tu pourrais aussi utiliser un VLA si tu es en C99.

Citation:

---

Tu n'aimes pas la notation hongroise ? ^^

---

Si mais j'essaie (je n'y arrive pas toujours) d'utiliser plutôt la apps hungarian. (+préfixage des pointeurs, que je trouve indispensable dès qu'on touche à différents niveaux d'indirection/pointeurs de pointeurs).

Citation:

---

Envoyé par Médinoc

Pourquoi avoir déclaré tes chaînes en TCHAR?
Parce que je travaille sur un projet et mon boss veut que ça soit comme ça!! ^^
PS: Comme le dit Metalman, tu fais ton scanf() vers NULL, très mauvais.
PPPS: Pourquoi cette discussion est-elle marquée comme Résolue?

---

J'ai remodifié mon code et ça marche :

Code:

---

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

//TSerialStreamObject tSerialStream; TCONST TCHAR * pcCommandeAT="AT$SS="; TUCHAR * pucChaineConcat; void vFSend( TCONST TUCHAR * pucByte, TUCHAR ucSize ) { //concaténation de commande AT$SS avant le mot à envoyer pucChaineConcat = strcat( ( TCHAR *)pcCommandeAT, ( TCONST TCHAR *)pucByte );   // Taille de la chiane concaténée ucSize = strlen( pucChaineConcat );   //Emission du message tSerialStream.uxFTxWrite( pucLigneComplete , ucSize );   }

---

avec la fonction uxFTxWrite(TUCHAR *, TUCHAR) que mon boss a écrit (transmission d'octet)
Et du coup j'ai une autre question:
je veux rajouter à la fin de ma chaine de caractères un carriage return <CR>
mais ça me fais marche pas

Citation:

---

Envoyé par Médinoc

PS: Comme le dit Metalman, tu fais ton scanf() vers NULL, très mauvais.

---

Metalman? :mouarf::ccool:

Ouais... j'ai pas osé intervenir (je vais pas salir mon grade, hein ! :P), mais c'est Bysbobo qui a remonté le "NULL"...

Moi j'ai juste dit que ça devait être une taille fixe qui reçoit les chars du scanf ! :P

Bravo Bysbobo du coup ! :mouarf:

Bjr!!
Est ce qu'on peut utiliser sprintf pour concater 3 chaines a la fois?
ex:

Code:

---

1 2 3 4 5 6 7 8 9 10

int main() { char * buffer[80]; char * s1="hello"; char * s2; char * s3="bye"; printf("votre message?"); scanf(%s, &s2); sprintf(buffer, "%s", s1, s3 ); }

---

ou quelque chose dans ce genre?
Merci!

Oui, on peut.

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 37 38 39 40 41 42 43 44 45 46

/* Returns a nonzero value if found, zero if not. */ int truncate_newline(char *str) { int bRet=0; if(str!=NULL) { char *pNewline = strchr(str, '\n'); if(pNewLine!=NULL) { bRet = 1; *pNewLine = '\0'; } } return bRet; }   //Returns 0 if OK, a negative value if EOF. int fpurge(FILE *f) { int c; while((c=fgetc(f))!=EOF && c!='\n') { } return (c==EOF ? -1 : 0); }   /* Returns 0 if buffer is full, a positive value if line is complete,   a negative value if EOF (implies buffer full). */ int fclean(char *str, FILE *f) { int ret = 1; if(!truncate_newline(str)) ret = fpurge(f); return ret; }   int main() { char buffer[80]; char const * s1="hello"; char s2[72]; char const * s3="bye"; printf("votre message?"); fgets(s2, 72, stdin); fclean(s2, stdin); sprintf(buffer, "%s%s%s", s1, s2, s3 ); }

---