Discussion :

[ehben]

Bonjour,

j'ai un petit soucis avec une fonction de fusion de deux champs en C...

En entrée, on donne les deux champs qu'on doit concaténer + la longueur de ces deux champs + la longueur du champ de sortie
En sortie, on obtient le champ concatené et sa longueur

En prenant l'exemple suivant, j'ai un soucis :
champ1 = "Voiture" (longueur = 7) et
champ2 = "5556661" (longueur = 8)
et taille champ de sortie = 45
Lors de la fusion le dernier caractère le "1" disparaît... :/
Je ne comprends pas pourquoi...
A noter que cela fonctionne (pas d'effacement de dernier caractère) si on envoie "555666" (en gardant longueur = 8)

J'imagine qu'il doit y avoir un problème de taille...

voici la fonction

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
21
int FUSADR(char * strin1, char * strin2, char * strout, int * pilen1, int * pilen2, int * polen) 
{ 
     int i,slen; 
     int olen =  READINT(polen); 
     int ilen1 = READINT(pilen1); 
     int ilen2 = READINT(pilen2);
 
     for(i = 0; i<ilen1; i++) { 
         if(strin1[i] == ',') 
            strin1[i] = 0; 
     } 
     slen=strlen(strin1)+strlen(strin2); 
     if (slen > olen) 
        return(0); 
     strcpy(strout,strin1); 
     strcat(strout,", "); 
     strcat(strout,strin2); 
     slen=strlen(strout); 
     strout[slen]=' '; 
     return(1); 
}

merci !

[jeroman]

Ta chaîne de sortie n'est pas terminée par un '\0' mais par un caractère espace.

[Sve@r]

Bonjour,

j'ai un petit soucis avec une fonction de fusion de deux champs en C...

En entrée, on donne les deux champs qu'on doit concaténer + la longueur de ces deux champs + la longueur du champ de sortie
En sortie, on obtient le champ concatené et sa longueur

En prenant l'exemple suivant, j'ai un soucis :
champ1 = "Voiture" (longueur = 7) et
champ2 = "5556661" (longueur = 8)
et taille champ de sortie = 45

La remarque de jeroman est bonne. Et tu peux faire ta concaténation en une seule opération

Code :

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

sprintf(strout, "%s%s", strin1, strin2)

Et là, le sprintf() s'assurera lui-même du positionnement de '\0'

[ehben]

ok merci...
c'est donc ça

Code :

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

strout[slen]=' ';

qui pose un problème ?

Mais en quoi cet espace va éliminer le dernier caractère :/ ?

[jeroman]

Car tu écrases le '\0' avec cet espace :

Code :

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

1
2
     slen=strlen(strout);
     strout[slen]=' ';

Exemple en image. Si ta chaîne strout contient "Hello\0"

Code :

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

slen=strlen(strout);

slen sera égal à 5, car la chaîne contient 5 caractères. Or, strout[5] sera égal à '\0', car il ne faut pas oublier que l'indice 0 d'un tableau (au sens large) correspond à son premier élément :

Code :

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

1
2
3
4
5
6
7
indice -> caractère
0 -> H
1 -> e
2 -> l
3 -> l
4 -> o
5 -> \0

Par conséquent

Code :

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

strout[slen]=' ';

vire le '\0' pour le remplacer par un espace. En conclusion, ta chaîne a une longueur indéterminée et contient n'importe quoi.

(Après, je n'ai pas regardé en profondeur le reste du code, tu n'as pas expliqué en détail ce qu'il est censé faire)