C++

Bloqué sur mon programme relativement simple.

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Bloqué sur mon programme relativement simple.

aizekyel

Noob.

Bonjour,

Je me suis lancé sur la création d'un petit jeu utilisant la console pour débuter a voler de mes propres ailes en C++ (après quelques cours très scolaires et mal expliquer en ISN).
Ainsi je suis plutôt fier de mon petit programme que j'ai réalisé en peu de temps..

Mais j'ai voulus implémenter une variable de monnaie, passant par une fonction simple.

Code :

int sous(int dummy, int gain) {
return dummy + gain;
}

J'utilise cette fonction dans ma fonction de "combat" de mon mini jeu, j'ai pas mal brodé autour parce que je suis bloqué dessus depuis quelques heures :

Code :

int combat(int attqJ, int pvJ, int pvmax, int attqM, int pvM, int dummy) {
srand(time(0));
int rep, degats_sub, degats_inf, pause, gain;
cout<<"Vous avez "<<pvJ<<" PV !"<<endl;
cout<<""<<endl;
cout<<"Un monstre sauvage apparait ! Il possede " << attqM <<" d'attaque et " << pvM <<" PV !"<<endl;
cout<<""<<endl;
cout<<"Que voulez vous faire ? ATTAQUER [1] / FUIR [2]"<<endl;
cin>>rep;
if (rep == 1) {
do {
if (pvJ > 0) {
system("cls" );
degats_inf = attqJ + ( rand() %5 + 1 );
pvM = pvM - degats_inf;
cout<<"Vous attaquez le monstre, lui infligeant "<<degats_inf<<" de degats !"<<endl;
cout<<"Il lui reste "<<pvM<<" PV !"<<endl;
degats_sub = attqM + (rand() %5 + 1);
pvJ = pvJ - degats_sub;
cout<<"Il riposte en vous infligeant "<<degats_sub<<" de degats !"<<endl;
cout<<"Il vous reste "<<pvJ<<" PV !"<<endl;
cout<<""<<endl;
if (pvM <= 0) {
cout<<"Le monstre a ete vaincu !"<<endl;
cout<<""<<endl;
gain = rand() %75 + 1;
cout<<"Felicitations ! Vous remportez "<<gain<<" pieces !"<<endl ;
sous(dummy, gain);
dummy = sous(dummy, gain);
cout<<"Votre argent total est de : "<<dummy<<endl;
cout<<""<<endl;
if (pvJ < pvmax) {
cout<<"Il semblerait que vous ayez subit des degats."<<endl;
cout<<"Passez a une auberge pour vous soigner !"<<endl;
break;
}
else {
break;
}
return dummy;
}
else if (pvJ < 0) {
cout<<"Un villageois vous a retrouver evanoui..."<<endl;
cout<<"Il semblerait qu'il vous a deplacer dans une auberge..."<<endl;
break;
}
cout<<"Appuyez sur [1] pour continuer"<<endl;
cin>>pause;
pause=0;
system("cls" );
}
}while((pvJ > 0) && (pvM > 0));
}
else {
cout<<"Vous fuyez..."<<endl;
cout<<""<<endl;
cout<<"Appuyez sur [1] pour continuer"<<endl;
cin>>pause;
pause=0;
system("cls" );
return 0;
}
return 0;
}

Là mon programme m'affiche bien la valeur de la fonction "sous" (qui a un nom très original d'ailleurs, il faut le dire !).

Mais c'est dans mon main que tout cela me casse les bonbons.

Exemple d'un cas 1 pour un combat :

Code :

