Comprendre comment écrire une fonction

Version imprimable

Bonjour,

Je dois écrire une fonction appelée conversion qui prend en paramètre une note (type int) et renvoie un caractère (type char).
Si la note est comprise entre 0 et 20, elle sera convertie dans la lettre correspondante. Si elle n'est pas comprise dans cet intervalle, le caractère renvoyé sera 'X'. La fonction ne doit faire ni lecture au clavier ni affichage à l'écran.
un main qui lit au clavier un int et affiche le résultat de la conversion de cet entier par la fonction conversion .
On va leur fournir un convertisseur qui leur permettra de convertir les anciennes notes entre 0 et 20 en nouvelles notes entre A et F.
La table de conversion est la suivante:
Notes de 16 à 20: A
Notes de 13 à 15: B
Notes de 10 à 12: C
Notes de 8 à 9: D
Notes de 6 à 7: E
Notes de 0 à 5: F

J'ai commencé à écrire le programme
Je pense que le main est OK. Il me manque une boucle if pour le renvoi du X si la valeur tapée au clavier est inférieure à 0 ou supérieure à 20.
C'est la fonction convert qui me pose quelques soucis d'écritures. Je ne comprends pas comment intégrer la table de conversion noté ci-dessus.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 #include <stdio.h> #include <stdlib.h> int convert(int note) { int a = 16; char aaa = aa; printf("%c", aaa); } int main() { int note; note=0; printf("une note comprise entre 0 et 20\n"); scanf("%d",&note) ; printf ("conversion de la note de %d\n", note); convert (xxxx) return 0; }
Soyez indulgent, je débute en programmation ;-)
Merci
Yohan

Bonjour,

Pour commencer, prends tout de suite une bonne habitude : utiliser le bon vocabulaire.

if n'est pas une boucle ! :)

Pour ton problème, écris ce que tu ferais en français, genre :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 si condition1 alors ce qu'il faut faire quand la condition1 est remplie sinon ce qu'il faut faire quand la condition1 n'est pas remplie (on passe à la suivante) si condition2 alors ce qu'il faut faire quand la condition2 est remplie sinon ce qu'il faut faire quand la condition2 n'est pas remplie (on passe à la suivante) etc.

03/05/2017, 22h26
dalfab

Bonjour,

Le début de l'énoncé "Je dois écrire une fonction appelée conversion qui prend en paramètre une note (type int) et renvoie un caractère (type char). ". Je commencerais donc par écrire char conversion( int note ) au lieu de ton int convert(int note).
Et au sujet du main, l'énoncé "un main qui lit au clavier un int et affiche le résultat de la conversion de cet entier par la fonction conversion", le code lit bien un int, puis ni conversion de cet entier, ni récupération de la conversion, ni affichage du résultat. Non il n'est pas OK!

Fonction !!

Bonjour,
Merci pour votre aide ;-)
J'ai avancé un peu.
Il me reste encore a comprendre comment construire le tableau de conversion et voir pourquoi la fonction ne renvoie pas la lettre A lorsque je renseigne une note entre 16 et 20.

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
 
#include <stdio.h>
 
char conversion(int noteo) //noteo  correspond à la note enregistrer
{
 int noter; //initialise la variable note resultat
 char notec; // note changé en char de type A B C 
 notec=0;  //on met la variable à 0
 
 scanf("%d",&noteo); //enregistre la valeur entrée au clavier
 if (noteo>0 && noteo<20) // premier test
 {
      printf("la note choisie est %d\n" ,noteo);
 }
            if (noteo>16 && noteo<20) // test si l'entrée est entre 16 et 20
               { 
                   noter= 'A'; // on enregistre la note A pour les notes de 16 à 20
               }
                else /// a construire
                {
 printf("X\n"); // sort de la premier boucle
                }
return noter;
printf("La conversion en char est %d\n", noter);
}
 
 
int main() {
 int noteo;
 noteo = 0;
 printf("entrer une note comprise entre 0 et 20 \n"); //Demande d'entrer une valeur entre 0 et 20
 noteo=conversion(noteo); //appel de la fonction qui donne le résultat, la note en A, B , C, D, ....
 return 0;
}

Est-ce que c'est ce genre de chose que tu dois produire ? Si j'ai bien compris ça correspond au cahier des charges.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
#include <stdio.h>
 
char conversion(int note){
    if (note<0 || note>20) return 'x';
    if (note>=16) return 'a';
    if (note>=13) return 'b';
    if (note>=10) return 'c';
    if (note>=8) return 'd';
    if (note>=6) return 'e';
    return 'f';
}
 
