Probleme avec malloc sur tableau de chaine

**grex1** · 12/09/2010, 20h48

Bonjour,
J'ai toujour eu du mal à utiliser les malloc pas forcement par se que je comprend pas mais par se que sa ajoute beaucoup de texte pour pas grand chose.
Comme je suis dyslexique ses déja difficil pour moi de tout mettre en ordre la ses pire.

j'ai créer 2 fonctions la premier ajoute une ligne a la suite d'un autre dans un fichier, la 2 fonctionne presque de la même fasson à la différence qu'elle n'ajoute pas directement a la suite de se cette ligne mais quant elle rencontre un espace.
Voici le code des 2 fonctions(premier mots ses la ligne a rechercher dans le fichier secont mot ses la lignre qui va être ajouter dans le fichier):

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
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
void ajouter_ligne_fichier2(FILE *fichier2, char *premiermot, char *secondmot)
{
 
 
    char chaine[1000000] = "";
    char sautligne[10000] = "\n";
 
    char **variable = NULL;
    long i2= 0,bite=0;
    variable = malloc(nombredeligne * siqeof(*variable)); //Allocation de la "première dimension"
    strcat(secondmot ,sautligne); // On concatène chaine2 dans chaine1
 
 
    if (fichier2 != NULL)
    {
        while (fgets(chaine, 1000000, fichier2) != NULL) // On lit le fichier tant qu'on ne reçoit pas d'erreur (NULL)
        {
 
 
            if (strcmp(chaine, premiermot) == 0) // Si strcmp renvoie 0 (chaînes identiques)
            {
                if (!variable) exit(-1);  //Ou autres notifications d'erreurs
 
                for (i=0 ; i2 < nombredeligne ; i2++)         //Boucle d'allocation de la "deuxième dimension"
                {
                    variable[i2]=strdup(chaine);
 
                    if (!variable[i2]) //En cas d'erreur d'allocation
                    {
                        my_free(variable,i2);     //On libère ce qu'on a réussi à allouer (à coder par toi même)
                        exit(-2);  //Ou autres notifications d'erreurs
                    }
                }
 
 
                bite=1;
 
            }
            else
            {
                if (strcmp(chaine, premiermot) == 0) // Si strcmp renvoie 0 (chaînes identiques)
                {
                    if (!variable) exit(-1);  //Ou autres notifications d'erreurs
 
                    for (i=0 ; i2 < nombredeligne ; i2++)         //Boucle d'allocation de la "deuxième dimension"
                    {
                        variable[i2]=strdup(chaine);
 
                        if (!variable[i2]) //En cas d'erreur d'allocation
                        {
                            my_free(variable,i2);     //On libère ce qu'on a réussi à allouer (à coder par toi même)
                            exit(-2);  //Ou autres notifications d'erreurs
                        }
                    }
 
 
 
 
                }
 
                if (bite)
                {
 
                    if (strcmp(chaine, sautligne) == 0) // Si strcmp renvoie 0 (chaînes identiques)
                    {
                        if (!variable) exit(-1);  //Ou autres notifications d'erreurs
 
                        for (i=0 ; i2 < nombredeligne ; i2++)         //Boucle d'allocation de la "deuxième dimension"
                        {
                            variable[i2]=strdup(secondmot);
 
                            if (!variable[i2]) //En cas d'erreur d'allocation
                            {
                                my_free(variable,i2);     //On libère ce qu'on a réussi à allouer (à coder par toi même)
                                exit(-2);  //Ou autres notifications d'erreurs
                            }
 
                            i2++;
 
                            if (!variable) exit(-1);  //Ou autres notifications d'erreurs
 
                            for (i=0 ; i2 < nombredeligne ; i2++)         //Boucle d'allocation de la "deuxième dimension"
                            {
                                variable[i2]=strdup(sautligne);
 
                                if (!variable[i2]) //En cas d'erreur d'allocation
                                {
                                    my_free(variable,i2);     //On libère ce qu'on a réussi à allouer (à coder par toi même)
                                    exit(-2);  //Ou autres notifications d'erreurs
                                }
 
                                bite=0;
 
                            }
 
                        }
 
 
 
                        if (fichier3 != NULL)
                        {
 
 
 
                            fseek(fichier3, 0, SEEK_END);
 
                            for (i=0 ; i2 < nombredeligne ; i2++)         //Boucle d'allocation de la "deuxième dimension"
                            {
                                // On l'écrit dans le fichier
                                fprintf(fichier3, "%s", variable[i]);
 
                                if (!variable[i2]) //En cas d'erreur d'allocation
                                {
                                    my_free(variable,i2);     //On libère ce qu'on a réussi à allouer (à coder par toi même)
                                    exit(-2);  //Ou autres notifications d'erreurs
                                }
 
 
 
                                fclose(fichier3);
                            }
 
                        }
 
                        int ajouter_ligne_fichier2(FILE *fichier2, char *premiermot, char *secondmot)
                        {
 
                            char chaine[1000000] = "";
                            char sautligne[10000] = "\n";
 
                            char   *variable[100000];
                            long i2= 0,bite=0;
 
                            strcat(secondmot ,sautligne); // On concatène chaine2 dans chaine1
 
 
                            if (fichier2 != NULL)
                            {
                                while (fgets(chaine, 1000000, fichier2) != NULL) // On lit le fichier tant qu'on ne reçoit pas d'erreur (NULL)
                                {
 
 
                                    if (strcmp(chaine, premiermot) == 0) // Si strcmp renvoie 0 (chaînes identiques)
                                    {
 
                                        variable[i2]=strdup(chaine);
                                        bite=1;
 
                                    }
                                    else
                                    {
 
                                        variable[i2]=strdup(chaine);
 
 
                                    }
 
                                    if (bite)
                                    {
 
                                        if (strcmp(chaine, sautligne) == 0) // Si strcmp renvoie 0 (chaînes identiques)
                                        {
 
                                            variable[i2]=strdup(secondmot);
                                            i2++;
                                            variable[i2]=strdup(sautligne);
                                            bite=0;
 
                                        }
 
                                    }
                                    i2++;
 
 
 
                                }
 
                                fclose(fichier2);
                            }
 
                            FILE* fichier3 =NULL;
                            fichier3 = fopen("codesource.txt","w");
 
 
 
 
 
 
 
                            if (fichier3 != NULL)
                            {
 
 
                                long i=0;
 
                                while (i!=i2)
                                {
                                    fseek(fichier3, 0, SEEK_END);
                                    // On l'écrit dans le fichier
                                    fprintf(fichier3, "%s", variable[i]);
                                    i++;
                                }
 
                                fclose(fichier3);
                            }
 
                        }

J'ai besoin qu'on me corrige le code de une de ses 2 fonctions si possible la 2 eme pour que variable(que j'aurais du nommer ligne) soi allouer dynamiquement.
J'ai déja tentté le cous mais j'ai tout mélanger ses le bordel dans 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
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
void ajouter_ligne_fichier2(FILE *fichier2, char *premiermot, char *secondmot)
{
 
 
    char chaine[1000000] = "";
    char sautligne[10000] = "\n";
 
    char **variable = NULL;
    long i2= 0,bite=0,nombredeligne=0;
    variable = malloc(nombredeligne * siqeof(*variable)); 
    strcat(secondmot ,sautligne); // On concatène chaine2 dans chaine1
 
 
    if (fichier2 != NULL)
    {
        while (fgets(chaine, 1000000, fichier2) != NULL)  
        {
 
 
            if (strcmp(chaine, premiermot) == 0) 
            {
                if (!variable) exit(-1);  
 
                for (i2=0 ; i2 < nombredeligne ; i2++)        
                {
                    variable[i2]=strdup(chaine);
 
                    if (!variable[i2]) 
                    {
                        my_free(variable,i2);     
                        exit(-2);  
                    }
                }
 
 
                bite=1;
 
            }
            else
            {
                if (strcmp(chaine, premiermot) == 0) 
                {
                    if (!variable) exit(-1);  
                    for (i2=0 ; i2 < nombredeligne ; i2++)        
                    {
                        variable[i2]=strdup(chaine);
 
                        if (!variable[i2]) 
                        {
                            my_free(variable,i2);    
                            exit(-2); 
                        }
                    }
 
 
 
 
                }
 
                if (bite)
                {
 
                    if (strcmp(chaine, sautligne) == 0) 
                    {
                        if (!variable) exit(-1);  
                        for (i2=0 ; i2 < nombredeligne ; i2++)        
                        {
                            variable[i2]=strdup(secondmot);
 
                            if (!variable[i2]) 
                            {
                                my_free(variable,i2);     
                                exit(-2);  
                            }
 
                            i2++;
 
                            if (!variable) exit(-1);  
 
                            for (i2=0 ; i2 < nombredeligne ; i2++)         
                            {
                                variable[i2]=strdup(sautligne);
 
                                if (!variable[i2]) 
                                {
                                    my_free(variable,i2);    
                                    exit(-2);  //Ou autres notifications d'erreurs
                                }
 
                                bite=0;
 
                            }
 
                        }
 
 
                        FILE* fichier3 =NULL;
                        fichier3 = fopen("codesource.txt", "r+");
 
                        if (fichier3 != NULL)
                        {
 
 
 
                            fseek(fichier3, 0, SEEK_END);
 
                            for (i2=0 ; i2 < nombredeligne ; i2++)      
                            {
                                // On l'écrit dans le fichier
                                fprintf(fichier3, "%s", variable[i2]);
 
                                if (!variable[i2]) 
                                {
                                    my_free(variable,i2);   
                                    exit(-2);  
                                }
 
 
 
                                fclose(fichier3);
                            }
 
                        }

**Sve@r** · 12/09/2010, 21h49

Désolé, vraiment désolé mais je pige que dalle à tes fonctions.
A un moment, tu compares chaine avec premier mot et si c'est égal tu fais un truc, puis; si c'est différent (else), alors tu recompares ta chaine avec ton premier mot et si c'est égal, alors tu refais le même truc. Mais ça peut pas être égal puisqu'on est dans le else !!!

Ce que tu as fait de bien, c'est que t'as pensé à libérer ce qui est déjà alloué en cas de problème intermédiaire (le "my_free"). Mais pour le reste...

Alors essayons de reprendre le problème depuis le début. Tu veux rajouter, dans un fichier, une ligne à la suite d'une autre. Qu'est-ce qui caractérise la ligne qui sert de repère ? Et la ligne qu'on rajoute, est-ce qu'elle s'insère ou bien est-ce qu'elle remplace la ligne suivante ???

En tout état de cause, quand on veut faire une modif dans un fichier on n'a pas vraiment besoin de malloc. On se contente de lire le fichier 1 ligne par ligne (on a donc une zone de taille jugée suffisante pour stocker une simple ligne) et d'écrire la ligne originale ou bien la ligne modifiée ou bien la ligne insérée dans un fichier 2. En fin de traitement, le fichier 2 remplace le fichier 1...

PS: le NULL de fgets n'est pas forcément une erreur (ton commentaire). C'est que le fichier a été lu entièrement et qu'il n'y a plus rien à lire et donc qu'on est en fin de fichier...

**grex1** · 12/09/2010, 22h36

Le mieu pour tenté de comprendre ne regarde pas les 2 code que j'ai poster il s'agis de la même fonction.
Le 2 code ses la même fonction ou j'ai tenté d'allouer dinamiquement variable même moi je m'y retrouve plus.
Je vient de voir que je n'ai pas envoyais le code de la premier fonction donc je le donne maintenant(j'ai mis des commentaire est modifier le nom des chaines pour que sa soi compréhensible):

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
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
int ajouter_ligne_fichier(FILE *fichier2, char *ligne_a_rechercher, char *ligne_a_coller_a_la_suite_de_ligne_a_rechercher)
{
 
    char chaine[100000] = "";
    char sautligne[10000] = "\n";
 
    char   *ligne[10000];// tableau de chaine pour acceuillir tout les lignes du fichier plus les nouvelle qu'on ajoute
    long i2= 0;
 
    strcat(ligne_a_coller_a_la_suite_de_ligne_a_rechercher ,sautligne); // On concatène chaine2 dans chaine1
 
 
    if (fichier2 != NULL)
    {
        while (fgets(chaine, 100000, fichier2) != NULL) // On lit le fichier tant qu'on ne reçoit pas d'erreur (NULL)
        {
 
 
            if (strcmp(chaine, ligne_a_rechercher) == 0) // Si la ligne a rechedcher correspond a la ligne actuel du fichier
            {
                ligne[i2]=strdup(chaine);// on ajoute la ligne du fichier dans ligne
                i2++;// on ajoute aussi a la suite la nouvelle migne qu'on souhaitre introduire
                ligne[i2]=strdup(ligne_a_coller_a_la_suite_de_ligne_a_rechercher);
 
            }
            else
            {
 
                ligne[i2]=strdup(chaine);// on ajoute juste la ligne du fichier dans ligne
 
 
            }
 
 
            i2++;
 
 
 
        }
 
        fclose(fichier2);
    }
 
// pour évitter d'avoir a créer un fichier tempon je redéclare le même fichier sous un autre nom
    FILE* fichier3 =NULL;
    fichier3 = fopen("codesource.txt","w");
 
 
 
 
 
// on remplasse tout le contenu du fichier par ligne
 
 
    if (fichier3 != NULL)
    {
 
 
        long i=0;
 
        while (i!=i2)
        {
            fseek(fichier3, 0, SEEK_END);
            // On l'écrit dans le fichier
            fprintf(fichier3, "%s", ligne[i]);
            i++;
        }
 
        fclose(fichier3);
    }
 
}

Voici un exemple de code ou j'utilise cette 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
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
50
51
52
    FILE* fichiersource =NULL;
    FILE* fichiersource2 =NULL;
    FILE* fichiersource3 =NULL;
    FILE* fichiersource4 =NULL;
    FILE* fichiersource5 =NULL;
 
    fichiersource= fopen("codesource.txt", "r+");
    fichiersource2 = fopen("codesource.txt", "r+");
    fichiersource3 = fopen("codesource.txt", "r+");
    fichiersource4 = fopen("codesource.txt", "r+");
    fichiersource5 = fopen("codesource.txt", "r+");
 
 
 
    char chainechercher[1000] = "/* Création des Widget */\n";
    char *creationwidget = g_strdup_printf("GtkWidget *%s; ", sText);
 
 
    ajouter_ligne_fichier(fichiersource,chainechercher,creationwidget);
 
    char chainechercher2[10000] = "/* Creation des GtkEntry */\n";
    char *creationlabel = g_strdup_printf("%s=gtk_entry_new();", sText);
 
 
    ajouter_ligne_fichier(fichiersource2,chainechercher2,creationlabel);
 
    char chainechercher3[10000] = "/* ajouter dans box */\n";
    char *ajoutbox = g_strdup_printf("gtk_box_pack_start(GTK_BOX(pVBox),%s , TRUE, TRUE, 0);", sText);
 
 
    ajouter_ligne_fichier(fichiersource3,chainechercher3,ajoutbox);
 
    char chainechercher4[10000] = "/*declaration des copies des labels*/\n";
    char *creationcopiewidget = g_strdup_printf("GtkWidget *%s; ", sText);
 
 
    ajouter_ligne_fichier(fichiersource4,chainechercher4,creationcopiewidget);
 
    char chainechercher5[10000] = "/*suivent*/\n";
    char *elementsuivent = g_strdup_printf("pList = g_list_next(pList); %s= GTK_WIDGET(pList->data);", sText);
 
 
    ajouter_ligne_fichier(fichiersource5,chainechercher5,elementsuivent);
 
 
 
    /* Liberation de la memoire utilisee par la liste */
    g_free(creationwidget);
    g_free(creationlabel);
    g_free(ajoutbox);
    g_free(creationcopiewidget);
    g_free(elementsuivent);'appelle a la fonction:

Comme expliquer precedament j'émerais juste que ligne soit allouer dynamiquement.
Oublis les autres code suis la est beaucoup plus claire.

Pour répondre a t'a question la ligne s'inssaire a la suite et ne remplasse pas l'enssienne.

Probleme avec malloc sur tableau de chaine

C

Discussions similaires

Partager

Partager