Discussion :

[Jiraiya42]

Salut,

J'ai commencé le C depuis peu et je dois faire un programme pour calculer le périmètre et l'aire d'un cercle. J'ai lu plein de tutoriels et de cours sur le C mais je n'arrive toujours pas à récupérer le résultat des fonctions...

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
 
#include <stdio.h>
PI=3.14
 
void main()
   {
    float calculperimetre(r)
    float calculaire(r)
 
    clrscr();
    printf("rayon:\n");
    scanf("%f", &rayon);      
 
/*c'est là qu'est mon pb je veux que    l'utilisateur entre le rayon et que cette valeur soit utilisée dans les 2 fonctions. Une fois le traitement fini dans les fonctions je voudrais récupérer les résultats et les afficher comme rayon=... périmètre=... aire...*/
 
   }
 
 
float calculperimetre(float r)
   {
    resultat=2*PI*r
    return resultat
   }
 
float calculaire(float r)
   {
    res=PI*r²
    return res
   }

Donc est-ce que quelqu'un peut m'expliquer comment je peux passer la valeur du rayon à la fonction et récupérer les 2 résultats pour les afficher dans le programme principal svp ?

[Emmanuel Delahaye]

J'ai commencé le C depuis peu et je dois faire un programme pour calculer le périmètre et l'aire d'un cercle. J'ai lu plein de tutoriels et de cours sur le C mais je n'arrive toujours pas à récupérer le résultat des fonctions...

Joli score !

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
 
main.c:2: warning: type defaults to `int' in declaration of `PI'
 
main.c:4: error: syntax error before "void"
 
main.c:10: error: syntax error before string constant
main.c:10: warning: type defaults to `int' in declaration of `printf'
main.c:10: warning: function declaration isn't a prototype
main.c:10: error: conflicting types for 'printf'
main.c:10: note: a parameter list with an ellipsis can't match an empty parameter name list declaration
main.c:10: error: conflicting types for 'printf'
main.c:10: note: a parameter list with an ellipsis can't match an empty parameter name list declaration
 
main.c:10: warning: data definition has no type or storage class
main.c:11: error: syntax error before string constant
main.c:11: warning: type defaults to `int' in declaration of `scanf'
main.c:11: warning: function declaration isn't a prototype
main.c:11: error: conflicting types for 'scanf'
main.c:11: note: a parameter list with an ellipsis can't match an empty parameter name list declaration
main.c:11: error: conflicting types for 'scanf'
main.c:11: note: a parameter list with an ellipsis can't match an empty parameter name list declaration
main.c:11: warning: data definition has no type or storage class
main.c:19: warning: no previous prototype for 'calculperimetre'
main.c: In function `calculperimetre':
main.c:20: error: `resultat' undeclared (first use in this function)
 
main.c:20: error: (Each undeclared identifier is reported only once
main.c:20: error: for each function it appears in.)
main.c:21: error: syntax error before "return"
main.c: At top level:
main.c:25: warning: no previous prototype for 'calculaire'
main.c: In function `calculaire':
 
main.c:26: error: `res' undeclared (first use in this function)
main.c:26: error: stray '\178' in program
main.c:27: error: syntax error before "return"

On pourrait avoir la version qui compile ?

Code commenté et en partie corrigé :

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
 
/* -ed-
   reformattage...
   modifie' la disposition des fonctions.
 */
#include <stdio.h>
 
/* -ed-
   PI=3.14
 
   Le type implicite est int, ce qui va faire 3.
   Je conseille 'double'.
   Il manque un ';'
 */
double PI = 3.14;
 
/* -ed-
   void main()
   main() retourne int. Toujours.
 
   <a href="http://emmanuel-delahaye.developpez.com/notes.htm" target="_blank">http://emmanuel-delahaye.developpez.com/notes.htm</a>
 
 */
 
/* -ed-
   2 fonctions deplacees.
   Ajoute static (fonctions locales)
 */
static float calculperimetre (float r)
{
/* -ed-
   manquaient 2 ';'
   'resultat' non definie. Ajoute 'float'
 */
   float resultat = 2 * PI * r;
   return resultat;
}
 
static float calculaire (float r)
{
/* -ed-
   manquaient 2 ';'
   C'est quoi ce petit 2 ? (supprime) Quelle est la definition d'un carre ?
   'res' non definie. Ajoute 'float'
 */
   float res = PI * r;
   return res;
}
 
