Discussion :

[don1design]

bonjour, tout monde

j'ai un fichier service.txt qui contient des infos comme suit

Impression_noir 10
Impression_couleur 25
Scannarisation 15
Plastification_Petite 20
Plastification_Grande 40
Reliure_Petit 30
Reliure_Grande 50

jaimerais charge chaque service plus son tarif dans une liste chainee.
jai ecrit le programme que jai compile avec code::blocks
qui ne me montre aucune erreur mais lorsque je lexecute il se plante
pourriez vous bien maider a trouver lerreur.

voici le 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
#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
 
#define TAILLE_MAX 100
 
//les prototypes
int calculNbrDeService();
void chargement();
void affichage();
 
//les variables globales
int qtyService=0, i;
//services service[qtyService];
 
 
//Declaration de la structure SERVICE
typedef struct services services;
struct services
{
    char type[50];
    int tarif;
 
    struct services *precedent;
    struct services *suivant;
};
 
//Declation et initialisation de variable structuree
services *debut=NULL;
services *fin=NULL;
services *nouveau;
services *courant;
 
//FONCTION principale
int main()
{
    int calculNbrDeService();
 
    printf("Le nombre de service est ");
 
    qtyService = calculNbrDeService();
 
    printf("%d", qtyService);
    printf("\nLes service sont\n");
 
    chargement();
    affichage();
 
    return 0;
}
 
//Calcul du nombre de services disponible
int calculNbrDeService()
{
    int nbrService=0;
 
    FILE* fichier = NULL;
    char chaine[TAILLE_MAX] = "";
 
    fichier = fopen("service.txt", "r");
 
    if (fichier != NULL)
    {
        while (fgets(chaine, TAILLE_MAX, fichier) != NULL) // On lit le fichier au complet
        {
            nbrService++; //augmentation du nombre de service
        }
        fclose(fichier);
    }
    return nbrService;
}
 
//CHARGEMENT des services dans la liste chainee
void chargement()
{
    //CREATION du pointeur FILE
    FILE* fichier = NULL;
    char ttype[50];
    int ttarif;
 
    //OUVERTURE du fichier service.txt
    fichier = fopen("service.txt", "r");
 
    //Allocation d. de memoire
    nouveau=(services*)malloc(sizeof(services));
    if(nouveau!=NULL)
    {
 
        for ( i=0 ; i<qtyService ; i++ )
        {
            fscanf(fichier, "%s %d\n", ttype, &ttarif);
            nouveau->type[0]=&ttype[0];
            nouveau->tarif=ttarif;
            nouveau->precedent=NULL;
            nouveau->suivant=debut;
            debut->precedent=nouveau;
            debut=nouveau;
        }
    }
    fclose(fichier);
}
 
//Affichage de la liste
void affichage()
{
    courant=debut;//Point de depart
    while(courant!=NULL)
    {
        printf("%s %d", courant->type, courant->tarif);
        courant=courant->suivant;
    }
    getch();
}

[snowpy]

je te conseil de faire une par une tes fonctions déjà et de print le résultat quand tu peux genre nbservice, incorpore des printf aussi quand le fichier a bien été ouvert et quand le fichier n'a pas été ouvert

[diogene]

*

Code :

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

1
2
3
4
//les prototypes
int calculNbrDeService();
void chargement();
void affichage();

Ce ne sont pas des prototypes. Un prototype doit donner le type de chaque paramètre ou void si il n'y en a pas :

Code :

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

1
2
int calculNbrDeService(); // n'est pas un prototype
int calculNbrDeService(void); // est un prototype.

*

Code :

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

nouveau->type[0]=&ttype[0];

est faux et un compilateur bien règlé aux niveaux des avertissements doit le signaler :
&ttype[0] est une adresse char *
nouveau->type[0] est un entier char
Probablement tu voulais faire strcpy(nouveau->type, ttype)

* dans chargement(), un et un seul services est créé par allocation dynamique (le malloc() est en dehors de la boucle)

* Le code est mal construit :
- La fonction calculNbrDeService() ne sert à rien : tu n'as pas besoin de connaitre à priori le nombre de services avant d'appeler chargement() et chargement() peut, dans la foulée, calculer et renvoyer le nombre de services qu'il a mis dans la liste si on a besoin de cette grandeur ultérieurement.
- Certaines des variables sont scandaleuses en tant que globales (i, nouveau, courant) ou inutilisées (fin).

* Pour partir sur des bases plus saines, je te suggère
- d'éliminer toutes les variables globales. Le code sera bien plus clair si on voit les arguments des fonctions et ceci rendra chaque fonction autonome.
- de supprimer calculNbrDeService() et d'incorporer ce calcul, si besoin est, dans chargement()
- de créer un type représentant la liste. Par exemple :

Code :

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

1
2
3
4
5
6
typedef struct
{
 services *debut;
 services *fin;   // si besoin
 int qtyService ; // si besoin
 }ListeServices;

avec un main() qui prend l'allure :

Code :

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

