bug à l'affichage des structures

**Magnum8760** · 01/04/2019, 11h58

Bonjour, pour le travail j'essaye de fabriquer un petit programme mais bien qu'il n'y ai apparemment pas d'erreurs dans ma partie "Analyse" tout en bas, C ne veux pas restituer mes valeurs (de la structure paddock_data) !

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
164
165
166
167
168
169
170
171
172
173
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
 
struct PADDOCK { 
    unsigned int type; 
    char *name; 
    float paddock_size; 
    float hauteur_mesuree;
    char *commentaire;
}paddock_data; 
 
/* Les fonctions export sous format linéaire dans un csv (revoir) */
 
void export_row(int number_of_data,char const *fileName, int data_array){
        for (int i = 0 ; i < number_of_data ; i++)
    {
        FILE *fic = fopen(fileName, "a"); 
        if (fic == NULL)
        {
            exit(1);
        }      
        fprintf(fic, "%d ;", data_array); 
        //printf("P%d : %d\n", i+1, paddock_data[i].paddock_size);
        fclose(fic); 
    }
}
/*
    for (i = 0 ; i < number_of_paddock ; i++)
    {
        FILE *fic = fopen("Data.csv", "a"); 
        if (fic == NULL)
        {
            exit(1);
        }      
        fprintf(fic, "%d ;", paddock_data[i].paddock_size); 
        //printf("P%d : %d\n", i+1, paddock_data[i].paddock_size);
        fclose(fic); 
    }
*/
int main(int argc, char** argv)
{
    /* La fonction temps */
 
    time_t seconds = time(NULL);
    struct tm * timeinfo = localtime(&seconds);
 
    /* 
     * Partie 0 : Introduction
     */
 
    printf("\n");
    printf("\t\tMESURES D'HERBE\n\t\t%02d-%02d-%04d %02d:%02d:%02d\n", timeinfo->tm_mday, timeinfo->tm_mon+1, 1900+timeinfo->tm_year, timeinfo->tm_hour, timeinfo->tm_min, timeinfo->tm_sec);
    printf("\t\tQuitter : \"ctrl + c\"\n");
    printf("\n");
 
    printf("\t Bonjour, je suis un petit programme test \n\t sur l estimation de la croissance de l herbe en paturage.\n\t Actuellement, je n enregistre pas de donnees car ma\n\t fonction \"exporter\" n est pas encore au point. Neanmoins je calcul\n\t , avec vous, simplement et rapidement vos mesures d'herbes. :)\n\n");
 
    /* 
     * Partie 1 : Les variables d'exploitation. 
     */
 
    struct PADDOCK *paddock_data = NULL;
    unsigned int number_of_paddock = 0;
    unsigned int i, nombre_de_jour, nombre_de_VL, ingestion ;
    float hauteur_paturage = 0 ;
    char c;
    int densite = 220;
 
    /* 
     * Partie 1 - bis : Les variables de traitement de données. 
     */
 
    float hauteur_utile;
 
    /* 
     * Partie 2 : Début du questionnaire. 
     */
/*
    printf("\t1.- Combien de jours depuis la derniere mesure d'herbe ? ");
    scanf("%d", &nombre_de_jour);
    printf("\n");
 
    do{
            printf("\t2.- Acceptez vous une densite de 220 kgMS/cm/ha (o ou n) ? ");
            scanf(" %c", &c);
    }while(c != 'o' && c != 'n');
    //printf("\n");
 
    if(c=='n')
    {
            printf("\t2-bis.- Si non, quel densite souhaite vous utiliser ? ");
            scanf("%d",&densite);
    }
    printf("\n");
 
    printf("\t3.- Nombre de vaches laitieres ? ");
    scanf("%d", &nombre_de_VL);
    printf("\n");
 
    printf("\t4.- Ingestion par vache laitiere (kgMS /cm/ha) ? ");
    scanf("%d", &ingestion);
    printf("\n");
 
    printf("\t5.- Hauteur de paturage (cm) ? ");
    scanf("%f", &hauteur_paturage);
    printf("\n");
*/
    printf("\t6.- Combien de parcelles ? ");
    scanf("%d", &number_of_paddock);
 
    if(number_of_paddock > 0)
    {
        paddock_data = malloc(sizeof(int)*number_of_paddock);
 
        /* si l`allocation a echoue, on quitte le programme. */
        if(paddock_data == NULL)
            exit(1); 
 
        /* Premier paramètre : la taille des parcelles */
        for (i = 0 ; i < number_of_paddock ; i++)
        {
            //do{
                    if(i+1 == 1)
                        printf("\t%dere parcelle", i + 1);
                    else
                        printf("\t%deme parcelle", i + 1);
                    printf("\n");
                    printf("\ta. -  Donnez la taille (ha), son nom et son type (ex: 2 Charlotte 3) ? ");
                    scanf("%d %s %d", &paddock_data[i].paddock_size, &paddock_data[i].name, &paddock_data[i].type);
            //}while((int)(paddock_data[i].paddock_size)%10 == 0 && (int)(paddock_data[i].type)%10 == 0);
            //printf("\n");
 
            //do{
                    printf("\tb. - Hauteur d'herbe mesuree ? ");
                    scanf("%f", &paddock_data[i].hauteur_mesuree);
            //}while(paddock_data[i].hauteur_mesuree%10 > 0);
            //printf("\n");
 
            printf("\tc. - Evenement de la semaine (commentaire) ? ");
            scanf("%s", &paddock_data[i].commentaire);
            printf("\n");
        }
        //export_row(number_of_paddock,"Data.csv",paddock_data[i].paddock_size);
    }
 
    do{
            printf("\t7.- Connaissez vous la hauteur precedente (o ou n) ? ");
            scanf(" %c", &c);
    }while(c != 'o' && c != 'n');
            printf("\n");
 
    if(c=='o')
    {
            printf("\t7-bis.- Quel est cette hauteur precedente ? ");
            scanf("%d",&densite);
    }
    if(c == 'o')
    printf("\n");
 
    /* 
     * Partie 3 : Début des analyses parcelles. 
     */
    printf("\tAinsi nous pouvons derteminer : ");
    printf("\n\t1. - hauteur utile : \n");
    for(i=0; i < number_of_paddock; i++){
            hauteur_utile = paddock_data[i].hauteur_mesuree - hauteur_paturage;
            printf("\t\t%s [%.2f ha] : %.2f\n", paddock_data[i].name, paddock_data[i].paddock_size, hauteur_utile);
    }
 
    free(paddock_data);
    return 0;
}