int main (void)
{
/* -ed-
   float calculperimetre (r)
   float calculaire (r)
 
   Il manque les ';'...
   A de rares exceptions pres, les prototypes sépare's doivent être dans un
   header. Sinon, il suffit de definir les fonctions avant de les utiliser.
   Supprimes.
 */
 
/* -ed-
   clrscr ();
 
   Fonction non standard, non declaree, et probablement inutile.
   Supprimmee.
 */
 
   printf ("rayon:\n");
 
/* -ed- variable 'rayon' non definie. Ajoute */
   {
      float rayon;
      scanf ("%f", &rayon);
 
/* c'est là qu'est mon pb je veux que l'utilisateur entre le rayon
   et que cette valeur soit utilisée dans les 2 fonctions.
   Une fois le traitement fini dans les fonctions je voudrais
   récupérer les résultats et les afficher comme
   rayon=... périmètre=... aire...
 
 */
 
      /* -ed-
         Ben oui, il faut ecrire le code. Tu as certainement eu un cours qui
         explique comment faire. On ne fera pas ton travail a ta place.
       */
 
   }
   /* -ed- parce que main() retourne int : */
 
   return 0;
}

[Jiraiya42]

Je l'ai pas encore compilé vu qu'il n'est pas fini

[Fiquet]

Aïe déjà au vol je vois 2 erreurs. Ton PI n'est pas déclarer il faut faire style:

Code :

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

1
2
 
const double PI = 3.14159;

Et tu ne peut pas faire une puissance simplement en mettant le chiffre en exposant! Ce n'est pas directement intégré au language et ca vient du fait que les processeurs ne peuvent pas directement faire une puissance au niveau basique. Il y a des fonction pour ça dans les libraires standards (regarde du côté de pow() dans math.h)...

[Jiraiya42]

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
 
#include <stdio.h>
const double PI = 3.14159;
 
void main()
   {
    float calculperimetre(r)
    float calculaire(r)
 
    clrscr();
    printf("rayon:\n");
    scanf("%f", &rayon);
 
   }
 
 
 
 
float calculperimetre(float r)
   {
    resultat=2*PI*r
    return resultat
   }
 
float calculaire(float r)
   {
    res=PI*(r*r)
    return res
   }

Là j'ai modifié comme tu as dis et il n'y a plus de puissance mais est-ce que tu peux m'expliquer comment je peux passer la valeur du rayaon aux fonctions et récupérer les résultats stp ?

[Emmanuel Delahaye]

Je l'ai pas encore compilé vu qu'il n'est pas fini

A faire d'urgence. On va pas s'amuser à corriger tes erreurs basiques à chaque fois....

Une bonne regle de codage. A chaque étape du codage, le code doit être compilable et exécutable.

1

Code :

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

1
2
3
4
int main (void)
{
   return 0;
}

2

Code :

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

1
2
3
4
5
6
#include <stdio.h>
int main (void)
{
   printf ("rayon:\n");
   return 0;
}

etc.
Y'a que comme ça que ça marche rapidement (eXtrem Programming).

[Emmanuel Delahaye]

Et tu ne peut pas faire une puissance simplement en mettant le chiffre en exposant!

Ce n'est pas directement intégré au language et ca vient du fait que les processeurs ne peuvent pas directement faire une puissance au niveau basique.

Rien a voir. Le langage n'a que faire du processeur.

Il y a des fonction pour ça dans les libraires standards (regarde du côté de pow() dans math.h)...

Je ne suis pas très fort en math, mais sortir pow() pour faire un carré, y'a de l'abus, non ? Le carré de a c'est pas a x a ?

[Marcin]

