Posez votre question Signaler

Erreur en prog C [Résolu]

KunGu 1187Messages postés 7 septembre 2007Date d'inscription 5 juin 2011Dernière intervention - Dernière réponse le 12 mars 2008 à 13:42
Bonjour,
je vien de commencer le langage C, j'ai suivi les cours du site zéro et je me retrouve avec ce code. Code:Blocks de me détecte pas d'erreur mais quand le prog arrive à la fin pour demander s'il veut faire une nouvelle partie, windows me dit qu'il doit fermer :-(.
Le voilà :
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
int main(int argc, char *argv[])
{
const long MAX = 100, MIN = 1;
long nombreEntre = 0, nombreMystere = 0, compteur = 0;
int continuerPartie = 0, erreur = 1;
srand(time(NULL));
nombreMystere = (rand() % (MAX - MIN +1)) + MIN;
do
{
do
{
printf("Quel est le nombre ? ");
scanf("%ld", &nombreEntre);
if (nombreMystere > nombreEntre) {
printf("C'est plus !\n\n");
}
else if (nombreMystere < nombreEntre) {
printf("C'est moins !\n\n");
}
else if (nombreEntre == nombreMystere) {
printf("Bravo ! Vous avez trouve le nombre mystere en %ld coups !\n\n", compteur);
}
compteur++;
}
while (nombreMystere != nombreEntre);
printf("Voulez-vous faire une nouvelle partie ? Tapez 1 pour Oui ou 0 pour Non");
scanf("%ld", continuerPartie);
do
{
if (continuerPartie != 1 || 0)
printf("Veuillez taper 1 ou 0 !");
else (erreur = 0);
}
while (erreur == 1);
}
while (continuerPartie == 1);
printf("Au revoir !");
return 0;
}
Si pouviez me dire ce qu'il ne vas pas ce serait pas mal.
Merci
Lire la suite 

Erreur en prog C »

17 réponses
Réponse
+0
moins plus
Bonjour.
Ton test if (continuerPartie != 1 || 0) ne sert à rien. En effet le || renvoie vrai si l'une des expressions est vraie. Or quand tu fait || 0, 0 est vrai donc il passe pas au bon endroit.
Il vaut mieu mettre

if (continuerPartie != 1 || continuerPartie != 0)

Voilou en espérant que cela t'aura aidé.
KunGu - 11 mars 2008 à 16:06
Je crois qu'on est pas sur la même longueur d'ondes ^^
En fait j'ai pas mis de boucle while, j'ai mis une boucle DO ... WHILE donc while est à la fin de la boucle est n'est pas fait pour prendre en compte le Au revoir !
hamzafreto - 11 mars 2008 à 19:32
apres la boucle
while(......);
c'est faut il n'y a pas le ;
le juste :
while(...)
{
}
mype - 11 mars 2008 à 20:12
toi ta pas bien compris...
je t'invite a relire le post 4 c'est une boucle do...while pas une boucle while donc il y a bien un ";"
Ajouter un commentaire
Réponse
+0
moins plus
Youps la.....la tabulation manquante et c'est le drame dans la relecture.
Ben j'ai pu d'idées la désolé.
KunGu - 11 mars 2008 à 16:12
Bon c'est pas grave on verra si quelq'un d'autre pourra m'aider ! Merci quand même !
Au fait pour la tabulation quand j'ai collé le code elle y était mais elle a disparu :-(
Ajouter un commentaire
Réponse
+0
moins plus
essaye en mettant
scanf("%d", continuerPartie); 

puisque tu as déclaré continuerPartie comme un int et pas un long

et pour que les tabulations reste il faut que tu utilise les balises a coté du souligné en haut a gauche du cadre ou tu écris et tu mets ton code entre les deux
KunGu - 11 mars 2008 à 17:02
C'est toujours la même chose windows me met que main.exe a rencontré un prob et doit fermer.


Je reposte le code avec les tabulations :

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int main(int argc, char *argv[])
{
    const long MAX = 100, MIN = 1;
    long nombreEntre = 0, nombreMystere = 0, compteur = 0;
    int continuerPartie = 0, erreur = 1;

    srand(time(NULL));
    nombreMystere = (rand() % (MAX - MIN +1)) + MIN;

    do
    {
        do
        {
        printf("Quel est le nombre ?  ");
        scanf("%ld", &nombreEntre);
            if (nombreMystere > nombreEntre) {
                printf("C'est plus !\n\n");
            }
            else if (nombreMystere < nombreEntre) {
                printf("C'est moins !\n\n");
            }
            else if (nombreEntre == nombreMystere) {
            printf("Bravo ! Vous avez trouve le nombre mystere en %ld coups !\n\n", compteur);
            }

        compteur++;
        }
        while (nombreMystere != nombreEntre);

    printf("Voulez-vous faire une nouvelle partie ? Tapez 1 pour Oui ou 0 pour Non");
    scanf("%d", continuerPartie);
        do
        {
            if (continuerPartie != 1 || continuerPartie != 0)
            printf("Veuillez taper 1 ou 0 !");
            else (erreur = 0);
        }
        while (erreur == 1);
    }
    while (continuerPartie == 1);
    printf("Au revoir !");

return 0;
}
freto - 11 mars 2008 à 17:05
dans cette partie
if (continuerPartie != 1 || continuerPartie != 0)
printf("Veuillez taper 1 ou 0 !");
else (erreur = 0);
enlève le else, la syntaxe sera correcte.
Mais si c'est windows qui te sort une erreur c'est bizarre, ce n'est pas plutôt le compilo qui hurle??
Si oui, peut tu nous poster l'erreur de compil??
KunGu - 11 mars 2008 à 17:32
Le else permet de faire stocker le nombre 0 dans la variable erreur, donc si l'utilisateur tape 1 ou 2, le else inscrira 0 dans la variable, donc le while ne fera pas de boucle donc continuera vers while (continuerPartie == 1);

Windows me dit d'envoyer un rapport d'erreur ! Quand je met "ne pas envoyer" le main.exe ne se ferme pas mais le jeu se termine.


PS: Pourrais-tu essayer de le compiler pour voir si ça fait pareil chez toi ?
Ajouter un commentaire
Réponse
+0
moins plus
je viens de voir que la syntaxe de ton scanf n'est pas correct il te manque le &
essaye avec
scanf("%d", &continuerPartie);
KunGu - 11 mars 2008 à 18:07
Et encore merci pour vos aides !
freto - 12 mars 2008 à 09:23
Mais de rien...parfois la solution est sous notre nez et on ne la trouve pas.
Bonne continuation a toi
mypeKunGu - 12 mars 2008 à 13:42
oui mais sans cette boucle si l'utilisateur ne rentre pas 1 ou 0 ton programme ne gere plus ce cas la...
tu aurait pu remplace le OU par un ET ça devrait éviter la boucle infini
Ajouter un commentaire
Ce document intitulé « Erreur en prog C » issu de CommentCaMarche (www.commentcamarche.net) est mis à disposition sous les termes de la licence Creative Commons. Vous pouvez copier, modifier des copies de cette page, dans les conditions fixées par la licence, tant que cette note apparaît clairement.
Dossier à la une
5 extensions si vous voulez revenir à l'ancien Facebook