Problème sur un prog qui se ferme alors qu'il ne devrait pas

Version imprimable

Bonjour
mon prog (Nombre Mysère)une fois que j'ai choisi oui ou non pour accepter ou non le jeu se referme alors qu'avec le oui il devrait se poursuivre.Pourais-t-on m'aider ? Merci
Voici le prog :

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
60
61
62
63
64
65
66
67
68
69
70
71
#include<stdio.h>
#include<stdlib.h>
#include<time.h>
 
 
main()
{
     system("cls");
     int re,nbre_alea,nbre_saisi,quitter;
     char cont[3];
 
     //variable aléatoire
     srand((int)time(NULL));        //randomize : init la fonction rand
     nbre_alea=rand()%51;               // fonction qui renvoie une variable aléatoire entre 0 et 50           
 
     printf ("%55s","Le jeux consiste a trouver le nombre mystere compris entre 0 et 50.\n\n\n");
     printf ("Veux-tu y jouer ? Ecrit oui  ou  non\n\n");
     scanf ("%c",cont);
 
     if (cont=="oui")
     {
     system ("cls");
     printf("le nombre = %d",nbre_alea);
 
 
     printf("\n\n\n");
     printf("Saisir  un nombre entre 0 et 50 :  ");
     scanf("%d",&nbre_saisi);
 
 
     while (nbre_alea!=nbre_saisi)
     {
           if (nbre_saisi>nbre_alea & nbre_saisi<50)
           {           
                       printf("\n\n");
                       printf ("%30s","Trop Grand !\n\n");
                       printf("Saisir  un nombre entre 0 et 50 :");
                       scanf("%d",&nbre_saisi);
            }
            if (nbre_saisi<nbre_alea && nbre_saisi<50)
            {
                       printf("\n\n");
                       printf ("%30s","Trop Petit !\n\n");
                       printf("Saisir  un nombre entre 0 et 50 :");         
                       scanf("%d",&nbre_saisi);
            }
            if (nbre_saisi >50)
            {
                       printf("\n\n");
                       printf ( "%60s", " Ne sois pas bete ! c'est entre 0 et 50\n");
                       scanf("%d",&nbre_saisi);
            }
      }
      system("cls");
      printf ("\n\n\n\n\n\n\n");
      printf ("%55s","Bravo !!!  Le nombre choisi est le bon\n\n");
 
      printf ("\n\n\n\n");
      printf ("%50s","Veut-tu rejouer ? 1=oui ou 0=non\n");
      scanf("%d",&re);
 
      if (re==1) {return main(); }
      else {return 0; }
      }
      else
 
      {return 0;}
 
 
 
}

25/09/2005, 17h39
diogene
Code:

if (cont=="oui")

on ne peut pas comparer des chaînes de caractères ainsi. Il faut utiliser une fonction syle strcmp.(d'ailleurs tu ne lis qu'un caractère)

Code:

scanf ("%c",cont);

essaie qq chose du genre

Code:

if ((cont=='o') ||(cont=='O' ))

Code:

if (nbre_saisi>nbre_alea & nbre_saisi<50)

erreur :

Code:

if (nbre_saisi>nbre_alea && nbre_saisi<50)
Code:

1 2 if (re==1) {return main(); } else {return 0; }
Tu ne peux pas rappeler la fonction main pour réexécuter le programme. Entoure le code à répéter d'un grand
Code:

1 2 3 4 5 do { ...... ...... } while(re !=1);
Et utilise les balises code STP
25/09/2005, 17h48
gege2061

Bonjour,

Citation:

Envoyé par diogene

Tu ne peux pas rappeler la fonction main pour réexécuter le programme.

si c'est possible, après tout main n'est qu'une fonction comme une autre, elle peut faire l'objet d'une récurtion mais il faut au moins donner son prototype en entier :

Code:

int main (void)

Mais ce n'est pas chose courrante (mis à part pour rendre le code illisible) et une boucle est bien plus pratique.
25/09/2005, 17h49
Médinoc

Code:

scanf("%c", cont);