Il y a un problème de logique dans ta fonction main. Tu essaies (si j'a bien compris) d'abord d'appeler les fonctions 'calculperimetre' et 'calculaire' et seulement ensuite tu demandes le rayon à l'utilisateur.... (au passage, la variable 'rayon' doit être declarée !).

Tu dois faire exactement l'inverse - d'abord recupérer le rayon et ensuite appeler les deux fonctions avec 'rayon' comme argument...

Autre remarque : 'main' doit toujours renvoyer un 'int' :

Code :

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

1
2
3
4
int main () {
....
return 0
}

Enfin, il te manque des ';', et les variables 'res' et 'resultat' ne sont pas declarées.

[afrikha]

Là j'ai modifié comme tu as dis et il n'y a plus de puissance mais est-ce que tu peux m'expliquer comment je peux passer la valeur du rayaon aux fonctions et récupérer les résultats stp ?

tu dois faire appels à ces fonctions dans la fonction main.
du style:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
#include...
main{
....
perimetre(r);
surface(r);
/*tu peux eventuellement les afficher avec printf*/
...
return 0;
}

[Neilos]

1 - les instructions en C ça se termine par un ;

Code :

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

1
2
3
4
5
6
7
8
9
float calculperimetre(float r)
   {
    return 2*PI*r;
   }
 
float calculaire(float r)
   {
    return PI*(r*r);
   }

2 - ça sert à rien d'appeler les deux fonctions ci-dessus avec un r non initialisé

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
 
void main()
   {
        float perimetre;
        float aire;
        float rayon;  // Faudrait penser à le déclarer lui aussi
 
        clrscr();
        printf("rayon:\n");
        scanf("%f", &rayon);
 
        perimetre = calculperimetre(r);
        aire = calculaire(r);
 
        printf("Perimetre : %f\n", perimetre);
        printf("Aire : %f\n", aire);
   }

Bon dans le main tu peux aussi directement mettre

Code :

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

1
2
printf("Perimetre : %f", calculperimetre(rayon));
        printf("Aire : %f, calculaire(rayon));

Bref à mon avis faudrait bien relire tes tutoriels de C et refaire les exemples qu'il y dedant...bon courrage !

[diogene]

je peux passer la valeur du rayaon aux fonctions et récupérer les résultats stp ?

Après corrections des erreurs signalées par ailleurs

Code :

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

1
2
3
float perimetre;
 .....
perimetre =calculperimetre(rayon);

[afrikha]

marcin a dit:

Il y a un problème de logique dans ta fonction main. Tu essaies (si j'a bien compris) d'abord d'appeler les fonctions 'calculperimetre' et 'calculaire' et seulement ensuite tu demandes le rayon à l'utilisateur.... (au passage, la variable 'rayon' doit être declarée !).

non,il faut declarer les fonctions dans main si elles sont definies apres main justement.

[Marcin]

marcin a dit:

Il y a un problème de logique dans ta fonction main. Tu essaies (si j'a bien compris) d'abord d'appeler les fonctions 'calculperimetre' et 'calculaire' et seulement ensuite tu demandes le rayon à l'utilisateur.... (au passage, la variable 'rayon' doit être declarée !).

non,il faut declarer les fonctions dans main si elles sont definies apres main justement.

Dans le main????

Je dirai avant (les prototypes), mais pas à l'intérieur...

[Emmanuel Delahaye]

non,il faut declarer les fonctions dans main si elles sont definies apres main justement.

Non, pas dans main() (ni aucune fonction d'ailleurs). Bien que ce soit techniquement possible, c'est considéré comme du mauvais style.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
/* mauvais style */
int main (void)
{
   int f(void);
}
 
int f(void)
{
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
/* mieux, mais absurde */
int f(void);
 
int main (void)
{
}
 
int f(void)
{
}

Code :

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

1
2
3
4
5
6
7
8
9
 
/* correct, mais ameliorable */
int f(void)
{
}
 
int main (void)
{
}

Code :

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

1
2
3
4
5
6
7
8
9
 
/* correct */
static int f(void)
{
}
 
int main (void)
{
}

[Jiraiya42]

Merci pour toutes vos réponses ça m'a permi de comprendre ce que je voulais je crois que je vais refaire un pti tour vers les tutoriels je crois oui merci encore

[afrikha]

apparemment mon tutorial m'apprend des mauvais styles!!
question: c'est quoi un mauvais style?et sur quoi tu te bases pour classer les styles?
si ils fonctionnent tous,ils devraient etre tous equivalents,non?

[AjJi]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
/* mieux, mais absurde */
int f(void);
 
int main (void)
{
}
 
int f(void)
{
}

Bah par exemple comme l'as dit Emmanuel, cette ecriture est Absurde vu que dans ce cas on a une ou deux fonctions, mais généralement on préfere acceder au main sans à faire travailler la roulette de la souris pour y arriver donc dans le cas où on a pas mal de fonctions, il vaut mieux faire comme ça je pense.

Code :

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

1
2
3
4
5
6
7
8
9
 
int main (void) 
{
  int f(void);
}
 
int f(void)
{
}

Celle-là est fausse même non :

[Emmanuel Delahaye]

apparemment mon tutorial m'apprend des mauvais styles!! question: c'est quoi un mauvais style?et sur quoi tu te bases pour classer les styles?

L'expérience, petit scarabé...

si ils fonctionnent tous,ils devraient etre tous equivalents,non?

Techniquement, oui. Mais il y a d'autres considérations, notamment l'organisation du code.

En gros on a 2 possibilités 'correctes'.

Tout dans un seul fichier (une seule unité de compilation)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
/* main.c */
/* fonctions privees */
static void f(void)
{
}
 
/* point d'entree unique */
int main(void)
{
   f();
   return 0;
}

Compilation separée

Code :

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

1
2
3
4
5
6
7
8
9
10
 
/* main.c */
#include "f.h"
 
/* point d'entree unique */
int main(void)
{
   f();
   return 0;
}

Code :

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

1
2
3
4
5
6
7
8
9
 
/* f.h - interface */
#ifndef H_F
#define H_F
 
/* protypes des fonctions publiques */
void f (void);
 
#endif /* guard */

Code :

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

1
2
3
4
5
6
7
8
 
/* f.c - implementation */
#include "f.h"
 
/* fonctions publiques */
void f(void)
{
}

Evidemment, rien n'empêche que les sources main.c et f.c aient leurs propes fonctions privées (static).

[afrikha]

je dormirai moins con ce soir
merci pour ces explications