Problèmes de mémoire sur un projet en C++ (en fait C)

Version imprimable

Bonjour à tous !

Je jette une bouteille à la mer car je suis désespéré, je débute en programmation en C++ et je dois effectuer pour l'école un projet qui consiste à faire un qcm sur base d'un fichier csv composé comme suis : Question;Proposition1;Proposition2;Score1;Score2 . Sachant également qu'il peut y avoir + ou - de propositions et que plusieurs d'entre elles peuvent être correct donc il faut la possibilité à l'utilisateur de sélectionner plusieurs propositions.
Je pense que mon projet comporte beaucoup d'erreurs car il ne tourne pas. Ce sont principalement des erreurs d'allocations de mémoire.
Si quelqu'un saurait relire mon code et repérer d'éventuelles fautes ce serait très aimable.
J'ai placé quelques printf inutile au projet pour essayer de localisé l'erreur mais je n'y parviens pas.

voici mon code :

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
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
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>
#include "..\..\Modules\Lecture\lecture.h"
 
void LibereMemoireTableau(char** tableau, int taille);
void DebutChaine(char chaine[], char retour[]);
void ExtraireInfoLigne(char chaine[], char** info);
void PermutationAleatoire(int n, int tab[]);
char** ConversionFichier(const char* fichier, int* nblignes);
 
