Problème avec mon code C

Salut,
Je suis actuellement en train d'apprendre le langage C, et j'ai voullu essayer d'utiliser la librairie math.h mais j'ai un petit problème.
Mon code se compile bien, mais me donne pas les bons résultat voici le code

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

#include <stdio.h> #include <stdlib.h> #include <math.h> //Inclusion de la librairie math.h   int main(int argc, char *argv[]) { long entree = 0; //Déclaration et initialisation des variables utilisées par le programme. long cosinus = 0; long valeurAbsolue = 0; long auDessus = 0;   printf("Entrez un nombre "); scanf("%lf", &entree); //On saisi un nombre. Les nombres décimaux sont tolérés. printf("Vous avez saisi %lf \n", entree); //Affichage du nombre saisi.   cosinus = cos(entree); //Traitement de la variable entree, puis stockage des résultat dans les variables correspondantes. valeurAbsolue = fabs(entree); auDessus = ceil(entree);   printf("Les resultat sont : \n"); //Affichage des résultats. printf("Le cosinus de %lf est %lf \n", entree, cosinus); printf("La valeure absolue de %lf est %lf", entree, valeurAbsolue); printf("L'entier au dessus de %lf, est %lf", entree, auDessus);       system("PAUSE"); return 0; }

---

]

Donc plutôt que de décrire ce qu'il me renvoie voici un screen du résultat compilé.

http://img182.imageshack.us/img182/9895/mathjl1.jpg

Voila si quelqu'un peut me donner un coup de main, bha je veut bien.
Merci d'avance :D

si tu passes tes valeurs avec des doubles, peut etre que ton probleme va s'arranger.

Voici ton code corrigé:

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

#include <stdio.h> #include <stdlib.h> #include <math.h>   int main(void) { long entree = 0; long cosinus = 0; long valeurAbsolue = 0; long auDessus = 0;   printf("Entrez un nombre "); scanf("%ld", &entree); printf("Vous avez saisi %ld \n", entree);   cosinus = cos(entree); valeurAbsolue = fabs(entree); auDessus = ceil(entree);   printf("Les resultat sont : \n"); printf("Le cosinus de %ld est %ld \n", entree, cosinus); printf("La valeure absolue de %ld est %ld\n", entree, valeurAbsolue); printf("L'entier au dessus de %ld, est %ld\n", entree, auDessus);     return 0; }

---

Ton erreur était que tu utilisait le format d'un double %lf alors que tes variables sont des long qui ont le format %ld. Tu pourrais aussi résoudre le problème en mettant double à la place de long.

Petit conseil si tu veux que ton code soit portable ne fait pas de commentaires // mais plutôt /* */ et n'utilise pas de fonction system.

Voila en espérant t'avoir aider:)

Citation:

---

Voila en espérant t'avoir aider

---

Oui un grand merci à vous deux d'avoir pris le temps de jeter un coup d'œil à mon code et de l'avoir corrigé :)
Et merci pour les conseils aussi.

la fonction cos() retourne un double, je ne suis pas certain que %ld soit juste.

Citation:

---

Envoyé par snake264

Ton erreur était que tu utilisait le format d'un double %lf alors que tes variables sont des long qui ont le format %ld. Tu pourrais aussi résoudre le problème en mettant double à la place de long.

---

Oui, enfin mettre le résultat d'un cosinus dans un entier c'est assez moyen cote précision.
L'utilisation d'un double est ici indispensable.

C'est vrai sachant que les fonctions qu'il utilise renvoient des double c'était la meilleur solution à prendre:) petite imprudence de ma part:oops:

Donc voici la version de snake264 modifiée (et retouchée ...) :)

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

#include <stdio.h> #include <math.h>   int main(void) { double entree, cosinus, valeurAbsolue, auDessus;   printf("Entrez un nombre : "); scanf("%lf", &entree); printf("Vous avez saisi %lf \n", entree);   cosinus = cos(entree); valeurAbsolue = fabs(entree); auDessus = ceil(entree);   printf("Les resultat sont : \n"); printf("Le cosinus de %lf est %lf \n", entree, cosinus); printf("La valeur absolue de %lf est %lf\n", entree, valeurAbsolue); printf("L'entier au dessus de %lf est %lf\n", entree, auDessus);   return 0; }

---

Pas besoin de stdlib, pas besoin d'initialiser les variables. Dans printf (seulement dans printf), on peut mettre simplement %f à la place de %lf.

En sécurisant un peu la saisie, cela donne:

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

#include <stdio.h> #include <math.h>   /* -tc- Elimine les caracteres non lus par scanf() */ void m_purge(void) { int c;   while ((c = fgetc(stdin)) != '\n' && c != EOF) { } }   int main(void) { double entree; double cosinus; double valeurAbsolue; double auDessus; int rv; /* -tc- Valeur de retour de scanf() */   /* -tc- Securiser un peu la saisie... */ do { printf("Entrez un nombre : "); /* -tc- Si l'invite ne se termine pas par un caractere de fin de ligne, il   recommande de force explicitement l'affichage a l'aide de fflush() */ fflush(stdout); /* -tc- Il est egalement recommande de toujours verifier la valeur retournee   par scanf() */ rv = scanf("%lf", &entree); /* -tc- Apres chaque appel de scanf(), on elimine les caracteres non lus en   prevision de la saisie suivante */ m_purge(); } while (rv != 1);   /* -tc- Le specificateur de format %lf n'est pas defini en C90 et est tolere   en C99. Je recommande plutot d'utilise %f pour afficher la valeur d'un   flottant de type double*/ printf("Vous avez saisi %f \n", entree);   cosinus = cos(entree); valeurAbsolue = fabs(entree); auDessus = ceil(entree);   printf("Les resultat sont : \n"); printf("Le cosinus de %f est %f \n", entree, cosinus); printf("La valeur absolue de %f est %f\n", entree, valeurAbsolue); printf("L'entier au dessus de %f est %f\n", entree, auDessus);   return 0; }

---

Thierry

En tous cas c'est très sympa de m'aider :D