%s, non? (surtout si ensuite c'est puor une comparaison de chaîne)
25/09/2005, 17h53
Goundy

pour tes printf pourquoi tu ne fais pas tout simplement :

Code:

printf("Trop grand \n \n");

....etc ?

tiens je crois que c'est mieu

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
60
61
62
63
64
65
66
67
68
69
70
71
 
#include<stdio.h>
#include<stdlib.h>
#include<time.h>
 
 
int main(void)
{
    /* system("cls"); sert à rien , l'ecran est déja vide*/
     int re,nbre_alea,nbre_saisi,quitter;
     char cont;
 
     //variable aléatoire
     srand((int)time(NULL));        //randomize : init la fonction rand
     nbre_alea=rand()%51;               // fonction qui renvoie une variable aléatoire entre 0 et 50           
 
     printf ("Le jeux consiste a trouver le nombre mystere compris entre 0 et 50.\n\n\n");
     printf ("Veux-tu y jouer ? Ecrit O/oui  ou  N/non\n\n");
     scanf ("%c",&cont);
 
     if (cont=='o'||cont=='O')
     {
     system ("cls");
     printf("le nombre = %d",nbre_alea);
 
 
     printf("\n\n\n");
     printf("Saisir  un nombre entre 0 et 50 :  ");
     scanf("%d",&nbre_saisi);
 
  for(;;){
     while (nbre_saisi!=nbre_alea)
     {
           if (nbre_saisi>nbre_alea && nbre_saisi<50)
           {           
                       printf("\n\n");
                       printf ("Trop Grand !\n\n");
                       printf("Saisir  un nombre entre 0 et 50 :");
                       scanf("%d",&nbre_saisi);
            }
              else
            if (nbre_saisi<nbre_alea && nbre_saisi<50)
            {
                       printf("\n\n");
                       printf ("Trop Petit !\n\n");
                       printf("Saisir  un nombre entre 0 et 50 :");         
                       scanf("%d",&nbre_saisi);
            }
             else
            if (nbre_saisi >50)
            {
                       printf("\n\n");
                       printf (" Ne sois pas bete ! c'est entre 0 et 50\n");
                       scanf("%d",&nbre_saisi);
            }
      }
      system("cls");
      printf ("\n\n\n\n\n\n\n");
      printf ("Bravo !!!  Le nombre choisi est le bon\n\n");
 
      printf ("\n\n\n\n");
      printf ("Veut-tu rejouer ? 1=oui ou 0=non\n");
      scanf("%d",&re);
 
      if (re!=1)
       break; 
      }
      }
 
 
}

Pour réduire un peu la redondance du code de tous ces if à l'intérieur du while :

Code:

1
2
3
4
5
6
printf("\n\n");
if (nbre_saisi >50)   printf (" Ne sois pas bete ! c'est entre 0 et 50\n");           
  else  if (nbre_saisi>nbre_alea )  printf ("Trop Grand !\n\n"); 
          else  printf ("Trop Petit !\n\n"); 
printf("Saisir  un nombre entre 0 et 50 :");                         
scanf("%d",&nbre_saisi);

pardon mon code est mal corrigé voilà :

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
60
61
62
63
64
65
66
67
68
69
70
#include<stdio.h>
#include<stdlib.h>
#include<time.h>
 
 
int main(void)
{
    /* system("cls"); sert à rien , l'ecran est déja vide*/
     int re,nbre_alea,nbre_saisi,quitter;
     char cont;
 
 
     printf ("Le jeux consiste a trouver le nombre mystere compris entre 0 et 50.\n\n\n");
     printf ("Veux-tu y jouer ? Ecrit O/oui  ou  N/non\n\n");
     scanf ("%c",&cont);
 
     if (cont=='o'||cont=='O')
     {
           for(;;){                      
     system ("cls");
          //variable aléatoire
     srand((int)time(NULL));        //randomize : init la fonction rand
     nbre_alea=rand()%51;               // fonction qui renvoie une variable aléatoire entre 0 et 50  
     printf("le nombre = %d",nbre_alea);
 
 
     printf("\n\n\n");
     printf("Saisir  un nombre entre 0 et 50 :  ");
     scanf("%d",&nbre_saisi);
 
     while (nbre_saisi!=nbre_alea)
     {
           if (nbre_saisi>nbre_alea && nbre_saisi<50)
           {           
                       printf("\n\n");
                       printf ("Trop Grand !\n\n");
                       printf("Saisir  un nombre entre 0 et 50 :");
                       scanf("%d",&nbre_saisi);
            }
              else
            if (nbre_saisi<nbre_alea && nbre_saisi<50)
            {
                       printf("\n\n");
                       printf ("Trop Petit !\n\n");
                       printf("Saisir  un nombre entre 0 et 50 :");         
                       scanf("%d",&nbre_saisi);
            }
             else
            if (nbre_saisi >50)
            {
                       printf("\n\n");
                       printf (" Ne sois pas bete ! c'est entre 0 et 50\n");
                       scanf("%d",&nbre_saisi);
            }
      }
      system("cls");
      printf ("\n\n\n\n\n\n\n");
      printf ("Bravo !!!  Le nombre choisi est le bon\n\n");
 
      printf ("\n\n\n\n");
      printf ("Veut-tu rejouer ? 1=oui ou 0=non\n");
      scanf("%d",&re);
 
      if (re!=1)
       break; 
      }
      }
 
 
}

25/09/2005, 19h19
jejeb

Merci goundy il marche nickel à part que tu avait rajouter 2 else qu'il fallait pas.

Merci bcp

@+ jérémy
25/09/2005, 19h24
jejeb

J'ai un petit problème je suis débutant et il a un morceau que je ne comprend pas bien. Pourquoi après :

Code:

if (cont=='o'||cont=='O')

Tu as écrit :

Code:

for(;;){

et aussi pk tu as écrit :

Code:

break

pour finir . on obtient quoi avec un break

Merci d'avance

@= Jérémy
25/09/2005, 19h45
Goundy

le

Code:

if (cont=='o'||cont=='O')

veut dire :
si cont est = à o(min) ou cont est = à O (maj)
c'est pour au cas ou l'utilisateur entre un o miniscule ou majuscule.

Code:

for(;;){

boucle infenie pour pouvoir rejouer jusqu'a ce que l'utilisateur décide de quitter .

Code:

break;

pour quitter la boucle for(;;) ;)

pas la peine de répéter
Code:

1 2 3 printf("Saisir un nombre entre 0 et 50 :"); scanf("%d",&nbre_saisi);
à chaque fois, tu peux bien l'écrire une seule fois à la fin du while non :?:

25/09/2005, 21h30
Emmanuel Delahaye

Citation:

Envoyé par jejeb

Code:

for(;;){

C'est une boucle inconditionelle. Equivallent à while(1). Disons que c'est la version 'gourou!'

Citation:

Code:

break

pour finir . on obtient quoi avec un break

Ca, c'est écrit dans ton livre de C. break est une instruction qui fait quitter la boucle courante.
25/09/2005, 21h36
artatum

bonsoir,
je me permets de poster un petit quelque chose, mais j'ai vérifié d'abord :wink:

c'est juste une petite remarque pour Jejeb, sur le code comme il était ds le premier post:

ton tableau cont, tu aurais du le déclarer cont[4] pas cont[3], sinon le \n de fin de chaîne n'est pas mis en mémoire, et à l'appel du if la lecture ne s'arrête pas.

normalement avec cont[4] tu dois pouvoir utiliser oui/non
25/09/2005, 21h59
Emmanuel Delahaye

Citation:

Envoyé par artatum

ton tableau cont, tu aurais du le déclarer cont[4] pas cont[3], sinon le \n de fin de chaîne n'est pas mis en mémoire, et à l'appel du if la lecture ne s'arrête pas.

normalement avec cont[4] tu dois pouvoir utiliser oui/non

Beaucoup de confusions...

Si il faut stocker le mot "oui", il faut 3+1 char. En effet, la chaine "oui" est codée ainsi :

Code:

char s[] = {'o', 'u', 'i', 0};

Je rappelle que les chaines C sont terminées par un 0.

Maintenant si il n'y a pas la place suffisante, il y a débordement de tableau et le comportement est indéfini, et non pas "de fin de chaîne n'est pas mis en mémoire".

La fin de chaine n'est pas '\n' (qui est la fin de ligne) mais 0.

Enfin, scanf() ne met pas le '\n' dans la chaine saisie.
25/09/2005, 23h31
artatum

à emmanuel:

salut :wink:

cette erreur m'avait sauté aux yeux, mais vu mes imprécisions sur les derniers posts, j'ai préféré vérifier.

je suis au boulot, mais en rentrant chez moi demain je testerai son code avec cont[4], (et la correction du %s), normalement ça devrait marcher.

j'avais déjà écrit ce code il y a qq temps, mais j'ai la mauvaise habitude de toujours déclarer des chaînes trop grandes donc je ne m'étais pas retrouvé confronté au problème.

en fait les fins de chaînes sont terminées par '\0', donc 0 en ASCII, tu avais bien sûr raison :wink: j'ai corrigé une erreur, merci. 8)