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

**jejeb** · 25/09/2005, 17h19

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 :

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
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;}
 
 
 
}

**diogene** · 25/09/2005, 17h39

Code :

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

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 :

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

scanf ("%c",cont);

essaie qq chose du genre

Code :

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

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

Code :

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

if (nbre_saisi>nbre_alea & nbre_saisi<50)

erreur :

Code :

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

if (nbre_saisi>nbre_alea && nbre_saisi<50)

Code :

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

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 :

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

1
2
3
4
5
do
{
......
......
} while(re !=1);

Et utilise les balises code STP

**gege2061** · 25/09/2005, 17h48

Bonjour,

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 :

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

int main (void)

Mais ce n'est pas chose courrante (mis à part pour rendre le code illisible) et une boucle est bien plus pratique.

**Médinoc** · 25/09/2005, 17h49

Code :

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

scanf("%c", cont);

%s, non? (surtout si ensuite c'est puor une comparaison de chaîne)

**Goundy** · 25/09/2005, 17h53

pour tes printf pourquoi tu ne fais pas tout simplement :

Code :

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

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

....etc ?

**Goundy** · 25/09/2005, 18h01

tiens je crois que c'est mieu

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
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; 
      }
      }
 
 
}

**diogene** · 25/09/2005, 18h13

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

Code :

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

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);

**Goundy** · 25/09/2005, 18h22

pardon mon code est mal corrigé voilà :

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
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; 
      }
      }
 
 
}

**jejeb** · 25/09/2005, 19h19

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

Merci bcp

@+ jérémy

**jejeb** · 25/09/2005, 19h24

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 :

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

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

Tu as écrit :

Code :

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

for(;;){

et aussi pk tu as écrit :

Code :

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

break

pour finir . on obtient quoi avec un break

Merci d'avance

@= Jérémy

**Goundy** · 25/09/2005, 19h45

le

Code :

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

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 :

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

for(;;){

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

Code :

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

break;

pour quitter la boucle for(;

**AjJi** · 25/09/2005, 20h40

pas la peine de répéter

Code :

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

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

:

**Emmanuel Delahaye** · 25/09/2005, 21h30

Envoyé par jejeb

Code :

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

for(;;){

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

Code :

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

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.

**artatum** · 25/09/2005, 21h36

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

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

**Emmanuel Delahaye** · 25/09/2005, 21h59

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 :

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

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.

**artatum** · 25/09/2005, 23h31

à emmanuel:

salut

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

j'ai corrigé une erreur, merci. 8)

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

C

Discussions similaires

Partager

Partager