methode avec struct

**l1informatique** · 01/12/2015, 11h38

Bonjour,

J'essaye de répondre a cette question :Ecrivez un programme qui lit une suite d'adresses dans un fichier et les met dans une table de structures.

Code :

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

1
2
3
4
mon fichier contient deux lignes d'adresses suivantes:
 
6 rue flobet 352100
5 rue guy de ronoit 65200

j'ai essayé de réaliser ce code mais il m'envoit erreur de segmentation

Code :

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

Erreur de segmentation (core dumped)

pourriez vous m'aider je bloque sur ça depuis des jours je ne trouves pas ou est le problème .

Merci

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
typedef struct adresse adresse1;
struct adresse {
int numero;
int postale;
char *voie;
char *nomdelavoie;
char *ville;
};
 
int main(void) {
int n = 0, i;
struct adresse *adresse1;
    FILE *f = fopen("adresses.txt","r");
    if (!f) {
        printf("Impossible d'ouvrir le fichier\n");
        return 1;
    }
    char ligne[256];
    while (fgets(ligne,256,f)) {
        if (n == 0) {
            adresse1 = malloc(sizeof(struct adresse)*(n+1));
        }
        else {
            adresse1 = realloc(adresse1,sizeof(struct adresse)*(n+1));
        }
 
fscanf(f, "%d",&adresse1[n].numero);
fscanf(f, "%s",adresse1[n].voie);
fgets(adresse1[n].nomdelavoie , sizeof(adresse1[n].nomdelavoie), f);
fscanf(f, "%s", adresse1[n].ville);		
fscanf(f, "%d",&adresse1[n].postale);
        n++;
    }
    for (i = 0; i < n; i++)
    {
		printf("Numero : %d\n", adresse1[i].numero);
		printf("voie : %s\n", adresse1[i].voie);
		printf("Nom de la voie :%s\n", adresse1[i].nomdelavoie);
		printf("ville :%s\n", adresse1[i].ville);
		printf("postale : %d\n", adresse1[i].postale);
	fclose(f);
    free(adresse1);
    }
 
    return 0;
}

**ternel** · 01/12/2015, 11h44

Une erreur de segmentation signifie que tu as tenté d'écrire dans de la mémoire en lecture seule, ou de déréférencer un pointeur vers de la mémoire interdite d'accès
typiquement, c'est déréférencer un pointeur NULL ou écrire dans un pointeur sur de la mémoire non allouée

pointeur = malloc.

Comment veux-tu faire un scanf (ou autre fgets) si tu n'a pas alloué la chaine à écrire?

Par ailleurs, realloc peut couter très cher, envisage de faire une liste chainée de struct adresse.

**nibors** · 01/12/2015, 14h40

Je ne suis pas expert, mais a mon avis fclose(f) et free(adresse1) a l'interieur de ton FOR vont poser probleme aussi

**o_Nejma** · 01/12/2015, 22h55

Bonjour,

Code :

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

1
2
3
4
5
 
              adresse1 = malloc(sizeof(struct adresse)*(n+1));
        }
        else {
             adresse1 = realloc(adresse1,sizeof(struct adresse)*(n+1));

est ce que cette allocation suffit pour que tu puisse accéder à l'espace mémoire pointé par:
char *voie;
char *nomdelavoie;
char *ville;
?

Cordialement,

**foetus** · 02/12/2015, 00h02

En C, il n'y a pas de constructeurs, mais on peut coder les initialisations proprement

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
void init_adresse1(adresse1** new_address) {
    if (new_address == NULL) { return; }
 
//  Maybe test (*new_address)
 
    (*new_address) = malloc( sizeof(struct adresse) );
 
    if ((*new_address) != NULL) {
        (*new_address)->numero  = 0;
        (*new_address)->postale = 0;
        (*new_address)->voie        = NULL;
        (*new_address)->nomdelavoie = NULL;
        (*new_address)->ville       = NULL;
    }
}
 
 
void init_adresse1_with_city(adresse1** new_address, const char* city) {
    if (new_address == NULL) { return; }
 
//  Maybe test (*new_address)
 
    (*new_address) = malloc( sizeof(struct adresse) );
 
    if ((*new_address) != NULL) 
        (*new_address)->numero  = 0;
        (*new_address)->postale = 0;
        (*new_address)->voie        = NULL;
        (*new_address)->nomdelavoie = NULL;
 
        if (city != NULL) {
            size_t len = strlen(city);
 
            if (len > 0) {
                (*new_address)->ville = malloc((len + 1) /** sizeof(char)*/);
 
                if ((*new_address)->ville != NULL) {
                    strncpy((*new_address)->ville, len, city);
// or               strcpy((*new_address)->ville, city);
 
                    (*new_address)->ville[len] = '\0';
                }
            } else {
                (*new_address)->ville = NULL;
            }
        } else {
            (*new_address)->ville = NULL;
        }
    }
}

