fonction et structure

Version imprimable

Bonjour
quand j'essaye de compiler ce programme , j'ai un message d'erreur à la ligne 45. Pouvez-vous m'aider?

Code:

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
#include<stdio.h>
// EXERCICE 1
//1)
#define NP 20
//2
struct POINT
{
    char P;
    float x;
    float y;
};
//3
struct POINT COURBE[NP];
//4
void AFFICHERPOINT( struct POINT );
void AFFICHERPOINT( struct POINT T )
{
    printf ("%s\t%f\t%f\t\n",T.P,T.x,T.y);
 
}
int main()
{
    int k, n;
    printf("Donnez le nombre de points: ");
    scanf ("%d",&n);
    printf("\n");
        for (k=1;k<=n;k++)
        {
          printf ("point:%d",k);
 
          struct POINT SAISIEPOINT(void)
 
            {struct POINT Q;
              printf("P:");
              scanf("%s",&Q.P);
              print("\n");
              printf("x:");
              scanf("%f",&Q.x);
              print("\n");
              printf("y:");
              scanf("%f",&Q.y);
              print("\n");
              return Q;
            };
             COURBE[k] = SAISIEPRODUIT();
         };
 
        printf ("P\t x\t y\t");
        for (k=1 ; k<=n; k++);
        {
            AFFICHERPOINT(COURBE[k]);
            printf("\n");
        };
        return 0;
}

06/12/2022, 11h14
gerald3d

Bonjour.

Où est la fonction SAISIEPRODUIT(); ?

Et pour information on n’écrit pas les fonctions en majuscule 😉.
06/12/2022, 11h19
foetus

Où as-tu appris à coder en C ? :ptdr:

On ne code pas 1 fonction dans 1 boucle :aie: :aie:

Et en passant en paramètre des structures, on fait 1 copie.

Bonjour

