Discussion :

[crashtib]

salut a tous,

Voici le code de deux fonctions, toutes simples :

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
/test/src> more parser.c
#include "biblio.h"

void* parser(void *p)
{
        int i;
        printf("debut du parsing des fichiers GRE et FNR...\n");
        int lignedebut = 1;
        int lignefin = 5;
        char * entreeformatee;
        char ** donnees;

        donnees = lireligne("test.csv", lignedebut, lignefin);

        for (i = 0; i < (lignefin - lignedebut); i++)
        {
printf("\n%s\n", donnees[i]);
                entreeformatee = formaterentree(donnees[i]);
printf("\n%s\n",entreeformatee);
                ajouterentree(CHEMINFIC_DONNEES_COMP, entreeformatee);
        }
        (void)p;
        return NULL;
}


/test/src> more ajouterentree.c
#include "biblio.h"

void ajouterentree(char * cheminfichier, char * nouvellentree)
{

        printf("%s\n", nouvellentree);
        FILE * fichier;
        if ((fichier = fopen(cheminfichier, "a"))!=NULL)
        {
                fprintf(fichier, "%s\n", nouvellentree);
                fclose(fichier);
        }



}

j'ai mis deux lignes en rouge.
le résultat de cette première ligne est toto, par exemple. ou un truc beaucoup plus long. peu importe.
le résultat de la deuxième ligne est "", soit rien.



Pourquoi?

[souviron34]

il faudrait avoir le code de formaterentree

[crashtib]

le voici

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
/test/src> more formaterentree.c
#include "biblio.h"
 
 
char * formaterentree(char * donnees)
{
        char * temp;
        char ** temp2;
        char donneefin[200];
        FILE * fichier;
        int i = 0;
 
        //remplir donneefin en fonction de donnnees P3G
 
        //MSISDN
        strcat(donneefin, strtok(donnees, ";"));
        strcat(donneefin, ";");
 
        //IMSI
        strcat(donneefin, strtok(NULL, ";"));
        strcat(donneefin, ";");
 
        //NO_HLR_REF
        temp = strtok(NULL, ";");
        strcat(donneefin, temp);
        strcat(donneefin, ";");
 
 
        //HLR_REF
        if (strcmp(temp, "")!=0)
        {
                i = chercher(CHEMINTAB_ASS_HLR, temp);
                if (i>0)temp2 = lireligne (CHEMINTAB_ASS_HLR, i, i);
                strtok(temp2[0], ";");
                strcat(donneefin, strtok(NULL, ";"));
        }
        strcat(donneefin, ";");
 
 
        //remettage du curseur de strtok a l'endroit precedent le HLR_REF
        strtok(donnees, ";");
        for(i = 0; i < 2; i++)strtok(NULL, ";")
 
        donnees = donneefin;
        return donnees;
}

le rendu? ça donne quelquchose comme ça :

33660141414;208209865774349;HLR02;11089;

[souviron34]

beaucoup de choses qui ne vont pas :

• strtok modifie (en mettant des NULL) la chaîne en entrée
• la chaine en entrée n'est pas passée par pointeur, donc son adresse n'est pas modifiable
• tu assignes à une chaine ressortante un buffer local, qui donc n'a plus d'existence en sortie de la fonction.

Corrige ça, et ça marchera..

[diogene]

Tu peux ajouter à la liste des erreurs celle-ci :

Code :

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

1
2
3
4
5
6
char * formaterentree(char * donnees)
{
....
        char donneefin[200];
....
        strcat(donneefin, strtok(donnees, ";"));

strcat suppose que les DEUX arguments soient des chaines correctement formées, c'est à dire terminée par '\0'. Ce n'est pas le cas ici du tableau (local) non initialisé donneefin.
- initialise char donneefin[200] = "" ou donneefin[0] = '\0';
- ou utilise strcpy

* L'erreur de principe la plus grave est signalée par souviron34 : tu retournes l'adresse d'une variable locale (donneefin)

[crashtib]

je vous remercie de vos conseils et je vais immédiatement les mettre avec exécution. Toutefois, une question se pose : comment se fait-il que même à ce paquet d'erreurs, je suis capable d'obtenir de la fonction appelante (parser.c) la chaîne voulue, sans défaut? en gros quand je vais printf(donneeformatee); dans parser(), ça m'écrit quand même en console la chaîne formatée correctement?