int main() {
    int nbrquestions;
    char** fichier;
 
    // Lecture du fichier
    fichier = ConversionFichier("ExempleQCM.csv", &nbrquestions);
    if (fichier == NULL) {
        printf("Erreur d'allocation ou de lecture du fichier!\n");
        return 1;
    }
    printf("Nombre de questions lues: %d\n", nbrquestions);
 
    // Génération d'un ordre aléatoire pour les questions
    int* ordre = (int*)malloc(nbrquestions * sizeof(int));
    if (ordre == NULL) {
        printf("Erreur d'allocation de mémoire pour l'ordre des questions!\n");
        LibereMemoireTableau(fichier, nbrquestions);
        return 1;
    }
    for (int t = 0; t < nbrquestions; t++)
        ordre[t] = t;
    PermutationAleatoire(nbrquestions, ordre);
 
    float scorefinal = 0;
    float scoremax = 0;
 
    // Boucle pour chaque question
    for (int i = 0; i < 8; i++) {
        int n = ordre[i];
        printf("Traitement de la question %d (index %d)\n", i + 1, n); //!!!!!!!!!!!!!temporaire
 
        // Déterminez la taille nécessaire pour info
        int tailleInfo = 0;
        char* ligne = fichier[n];
        for (int pos = 0; ligne[pos] != '\0'; pos++) {
            if (ligne[pos] == ';') {
                tailleInfo++;
            }
        }
        tailleInfo++; // Compte pour le dernier élément après le dernier séparateur
 
        printf("Taille d'info pour la question %d : %d\n", i + 1, tailleInfo);//!!!!!!!!!!!!!!!!!!!!temporaire
 
        char** info = (char**)malloc(tailleInfo * sizeof(char*));
        if (info == NULL) {
            printf("Erreur d'allocation de mémoire pour info!\n");
            free(ordre);
            LibereMemoireTableau(fichier, nbrquestions);
            return 1;
        }
        for (int j = 0; j < tailleInfo; j++) {
            info[j] = (char*)malloc(strlen(ligne) + 1);
            if (info[j] == NULL) {
                printf("Erreur d'allocation de mémoire pour info[%d]!\n", j);
                LibereMemoireTableau(info, j);
                free(info);
                free(ordre);
                LibereMemoireTableau(fichier, nbrquestions);
                return 1;
            }
        }
        printf("Extraction des infos pour la ligne %d\n", i + 1); //!!!!!!!!!!!!!temporaire
        ExtraireInfoLigne(fichier[n], info);
 
        // Allocation de mémoire pour question et autres éléments
        char* question = (char*)malloc(strlen(info[0]) + 1);
        if (question == NULL) {
            printf("Erreur d'allocation de mémoire pour question!\n");
            LibereMemoireTableau(info, tailleInfo);
            free(info);
            free(ordre);
            LibereMemoireTableau(fichier, nbrquestions);
            return 1;
        }
        strcpy(question, info[0]);
 
        int nbrprop = atoi(info[1]);
 
        printf("Nombre de propositions pour la question %d: %d\n", i + 1, nbrprop); //!!!!!!!!!!!!!!!!!!temporaire
 
        float* points = (float*)malloc(nbrprop * sizeof(float));
        if (points == NULL) {
            printf("Erreur d'allocation de mémoire pour points!\n");
            free(question);
            LibereMemoireTableau(info, tailleInfo);
            free(info);
            free(ordre);
            LibereMemoireTableau(fichier, nbrquestions);
            return 1;
        }
        char** propositions = (char**)malloc(nbrprop * sizeof(char*));
        if (propositions == NULL) {
            printf("Erreur d'allocation de mémoire pour propositions!\n");
            free(points);
            free(question);
            LibereMemoireTableau(info, tailleInfo);
            free(info);
            free(ordre);
            LibereMemoireTableau(fichier, nbrquestions);
            return 1;
        }
 
        for (int j = 0; j < nbrprop; j++) {
            propositions[j] = (char*)malloc(strlen(info[j + 2]) + 1);
            if (propositions[j] == NULL) {
                printf("Erreur d'allocation de mémoire pour propositions[%d]!\n", j);
                LibereMemoireTableau(propositions, j);
                free(propositions);
                free(points);
                free(question);
                LibereMemoireTableau(info, tailleInfo);
                free(info);
                free(ordre);
                LibereMemoireTableau(fichier, nbrquestions);
                return 1;
            }
            strcpy(propositions[j], info[j + 2]);
            points[j] = atof(info[nbrprop + j + 2]);
        }
 
        float maxpoints = 0;
        for (int j = 0; j < nbrprop; j++) {
            if (points[j] > 0) {
                maxpoints += points[j];
            }
        }
        scoremax += maxpoints;
 
        int* ordreprop = (int*)malloc(nbrprop * sizeof(int));
        if (ordreprop == NULL) {
            printf("Erreur d'allocation de mémoire pour ordreprop!\n");
            LibereMemoireTableau(propositions, nbrprop);
            free(propositions);
            free(points);
            free(question);
            LibereMemoireTableau(info, tailleInfo);
            free(info);
            free(ordre);
            LibereMemoireTableau(fichier, nbrquestions);
            return 1;
        }
        for (int t = 0; t < nbrprop; t++)
            ordreprop[t] = t;
        PermutationAleatoire(nbrprop, ordreprop);
 
        printf("Question %d: %s\n", i + 1, question);
        for (int j = 0; j < nbrprop; j++)
            printf("%d: %s\n", j + 1, propositions[ordreprop[j]]);
 
        float totalpoints = 0;
        int rep=1;
        do {
            printf("Entrez vos réponses (entrez 0 pour passer à la question suivante) : ");
            scanf("%d", &rep);
            if (rep >= 1 && rep <= nbrprop)
                totalpoints += points[ordreprop[rep - 1]];
        } while (rep != 0);
        scorefinal += totalpoints;
 
        // Libération de la mémoire pour chaque question
        for (int j = 0; j < nbrprop; j++) {
            free(propositions[j]);
        }
        free(propositions);
        free(points);
        free(ordreprop);
        free(question);
        LibereMemoireTableau(info, tailleInfo);
    }
 
    printf("Votre score final est %f sur %f\n", scorefinal, scoremax);
 
    // Libération de la mémoire
    LibereMemoireTableau(fichier, nbrquestions);
    free(fichier);
    free(ordre);
 
    return 0;
}
 
// Fonction pour libérer la mémoire allouée pour un tableau de chaînes de caractères
void LibereMemoireTableau(char** tableau, int taille) {
    if (tableau != NULL) {
        for (int i = 0; i < taille; i++) {
            free(tableau[i]);
        }
        free(tableau);
    }
}
 
void DebutChaine(char chaine[], char retour[]) {
    int i = 0;
    while (chaine[i] != '\0' && chaine[i] != ';') {
        retour[i] = chaine[i];
        i++;
    }
    retour[i] = '\0';
}
 