[Pleins de définition de variables pas ici...]
int coins = sous(0,0);
[Là y'a tout mon code pour les menus etc...]
case 1: {
system("cls" );
cout<<"Vous entrez dans les bois !"<<endl;
cout<<""<<endl;
int monster[] = {rand()%10 + 1, rand()%50 + 25};
combat(stats[0], stats[1], stats[2], monster[0], monster[1], coins);
cout<<"Tapez [1] pour continuer !"<<endl;
cin>>pause;
pause = 0;
menu = 0;
break;
}

Et alors mon problème c'est que j'aimerais que "coins" soit à la valeur de ma fonction sous, mais j'arrive pas a récuperer une bonne valeur.

Pourtant dans la fonction combat j'ai bien dummy = 86 (par exemple) j'arrive bien a récuperer la valeur... mais alors dans le main y'a pas moyen.

Si quelqu'un pourrait m'expliquer comment faire, pas juste me donner une réponse s'il vous plaît.

Merci bien !

Cordialement,

Aizekyel.

PS : Je conçois que ça puisse paraître très simple pour certains mais je suis a mes débuts en C++.

Publicité

rufo

Pas me confondre avec Lycos!

Ligne 29 :pt1cable:
Tu ne stockes pas la valeur retournée dans une variable.

Au passage, si c'est pour faire ce genre de structure de programme, laisse tomber le C++ et fait juste du C.

Par ailleurs mettre des break et des return en plein milieu comme tu le fais, c'est un coup à avoir pleins de bugs. Il faut mettre en place des structures de contrôles propres dans tes boucles; en gros, que les critères d'arrêts soient clairs quitte à utiliser des booléens dont la valeur changera en fonction de tests réalisés dans le corps de la boucle. Mais au moins, qu'on ait tous les critères d'arrêt au même endroit.

Ex :
if (pvJ < pvmax) {
aSubiDegats = TRUE; // TRUE = 1
}

else if (pvJ < 0) {
estEvanui = TRUE;
}

Et dans ton while, tu mettras !aSubiDegats et !estEvanui.

Edit : j'avais pas vu la ligne 30. Mais du coup, pourquoi conserver la ligne 29 :??: Du coup, je pense que ça vient de tous des critères de sortie de boucle ou de return multiples. Tu dois probablement pas passer là où tu penses.
Rappel : une fonction, un seul return. Et le break, c'est que pour le switch()

Message édité par rufo le 08-02-2016 à 13:22:25

---------------
Astres, outil de help-desk GPL : http://sourceforge.net/projects/astres, ICARE, gestion de conf : http://sourceforge.net/projects/icare, Outil Planeta Calandreta : https://framalibre.org/content/planeta-calandreta

aizekyel

Noob.

Bonjour,

Merci à toi pour ta réponse..

Pour les break ,a vrai dire notre prof nous avait apprit ça comme ça et après m'être pas mal documenté et ainsi qu'avec ton aide, ça paraît beaucoup plus logique.

Je vais refaire ça de mon coté et si jamais j'ai encore un soucis je reviendrais vers vous, mais un grand merci à toi.

rufo

Pas me confondre avec Lycos!

"a vrai dire notre prof nous avait apprit ça comme ça" -> c'est une honte d'enseigner des structures comme ça :fou:

aizekyel

Noob.

Bah le prof est pas prof de programmation et tout ce qu'il fait en cours c'est distribuer un paquet de feuille, le lire, et parfois il découvre des trucs lui même.

TotalRecall

Dans ce cas c'est une honte qu'il soit prof tout court :whistle:

Il y a vraiment de meilleures façons de structurer un algo et la logique générale d'un programme, celle là est ce qui mène le plus vite à la confusion et à des bugs vicieux, et je ne parle même pas de la maintenabilité...

Message édité par TotalRecall le 08-02-2016 à 16:46:49

---------------
Topic .Net - C# @ Prog

FORUM HardWare.fr

Programmation

C++

Bloqué sur mon programme relativement simple.

Sujets relatifs
Problème avec mon programme	Programme que je n'arrive pas a finir
Retourner une variable à un programme VBA appelant	Ecrire totalité des resultats d'un programme dans un fichier en C
Découpe d'un programme c++	Besoin d'aide pour programme sous Java's Cool !
[Divers]Création d'un programme pour tester connaissances linguistique	Comment créer un site web simple ?
[PowerShell] Lister rep. dont l'héritage des Permis. NTFS est bloqué	Appel concurent de procedure stocké oracle dans mon programme java
Plus de sujets relatifs à : Bloqué sur mon programme relativement simple.

Page générée en 0.088 secondes