int main(){
    int note=0;
    printf("entrer une note comprise entre 0 et 20: ");
    scanf("%d",&note);
    printf("resultat: %c\n",conversion(note));
    return 0;
}

04/05/2017, 08h26
dalfab

Bravo hp41c,
@bo_yoyo, le code de hp41c a très bien utilisé la chronologie pour arriver à un code optimal, le tiens comporte beaucoup d'erreurs dues essentiellement à une non maîtrise de cette chronologie. Relis le bien et comprend le.

Citation:
Envoyé par bo_yoyo
Je dois écrire une fonction appelée conversion qui prend en paramètre une note (type int) et renvoie un caractère (type char).
Si la note est comprise entre 0 et 20, elle sera convertie dans la lettre correspondante. Si elle n'est pas comprise dans cet intervalle, le caractère renvoyé sera 'X'. La fonction ne doit faire ni lecture au clavier ni affichage à l'écran.
un main qui lit au clavier un int et affiche le résultat de la conversion de cet entier par la fonction conversion .
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 #include <stdio.h> char conversion(int noteo) //noteo correspond à la note enregistrer { int noter; //initialise la variable note resultat char notec; // note changé en char de type A B C notec=0; //on met la variable à 0 scanf("%d",&noteo); //enregistre la valeur entrée au clavier if (noteo>0 && noteo<20) // premier test { printf("la note choisie est %d\n" ,noteo); } if (noteo>16 && noteo<20) // test si l'entrée est entre 16 et 20 { noter= 'A'; // on enregistre la note A pour les notes de 16 à 20 } else /// a construire { printf("X\n"); // sort de la premier boucle } return noter; printf("La conversion en char est %d\n", noter); }
Soyez indulgent, je débute en programmation ;-)
Bonsoir

Débuter en programmation ne te dispense pas de lire l'énoncé. Et ne te dispense pas non plus de lire les messages qu'on te fait (surtout celui de droggo qui te dit que "if" n'est pas une boucle !!!).

Mis à part ça, les erreurs de ton programme c'est tout d'abord une trop grande quantité de variables. Tu en places "en veux-tu, en voilà" sans trop savoir à quoi elles vont te servir. Et plus tu as de variables plus ça devient complexe.
Autre chose: quand on écrit un code C, on l'écrit proprement surtout au niveau de l'indentation. Ca permet entre autres de s'y retrouver à l'écriture et surtout quand on y revient pour le faire évoluer. Regarde par exemple le code de hp41c. Sans même présumer de son efficacité, rien que dans l'indentation on le lit 100 fois mieux et 100 fois plus vite que le tien.

Et sinon, si l'instruction return te fait quitter la fonction, ça ne sert alors à rien de rajouter un printf() derrière.

06/05/2017, 15h38
matt42

Citation:

Envoyé par dalfab

Bravo hp41c

Je ne pense pas que lui donner la fonction toute faite lui soit d'une quelconque aide.

Attention :

Code:

sizeof('a') == sizeof(int)
06/05/2017, 17h30
Sve@r

Citation:

Envoyé par matt42

Je ne pense pas que lui donner la fonction toute faite lui soit d'une quelconque aide.

Dans l'absolu tu as absolument raison et je pense comme toi.

Toutefois, il peut arriver que parfois un petit exemple serve de point de départ et soit bénéfique (pourvu que ça ne devienne pas systématique). Et si ce n'est pas le cas, alors sa réussite à son exo grace à cet exemple ne sera qu'éphémère et sombera rapidement dans l'oubli. Et donc j'ai trouvé le code de hp41c tellement élégant que je le range dans cette catégorie d'exemple qui "peut" être ici plus bénéfique que néfaste.

Bref c'est à considérer comme "exception qui confirme la règle" quoi... 8-)
09/05/2017, 20h02
bo_yoyo

Merci

Bonjour
J’enregistre vos réponses et vos recommandations.
Je suis en train de décortiquer cette fonction et d'essayer de mieux comprendre.
En résumé, If n'est pas une condition, trop de variable (une variable est seulement nécessaire) et la mauvaise chronologie de ma fonction.
Bref, j'ai du travail devant moi.
Merci a vous.
Yohan

Une autre façon de faire, pour éviter les répétitions de code if .. return, est de stocker le barème dans un tableau, et de faire un truc comme cela :
Code:

1 2 3 4 5 6 7 8 9 int bareme[] = { 16, 13, 10, 8, 6 }; /* correspond à : a b c d e */ char conversion(int note){ int n = 0; if (note < 0 || note > 20) return 'x'; while (note < bareme[n++]); return 'a' -1 + n; }
Avec l'avantage que pour modifier le barème appliqué, il suffit de modifier les données du tableau