void ExtraireInfoLigne(char chaine[], char** info) {
    int pos = 0;
    int i = 0;
    while (chaine[pos] != '\0') {
        DebutChaine(&chaine[pos], info[i]);
        pos = pos + strlen(info[i]) + 1;
        i++;
    }
}
 
void PermutationAleatoire(int n, int tab[]) {
    srand((unsigned)time(NULL));
    for (int i = 0; i < n - 1; i++) {
        int j = i + rand() % (n - i);
        int k = tab[i];
        tab[i] = tab[j];
        tab[j] = k;
    }
}
 
char** ConversionFichier(const char* fichier, int* nblignes) {
    FILE* QCM;
    char ligne[512 + 1];
    QCM = fopen(fichier, "r");
    if (QCM == NULL)
        return NULL;
 
    *nblignes = 0;
    while (fgets(ligne, sizeof(ligne), QCM) != NULL)
        (*nblignes)++;
    rewind(QCM);
 
    char** tlignes = (char**)malloc(*nblignes * sizeof(char*));
    if (tlignes == NULL) {
        fclose(QCM);
        return NULL;
    }
 
    for (int i = 0; i < *nblignes; i++) {
        if (fgets(ligne, sizeof(ligne), QCM) == NULL) {
            for (int j = 0; j < i; j++)
                free(tlignes[j]);
            free(tlignes);
            fclose(QCM);
            return NULL;
        }
        tlignes[i] = (char*)malloc(strlen(ligne) + 1);
        if (tlignes[i] == NULL) {
            for (int j = 0; j < i; j++)
                free(tlignes[j]);
            free(tlignes);
            fclose(QCM);
            return NULL;
        }
        strcpy(tlignes[i], ligne);
    }
    fclose(QCM);
    return tlignes;
}

05/08/2024, 19h16
bacelar

Citation:

J'ai placé quelques printf inutile au projet pour essayer de localisé l'erreur mais je n'y parviens pas.

Heu, sinon, l'utilisation d'un débogueur ? c'est pour l'année prochaine, ou c'est pour les faibles ??? :weird:

Franchement, ce code, c'est une catastrophe de code C qui essaye de se faire compiler par un pauvre compilateur C++ qui fait tous pour vous montrer comment "mieux" coder en C++.

