Fonction du jeu plus ou moin

Ma fonction/présentation est-elle correcte

3 janvier 2008 à 2:48:47

Bonjour ou plutôt bonsoir a tous le monde

Voila je ne suis pas sur de moi par rapport à ma 2 eme fonction compareNombre, je voudrais avoir l'avis d'un programmeur un peu plus instruit car j'ai commencé la programmation depuis moin d'une semaine et j'aimerais avoir des conseils critique sur ma présentation ou autre.

Merci d'avance pour votre aide.

#include <stdio.h>
#include <stdlib.h>
#include <time.h>
long genereNombre(long MIN, long MAX)
{
srand(time(NULL));
return (rand() % (MAX - MIN+1)) + MIN;
}
long compareNombre (long nombreEntre, long nombreMystere, long compteur, long continuer)
{
while(continuer)
{
do
{
printf("\nEntre un nombre : ");
scanf("%ld", &nombreEntre);
compteur++;
if(nombreEntre<nombreMystere)
printf("C'est plus !\n\n");
else if (nombreEntre>nombreMystere)
printf("C'est moin !\n\n");
else
{
printf("\nBravo vous avez trouver le nombre Mystere qui etait %ld, le tout en %ld coups", nombreEntre, compteur);
printf("Vous avez maintenant le choix entre : \n\n");
printf("0 : Arreter et vous reposer les neurones\n");
printf("1 : Continuer a vous exercer, youpi !\n");
printf("ATTENTION toute erreur de frappe entraine une nouvelle partie !!\n\n");
printf("Alors que voulez vous faire ? ");
scanf("%ld", &continuer);
}
}while(nombreEntre =! nombreMystere);
}
}
int main (void)
{
long choixMode = 0, choixNiveau = 0, continuer = 1, MIN = 1, MAX = 100, compteur = 0, nombreEntre = 0, nombreMystere = 0, nombreJ1 = 0, nombreJ2 = 0;
printf("*********** Bienvenue dans le jeu du Nombre Mystere ***********\n\n");
printf("===Menu Joueur===\n\n");
printf("1 : Mode 1 joueur\n");
printf("2 : Mode 2 joueur\n\n");
printf("Votre choix ? ");
scanf("%ld", &choixMode);
if(choixMode == 1)
{
printf("\nVous avez choisis le mode 1 joueur, quel difficulter voulez vous prendre ?\n\n");
printf("1 ==> Niveau 1 : Entre 1 et 100\n");
printf("2 ==> Niveau 2 : Entre 1 et 1000\n");
printf("3 ==> Niveau 3 : Entre 1 et 10000\n");
printf("4 ==> Niveau 4 : Entre 1 et 100000\n");
printf("5 ==> Niveau 5 : Entre 1 et 1000000 pour les dur !\n\n");
printf("Que choisissais vous ? ");
scanf("%ld", &choixNiveau);
switch(choixNiveau)
{
case 1:
MAX=100; break;
case 2:
MAX=1000; break;
case 3:
MAX=10000; break;
case 4:
MAX=100000; break;
case 5:
MAX=1000000; break;
default:
printf("Veuillez choisir 1, 2, 3, 4 ou 5\n\n"); break;
}
nombreMystere = genereNombre(MIN, MAX);
compteur = 0;
compareNombre(nombreEntre, nombreMystere, compteur, continuer);
}
else
{
printf("\nVous avez choisis le mode 2 joueur, pas de niveau de difficulte a vous de jouer !\n\n");
while(continuer)
{
compteur = 0;
printf("Joueur 1 veuillez entrer un nombre attention il doit rester secret : ");
scanf("%ld", &nombreJ1);
system("cls");
printf("\nJoueur 2 c'est a vous, touver le nombre du joueur 1\n");
do
{
printf("\nEntrer un nombre : ");
scanf("%ld", &nombreJ2);
compteur++;
if(nombreJ2<nombreJ1)
printf("C'est plus !\n");
else if(nombreJ2>nombreJ1)
printf("C'est moin !\n");
else
{
printf("\nBravo vous avez trouver le nombre du joueur 1 qui etait %ld, le tout en %ld coups", nombreJ1, compteur);
printf("Vous avez maintenant le choix entre : \n\n");
printf("0 : Arreter et vous reposer les neurones\n");
printf("1 : Continuer a vous amuser, youpi !\n");
printf("ATTENTION toute erreur de frappe entraine une nouvelle partie !!\n\n");
printf("Alors que voulez vous faire ? ");
scanf("%ld", &continuer);
}
}while(nombreJ2 != nombreJ1);
}
}
return 0;
}

marc3757

3 janvier 2008 à 3:43:50

Est-ce que cela fonctionne? parce qu'en mettant srand(time(NULL)) dans ta fonction genereNombre, tu appelle srand plusieurs fois si tu fais plusieurs parties et il ne faut JAMAIS rappelé srand deux fois ou plus (mets le au début de main par exemple)

Sinon, je ne suis pas beaucoup expérimenté mais je trouve ca bien pour quelques raisons:

1. Tu ne mets pas tout dans la fonction main, ce qui démontre déja un peu plus d'organisation (bien sur tu pourrais mettre ca dans d'autres fichiers mais je ne suis pas sur que tu aie vu la programmation modulaire)

2. Ta 2e fonction est bien fait et elle prend aussi compte si l'utilisateur entre autre chose que le choix demandé

Bien sur c'est bien parti pour quelqu'un qui vient de débuter, mais continue a l'améliorer et toi-même tu vas t'améliorer!

J'espere t'avoir aidé!

OverCrash

3 janvier 2008 à 4:22:46

Oui mon programme fonctionne trés bien mais le seul point louche pour moi c'est belle et bien le sran(time(NULL)) qui lors d'une (2eme, 3 eme...) partie ne pose pas de prolème alors qu'il devrait vu qu'il est appelé plusieurs fois ici donc j'aimerais quelques explications

Merci pour tes points de vues sur mon programme mais je le perfectionne en avançant dans le cour de m@téo j'ai eu un soucis et mettant en pratique la programmation modulaire justement par rapport à ce fameux srand(time(NULL))

j'arrive au pointeur la et je crois qu'une bonne nuit de sommeil s'impose

Si quelqu'un aurait l'aimabilité de m'éclairé sur ce fameux srand(time(NULL)) enfin surtout de me dire pourquoi cela fonctionne alors que je fais apelle à lui plusieurs fois.

Bonne soirée à tous

Lumin0u

3 janvier 2008 à 11:06:18

alors si j'ai bien compris, srand fait un calcul sur un nombre (seed) avec son parametre.
puisque le parametre envoyé est ici time() (qui renvoie l'heure (différente à chaque appel)), le nombre que calcule srand est toujours différent.

enfin je me suis référé à cette page

-ed-

3 janvier 2008 à 12:42:50

Citation : OverCrash

Si quelqu'un aurait l'aimabilité de m'éclairé sur ce fameux srand(time(NULL)) enfin surtout de me dire pourquoi cela fonctionne alors que je fais apelle à lui plusieurs fois.

http://mapage.noos.fr/emdel/notes.htm#rand

Music only !