Merci et bonne journée à tous

**gerald3d** · 01/04/2019, 17h40

Bonjour.

ligne 114 :

Code :

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

paddock_data = malloc(sizeof(int)*number_of_paddock);

Tu n'alloues pas la place nécessaire pour ta structure. Tu as à la place number_of_paddock * la taille d'un int !

Essaies plutôt cà :

Code :

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

paddock_data = malloc(sizeof(struct PADDOCK) * number_of_paddock);

Petite remarque :
Évite d’écrire en majuscule le nom de tes structures. Même si cela importe peu pour le compilateur, par contre pour les humains que nous sommes, avec tous nos carcans intellectuels

, cela importe. En effet nous avons tendance à écrire en majuscule les #define.

Attention aussi à la déclaration de tes structures. Soit tu écris sans la déclaration finale :

Code :

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

1
2
3
4
5
6
7
struct PADDOCK { 
    unsigned int type; 
    char *name; 
    float paddock_size; 
    float hauteur_mesuree;
    char *commentaire;
}paddock_data;

Soit tu inclus la déclaration d'un nouveau type lors de la déclaration :

Code :

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

1
2
3
4
5
6
7
typedef struct  { 
    unsigned int type; 
    char *name; 
    float paddock_size; 
    float hauteur_mesuree;
    char *commentaire;
}SPaddock;

**Magnum8760** · 01/04/2019, 19h45

Merci je regarde demain mais la bonne façon d'écrire une structure, ce ne serrais pas :

Code :

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

1
2
struct s_paddock{
...}t_paddock

? (cette syntaxe apparaît quelques fois sur le forum)

**foetus** · 02/04/2019, 01h11

C'est en C++ que le typedef peut être omis

**Magnum8760** · 02/04/2019, 06h36

Ici l'auteur a omis de mettre son "struct Map" après typedef mais crée bien son nouveau type après les accolades. C'est courant comme syntaxe ou non ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
typedef struct{
    int Nb_Block_W, Nb_Block_H; /* Le nombre de bloc en largeur et hauteur (ce qui permet de donner les limites de la carte), définis dans le fichier .lvl qui contient le niveau. */
    uint16_t** LoadedMap;  /* LE BUT EST D'ALOUER SUFFISAMENT DE MÉMOIRE
                            * Uint16** : adresse d'un autre entier système contenant lui même l'adresse
                            * d'un tableau de N entiers sans signe sur 2 octets (le tout sur 4 octets).
                            * Uint16 est un nombre non-signé sur 16 bits, soit donc 2 octets.
                            * (int -32767 à +32767 et uint 0 à +65535 ont le point commun d'être sur 16 bits (2octets).
                            */
    uint16_t** Used;
    long int xscroll,yscroll;
}Map;

**gerald3d** · 02/04/2019, 08h07

Voila un peu lecture pour la déclaration d'une structure :
https://c.developpez.com/cours/berna...gne/node80.php

Et pour déclarer un nouveau type lié à une structure :
https://c.developpez.com/cours/berna...ne/node116.php

bug à l'affichage des structures

C

Vue hybride

Discussions similaires

Partager

Partager