Mon petit programme est-il robuste ?

Version imprimable

Bonjour. J'ai créé un programme de style (trouvez le nombre) et je souhaite que vous le testiez dans le but de savoir s'il est si robuste qu'il en à l'air :

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
 
#include <stdio.h>
#include <stdlib.h>
#include "myHeader.h"
int main(void)
{
  int hasard, argent;
  char code[2], saisie[2], choix[2];
 
  srand(time(NULL));
  hasard = rand() % (50+1);
 
  fprintf(stdout, "Samy Hocine - Programme\n");
  fprintf(stdout, "1 - Jouer\n");
  fprintf(stdout, "2 - Quitter\n\n");
  fprintf(stdout, "Votre choix:");
 
  fgets(choix, sizeof choix, stdin);
  fclean(choix, stdin);
 
  if(!strcmp(choix, "1")==0)
  {
                   return 0;
  }
  else
  {
 
  system(CLEAR);
 
  fprintf(stdout, "Entrez un nombre entre 10 et 20\n\nVotre nombre:");
  fgets(saisie, sizeof saisie, stdin);
  fclean(saisie, stdin);
  snprintf(code, sizeof code, "%i", hasard);
 
  if(strcmp (saisie, code)==0)
  {
            fprintf(stdout, "\n\nVous avez trouve le bon nombre\n\n");
  }
  else
  {
      fprintf(stdout, "\n\nVous n'avez pas trouve le bon nombre\n");
      fprintf(stdout, "Le bon nombre etait %i\n\n", hasard);
  }
 
  pause("Appuyez sur ENTRER pour continuer");
  system(CLEAR);
 
  return main();
 
  }
}

myHeader.h

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
#ifndef MYHEADER_H
#define MYHEADER_H
#ifdef WIN32
   #define CLEAR "CLS"
#elif defined (linux)
   #define CLEAR "clear"
#else
   #error Not defined for this plateforme
#endif
void fclean(char *s_buffer, FILE *stream);
void pause(char const *s_message);
#endif //MYHEADER_H

myHeader.cpp

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
 
#include <stdio.h>
#include <string.h>
#include <stddef.h>
#include "myHeader.h"
void fclean(char *s_buffer, FILE *stream)
{
    if (s_buffer != NULL && stream != NULL)
    {
        char *pc = strchr(s_buffer, '\n');
 
        if (pc != NULL) /* La saisie n'a pas été tronquée */
        {
             /* On remplace '\n' par le caractère nul '\0' */
             *pc = 0;
        }
        else
        {
            /* La saisie a été tronquée, on purge le flux d'entrée */
            int c;
            while ((c = fgetc(stream)) != '\n' && c != EOF)
            {
                  //Rien
            }
        }
    }
}
void pause (const char *s_message)
{
   if (s_message != NULL && strlen (s_message) != 0)
   {
      fprintf (stdout, "%s", s_message);
      fflush (stdout);
   }
   {
      char c[2];
      fgets (c, sizeof c, stdin);
      fclean (c, stdin);
   }
}

Merci d'avance :)

15/09/2007, 19h21
aoyou

Quelques petites remarques en vitesse...

myHeader.cpp je ne connais pas c'est l'extension d'un source C++. Renomme le myHeader.c

Inclus <string.h> et <time.h> pour strcmp et time. Ce sera mieux.