1
2
3
4
5
6
7
int main(void)
{
    ListeServices liste = {NULL,NULL,0};
    chargement("service.txt",&liste);
    affichage(&liste);
    return 0;
}

en modifiant en conséquence les fonctions chargement() et affichage()

[don1design]

je viens de porter des corrections
jarrive a lire le fichier txt mais il ne lit que la derniere ligne
en depit du fait que jai mit le curseur au debut tu fichier
je ne lis que la derniere ligen

voic 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
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
#include <stdio.h> 
#include <stdlib.h> 
#include <conio.h> 
#include <string.h> 
#include <time.h> 
 
#define TAILLE_MAX 100 
 
//les prototypes 
int calculNbrDeService(); 
void chargement(); 
void affichage(); 
void chargementSiVide(); 
 
//les variables globales 
int qtyService=0, i=0, taille=0; 
//services service[qtyService]; 
 
 
//Declaration de la structure SERVICE 
typedef struct services services; 
struct services 
{ 
char type[50]; 
int tarif; 
 
struct services *precedent; 
struct services *suivant; 
}; 
 
//Declation et initialisation de variable structuree 
services *debut=NULL; 
services *courant; 
services *nouveau; 
services *fin=NULL; 
 
//FONCTION principale 
int main() 
{ 
int calculNbrDeService(); 
 
printf("Le nombre de service est "); 
 
qtyService = calculNbrDeService(); 
 
printf("%d", qtyService); 
printf("\n\nLes service sont\n"); 
 
chargement(qtyService); 
affichage(); 
 
return 0; 
} 
 
//Calcul du nombre de services disponible 
int calculNbrDeService() 
{ 
int nbrService=0; 
 
FILE* fichier = NULL; 
char chaine[TAILLE_MAX] = ""; 
 
fichier = fopen("service.txt", "r"); 
 
if (fichier != NULL) 
{ 
while (fgets(chaine, TAILLE_MAX, fichier) != NULL) // On lit le fichier au complet 
{ 
nbrService++; //augmentation du nombre de service 
} 
 
fseek(fichier, 0, SEEK_SET); 
fclose(fichier); 
} 
return nbrService; 
} 
 
//CHARGEMENT des services dans la liste chainee 
void chargement() 
{ 
//CREATION du pointeur FILE 
FILE* fichier = NULL; 
 
//OUVERTURE du fichier service.txt 
fichier = fopen("service.txt", "r"); 
fseek(fichier, 0, SEEK_SET); 
 
//Allocation d. de memoire 
nouveau=(services*)malloc(sizeof(services)); 
if(nouveau!=NULL) 
{ 
//chargement quand il y a deja des elements 
if (taille!=0) 
{ 
for ( i=0 ; i<qtyService ; i++ ) 
{ 
fscanf(fichier, "%s %d\n", &nouveau->type, &nouveau->tarif); 
nouveau->precedent=NULL; 
nouveau->suivant=debut; 
debut->precedent=nouveau; 
debut=nouveau; 
taille++; 
} 
} 
else 
{ 
//chargementSiVide 
for ( i=0 ; i<qtyService ; i++ ) 
{ 
fscanf(fichier, "%s %d\n", &nouveau->type, &nouveau->tarif); 
nouveau->precedent=debut; 
nouveau->suivant=fin; 
debut=nouveau; 
fin=nouveau; 
taille++; 
} 
} 
} 
fclose(fichier); 
} 
 
//Affichage de la liste 
void affichage() 
{ 
courant=debut;//Point de depart 
for ( i=0 ; i<qtyService ; i++) 
{ 
printf("%d.- %s %d\n", i, courant->type, courant->tarif); 
courant=courant->suivant; 
} 
getch(); 
} 
 
 
void freeList() 
{ 
services*suivant; 
while(debut!=NULL) 
{ 
suivant=debut->suivant; 
free(debut); 
debut=suivant; 
} 
printf("It's free!!;)"); 
}

[diogene]

Si tu préfères ne pas tenir compte des observations qui te sont faites, comment peut-on t'aider efficacement?

@diogene :

* Ce ne sont pas des prototypes. Un prototype doit donner le type de chaque paramètre ou void si il n'y en a pas :

Code :

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

1
2
int calculNbrDeService(); // n'est pas un prototype
int calculNbrDeService(void); // est un prototype.

* Dans chargement(), un et un seul services est créé par allocation dynamique (le malloc() est en dehors de la boucle)
* Pour partir sur des bases plus saines, je te suggère
- d'éliminer toutes les variables globales. Le code sera bien plus clair si on voit les arguments des fonctions et ceci rendra chaque fonction autonome.
- de supprimer calculNbrDeService() ....
....

De plus, il s'ajoute maintenant quelques remarques :
* les fseek() sont inutiles
* appel incohérent :

Code :

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

1
2
3
chargement(qtyService); 
...
void chargement() ....

* on a encore récupéré une globale de plus (taille). Ca n'améliore pas le code.

Ne continue pas à développer ce code mal fichu : fais les modifications qui s'imposent et poste le code si tu as des problèmes.