concatenation de 2 chaines

**rigole** · 09/07/2013, 09h29

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** · 09/07/2013, 09h37

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** · 09/07/2013, 09h46

Envoyé par rigole

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** · 09/07/2013, 13h24

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** · 09/07/2013, 14h05

Envoyé par Médinoc

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

Tu n'aimes pas la notation hongroise ? ^^

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.

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 :

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** · 09/07/2013, 15h47

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** · 09/07/2013, 17h10

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 :

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** · 10/07/2013, 09h26

Envoyé par Médinoc

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

Metalman?

**Metalman** · 10/07/2013, 15h20

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** · 11/07/2013, 12h04

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!

**Médinoc** · 11/07/2013, 12h39

Oui, on peut.

Code C :

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
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 );
}

concatenation de 2 chaines

C

Discussions similaires

Partager

Partager