fprintf(stdout peut être remplacé par printf(.

Les doubles négations

Code:

!strcmp(choix, "1")==0

ne facilitent pas la lecture. Préfère

Code:

strcmp(choix, "1") != 0

saisie n'est pas assez grand pour contenir un nombre entre 10 et 20.

Revois le calcul de hasard. Tu ne tires pas un nombre entre 10 et 20. Difficile de gagner par conséquent. rand() tire entre 0 et RAND_MAX.

Citation:

Dans Numerical Recipes in C: The Art of Scientific Computing (William H. Press, Brian P. Flannery, Saul A. Teukolsky, William T. Vetterling; New York: Cambridge University Press, 1990 (1st ed, p. 207)), le commentaire suivant apparaît :

"Si vous désirez engendrer un entier aléatoire entre 1 et 10, vous devez toujours procéder en utilisant les bits de poids forts, comme dans :

j=1+(int) (10.0*rand()/(RAND_MAX+1.0));

et jamais ainsi :

j=1+(rand() % 10);

(car cette derniere version utilise les bits de poids faibles)."

Code:

return main();

8O8O8O8O C'est la première fois que je vois ça. Utilise une boucle while() à la place. En plus, ça te fait appeler srand() plusieurs fois. Il ne faut appeler srand qu'une seule fois.

Tu ne contrôles pas que l'utilisateur rentre un nombre entre 10 et 20.

myHeader.cpp c'était une faute. En fait il se nomme bien myHeader.c (je code en c++ de temps en temps :) c'est pour ca)

Si j'utilise "votre code" pour la génération de mots de passes, j'obtiens toujours la même chose pas avec "ma méthode".

Je cherche à convertir un char en int (avez vous une idée ?) (dans le but de vérifier l'entrée)

En fait, je m'étais trompé, c'est de 0 à 50
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 54 #include <stdio.h> #include <stdlib.h> #include <time.h> #include <string.h> #include "myHeader.h" int main(void) { int hasard; char code[2], saisie[2], choix[2]; srand(time(NULL)); hasard=rand()%(50+1); fprintf(stdout, "%i\n\n", hasard); fprintf(stdout, "Samy Hocine - Programme\n"); fprintf(stdout, "1 - Jouer\n"); fprintf(stdout, "2 - Quitter\n\n"); fprintf(stdout, "Votre choix:"); fgets(choix, sizeof choix, stdin); fclean(choix, stdin); if(strcmp(choix, "1")!=0) { return 0; } else { system(CLEAR); fprintf(stdout, "Entrez un nombre entre 0 et 50\n\nVotre nombre:"); fgets(saisie, sizeof saisie, stdin); fclean(saisie, stdin); snprintf(code, sizeof code, "%i", hasard); if(strcmp (saisie, code)==0) { fprintf(stdout, "\n\nVous avez trouve le bon nombre\n\n"); } else { fprintf(stdout, "\n\nVous n'avez pas trouve le bon nombre\n"); fprintf(stdout, "Le bon nombre etait %i\n\n", hasard); } pause("Appuyez sur ENTRER pour continuer"); system(CLEAR); return main(); } }
Sinon, pour la boucle, je ne vois pas trop comment faire :( J'ai peur de voir apparaître indéfiniment mon programme dans la console :)

15/09/2007, 20h10
aoyou

Citation:

Si j'utilise "votre code" pour la génération de mots de passes, j'obtiens toujours la même chose pas avec "ma méthode".

Oups je suis désolé. William H. Press, Brian P. Flannery, Saul A. Teukolsky et William T. Vetterling se sont trompés tous les quatre.
Non, sérieusement ce code génére bien un nombre aléatoire. Ce n'est pas "mon code" ; c'est une "méthode" approuvée. Ton code de génération marche aussi mais a des défauts. Sinon, appelle srand () qu'une seule fois.

Citation:

Je cherche à convertir un char en int (avez vous une idée ?) (dans le but de vérifier l'entrée)

Regarde atoi. Tu veux convertir un char * pas un char.

Citation:

Sinon, pour la boucle, je ne vois pas trop comment faire J'ai peur de voir apparaître indéfiniment mon programme dans la console

Mais non mais non. Ta boucle doit tourner dans que l'utilisateur ne veux pas quitter. A toi de faire ce qu'il faut. Surtout qu'il n'y a pas grand chose à modifier... 3 lignes.
15/09/2007, 20h18
Emmanuel Delahaye
Citation:

Envoyé par _SamSoft_

Sinon, pour la boucle, je ne vois pas trop comment faire :( J'ai peur de voir apparaître indéfiniment mon programme dans la console :)

Un problème à la fois. Pour le moment, ne fait pas de boucle (ou utilise l'astuce que je t'ai montré avec la saisie du nom du répertoire dans un autre projet).

Déjà, je trouve que tu te disperses beaucoup. Mener 3 projets en même temps quand on débute, c'est pas très sérieux...

Ensuite, tu dois te concentrer sur un sujet à la fois. Ici, tu cherches à manipuler des valeurs numériques. OK.

Tu as réglé le problème de la saisie de chaine, c'est bien.

Maintenant, il faut apprendre à saisir une valeur numérique. Pour ça, le principe est simple. On utlise fgets() et fclean() comme d'habitude, puis on convertit la chaine avec
- strtol() si la valeur attendue est un entier signé (long)
- strtoul() si la valeur attendue est un entier non signé (unsigned long)
- strtod() si la valeur attendue est un réel (float)
Avant de poser 3000 questions, lire la doc puis expérimenter... (ça n'a pas été fait déjà ?)
15/09/2007, 20h39
_SamSoft_

Merci :) Je continuerai sûrement ca demain (après mes devoirs de maths :D ) ou mercredi prochain.

Bonjour, j'ai plein de questions :D

Déjà le code (j'ai juste ajouté ce qu'a dis Emmanuel) :
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 54 55 #include <stdio.h> #include <stdlib.h> #include <time.h> #include <string.h> #include "myHeader.h" int main(void) { int hasard; char code[2], saisie[2], choix[2], end; srand(time(NULL)); hasard=rand()%(50+1); fprintf(stdout, "%i\n\n", hasard); fprintf(stdout, "Samy Hocine - Programme\n"); fprintf(stdout, "1 - Jouer\n"); fprintf(stdout, "2 - Quitter\n\n"); fprintf(stdout, "Votre choix:"); fgets(choix, sizeof choix, stdin); fclean(choix, stdin); end=(saisie[0]=='\0'); if(!end) if(strcmp(choix, "1")!=0) { return 0; } else { system(CLEAR); fprintf(stdout, "Entrez un nombre entre 0 et 50\n\nVotre nombre:"); fgets(saisie, sizeof saisie, stdin); fclean(saisie, stdin); snprintf(code, sizeof code, "%i", hasard); if(strcmp (saisie, code)==0) { fprintf(stdout, "\n\nVous avez trouve le bon nombre\n\n"); } else { fprintf(stdout, "\n\nVous n'avez pas trouve le bon nombre\n"); fprintf(stdout, "Le bon nombre etait %i\n\n", hasard); } pause("Appuyez sur ENTRER pour continuer"); system(CLEAR); return main(); } while(!end); }
Bon, j'ai essayé de découper mon programme en fonctions (pour supprimer cet affreux return main(); mais c'est une catastrophe, j'y arrive pas [une petite aide de ce côté serait la bienvenue).

Je continu de dire que le code des 4programmeurs américains sur la génération de mots de passes ne donne que le même mot de passe à chaque fois (pas avec ma méthode :aie:) :
Code:

1 2 3 srand(time(NULL)); hasard=(int) (50*rand()/(RAND_MAX+1));
Emmanuel: Pourquoi dois-je utiliser strtol() alors que snprintf fait très bien son travail ?

Pour les fprintf au lieu de printf, c'est parce que je préfère.

Emmanuel: Je n'ai que deux projets :) "Listeur" et "Plus ou Moins". Deux tout petits en C c'est raisonnable maintenant si je voulais me lancer dans la création d'un os d'un côté et la création d'un langage de programmation de l'autre, là ce serait totalement idiot !

Merci d'avance :)

16/09/2007, 17h04
aoyou
Citation:
Envoyé par _SamSoft_

Je continu de dire que le code des 4programmeurs américains sur la génération de mots de passes ne donne que le même mot de passe à chaque fois (pas avec ma méthode :aie:) :

Code:

1 2 3 srand(time(NULL)); hasard=(int) (50*rand()/(RAND_MAX+1));
Le code que tu fournis n'est pas celui des 4 américains. Regardes-tu les warnings de temps en temps ?

Code:

warning: integer overflow in expression

50*rand()/(RAND_MAX+1), ce n'est pas la même chose que 50.0*rand()/(RAND_MAX+1)
Revois l'utilisation de l'opérateur / avec les entiers et les flottants.

On ne parle pas de la génération de mots de passe mais de la génération d'un nombre pseudo-aléatoire.

Ton tableau saisie est toujours trop petit pour contenir en entier entre 0 et 50.
16/09/2007, 17h14
Emmanuel Delahaye
Citation:

Envoyé par _SamSoft_

Bon, j'ai essayé de découper mon programme en fonctions (pour supprimer cet affreux return main(); mais c'est une catastrophe, j'y arrive pas [une petite aide de ce côté serait la bienvenue).

Le principe est simple. On utilise une bête boucle avec une condition de sortie :
Code:

1 2 3 4 5 6 do { /* traitement */ } while (!fin);
A toi de déterminer comment le traitement fait passer fin à 1.
Citation:
Je continu de dire que le code des 4programmeurs américains sur la génération de mots de passes ne donne que le même mot de passe à chaque fois (pas avec ma méthode :aie:) :

Code:

1 2 3 srand(time(NULL)); hasard=(int) (50*rand()/(RAND_MAX+1));
Un 'mot de passe', c'est pas un nombre, c'est du texte...
Cette formule est incorrecte.

Voici le principe :
1. La norme dit que srand() retourne une valeur comprise en 0 et RAND_MAX.
2. L'idée est de diviser cette valeur par RAND_MAX, afin d'obtenir un nombre en 0 et 1. : rand() / RAND_MAX
3. Evidemment, il faut éviter la division euclidienne, donc on force un des opérandes en double : (double) rand() / RAND_MAX
4. Pour obtenir un plage de 0 à N, on multiplie ce résultat par N et on prend la partie entière :
  
  Code:
  
  int rnd = (int) (N * ((double) rand() / RAND_MAX));
  
  après, on peut ajouter des décalages (offset) etc.
Citation:

Emmanuel: Pourquoi dois-je utiliser strtol() alors que snprintf fait très bien son travail ?

J'avais compris que tu voulais convertir une chaine en valeur numérique.

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
54
55
56
57
58
59
 
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include <string.h>
#include "myHeader.h"
int main(void)
{
  int hasard;
  char code[2], saisie[2], choix[2], end;
 
  srand(time(NULL));
  hasard=rand()%(50+1);
  fprintf(stdout, "%i\n\n", hasard);
 
  do
  {
  do
  {
  fprintf(stdout, "Samy Hocine - Programme\n");
  fprintf(stdout, "1 - Jouer\n");
  fprintf(stdout, "2 - Quitter\n\n");
  fprintf(stdout, "Votre choix:");
 
  fgets(choix, sizeof choix, stdin);
  fclean(choix, stdin);
  end=(saisie[0]=='\0');
  if(strcmp(choix, "1")!=0)
  {
                   return 0;
  }
  else
  {
 
  system(CLEAR);
 
  fprintf(stdout, "Entrez un nombre entre 0 et 50\n\nVotre nombre:");
  fgets(saisie, sizeof saisie, stdin);
  fclean(saisie, stdin);
  snprintf(code, sizeof code, "%i", hasard);
 
  if(strcmp (saisie, code)==0)
  {
            fprintf(stdout, "\n\nVous avez trouve le bon nombre\n\n");
  }
  else
  {
      fprintf(stdout, "\n\nVous n'avez pas trouve le bon nombre\n");
      fprintf(stdout, "Le bon nombre etait %i\n\n", hasard);
  }
 
  pause("Appuyez sur ENTRER pour continuer");
  system(CLEAR);
  }
  }
  while(1);
  }
  while(!end);
}

Pour la génération de nombres pseudo-aléatoire, rien n'y fait :

Code:

1
2
3
4
srand(time(NULL));
long rnd;
rnd=(int) (50.0 * ((double) rand() / RAND_MAX));

Me donne toujours 31 8O

Emmanuel : c'est bon j'ai trouvé un autre truc pour contourner le problème

Citation:

Envoyé par _SamSoft_

Pour la génération de nombres pseudo-aléatoire, rien n'y fait :

Code:

1
2
3
4
 
srand(time(NULL));
long rnd;
rnd=(int) (50.0 * ((double) rand() / RAND_MAX));

Me donne toujours 31

Mais qu'est-ce que tu racontes ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
#include <stdlib.h>
#include <stdio.h>
#include <time.h>
 
int main (void)
{
   srand (time (NULL));
 
   int i;
   for (i = 0; i < 10; i++)
   {
      int rnd = (int) (50 * ((double) rand () / RAND_MAX));
      printf ("rnd = %d\n", rnd);
   }
   return 0;
 
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
rnd = 6
rnd = 34
rnd = 41
rnd = 12
rnd = 5
rnd = 37
rnd = 8
rnd = 33
rnd = 2
rnd = 1
 
Press ENTER to continue.

Citation:

Envoyé par _SamSoft_

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
54
55
56
57
58
59
 
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include <string.h>
#include "myHeader.h"
int main(void)
{
  int hasard;
  char code[2], saisie[2], choix[2], end;
 
  srand(time(NULL));
  hasard=rand()%(50+1);
  fprintf(stdout, "%i\n\n", hasard);
 
  do
  {
  do
  {
  fprintf(stdout, "Samy Hocine - Programme\n");
  fprintf(stdout, "1 - Jouer\n");
  fprintf(stdout, "2 - Quitter\n\n");
  fprintf(stdout, "Votre choix:");
 
  fgets(choix, sizeof choix, stdin);
  fclean(choix, stdin);
  end=(saisie[0]=='\0');
  if(strcmp(choix, "1")!=0)
  {
                   return 0;
  }
  else
  {
 
  system(CLEAR);
 
  fprintf(stdout, "Entrez un nombre entre 0 et 50\n\nVotre nombre:");
  fgets(saisie, sizeof saisie, stdin);
  fclean(saisie, stdin);
  snprintf(code, sizeof code, "%i", hasard);
 
  if(strcmp (saisie, code)==0)
  {
            fprintf(stdout, "\n\nVous avez trouve le bon nombre\n\n");
  }
  else
  {
      fprintf(stdout, "\n\nVous n'avez pas trouve le bon nombre\n");
      fprintf(stdout, "Le bon nombre etait %i\n\n", hasard);
  }
 
  pause("Appuyez sur ENTRER pour continuer");
  system(CLEAR);
  }
  }
  while(1);
  }
  while(!end);
}

Pourquoi une indentation aussi horrible ?
Pourquoi 2 boucle imbriquées ?
Pourquoi cet horrible 'return 0;' alors qu'il suffit de faire 'end=1' ?
Si l'appel à rand() n'est pas dans la boucle, le nombre ne risque pas changer...

indentation : Dev-Cpp me met en colère (son système de gestion de l'appuie sur tab n'est pas très correct ! ) Je vais bientôt installer C::B de toute façon .

Boucle : Si je n'en met qu'une (c.f code plus bas) ca ferme lors de "Appuyez sur entrer pour continuer"

return 0 : J'ai corrigé (mais quel est le problème ?)

rand() : En fait je croyais qu'à chaque lancement le nombre changeait. C'est en tout cas ce qui se passait avec la méthode que j'utilisais.

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
 
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include <string.h>
#include "myHeader.h"
int main(void)
{
  int hasard;
  char code[2], saisie[2], choix[2], end;
 
  srand(time(NULL));
 
   do
   {
 
   hasard=(int) (50 * ((double) rand() / RAND_MAX));
     fprintf(stdout, "Samy Hocine - Programme\n");
    fprintf(stdout, "1 - Jouer\n");
     fprintf(stdout, "2 - Quitter\n\n");
      fprintf(stdout, "Votre choix:");
 
    fgets(choix, sizeof choix, stdin);
    fclean(choix, stdin);
    end=(saisie[0]=='\0');
 
    if(strcmp(choix, "1")!=0)
     {
                 end=1;
     }
     else
     {
 
        system(CLEAR);
 
       fprintf(stdout, "Entrez un nombre entre 0 et 50\n\nVotre nombre:");
       fgets(saisie, sizeof saisie, stdin);
       fclean(saisie, stdin);
       snprintf(code, sizeof code, "%i", hasard);
 
       if(strcmp (saisie, code)==0)
       {
                  fprintf(stdout, "\n\nVous avez trouve le bon nombre\n\n");
        }
       else
      {
            fprintf(stdout, "\n\nVous n'avez pas trouve le bon nombre\n");
         fprintf(stdout, "Le bon nombre etait %i\n\n", hasard);
      }
 
       pause("Appuyez sur ENTRER pour continuer");
       system(CLEAR);
    }
  }
  while(!end);
}

J'ai modifié l'indentation

EDIT: Sur mon code tout est parfait, ici ca à l'air un peu de zigzaguer -_-

Citation:

Envoyé par _SamSoft_

indentation : Dev-Cpp me met en colère (son système de gestion de l'appuie sur tab n'est pas très correct ! ) Je vais bientôt installer C::B de toute façon .

Tu peux aussi compiler GNUIndent 1.91 et l'installer comme outil externe à Dev-C++ ...

http://emmanuel-delahaye.developpez....tm#indentation

Ca 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 54 55 56 57 #include <stdio.h> #include <stdlib.h> #include <time.h> #include <string.h> #include "myHeader.h" int main (void) { int hasard; char code[2], saisie[2], choix[2], end = 0; srand (time (NULL)); do { hasard = (int) (50 * ((double) rand () / RAND_MAX)); fprintf (stdout, "Samy Hocine - Programme\n"); fprintf (stdout, "1 - Jouer\n"); fprintf (stdout, "2 - Quitter\n\n"); fprintf (stdout, "Votre choix:"); fgets (choix, sizeof choix, stdin); fclean (choix, stdin); end = (saisie[0] == '\0'); if (strcmp (choix, "1") != 0) { end = 1; } else { system (CLEAR); fprintf (stdout, "Entrez un nombre entre 0 et 50\n\nVotre nombre:"); fgets (saisie, sizeof saisie, stdin); fclean (saisie, stdin); snprintf (code, sizeof code, "%i", hasard); if (strcmp (saisie, code) == 0) { fprintf (stdout, "\n\nVous avez trouve le bon nombre\n\n"); } else { fprintf (stdout, "\n\nVous n'avez pas trouve le bon nombre\n"); fprintf (stdout, "Le bon nombre etait %i\n\n", hasard); } pause ("Appuyez sur ENTRER pour continuer"); system (CLEAR); } } while (!end); return 0; }
(j'ai initialisé end, sinon, le comportement est indéterminé. J'ai placé le return 0 correctement.).

Citation:

Boucle : Si je n'en met qu'une (c.f code plus bas) ca ferme lors de "Appuyez sur entrer pour continuer"

Probablement parce que end n'était pas initialisé...

Citation:

return 0 : J'ai corrigé (mais quel est le problème ?)

Le langage C utilise les principes de la programmation structurée. Un des principes est "une fonction n'a qu'un seul point de sortie". Ca permet d'écrire du code plus sûr et plus fiable.

Donc, pas de return n'importe où. Juste en dernière instruction de la fonction.

Citation:

rand() : En fait je croyais qu'à chaque lancement le nombre changeait. C'est en tout cas ce qui se passait avec la méthode que j'utilisais.

Je ne sais pas de quoi tu parles, mais ce code se comporte correctement.

Merci pour indent mais lors de la copie (dev-cpp ici, ca me met des tabulations étranges -_- );

Citation:

Probablement parce que end n'était pas initialisé...

Code:

1
2
 
char end = 0; /* ceci est 'etrange, un char = 0 ? */

Pour rand() : pas grâve je me comprend et j'ai compris le problème (ca ne vient pas de moi :) )

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
54
55
56
57
 
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include <string.h>
#include "myHeader.h"
 
int main (void)
{
   int hasard;
   char code[2], saisie[2], choix[2], end = 0;
 
   srand (time (NULL));
 
   do
   {
      hasard = (int) (50 * ((double) rand () / RAND_MAX));
      fprintf (stdout, "Samy Hocine - Programme\n");
      fprintf (stdout, "1 - Jouer\n");
      fprintf (stdout, "2 - Quitter\n\n");
      fprintf (stdout, "Votre choix:");
 
      fgets (choix, sizeof choix, stdin);
      fclean (choix, stdin);
      end = (saisie[0] == '\0');
 
      if (strcmp (choix, "1") != 0)
      {
         end = 1;
      }
      else
      {
         system (CLEAR);
 
         fprintf (stdout, "Entrez un nombre entre 0 et 50\n\nVotre nombre:");
         fgets (saisie, sizeof saisie, stdin);
         fclean (saisie, stdin);
         snprintf (code, sizeof code, "%i", hasard);
 
         if (strcmp (saisie, code) == 0)
         {
            fprintf (stdout, "\n\nVous avez trouve le bon nombre\n\n");
         }
         else
         {
            fprintf (stdout, "\n\nVous n'avez pas trouve le bon nombre\n");
            fprintf (stdout, "Le bon nombre etait %i\n\n", hasard);
         }
 
         pause ("Appuyez sur ENTRER pour continuer");
         system (CLEAR);
      }
   }
   while (!end);
 
   return 0;
}

17/09/2007, 18h54
Thierry Chappuis
Citation:
Envoyé par _SamSoft_

Code:

1 2 char end = 0; /* ceci est 'etrange, un char = 0 ? */
C'est équivalent à:

Code:

char end = '\0';

qui correspond au caractère nul ou caractère de valeur 0 ('\0' est en fait la représentation octale de la valeur 0).

Thierry
17/09/2007, 18h57
Emmanuel Delahaye
Citation:
Envoyé par _SamSoft_

Code:

1 2 char end = 0; /* ceci est 'etrange, un char = 0 ? */
Oui, j'aurais mis un int. Toujours cette manie de factoriser les types...

A part ça, c'est quoi la question ?

(à part qu'un jeu de devinette non inter-actif, c'est sans intérêt...). Voir plutôt le +ou- du SdZ...
17/09/2007, 19h14
_SamSoft_

Je poursuivrai ce projet plus tard.