Invité · 02/12/2015, 00h23

Envoyé par l1informatique

Bonjour,

J'essaye de répondre a cette question :Ecrivez un programme qui lit une suite d'adresses dans un fichier et les met dans une table de structures.

Code :

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

1
2
3
4
mon fichier contient deux lignes d'adresses suivantes:
 
6 rue flobet 352100
5 rue guy de ronoit 65200

j'ai essayé de réaliser ce code mais il m'envoit erreur de segmentation

Code :

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

Erreur de segmentation (core dumped)

pourriez vous m'aider je bloque sur ça depuis des jours je ne trouves pas ou est le problème .

Merci

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
typedef struct adresse adresse1;
struct adresse {
int numero;
int postale;
char *voie;
char *nomdelavoie;
char *ville;
};
 
int main(void) {
int n = 0, i;
struct adresse *adresse1;
    FILE *f = fopen("adresses.txt","r");
    if (!f) {
        printf("Impossible d'ouvrir le fichier\n");
        return 1;
    }
    char ligne[256];
    while (fgets(ligne,256,f)) {
        if (n == 0) {
            adresse1 = malloc(sizeof(struct adresse)*(n+1));
        }
        else {
            adresse1 = realloc(adresse1,sizeof(struct adresse)*(n+1));
        }
 
fscanf(f, "%d",&adresse1[n].numero);
fscanf(f, "%s",adresse1[n].voie);
fgets(adresse1[n].nomdelavoie , sizeof(adresse1[n].nomdelavoie), f);
fscanf(f, "%s", adresse1[n].ville);		
fscanf(f, "%d",&adresse1[n].postale);
        n++;
    }
    for (i = 0; i < n; i++)
    {
		printf("Numero : %d\n", adresse1[i].numero);
		printf("voie : %s\n", adresse1[i].voie);
		printf("Nom de la voie :%s\n", adresse1[i].nomdelavoie);
		printf("ville :%s\n", adresse1[i].ville);
		printf("postale : %d\n", adresse1[i].postale);
	fclose(f);
    free(adresse1);
    }
 
    return 0;
}

