Discussion :

[rigole]

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 :

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

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?

[Metalman]

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)

[Bysbobo]

Code :

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

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

[Médinoc]

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?

[Bktero]

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

Tu n'aimes pas la notation hongroise ? ^^

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

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

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 :

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

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.

[Médinoc]

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).

[rigole]

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 :

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

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

[Bysbobo]

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

Metalman?

[Metalman]

Ouais... j'ai pas osé intervenir (je vais pas salir mon grade, hein ! ), 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 !

Bravo Bysbobo du coup !

[Susette]

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

Code :

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

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!