Discussion :

[1debutante]

Bonjour,je suis une debutante en programmation ,j ai ecris ce pgr mais je n arrive pas a trouver l erreur,le but etant de calculer les termes de la suite:1/2!+2p2/5!+3p3/8!....+t avec un epsilon donne par l utilisateur,(2p2=2 puissance 2)

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<conio.h>
#include<math.h>
main()
{
 int i,j,k;
 float eps,s,p;
 do
 {
  printf("Quelle est la precision desiree?(0<eps<1)");
  scanf("%f",&eps);
 }while((eps<=0)||(eps>=1));
  s=0;
 for(i=1,j=2;i<j*eps,j<i/eps;i++,j+=3)
 {
  p=1;
  for(k=2;k<=j;k--)
  {
   p=p*k;
  }
  s=s+(pow(i,i)/p);
 
 }
 
 printf("s= %f (avec eps= %f)",s,eps);
 getch();
 }

Biensur tout est elementaire ds ce pgr ,vu que je n en suis qu au tout debut
Merci d avance pour votre aide

[Trap D]

Et d'une : dans : i<j*eps,j<i/eps , (ton test de boucle for) il y a comme un problème, puisque j < i / eps est sensiblement identique à j * eps < i, donc il faut choisir

• ou i < j * eps
• ou i > j * eps

mais pas les deux en même temps.
Et de deux, dans l'enchainement i < j * eps, j < i / eps, la valeur retournée par l'évaluation, en C, est celle de j < i / eps.

Donc réfléchis bien à ce que tu veux faire et enlève une des deux inégalités.

Maintenant, le prototype de main dans ton cas est int main(void), il ne faut pas écrire main(), c'est du mauvais C.

Enfin, (pour le moment), es-tu sûr de cette boucle

Code :

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

1
2
3
4
  for(k=2;k<=j;k--)
  {
   p=p*k;
  }

Car soit tu ne rentres jamais dans la boucle si 2 est plus grand que j, soit tu n'en sors jamais puisque tu décrémentes k au lieu de l'augmenter, (k++)

[Thierry Chappuis]

Maintenant, le prototype de main dans ton cas est int main(void), il ne faut pas écrire main(), c'est du mauvais C.

C'est pas du mauvais, c'est pas du C (selon la norme actuelle du langage).

Thierry

[gege2061]

C'est pas du mauvais, c'est pas du C (selon la norme actuelle du langage).

Oui mais selon la norme la plus utilisée (C90), c'est authorisé, en l'absence de type de retour, c'est int par défaut (c'est moins pire que mettre void)

[1debutante]

Ah oui ,c est en effet vrai,donc i<j*eps est suffisante si j ai bien compris?
pour la décrémentation de k c est k++
En ce qui concerne le main(),c est vrai que dans un .c il y aurait un warning par contre dans un .cpp il l'accepte.Et je suis d'accord sur le fait que ce n est pas rigoureux ,je me demande d'ailleurs pourquoi on ns apprend a programmer avec C ds des fichiers cpp ?!
merciii bppp pour votre aide

[gege2061]

En ce qui concerne le main(),c est vrai que dans un .c il y aurait un warning par contre dans un .cpp il l'accepte.

Non c'est l'inverse : Les incompatibilités entre le C et le C++

[rebel64]

Effectivement il faut k++ pour le calcul de la factoriel
p=1;
for(k=2;k<=j;k--)
{
p=p*k;
}

Mais aussi les tests d'arrêt i<j*eps,j<i/eps à la précision epsilon de ta série risque de poser un problème de convertion (je n'en suis pas certain):

i,j sont des entiers divisés ou mutipliés par un flottant compris dans ]0 1[