Discussion :

[mak09]

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

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
#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;
}

[gerald3d]

Bonjour.

Où est la fonction SAISIEPRODUIT(); ?

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

[foetus]

Où as-tu appris à coder en C ?

On ne code pas 1 fonction dans 1 boucle

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

[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 c :

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
#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.

[Sve@r]

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.

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...

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...

[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.

[edgarjacobs]

(....) 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 :

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

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)

[mak09]

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é.

Où as-tu appris à coder en C ?

On ne code pas 1 fonction dans 1 boucle

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.

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 c :

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
#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