C'est quoi ce programme ??? Alors (liste non exhaustive)
définition de fonctions dans le code (remarque, en réalité ce n'est pas interdit depuis qu'on est passé en C99 dans lequel on peut définir des fonctions dans des fonctions mais ça reste quand-même bien débile => le programme, à chaque tour de boucle, il va devoir recréer la fonction pour pouvoir l'utiliser !!!)
saisie et affichage de char via "%s"
indices d'un tableau de n éléments variant de 1 à n

Me semble que c'est à peu près tout... enfin sans compter les détails comme l'utilisation de variables globales sans raison, ton aération totalement inexistante, ton passage de structure par valeurs (pour info la pile est limitée!!! là ça va que la structure fait 9 octets mais rajoute-lui un tab[5000000] et ton truc explose en vol!!!) et tes noms tout en majuscules. Donc commence par ouvrir un livre de C avant de vouloir faire du C...
Code:

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 #include <stdio.h> #define NP (20) typedef struct s_point { char n; float x; float y; } t_point; void point_affich(const t_point* const); void point_saisie(t_point* const); int main() { t_point points[NP]; printf("Donnez le nombre de points (pas plus de %u): ", NP); fflush(stdout); size_t n; scanf("%lu", &n); getchar(); // Ici faudrait tester n <= NP mais bon, je vais pas non plus tout te faire !!! size_t i; t_point* pt; for (i=0, pt=points; i < n; i++, pt++) point_saisie(pt); for (i=0, pt=points; i < n; i++, pt++) { printf("Point %lu/%lu: ", i+1, n); point_affich(pt); } return 0; } void point_saisie(t_point* const p) { fputs("Entrez p: ", stdout); fflush(stdout); scanf("%c", &p->n); getchar(); fputs("Entrez x: ", stdout); fflush(stdout); scanf("%f", &p->x); getchar(); fputs("Entrez y: ", stdout); fflush(stdout); scanf("%f", &p->y); getchar(); } void point_affich(const t_point* const p) { printf("n=[%c], x=%f, y=%f\n", p->n, p->x, p->y); }
Ce n'est pas un peu plus élégant ? Ou juste plus lisible ?? Ou seulement juste compilable ???

PS: le getchar() qui suit chaque scanf() c'est pour supprimer le <return> résiduel. C'est pas la solution la plus efficace ou la plus robuste mais c'est la plus simple à mettre en oeuvre pour pouvoir voir son code tourner.

Citation:

Envoyé par gerald3d

Bonjour.

Où est la fonction SAISIEPRODUIT(); ?

Et pour information on n’écrit pas les fonctions en majuscule 😉.

Bonsoir.
Merci beaucoup pour la reponse, pour la fonction SAISIEPRODUIT(), c'était une erreur de ma part. Mais pour l'utilisation des majuscules, c'est écrit comme tel dans l'énoncé.

Citation:

Envoyé par foetus

Où as-tu appris à coder en C ? :ptdr:

On ne code pas 1 fonction dans 1 boucle :aie: :aie:

Et en passant en paramètre des structures, on fait 1 copie.

Bonsoir .
Merci beaucoup pour la réponse. pour le codage de la fonction c'était une erreur de ma part à cause de la précipitation.
Et pour mon apprentissage, c'était en classe avec des méthodes dite "simple" par notre prof.
Citation:
Envoyé par Sve@r
Bonjour

C'est quoi ce programme ??? Alors (liste non exhaustive)
définition de fonctions dans le code (remarque, en réalité ce n'est pas interdit depuis qu'on est passé en C99 dans lequel on peut définir des fonctions dans des fonctions mais ça reste quand-même bien débile => le programme, à chaque tour de boucle, il va devoir recréer la fonction pour pouvoir l'utiliser !!!)
saisie et affichage de char via "%s"
indices d'un tableau de n éléments variant de 1 à n

Me semble que c'est à peu près tout... enfin sans compter les détails comme l'utilisation de variables globales sans raison, ton aération totalement inexistante, ton passage de structure par valeurs (pour info la pile est limitée!!! là ça va que la structure fait 9 octets mais rajoute-lui un tab[5000000] et ton truc explose en vol!!!) et tes noms tout en majuscules. Donc commence par ouvrir un livre de C avant de vouloir faire du C...
Code:

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 #include <stdio.h> #define NP (20) typedef struct s_point { char n; float x; float y; } t_point; void point_affich(const t_point* const); void point_saisie(t_point* const); int main() { t_point points[NP]; printf("Donnez le nombre de points (pas plus de %u): ", NP); fflush(stdout); size_t n; scanf("%lu", &n); getchar(); // Ici faudrait tester n <= NP mais bon, je vais pas non plus tout te faire !!! size_t i; t_point* pt; for (i=0, pt=points; i < n; i++, pt++) point_saisie(pt); for (i=0, pt=points; i < n; i++, pt++) { printf("Point %lu/%lu: ", i+1, n); point_affich(pt); } return 0; } void point_saisie(t_point* const p) { fputs("Entrez p: ", stdout); fflush(stdout); scanf("%c", &p->n); getchar(); fputs("Entrez x: ", stdout); fflush(stdout); scanf("%f", &p->x); getchar(); fputs("Entrez y: ", stdout); fflush(stdout); scanf("%f", &p->y); getchar(); } void point_affich(const t_point* const p) { printf("n=[%c], x=%f, y=%f\n", p->n, p->x, p->y); }
Ce n'est pas un peu plus élégant ? Ou juste plus lisible ?? Ou seulement juste compilable ???

PS: le getchar() qui suit chaque scanf() c'est pour supprimer le <return> résiduel. C'est pas la solution la plus efficace ou la plus robuste mais c'est la plus simple à mettre en oeuvre pour pouvoir voir son code tourner.
Bonsoir .
Merci beaucoup pour la réponse. pour le codage de la fonction dans la boucle c'était une erreur de ma part à cause de la précipitation.
Je ne pourrais pas changer les majuscules vu que c'est les consignes de l'énoncé mais je promet de faire beaucoup plus d'effort pour pouvoir ecrire des programme appréciable .
Merci pour les conseils et les explications

06/12/2022, 17h22
Sve@r

Citation:

Envoyé par mak09

Merci beaucoup pour la réponse. pour le codage de la fonction dans la boucle c'était une erreur de ma part à cause de la précipitation.

Trop de précipitation sont rarement une bonne idée. Et t'es pas obligé de réécrire tes posts à chacun d'entre nous, on se lit tous les uns les autres.

Citation:

Envoyé par mak09

Je ne pourrais pas changer les majuscules vu que c'est les consignes de l'énoncé

Bon. Je sens là des consignes venues d'un prof de C n'ayant jamais fait de C... ou n'ayant jamais été prof... ou n'ayant jamais été ni l'un ni l'autre... Enfin c'est peu important vu que mon code n'a que 2 fonctions, c'est facile à changer. Etonnant qu'il n'ait pas dit aussi d'écrire main() en majuscules...

Citation:

Envoyé par mak09

Et pour mon apprentissage, c'était en classe avec des méthodes dite "simple" par notre prof.

Ah voilà, un prof avec des méthodes "simple", tout s'explique...:roll:
06/12/2022, 20h04
edgarjacobs
Citation:

Envoyé par mak09

(....) Je ne pourrais pas changer les majuscules vu que c'est les consignes de l'énoncé (....)

Si on veut des majuscules, on peut écrire ceci
Code:

1 2 3 4 5 6 7 8 #define MAIN main #define INT int #define CHAR char #define RETURN return INT MAIN(INT ARGC, CHAR *ARGV[]) { RETURN 0; }
:(

Edit mais qu'est-ce que c'est c.. (au choix, on ou ul)
06/12/2022, 20h24
Christophe

Il n'est pas interdit d'écrire les noms de fonctions en C. Par contre c'est effectivement une mauvaise pratique et ça ne fait pas partie des conventions.