Discussion :

[Master C]

Bonjour voilà j'arrive à ajouter mon nouveau produit dans la structure mais pour l'affichage sequentiel , j'ai un gros gros problème
Au début j'ai 4 produit donc je nomme une variable nbel=4 puis je la fait passer en paramètre à ma fonction ajoute etc... puis incrémente nbel de 1 puis je fais un return pour la renvoyé au main..
je déclare un pointeur pt dans la fonction affiche pour qu'il pointe sur le nouveau nbel...
et ensuite while(i<*pt)
affiche etc....

C'est logique ?

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
#include<stdio.h>
#include<windows.h>
 
int Ajout(struct produit *,int);
void affiche(struct produit *,int *);
 
 
 
 
 
struct produit
{
 
long IDproduit;
char NomProduit[30];
char UniteMesure[10];
float Prix;
char TypeProduit[30];
int Categorie;
 
};
 
struct INDEX
{
long NumProduit ;
char NomProduit [30];
int *correspondant;
};
 
void main()
{
        int nbel=4,i,choix;
 
 
 
////////////////////////////////////////////////////////////////
 
        struct produit prod[100]=
        {
                {100,"Imprimante Jet d'encre","Piece",80,"fini",0},
                {101,"Ecran","Piece",300,"fini",0},
                {102,"Pommes","Kg",2,"fini",0},
                {103,"Ammoniac","Kg",150,"matiere premiere",1}
        };
 
        struct INDEX index[100]=
        {
 
                {103,"Ammoniac",},
                {101,"Ecran",},
                {100,"imprimante",},
                {102,"Pommes",},
 
        };
 
////////////////////////////////////////////////////////////////
i=0;
        while(i<1)
        {
        printf("\t\tMenu\n");
                printf("1. Ajout d'un produit\n");
                printf("2. Affichage sequentiel des produits\n");
                printf("3. Affichage indexe\n");
                printf("4. Quitter le programme\n");
 
                printf("Choix : ");
                scanf("%d",&choix);
                printf("\n");
 
                while(choix<1 || choix>4)
                {
                        printf("Veuillez saisir un nombe entre 1 et 4 ");
                        scanf("%d",&choix);
                        printf("\n");
 
 
                }
 
                switch(choix)
                {
                case 1:
                nbel=Ajout(&prod[4],nbel);
                break;
 
                case 2:
                affiche(&prod[0],&nbel);
                break;
                }
        }
 
 
}
 
 
 
int Ajout(struct produit *prod, int nbel)
{
                system("cls");
        printf("ID du Produit : ");
        scanf("%d",&(prod->IDproduit));
 
 
        printf("Nom du produit : ");
        fflush(stdin);
        gets (prod->NomProduit);
 
        printf("Unite de mesure : ");
        fflush(stdin);
        gets (prod->UniteMesure);
 
        printf("Type du produit : ");
        fflush(stdin);
        gets (prod->TypeProduit);
 
        printf("Prix : ");
        scanf ("%f",&(prod->Prix));
        printf("\n");  
 
 
 
 
        while(prod->Prix<0)
        {
 
        printf("erreur, le prix doit etre positif !\n");
        printf("Prix : ");
        scanf("%f",prod->Prix);
 
 
        }
        prod++;
        nbel++;
 
 
        return nbel;
 
 
 
 
 
 
}
 
void affiche(struct produit *pcProd, int *pt)
{
        int i;
 
        while(i<*pt)
                        {
 
                        printf("%d ", pcProd->IDproduit);
                        printf("%s\t", pcProd->NomProduit);
                        printf("%s\t ", pcProd->UniteMesure);
                        printf("%s\t ", pcProd->TypeProduit);
                        printf("%.2f\n", pcProd->Prix);
                        printf("\n");
                        pcProd++;
                        i++;
                        }      
 
 
 
}

Merci d'avance

[valefor]

Quelques remarques :
(1)

Code :

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

nbel=Ajout(&prod[4],nbel);

pourrait s'écrire

Code :

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

nbel=Ajout(&prod[nbel],nbel);

(2) Dans Ajout :
- Vérifie que le nb elem n'est pas trop grand pour ton tableau de produits
- pas la peine de faire prod++ à la fin.
- le nombre de produit est facultatif, d'ailleur tu ne t'en sert pas sauf pour retourner le nouveau nombre qui sera toujours l'ancien + 1, donc tu pourrait t'économiser cela. A voir suivant ce que tu veux faire après.

(3)

Code :

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

affiche(&prod[0],&nbel);

pas la peine de passer le nombre d'éléments par pointeur.

(4) homogénéise la notation (fonction commençant par une majuscule ou pas), revois le formatage (espaces, tabulations, etc).

[corentin59]

Code :

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

1
2
3
printf("Nom du produit : ");
        fflush(stdin);
        gets (prod->NomProduit);

"fflush(stdin);" induit un comportement indéfini et gets ne vérifie pas s'il y a eu trop de caractères saisis par rapport à la taille allouée. pour ces deux raisons, préfères l'utilisation de fgets.

[diogene]

Code :

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

1
2
3
4
5
void affiche(struct produit *pcProd, int *pt)
{
        int i;
 
        while(i<*pt)....

i est non initialisé. Le test n'a pas de sens