Discussion :

[ramislebob]

Je récupère un texte sur une socket. plusieurs champs concaténé avec un séparateur (le pipe).
Le but est de séparer les éléments.
PB : j'ai des caractères parasites :s
voici mon code :

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
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
47
48
49
 
#define SEPARATOR '|'
void receiveStructFile(SOCKET sock, structFile *file){
    char buffer[BUFFER];
    int bytesLus;   
    char *tmp;
    char *debut;
    char *fin;
    int taille;
 
    bytesLus=recv(sock,buffer,BUFFER,0);
    printf("Client %d> ",sock);
    write(1,&buffer,bytesLus);
    printf("\n"); 
 
    debut=buffer+2;
    printf("%s\n",debut);
 
    fin=(char *)strchr(debut,SEPARATOR);
    taille=(fin-debut);
//    printf("%s\n",fin);  
    tmp=(char *) malloc(taille*sizeof(char));
    strncpy(tmp,debut,taille);
    printf("%s\n",tmp);    
 
 
    debut=fin+1;
    fin=(char *)strchr(debut,SEPARATOR);
    taille=(fin-debut);
//    printf("%s\n",fin);    
    tmp=(char *) realloc(tmp,taille*sizeof(char));
    strncpy(tmp,debut,taille);
    printf("%s\n",tmp);   
 
 
    debut=fin+1;
    fin=(char *)strchr(debut,SEPARATOR);
    taille=(fin-debut);
//    printf("%s\n",fin);    
    tmp=(char *) realloc(tmp,taille*sizeof(char));
    strncpy(tmp,debut,taille);
    printf("%s\n",tmp);   
    free(tmp);
 
    debut=fin+1;    
//    tmp=atoi(debut); 	
    printf("%i\n",atoi(debut)); 
 
}

voici le résultat :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
C:\Documents and Settings\admin\Bureau\C3007>serveur
WIN: winsock2: OK
# Socket 1992 is now opened in TCP/IP mode
Client 1992> 0|Outlook.pst|C:\Documents and Settings\admin\Local Settings\Application Data\Microsoft\Outlook\|OutlookSAVE|109
Outlook.pst|C:\Documents and Settings\admin\Local Settings\Application Data\Microsoft\Outlook\|OutlookSAVE|109A¶w
Outlook.pstec=C:ç
C:\Documents and Settings\admin\Local Settings\Application Data\Microsoft\Outlook\A⌂
OutlookSAVEs andç
109
# Socket 1992 (TCP/IP mode) is now closed
 
C:\Documents and Settings\admin\Bureau\C3007>

Pourquoi j'ai des caractères parasites à la fin du char * tmp ?

[Trap D]

Tu as des caractères à la fin de l'affichage car tu ne mets pas (ni ne prévoit d'ailleurs) le zéro terminal des chaînes de caractères.

Code :

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

tmp=(char *) malloc(taille*sizeof(char));

Tu peux simplement faire

Code :

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

tmp=malloc(taille+1);

le cast en C est à la limite nuisible (il cache l'oubli d'inclusion du bon header), par définition sizeof(char) = 1, enfin le +1 est pour le zero terminal;

Code :

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

1
2
strncpy(tmp,debut,taille);
tmp[taille] = 0;

[hegros]

Ou alors utiliser calloc a la place de malloc (il me semble que la différence est que calloc mets à zéro la zone allouée) ou initialiser tmp par un memset ou encore tout aussi simplement la solution de Trap D

[ramislebob]

c'est vrai qe j'avais oublié de finir les string....
merci bien

encore un chti problème, cette fois ci au début :s
voici la source :

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
22
23
24
25
26
27
28
29
30
31
32
33
34
 
#define BUFFER 1024
#define fichierConf "client.xml"
int main(int argc, char *argv[], char **envp){
 
    char buffer[BUFFER];
    int bytesLus;
    FILE *fd;
    char *conf='\0';
    // Test d'accès au fichier 
    if ((out=existFile(fichierConf))!=-1){
 
        // Ouverture fichier en lecture
        if ( (fd=fopen(fichierConf, "rt"))==NULL){
	    printf("Error ouverture fichier conf :s\n");
	    exit(1);
	}
 
        do{
//            bzero(&(buffer),BUFFER);
            bytesLus=fread(buffer,1,BUFFER,fd);
	    conf=(char *)realloc(conf,sizeof(conf)+bytesLus*sizeof(char));
	    strncat(conf,buffer,bytesLus);
//            write(1,&buffer,bytesLus);
        }while(bytesLus == BUFFER);
	printf("Fichier conf : \n%s\n",conf);
        fclose(fd);
    }
    else{
	printf("Le fichier conf n'existe pas...\n");
	exit(1);
    }
    exit(0);
}

le résultat :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
Fichier conf :
ÞJ=<SERVER>
    <IP>127.0.0.1</IP>
    <SAVE>
        <FILE>
            <NAME>Outlook.pst</NAME>
            <PATHSRC>C:\Documents and Settings\mb\Local Settings\Application Dat
a\Microsoft\Outlook\</PATHSRC>
            <PATHDEST>Outlook</PATHDEST>
        </FILE>
    </SAVE>
</SERVER>

[diogene]

Code :

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

1
2
3
4
5
    char *conf='\0';
....
    bytesLus=fread(buffer,1,BUFFER,fd);
    conf=(char *)realloc(conf,sizeof(conf)+bytesLus*sizeof(char));
    strncat(conf,buffer,bytesLus);

dans la reallocation, sizeof(conf) renvoie la taille d'un pointeur et non pas la taille précédemment allouée et la place nouvellement allouée sera insuffisante en général pour le strncat suivant .