Discussion :

[elieto]

Bonjour à tous, je debute en C (2 jour que je "programme"). J'ai un probleme sur un exo tout simple qui me demande de d'afficher une table de multiplication d'un entier compris entre 0 et 20 avec un test sur l'entier en question. Je suis sous windows et j'utilise Dev-C++, voila le programme que j'ai ecrit:

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>
int main(void)
{
  int N, i=0, produit, Borne_Inf=0, Borne_Sup=20;
  do {
     printf ("Tapez l'entier compris entre 0 et 20 dont vous voulez la table de multiplication \n");
     scanf("%d",&N); // pas de test pour le caractere entier scanf(%d) prend seulement la partie entiere
     if (N < Borne_Inf || N > Borne_Sup)
        {printf ("L'entier choisit n'est pas compris entre 0 et 20!\n");
        }
     }
  while(N < Borne_Inf || N > Borne_Sup);
 
  for (i=Borne_Inf; i<=Borne_Sup; i++)
    {
   	produit = i * N;
	printf ("%d x %d = %d\n", N, i, produit);
    }
 
  system("PAUSE");
  return 0;
}

Quand je compile pas de soucis et quand je l'éxecute avec un entier quelqu'il soit ou un réel lui entre 0 et 20 pas de soucis non plus mais si je lui donnes un réel non compris dans [0;20] il boucle a l'infini !
Je ne comprend pas pourquoi...
Je suis allez voir les tutoriel de devellopez à propos de scanf ou de Do while et je n'arrive pas a voir l'erreur.
Merci d'avance pour vos reponse à mes questions!

[oswalidos]

j'ai essayé ton code et ca marche sans probleme ,je vois pas la boucle infinie !!
relance ton dev-c++ ou redemarre le pc ,je ne sais pas ce que tu dois faire mais ca marche la ,avec xp et code::blocks .

[FunK92]

Bonjour,

Pour oswalidos, as mon avis tu n'a pas testé completement sont programme, car effectivement pour un reel non compris dans les bornes, cela nous donne une belle boucle infini.

Sinon concernant le probleme tu declare ton N comme étant un entier de type "int", alors que tu lui affecte un reel (float/double) .

Met un float N, a la place d'un int N, sa devrait aller

[Emmanuel Delahaye]

Quand je compile pas de soucis et quand je l'éxecute avec un entier quelqu'il soit ou un réel lui entre 0 et 20 pas de soucis non plus mais si je lui donnes un réel non compris dans [0;20] il boucle a l'infini

Ce n'est pas vrai. Déjà, c'est pas 'réel', mais 'entier'... Le défaut se produit si on entre une valeur invalide ('a', par exemple). C'est dû au fait que scanf() laisse trainer un \n dans le flux qui le rend non bloquant. La règle de bon usage est de systématiquement vider le flux stdin après usage si on est pas certain que le dernier caractère lu était un '\n'.

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
 
#include <stdio.h>
 
static void fpurge (FILE * fp)
{
   int c;
   while ((c = fgetc (fp)) != '\n' && c != EOF)
   {
   }
}
 
int main (void)
{
   int N, i = 0, produit, Borne_Inf = 0, Borne_Sup = 20;
   do
   {
      N = -1;
      printf
         ("Tapez l'entier compris entre 0 et 20 dont vous voulez la table de multiplication \n");
/* pas de test pour le caractere entier scanf(%d) prend seulement la partie entiere */
      scanf ("%d", &N);
      fpurge (stdin);
      if (N < Borne_Inf || N > Borne_Sup)
      {
         printf ("L'entier choisit n'est pas compris entre 0 et 20!\n");
      }
   }
   while (N < Borne_Inf || N > Borne_Sup);
 
   for (i = Borne_Inf; i <= Borne_Sup; i++)
   {
      produit = i * N;
      printf ("%d x %d = %d\n", N, i, produit);
   }
 
   return 0;
}

On peut aussi envisager de faire ses saisies avec fgets() et la conversion qui va bien...

[jiddou]

Ton programme est syntaxiquement et algorithmiquement correct.Mais c'est langage C qui est comme ça.Quand tu valide pour la première fois à chaque fois qu'on repasse dans la boucle scanf trouve un \n dans le flux qui la valide donc c comme si cette instruction ne s'exécute plus.

Alors La solution:
après le scanf("%d",&N); mettre fflush(stdin);
J'espère avoir régler ton problème.

[Emmanuel Delahaye]

Alors La solution:
après le scanf("%d",&N); mettre fflush(stdin);
J'espère avoir régler ton problème.

Non.

http://c.developpez.com/faq/?page=cl...N_fflush_stdin

[jiddou]

Non.

http://c.developpez.com/faq/?page=cl...N_fflush_stdin

Et pourtant j'insiste parce qu'il va tester et ça va marcher et c'est plus propre.Au lieu d'écrire tout une fonction pour lire un entier.
Problème de portabilité mais je ne pense pas que c'est son soucis pour le moment :
En tout cas ça marche aussi bien sous Linux que sous windows alors bonne chance

[Emmanuel Delahaye]

Et pourtant j'insiste parce qu'il va tester et ça va marcher et c'est plus propre.Au lieu d'écrire tout une fonction pour lire un entier.
Problème de portabilité mais je ne pense pas que c'est son soucis pour le moment :
En tout cas ça marche aussi bien sous Linux que sous windows alors bonne chance

Euh, non, ça ne marche pas du tout sous Linux (comportement indéfini). Des cas de plantages ont été signalés...