Sinon, postez le code avec des balises [[CODE]] SVP. (le bouton [#] dans la "commande bar" de l'éditeur de ce forum)

Dans du code C++ moderne, on n'utilise quasiment jamais de pointeurs nues, on n'utilise jamais ni malloc, ni free, rarement de tableau à la C ou printf (et jamais pour débuguer) ou de scanf.
Les chaines C sont remplacées par des "std::string".
Les tableaux à la C sont très souvent remplacées par des "std::vector".

En résumé, votre code contient 99,9% de code C et 0.01 de code C++ (et c'est dans l'usage complètement inutile, voire faux, de #include)

Si c'est un prof. qui vous a appris à coder en C++ comme ça, c'est un charlatan qui a confondu le (mauvais) C (des années 70) avec le C++, qui a été normalisé pour la première fois en 1998.

Si vous voulez "sauver" votre code C, je pense que c'est plus dans la partie C du forum qu'ici, dans la partie C++ du forum, que vous trouverez des volontaires pour vous faire désapprendre bien des choses en C.

Si ce n'est pas un prof., je ne connais pas de source/cours C++ francophone tellement mauvais pour faire écrire de tels horreurs. Si c'est bien une source d'apprentissage, pouvez-vous nous la communiquer, SVP ?
07/08/2024, 17h13
Obsidian
Citation:

Envoyé par franbast10

Bonjour à tous ! Je jette une bouteille à la mer car je suis désespéré, je débute en programmation en C++

Bonjour et bienvenue.
Ton programme est effectivement écrit exclusivement en langage C et non C++. Je le déplace donc dans le forum approprié.

En effet, le C++ a beau avoir été construit « au dessus » du C et s'efforcer officiellement de rester compatible avec lui chaque fois que c'est possible, il s'agit bien d'un langage distinct avec une approche propre. On ne construit donc pas un programme C++ de la même façon qu'un programme C même si les postulats de base posés ici font qu'en pratique, un compilateur C++ sera capable de compiler la plupart des programmes C s'ils ne rentrent pas trop dans les détails.

Il est donc important de bien distinguer les deux et de savoir quel cours t'a été dispensé ici, C ou C++, pour savoir vers quelle branche on s'oriente.

Citation:

Je pense que mon projet comporte beaucoup d'erreurs car il ne tourne pas. Ce sont principalement des erreurs d'allocations de mémoire.
Si quelqu'un saurait relire mon code et repérer d'éventuelles fautes ce serait très aimable.
J'ai placé quelques printf inutile au projet pour essayer de localisé l'erreur mais je n'y parviens pas.

Effectivement, as-tu déjà appris à utiliser un débogueur ou pas encore ?

Après essai sur ma machine, ton programme compile sans erreur en C99. Il y a quelques remarques que l'on pourrait faire mais rien de critique pour le moment. Au lancement dans le débugueur, j'ai effectivement un crash à la fin de ton programme. La backtrace dit :
```
(gdb) bt
#0  0x00007ffff7e4a834 in __pthread_kill_implementation () from /lib64/libc.so.6
#1  0x00007ffff7df88ee in raise () from /lib64/libc.so.6
#2  0x00007ffff7de08ff in abort () from /lib64/libc.so.6
#3  0x00007ffff7de17d0 in __libc_message.cold () from /lib64/libc.so.6
#4  0x00007ffff7e547a5 in malloc_printerr () from /lib64/libc.so.6
#5  0x00007ffff7e56c6f in _int_free () from /lib64/libc.so.6
#6  0x00007ffff7e593de in free () from /lib64/libc.so.6
#7  0x0000000000401c0b in main () at ttt.c:188
```
… ce qui correspond à :
Code:

1 2 3 4 5 6 7 8 9 10 … 186: // Libération de la mémoire 187: LibereMemoireTableau(fichier, nbrquestions); 188: free(fichier); 189: free(ordre); 190: 191: return 0; }
Or, tu as écrit une fonction juste au dessus spécialement pour désallouer proprement les tableaux alloués en cascade. Au moment de l'appel à la ligne 188, la mémoire référencée par « fichier » a déjà été libérée par la ligne 187. Il suffit donc de supprimer cette ligne 188 pour que tout rentre dans l'ordre.

Par ailleurs, je n'ai pas eu besoin de #include "..\..\Modules\Lecture\lecture.h" ici mais peut-être ton code est-il en fait plus grand que l'extrait présenté ici.

Citation:

Envoyé par bacelar

En résumé, votre code contient 99,9% de code C et 0.01 de code C++ (et c'est dans l'usage complètement inutile, voire faux, de #include)
Si c'est un prof. qui vous a appris à coder en C++ comme ça, c'est un charlatan qui a confondu le (mauvais) C (des années 70) avec le C++, qui a été normalisé pour la première fois en 1998.

J'ai déplacé le fil dans le forum C.

Pour le reste, je trouve ces propos exagérés. Je n'ai pas encore parcouru son code en profondeur (qui n'est certes pas du C++) mais sur la forme, il me semble être un programme C relativement correct.
Par ailleurs, je n'ai justement trouvé aucune référence au C++, pas même dans les #include en question.
07/08/2024, 17h28
foetus

:whistle: Il faut fusionner les 2 threads qui semblent identiques : Problèmes de mémoire sur un projet en C
07/08/2024, 17h32
Obsidian

Citation:

Envoyé par foetus

:whistle: Il faut fusionner les 2 threads qui semblent identiques

Effectivement. J'attends le retour du PO d'abord et j'avise ensuite.
07/08/2024, 17h45
franbast10

Merci beaucoup pour vos conseils et pour votre aide !

En effet, on ne nous a jamais vraiment appris à utilisé un débogueur.

Et oui j'ai fait un autre canal dans la section C après avoir reçu un message disant que mon code était en C c'est pourquoi il y a 2 canaux similaires.