Discussion :

[ethan007]

Bonsoir à tous, je fais un projet et je viens de rencontrer un petit disfonctionnement je cherche des explications...
J'ai un fichier qui contient la définiton d'une matrice (en première ligne du fichier on a le nombre de ligne et le nombre de colonne (de la matrice), puis la suite c'est la matrice elle même)
EX:
3 3
1 2 3
4 5 6
7 8 9

Mon but est d'écrire une fonction qui lit le contenu du fichier et renvoie tous ce contenu (y compris la première ligne) sous forme de char* et voici mon code (qui fonctionne..fin je pense):
gestion_fichiers.c

Code c :

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "gestion_fichiers.h"
 
char *lire_fichier(char *nom_fichier){
    FILE *fichier = fopen(nom_fichier, "r");
    int nbreL = fgetc(fichier);
    nbreL = (int)nbreL - '0';//cast
    fgetc(fichier); //Espace entre le nombre de lignes et le nombre de colonnes au début du fichier
    int nbreC = fgetc(fichier);
    nbreC = (int)nbreC - '0';//cast
 
    int taille_ligne = nbreC + (nbreC -1);//La taille de la ligne à lire se calcule en incluant les espaces, pour n colonnes il y a n-1 espaces
    char *current_line_data = (char*)malloc(nbreC * sizeof(char));
    char *file_data = (char*)malloc(nbreC * nbreL * sizeof(char)+3); //will store the retrieved data # +3 is the nbreL space and nbreC
    printf("L = %d et C = %d", nbreL, nbreC);
    fseek(fichier, 0, SEEK_SET);//Get the cursor back to first ligne
    strcat(file_data, fgets(current_line_data, 4, fichier));//La première ligne du fichier
 
    if(fichier != NULL){
        while(fgets(current_line_data, taille_ligne +1, fichier)){
 
            strcat(file_data, current_line_data);//We are sure the space is enough for all datas because we calculated it earlier
        }
    }else{
        printf("The file %s doest not exist! exiting...", nom_fichier);
        exit(1);//Error can be handleled
    }
 
    free(current_line_data);
    return file_data;
}
 
void desallouer_donnees_fichier(char *ptr_datas){
    free(ptr_datas);
}

le .h:
gestion_fichiers.h

Code c :

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

1
2
3
4
5
 
#include <stdio.h>
#include <stdlib.h>
char *lire_fichier(char *nom_fichier);
void desallouer_donnees_fichier(char *ptr_datas);

Dans le main j'appelle ma fonction avec le code suivant:
main.c:

Code c :

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
 
#include <stdio.h>
#include <stdlib.h>
#include "gestion_fichiers.h"
 
int main(){
 
 
printf("\n-----------GESTION DU FICHIER!-------\n");
	char *donnees = lire_fichier("file.txt");
	printf("Affichage des données:\n %s", donnees);
	printf("\n-----------AUREVOIR !-------\n");
 
	//desallouer_matrice(mat);
	//desallouer_matrice(produit);
	desallouer_donnees_fichier(donnees);
	return 0;
}

Ce code marche bien mais lorsque je déboguait ailleurs j'ai constaté que la ligne 16 du fichier "gestion_fichiers.c" n'alloue pas un tableau de char de taille nbreC * sizeof(char)
En réalité j'a remplacé cette ligne par:

Code c :

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

1
2
 
char *current_line_data = (char*)malloc(1);

qui normalement devrait allouer 1 octet, ce qui devait me faire apparaitre une belle érreur de ségmentation (mon ami de tous les jours :p) à la ligne 24 mais au contraire tous ce passe bien : je me demande donc est ce que le fait que j'ai utilisé un malloc ou fgets le pousse recalculer la nouvelle taille à chaque tour de la boucle qui est à la lgine 24???

Best regards.

[Bousk]

Donc tu veux juste lire un fichier et stocker son contenu dans un char* ?
fopen
fseek
ftell
rewind
fread

[ethan007]

Bonsoir merci pour cette réponse rapide, en fait j'ai oublié de dire que c'est un projet d'école et les contraintes c'est d'utiliser uniquement fgets pour renvoyer le char*.
or fgets en boucle écrase à chaque itération.

[dalfab]

Bonjour,

Ligne 16, tu n'alloues pas assez de caractères, il faut aussi réserver de la place pour le terminateur et le caractère de retour chariot.
Ligne 17, je n'ai refait le calcul. Mais j'ai de gros doutes sur le nombre réservé.
Ligne 20, tu concatènes dans file_data, donc tu supposes qu'initialement elle est vide, mais où est-ce fait? Et pire qu'y a-t-il dans current_line_data? On n'y a jamais rien mis.

Maintenant pourquoi 1 semble suffire ligne 16? Si on n'alloue pas pas assez de place on a un "undefined behavior", ce qui veut dire que tout est possible et en particulier cela peut sembler fonctionner. Les caractères en trop se retrouvent dans un endroit non réservé, non prévu qui peut casser une autre partie du logiciel. Pour certains compilateurs quand on demande 1 octet on en obtient parfois 7 de plus en "cadeau", c'est ce qui doit ce passer dans ton cas.

[ethan007]

Bonsoir

Bonjour,

Ligne 16, tu n'alloues pas assez de caractères, il faut aussi réserver de la place pour le terminateur et le caractère de retour chariot.

Exact, merci

Ligne 17, je n'ai refait le calcul. Mais j'ai de gros doutes sur le nombre réservé.

En effet j'aurai dû penser aux espaces

Ligne 20, tu concatènes dans file_data, donc tu supposes qu'initialement elle est vide, mais où est-ce fait?

Bon je devrait lui affecter NULL avant de faire le malloc?

Et pire qu'y a-t-il dans current_line_data? On n'y a jamais rien mis.

current_line_data est rechargé à chaque tour de boucle while(gets..) de la ligne 23

Maintenant pourquoi 1 semble suffire ligne 16? Si on n'alloue pas pas assez de place on a un "undefined behavior", ce qui veut dire que tout est possible et en particulier cela peut sembler fonctionner. Les caractères en trop se retrouvent dans un endroit non réservé, non prévu qui peut casser une autre partie du logiciel. Pour certains compilateurs quand on demande 1 octet on en obtient parfois 7 de plus en "cadeau", c'est ce qui doit ce passer dans ton cas.

Ah soulagement je cherchait une explication là dessus, c'est donc peu être le fameux "undefined behavior" qui a de la chance d'aller dans des blocs inutilisés..
[/QUOTE]

Pour certains compilateurs quand on demande 1 octet on en obtient parfois 7 de plus en "cadeau", c'est ce qui doit ce passer dans ton cas

mais pourquoi est ce que le compilateur devrait allouer ne serait-ce que 1 octet de plus alors que j'en ai demandé un nombre exact? imaginons un programmeur sur un carte avec quelques Ko..

[Kannagi]

mais pourquoi est ce que le compilateur devrait allouer ne serait-ce que 1 octet de plus alors que j'en ai demandé un nombre exact? imaginons un programmeur sur un carte avec quelques Ko..

Cela n'est pas à cause de ton compilateur mais de ton processeur et c'est ce qu'on appelle l'alignement.

Sur une machine de quelque ko , les contraintes ne sont pas les mèmes et donc un processeur 8 bits n'a aucune mal a gérer la mémoire si elle n'est pas aligné.
(Mais si tu as vraiment besoin d'économiser la mémoire jusqu'a ce que la pile te pose souci , alors tu peux aller en assembleur).

[dalfab]

Bon je devrait lui affecter NULL avant de faire le malloc?

Avant malloc!

Code :

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

1
2
3
4
5
6
7
8
// que fait ce code ?
char *current_line_data = "hello";
current_line_data = NULL;
current_line_data = malloc(nbreC +2);
// ou, même question, que fait ce code ?
double a = 3.14159;
a = 0;
a = 5 * 3;

Après malloc, on peut initialiser le 1er caractère à '\0' ou commencer par strcpy() plutôt qur strcat().

current_line_data est rechargé à chaque tour de boucle while(gets..) de la ligne 23

Et par quelle magie, en ligne 20 c'est déjà rempli?

mais pourquoi est ce que le compilateur devrait allouer ne serait-ce que 1 octet de plus alors que j'en ai demandé un nombre exact? imaginons un programmeur sur un carte avec quelques Ko..

J'ajoute aux réponses précédentes que sur les cartes avec quelques Ko (c'est mon métier) on n'alloue jamais rien saufs de rares grands blocs (c-a-d quelques centaines d'octets), la perte relative est donc négligeable.