Discussion :

[Muesko]

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



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

[emprex]

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

[snake264]

Voici ton code 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
#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

[Muesko]

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.

[emprex]

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

[gl]

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.

[snake264]

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

[Melem]

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

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

[Thierry Chappuis]

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

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

[Muesko]

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