1) T'aurais du commencer par chercher sur developpez il y-a d'excellent cours sur les pointeurs ou sur le C lui même (je code pour le plaisir, je suis autodidacte et avant d'aborder un nouveau langage de programmation je me documente, oui monsieur, J'apprends au minimum les bases (pas de bibliothèque, pas de fonction nécessitant l'inclusion de bibliothèque))

2) Le langage C ne fera que ce que tu lui demande, il ne lie pas dans les pensés des gens, alors ne t'attends pas a des miracles, on t'as dit en haut que t'as erreur core dump parce que....
C'est simple à chaque fois que tu ton fameux:

Code :

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

 adresse1 = malloc(sizeof(struct adresse)*(n+1))

ou

Code :

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

adresse1 = alloc(sizeof(struct adresse)*(n+1))

Tu alloues de la mémoire que pour des structures de type adresse, or ta fameuse structure est composé de deux int et de trois pointeurs sur des char, (trois pointeurs et non trois chaînes de caractère), quand tu fait:

Code :

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

1
2
3
fgets(adresse1[n].nomdelavoie , sizeof(adresse1[n].nomdelavoie), f);
fscanf(f, "%s", adresse1[n].ville);		
fscanf(f, "%s",adresse1[n].voie);

tu envoies les char lu du fichier vers des espaces mémoire sensé contenir que des char pointaient respectivement par adresse1[n].ville, ...[n].voie, et ...nomdelavoie, et tes pointeurs ils pointent sur quoi? On ne s'est pas puisque tu n'as pas allouer de la mémoire, bref t'aurais du faire un petit:

Code :

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

1
2
3
4
 
adresse1[n].nomdelavoie = malloc( 256*sizeof( char));/* généralement sizeof(char) == 1*/
adresse1[n].ville = malloc( 256*sizeof( char));
adresse1[n].voie = malloc( 256*sizeof( char));

avant de tes scanf, en supposant que le nombre de caractère lu pour nomdelavoie, ville et voie ne dépasse pas 256 caractères (à chaque lecture).
Voilà c'étais pas aussi compliquer mais t'aurais du commencer par chercher.
Tant qu'on y est quand on fait un

Code :

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

 scanf( File *, "%s", char *)

la fonction rajoute un '\0' dans la chaîne de caractère ou est ce qu'on doit le faire explicitement? Et aussi est ce que c'est le cas dans toutes peut importe le compilateur, etc...?

**l1informatique** · 02/12/2015, 10h31

Mon fichier texte contient deux adresses :

Code :

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

1
2
3
 
5 rue Guy de la Brosse Paris 75005	
3 rue florent dancourt suresnes 92100

On suivant vos conseils si j'ai bien compris j'ai ajouté des modification au code mais je me retrouves avec des erreurs.

Code :

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

1
2
3
4
5
6
7
8
 
Numero : �:�7�8�9�
voie : rue
Nom de la voie :florent
ville :dancourt
postale : suresnes
*** Error in `./SD111': corrupted double-linked list: 0x08e93168 ***
Abandon (core dumped)

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
typedef struct adresse adresse1;
struct adresse {
char *numero;
char *postale;
char *voie;
char *nomdelavoie;
char *ville;
};
 
int main(void) {
int n = 0, i;
struct adresse *adresse1;
    FILE *f = fopen("adresses.txt","r");
    if (!f) {
        printf("Impossible d'ouvrir le fichier\n");
        return 1;
    }
    char ligne[256];
    while (fgets(ligne,256,f)) {
        if (n == 0) {
            adresse1 = malloc(sizeof(struct adresse)*(n+1));
        }
adresse1[n].numero = malloc( 256*sizeof( char));
adresse1[n].voie = malloc( 256*sizeof( char));
adresse1[n].nomdelavoie = malloc( 256*sizeof( char));/* généralement sizeof(char) == 1*/
adresse1[n].ville = malloc( 256*sizeof( char));
adresse1[n].postale = malloc( 256*sizeof( char));
 
fscanf( f , "%s %s %s %s %s", adresse1[n].numero, adresse1[n].voie, adresse1[n].nomdelavoie, adresse1[n].ville ,adresse1[n].postale);
 
 
n++;  
    }
 
    for (i = 0; i < n; i++)
    {
		printf("Numero : %s\n", adresse1[i].numero);
		printf("voie : %s\n", adresse1[i].voie);
		printf("Nom de la voie :%s\n", adresse1[i].nomdelavoie);
		printf("ville :%s\n", adresse1[i].ville);
		printf("postale : %s\n", adresse1[i].postale);
    fclose(f);
    free(adresse1);
    }
 
    return 0;
}

**foetus** · 02/12/2015, 10h53

Ligne 48

, réfléchis un peu

Ligne 47, pourquoi dans la boucle?

Édit: kdo

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
void free_adresse1(adresse1** new_address) {
    if (new_address == NULL) { return; }
 
    if ((*new_address) != NULL) {
        (*new_address)->numero  = 0;
        (*new_address)->postale = 0;
        if ((*new_address)->voie != NULL) { free((*new_address)->voie); (*new_address)->voie = NULL;  }
        if ((*new_address)->nomdelavoie != NULL) { free((*new_address)->nomdelavoie); (*new_address)->nomdelavoie = NULL;  }
        if ((*new_address)->ville != NULL) { free((*new_address)->ville); (*new_address)->ville = NULL;  }
 
        free((*new_address));
 
        (*new_address) = NULL;
    }
}

methode avec struct

C

Discussions similaires

Partager

Partager