conversion char* en char[]?

Version imprimable

Bonjour,

Encore une fois, j'aurais SVP besoin d'un peu d'aide...

Mon programme principal instancie une structure du type:

Code:

1
2
3
4
5
6
7
8
typedef struct {
        char        *  Nom_projet; 
        char        *  Chemin;
        char        *  DataFile;
        int         *  TypeJeu;
        int         *  UseFilterFile;
        char        *  FilterFile;
        } Project_STRUCT;

puis appelle une fonction qui lis un fichier .txt afin de compléter les champs du Projet_STRUCT...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
int main (int argc, char ** argv)
{
 
/******** Creation des elements necessaires au Projet *************/
 
   Project_STRUCT * Projet   = malloc(sizeof(Project_STRUCT));
 
        Projet->Nom_projet     = NULL; 
        Projet->Chemin         = NULL;
        Projet->DataFile       = NULL;
        Projet->TypeJeu        = NULL;
        Projet->UseFilterFile  = NULL;
        Projet->FilterFile     = NULL;
 
  lecture_txt(Projet);
}

avec

Code:

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
 
void lecture_txt (Project_STRUCT * Projet )
{    
    FILE *Out;    
    Out = fopen ("D:\toto.txt", "r");
 
    char firstline[4096];
    char unused1[128];
    char unused21[4096];
    char unused22[4096];
    char unused23[4096];
    char unused24[4096];
    int unused31=0;
    int unused32=0;
 
    if (Out != NULL)
    {
      fgets(firstline, sizeof firstline, Out);    
      fscanf(Out, "%s %s\n", unused1, unused21);
      Projet->Nom_projet = unused21;
 
      fgets(firstline, sizeof firstline, Out);     
      fscanf(Out, " %s %s\n ", unused1, unused22);         
      Projet->Chemin = unused22;
 
      fgets(firstline, sizeof firstline, Out);     
      fscanf(Out, " %s %s\n ", unused1, unused23);         
      Projet->DataFile = unused23;
 
      fgets(firstline, sizeof firstline, Out);     
      fscanf(Out, " %s %d\n ", unused1, &unused31);     
      Projet->TypeJeu = &unused31;    
 
      fgets(firstline, sizeof firstline, Out);     
      fscanf(Out, " %s %d\n ", unused1, &unused32);   
      Projet->UseFilterFile = &unused32;
 
....
 
      fclose (Out);
 
}

Le programme ci-dessus ne fonctionne pas car mes variables unused sont détruites en fin de fonction, et du coup, mes Projet->xxx pointent vers une adresse corrompue...

Le problème c'est que le fscanf veut une variable du type tableau de char... Et moi je dois avoir dans mon Project_STRUCT un char *....

Aussi, je ne vois pas comment faire....

Le programme ci-dessus ne fonctionne pas car mes variables unusedxx sont détruites en fin de fonction, et du coup, mes Projet->xxx pointent vers une adresse corrompue...

Alex

06/11/2009, 21h07
diogene
Citation:

Le programme ci-dessus ne fonctionne pas car mes variables unused sont détruites en fin de fonction, et du coup, mes Projet->xxx pointent vers une adresse corrompue...

Le problème c'est que le fscanf veut une variable du type tableau de char... Et moi je dois avoir dans mon Project_STRUCT un char *....

Fais de l'allocation dynamique pour tes tableaux.
Code:

1 2 3 Projet->Nom_projet = malloc(strlen(unused21)+1); strcpy(Projet->Nom_projet, unused21); ....
Du coup, tu n'as plus besoin de tous ces tableaux locaux, unused22[4096], unused23[4096] et unused24[4096];

j'ai modifié la fonction lecture_txt comme ceci:

Code:

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
 
void lecture_txt (Project_STRUCT * Projet )
{    
    FILE *Out;    
    Out = fopen ("D:\toto.txt", "r");
 
    int size = 250;
 
    if (Out != NULL)
    {
              Projet->Nom_projet = malloc(size);
              fgets(firstline, sizeof firstline, Out);    
              fscanf(Out, "%s %s\n", unused1, Projet->Nom_projet);
 
              Projet->Chemin = malloc(size);
              fgets(firstline, sizeof firstline, Out);     
              fscanf(Out, " %s %s\n ", unused1,Projet->Chemin );         
 
              Projet->DataFile = malloc(size);
              fgets(firstline, sizeof firstline, Out);     
              fscanf(Out, " %s %s\n ", unused1, Projet->DataFile);         
 
              Projet->TypeJeu = malloc(sizeof(int));
              fgets(firstline, sizeof firstline, Out);     
              fscanf(Out, " %s %d\n ", unused1, Projet->TypeJeu);      
 
              Projet->UseFilterFile = malloc(sizeof(int));
              fgets(firstline, sizeof firstline, Out);     
              fscanf(Out, " %s %d\n ", unused1, Projet->UseFilterFile);   
 
....
 
              fclose (Out);
 
}

Ca a l'air de fonctionner...

Y a t il une meilleur solution?

Alex

06/11/2009, 21h16
diogene

Citation:

Y a t il une meilleur solution?

Comme je l'ai suggéré dans mon post, au lieu d'allouer un taille fixe (250 dans ton cas) , tu peux allouer exactement la taille dont tu as besoin dans chaque cas. Ca peut avoir son importance si tu dois créer beaucoup de structures et/ou si la taille des chaines est très variable selon le cas.
06/11/2009, 21h31
alexglvr

merci pour ton aide !!!

J'ai un autre petit problème...

Si la chaine de caractère lue comporte des espaces genre
"C:\DOCUMENTS AND SETTINGS\..."

la lecture s'arrête au premier espace....

Sais tu comment je pourrais résoudre ca STP?

Citation:

Envoyé par diogene

Comme je l'ai suggéré dans mon post, au lieu d'allouer un taille fixe (250 dans ton cas) , tu peux allouer exactement la taille dont tu as besoin dans chaque cas. Ca peut avoir son importance si tu dois créer beaucoup de structures et/ou si la taille des chaines est très variable selon le cas.

Comme je l'ai fait pour une fonction de ma bibliothèque C_Str, une allocation par progression :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
            while (fgets (buff, BUFFER_SIZE, p_file) != NULL)
            {
               old_size = new_size;
               new_size += strlen (buff);
               err = C_Str_resize (self, new_size);
 
               if (err == C_STR_NO_ERR)
               {
                  memmove (&self->sz_string[old_size], buff, strlen (buff) + 1);
               }
               else
               {
                  /* Erreur durant le chargement du fichier dans l'objet. */
                  C_Str_destroy (& self);
                  break;
               }
            }

07/11/2009, 12h16
salseropom
Citation:
Envoyé par diogene

Fais de l'allocation dynamique pour tes tableaux.

Code:

1 2 3 Projet->Nom_projet = malloc(strlen(unused21)+1); strcpy(Projet->Nom_projet, unused21); ....
Ces 2 lignes ne correspondent-elles pas à strdup() ? Bien sûr, il faut penser à free() après chaque utilisation de strdup() ?
07/11/2009, 12h44
diogene
alexglvr :

Citation:

J'ai un autre petit problème...

Si la chaine de caractère lue comporte des espaces genre
"C:\DOCUMENTS AND SETTINGS\..."

la lecture s'arrête au premier espace....

Comme chaque ligne semble comporter plusieurs chaînes apparemment séparées par des blancs :
Code:

1 2 fgets(firstline, sizeof firstline, Out); fscanf(Out, "%s %s\n", unused1, Projet->Nom_projet);
comment fais-tu pour savoir si un blanc appartient à une chaine ou sépare une chaine de la suivante ?
La réponse à ta question dépend de ça.

salseropom :

Citation:

Ces 2 lignes ne correspondent-elles pas à strdup() ? Bien sûr, il faut penser à free() après chaque utilisation de strdup() ?

Oui.
09/11/2009, 09h59
alexglvr

L'idée c'était de couper la ligne en deux : le premier mot et le reste...

Maintenant, je peux aussi utiliser un autre séparateur, type ":" ou ";"...

Dans ce cas, comment définie t on ce caractère comme séparateur au lieu de "espace"?
09/11/2009, 10h47
Pouet_forever

Quelque chose comme ça :

Code:

fscanf(Out, "%[^:]%*c%[^\n]s", unused1, Projet->Nom_projet);

Ici j'ai défini le : tu peux mettre autre chose si tu le veux ;)
09/11/2009, 15h32
alexglvr

Citation:

Quelque chose comme ça :
Code :

fscanf(Out, "%[^:]%*c%[^\n]s", unused1, Projet->Nom_projet);

Ça marche super si je lis une chaine de caractère comme deuxième paramétre...

Par contre, je n'arrive pas à lire un entier....

j'ai aussi essayé
fscanf(Out, "%[^;]%*c%[^\n]d", unused1, Proj_main->Projet->TypeJeu);

sans succès!!!
09/11/2009, 15h42
diogene

Il y a une légère erreur dans le format proposé

Code:

fscanf(Out, "%[^:]%*c%[^\n]",....

Le s terminal a pu te faire croire que c'est lui qui spécifiait une chaine.
En fait, %[] spécifie une chaine et ne peut pas être utilisé pour charger un entier.
Toutefois, tu peux lire la chaine et la convertir ensuite en entier (ou utiliser un format pour entier comme %d)