Petits calculs pour les crédits, ne fonctionne pas [noob]

Marsh Posté le 15-05-2016 à 16:30:19

Salut à tous

Je débute avec le C++, je suis le cours d'OpenClassrooms.

Je veux faire un tout petit programme pour calculer le montant des mensualités et le cout d'un crédit, en fonction d'un capital a emprunter, d'un TEG et d'un nombre de mensualités.

Déjà, les 2 premières lignes apparaissent collées dans la console, malgré mon endl... Why ?!!

Ensuite et bien le calcul ne fonctionne pas...

Voici mon code :

Code :

#include <iostream>
//#include <string>
#include <cmath>
using namespace std;
// programme pour calculer les mensualités et le cout des crédits
int main()
{
int capitalAEmprunter, nombreMensualites;
double teg, montantMensualites, coutDuCredit;
cout << endl << "Petits calculs sur le credit";
cout << endl << "Capital a emprunter ? ";
cin >> capitalAEmprunter;
cout << endl << "Nombre de mensualites ? ";
cin >> nombreMensualites;
cout << endl << "TEG ? ";
cin >> teg;
montantMensualites=(capitalAEmprunter*teg/12)/(1-(pow(1+teg/12,-nombreMensualites)));
cout << endl << "Montant mensualites : " << montantMensualites;
coutDuCredit = montantMensualites*nombreMensualites-capitalAEmprunter;
cout << " Cout du credit : " << coutDuCredit << endl;
return 0;
}

Voici ce que j'ai dans la console :

Voici la formule mathématique que j'ai trouvé sur 2 sites :

Merci pour votre aide !

Message édité par Lt Ripley le 29-11-2016 à 18:38:54

Reply

Marsh Posté le 15-05-2016 à 16:30:19

Reply

Marsh Posté le 15-05-2016 à 16:47:37

Citation :

Déjà, les 2 premières lignes apparaissent collées dans la console, malgré mon endl... Why ?!!

Un endl place le cursor dans la ligne suivante - si tu veux une ligne vide faut donc mettre deux endl. Personnellement je les mettrais d'ailleurs plutôt à la fin du cout précédant (mais c'est une question de goût). Le cin rajoute automatiquement un saut de ligne.

Pour le calcul, le TEG est indiqué en % mais j'ai l'impression qu'il doit être sous forme décimale - soit 0,04 et non 4. Si je rajoute un teg=teg/100; dans le code le résultat à l'air bien plus plausible.

edit: Calcul confirmé par un autre site (sauf que ce dernier arrondit la mensualité). Pour le TEG il y a qu'a regarder l'exemple...

Message édité par rat de combat le 15-05-2016 à 16:50:08

Reply

Marsh Posté le 15-05-2016 à 17:01:07

Ahh oui ça fonctionne en décimal

Merci rat de combat !

Reply

Marsh Posté le 15-05-2016 à 17:14:51

Pour la mise en forme de la sortie de ton programme:

Code :

#include <iostream>
#include <iomanip>
#include <cmath>
// programme pour calculer les mensualités et le cout des crédits
int main() {
int capitalAEmprunter, nombreMensualites;
double teg, montantMensualites, coutDuCredit;
std::cout << "\nPetits calculs sur le credit" << std::endl;
std::cout << "\nCapital a emprunter ? " << std::flush;
std::cin >> capitalAEmprunter;
std::cout << "\nNombre de mensualites ? " << std::flush;
std::cin >> nombreMensualites;
std::cout << "\nTEG ? " << std::flush;
std::cin >> teg;
montantMensualites = (capitalAEmprunter * teg / 12) / (1 - (std::pow(1 + teg / 12, -nombreMensualites)));
coutDuCredit = montantMensualites * nombreMensualites - capitalAEmprunter;
std::cout << std::fixed << std::setprecision(2);
std::cout << "\nMontant mensualites : " << montantMensualites;
std::cout << "\tCout du credit : " << coutDuCredit << std::endl;
return 0;
}

Ce qui donne en sortie:

Petits calculs sur le credit

Capital a emprunter ? 200000

Nombre de mensualites ? 240

TEG ? 4

Montant mensualites : 66666.67 Cout du credit : 15800000.00

A+,

Message édité par gilou le 15-05-2016 à 17:16:50

---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --

Reply

Marsh Posté le 15-05-2016 à 20:16:44

Merci gilou !

Je vais étudier tout ça

Reply

Marsh Posté le 15-05-2016 à 20:32:05

Ahhh... il faut enlever using namespace std et puis du coup utiliser std:: devant les cout et les cin ?

Et puis mettre \n devant le texte ? Et pourquoi \t pour le dernier ?

Il y a une raison à tout cela, laquelle ?

Merci encore

Reply

Marsh Posté le 16-05-2016 à 01:02:57

> Ahhh... il faut enlever using namespace std et puis du coup utiliser std:: devant les cout et les cin ?
Non, mais on n'est pas obligé, et quand on apprend le C++, ça permet de savoir quoi est dans quel namespace.

>Et puis mettre \n devant le texte
Ça te fait un saut de ligne avant ton texte, c'est plus lisible que de coller de multiples endl non finaux

> Et pourquoi \t pour le dernier ?
Pour avoir une tabulation.

A+,

---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --

Reply

Marsh Posté le 18-05-2016 à 20:14:25

Merci gilou pour ces nouvelles infos, j'utilise maintenant les trois : \n , \t et endl

Je suis toujours le cours de OpenClassrooms, j'avance, ça me plait bien...

Reply

Marsh Posté le 19-05-2016 à 12:06:14

Donc j'en déduis que tu avais compris cette ligne
std::cout << std::fixed << std::setprecision(2);
Qui me semblait plus adaptée pour afficher tes derniers résultats.

A+,

---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --

Reply

Marsh Posté le 19-05-2016 à 21:17:35

Punaise non, elle est passée à travers celle là, et flush aussi, en fait j'ai trouvé le tout compliqué et du coup je suis passé à coté...

Flush c'est pour s'assurer de vider le buffer et de bien ecrire ce qu'on envoie, c'est ça ? il n'y en a besoin que quand tu fais autre chose qu'un cout après ? Pas besoin non plus en cas de endl on dirait...

:pt1cable:

Je suis étonné qu'on doivent écrire 25 fois std::

setprecision(2) c'est donc pour avoir seulement 2 chiffres après la virgule, mais le reste de la ligne je veux bien que tu m'expliques !

Reply

Marsh Posté le 19-05-2016 à 21:17:35

Reply

Marsh Posté le 20-05-2016 à 02:19:26

http://www.cplusplus.com/reference/ios/fixed/

> Flush c'est pour s'assurer de vider le buffer et de bien ecrire ce qu'on envoie, c'est ça ?
C'est utile pour écrire sans retour a la ligne. std::endl, grosso modo ca fait 1) un flush et 2) ça positionne le curseur en début de ligne suivante.

A+,

Message édité par gilou le 20-05-2016 à 02:23:57

---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --

Reply

Marsh Posté le 20-05-2016 à 12:55:15

ok je comprends mieux, merci

D'après ce que je vois std::ios c'est le "parent" de cin, cout et des ~~trucs~~ commandes de fstream, c'est ça ?

Message édité par Lt Ripley le 20-05-2016 à 13:10:16

Reply

Marsh Posté le 20-05-2016 à 13:37:53

Tu as une hiérarchie de classes illustrée par le diagramme suivant:

et cin, cout ne sont pas des classes, mais des objets (ie des instances de classes).

A+,

---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --

Reply

Marsh Posté le 20-05-2016 à 17:28:08

Oui je l'ai vu ce diagramme grâce à ton précédent lien.

Je n'imaginais pas être autant au début du début [:luxy:2]

Je ne risque pas de m'ennuyer !

Un grand merci pour toutes ces infos précieuses.

Ca tombe bien, je vois que les classes sont au programme du cours d'OpenClassroom que je suis (2 cours plus après que j'aie fini mon exo, sur lequel j'ai déjà passé pas loin de 8 ou 10 heures)

Ca me rassure [:kakashii]

Reply

Marsh Posté le 21-05-2016 à 11:59:19

Comment fait-on pour ajouter une tabulation à un groupe de lignes (un paragraphe quoi) d'un coup ? Quand j'inclus du code dans une boucle je tabule chaque ligne, y a t-il une fonction dans code::blocks pour cela ?

Reply

Marsh Posté le 21-05-2016 à 14:49:48

Pour tabuler plusieurs lignes il suffit de les sélectionner et d'appuyer la touche tab.

Reply

Marsh Posté le 21-05-2016 à 16:32:54

[:cerveau arf]

Merci

Reply

Marsh Posté le 21-05-2016 à 19:41:30

Bon, alors maintenant, les problèmes avec ton programme.
Il suppose que l'input utilisateur sera parfait, ce qui dans la pratique s'apparente à croire au père noël.

Il faut envisager le cas
1) d'une entrée non numérique quand un nombre est attendu (tapes "aaaa" en réponse à la première question de ton programme, pour voir)
2) de plusieurs entrées numériques quand une seule est attendue (tapes "200000 24 0.04" en réponse à la première question de ton programme, pour voir)

D’où une version améliorée:

Code :

#include <iostream>
#include <ios>
#include <istream>
#include <limits>
#include <iomanip>
#include <cmath>
template <typename CharT>
void ignore_line (std::basic_istream<CharT>& in) {
in.ignore(std::numeric_limits<std::streamsize>::max(), in.widen('\n'));
}
// programme pour calculer les mensualités et le cout des crédits
int main() {
int capitalAEmprunter, nombreMensualites;
double teg, montantMensualites, coutDuCredit;
std::cout << "\nPetits calculs sur le credit" << std::endl;
std::cout << "\nCapital a emprunter ? " << std::flush;
std::cin >> capitalAEmprunter;
if (!std::cin) {
std::cout << "\nErreur: Entree incorrecte!" << std::endl;
return(-1);
}
ignore_line(std::cin);
std::cout << "\nNombre de mensualites ? " << std::flush;
std::cin >> nombreMensualites;
if (!std::cin) {
std::cout << "\nErreur: Entree incorrecte!" << std::endl;
return(-1);
}
ignore_line(std::cin);
std::cout << "\nTEG ? " << std::flush;
std::cin >> teg;
if (!std::cin) {
std::cout << "\nErreur: Entree incorrecte!" << std::endl;
return(-1);
}
ignore_line(std::cin);
montantMensualites = (capitalAEmprunter * teg / 12) / (1 - (std::pow(1 + teg / 12, -nombreMensualites)));
coutDuCredit = montantMensualites * nombreMensualites - capitalAEmprunter;
std::cout << std::fixed << std::setprecision(2);
std::cout << "\nMontant mensualites : " << montantMensualites;
std::cout << "\tCout du credit : " << coutDuCredit << std::endl;
return 0;
}

Explication:
std::cin >> capitalAEmprunter;
// Si !std::cin l'input n'a pu être parsé comme une variable du type attendu
if (!std::cin) {
std::cout << "\nErreur: Entree incorrecte!" << std::endl;
return(-1);
}
// Sinon, s'il reste des trucs de tapés pas traités, on les lits et on n'en fait rien
ignore_line(std::cin);

Dans l'absolu, pour blinder complètement ton programme, il faudrait aussi vérifier que tes calculs numériques ne provoquent pas de débordements.

A+,

Message édité par gilou le 21-05-2016 à 20:00:19

---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --

Reply

Marsh Posté le 23-05-2016 à 13:51:56

Merci gilou

J'ai modifié mon programme en ajoutant tes améliorations, qui je le comprends bien, sont indispensables.

J'en comprends une partie mais pas tout, mais petit à petit ça viendra, et puis ça me permet de l'utiliser partout.

J'ai regardé un peu du coté des débordements, dits over/underflow si j'ai bon. Il faut que j'approfondisse... ça sert à ne pas atteindre les maximums d'origine des int, double etc du C++ c'est ça ? Et donc à ne pas être tronqué, ou autre ?

J'ai trouvé cette page qui en parle bien il me semble : http://www.developpez.net/forums/d [...] underflow/

Que peux tu me dire à ce sujet ?

Reply

Marsh Posté le 23-05-2016 à 15:37:34

Que malheureusement, il n'y a rien d'automatique.
Il y a une avancée pour les calculs en flottants avec <cfenv> mais ce n'est qu'une couche d'interface avec les flags du FPU, donc sa qualité dépend de celle du hardware.
cf http://stackoverflow.com/questions [...] -underflow

Ce pourquoi j'ai dit "dans l'absolu", parce que bien souvent, ce n'est pas blindé à ce niveau (ce serait un programme d'ingénierie, ou la fiabilité des calculs est primordiale, ce serait autre chose, mais pour un simple exercice en console...)
Je voulais juste te faire prendre conscience du problème potentiel.

A+,

---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --

Reply

Marsh Posté le 23-05-2016 à 20:20:26

Bon, il m'a fallu installer une nouvelle version de g++ (5.1) pour que cfenv soit activé, mais on peut aboutir à ceci:

Code :

#include <iostream>
#include <ios>
#include <istream>
#include <limits>
#include <iomanip>
#include <cfenv>
#include <cmath>
template <typename CharT>
void ignore_line (std::basic_istream<CharT>& in) {
in.ignore(std::numeric_limits<std::streamsize>::max(), in.widen('\n'));
}
// programme pour calculer les mensualités et le cout des crédits
int main() {
int capitalAEmprunter, nombreMensualites;
double teg, montantMensualites, coutDuCredit;
std::cout << "\nPetits calculs sur le credit" << std::endl;
std::cout << "\nCapital a emprunter ? " << std::flush;
std::cin >> capitalAEmprunter;
if (!std::cin) {
std::cerr << "\nErreur: Entree incorrecte!" << std::endl;
return(-1);
}
ignore_line(std::cin);
std::cout << "\nNombre de mensualites ? " << std::flush;
std::cin >> nombreMensualites;
if (!std::cin) {
std::cerr << "\nErreur: Entree incorrecte!" << std::endl;
return(-1);
}
ignore_line(std::cin);
std::cout << "\nTEG ? " << std::flush;
std::cin >> teg;
if (!std::cin) {
std::cerr << "\nErreur: Entree incorrecte!" << std::endl;
return(-1);
}
ignore_line(std::cin);
std::feclearexcept(FE_ALL_EXCEPT);
montantMensualites = (capitalAEmprunter * teg / 12) / (1 - (std::pow(1 + teg / 12, -nombreMensualites)));
coutDuCredit = montantMensualites * nombreMensualites - capitalAEmprunter;
if (std::fetestexcept(FE_OVERFLOW|FE_UNDERFLOW|FE_INVALID|FE_DIVBYZERO)) {
std::cerr << "\nUne erreur de calcul est survenue!" << std::endl;
return(-1);
}
std::cout << std::fixed << std::setprecision(2);
std::cout << "\nMontant mensualites : " << montantMensualites;
std::cout << "\tCout du credit : " << coutDuCredit << std::endl;
return 0;
}

(si tu entres un nombre de mensualités égal à 0, tu auras une division par 0 dans ton calcul de montantMensualites, si tu veux tester)
Note aussi que j'ai fait une correction par rapport à mon code précédent: les messages d'erreur avant interruption sont envoyés sur cerr et non sur cout, c'est plus propre conceptuellement.

Mais avant cela, il y a des choses de bon sens à faire:
vérifier que capitalAEmprunter > 0, nombreMensualites > 1 et 0 <= teg

Et éventuellement faire un message d'alerte si teg > 0.2005 (20.05% est le taux d'usure le plus élevé, et en fait, selon la catégorie du crédit, il y a un teg légal max, égal au taux d'usure http://www.cbanque.com/credit/taux-usure.php )

Tu peux aussi vouloir que le teg soit saisi en % et faire directement la conversion dans ton programme.

Code :

#include <iostream>
#include <ios>
#include <istream>
#include <limits>
#include <iomanip>
#include <cfenv>
#include <cmath>
template <typename CharT>
void ignore_line (std::basic_istream<CharT>& in) {
in.ignore(std::numeric_limits<std::streamsize>::max(), in.widen('\n'));
}
// programme pour calculer les mensualités et le cout des crédits
int main() {
int capitalAEmprunter, nombreMensualites;
double teg, montantMensualites, coutDuCredit;
std::cout << "\nPetits calculs sur le credit" << std::endl;
std::cout << "\nCapital a emprunter ? " << std::flush;
std::cin >> capitalAEmprunter;
if (!std::cin) {
std::cerr << "\nErreur: Entree incorrecte!" << std::endl;
return(-1);
}
if (capitalAEmprunter < 0) {
std::cerr << "\nMontant invalide!" << std::endl;
return(-1);
}
ignore_line(std::cin);
std::cout << "\nNombre de mensualites ? " << std::flush;
std::cin >> nombreMensualites;
if (!std::cin) {
std::cerr << "\nErreur: Entree incorrecte!" << std::endl;
return(-1);
}
if (nombreMensualites < 1) {
std::cerr << "\nNombre de mensualites invalide!" << std::endl;
return(-1);
}
ignore_line(std::cin);
std::cout << "\nTEG (%) ? " << std::flush;
std::cin >> teg;
if (!std::cin) {
std::cerr << "\nErreur: Entree incorrecte!" << std::endl;
return(-1);
}
if (teg < 0) {
std::cerr << "\nteg invalide!" << std::endl;
return(-1);
}
teg /= 100;
const double MAX_TEG = 0.2005;
if (teg > MAX_TEG) {
std::cout << "\nAttention, le teg semble trop important!" << std::endl;
}
ignore_line(std::cin);
std::feclearexcept(FE_ALL_EXCEPT);
montantMensualites = (capitalAEmprunter * teg / 12) / (1 - (std::pow(1 + teg / 12, -nombreMensualites)));
coutDuCredit = montantMensualites * nombreMensualites - capitalAEmprunter;
if (std::fetestexcept(FE_OVERFLOW|FE_UNDERFLOW|FE_INVALID|FE_DIVBYZERO)) {
std::cerr << "\nUne erreur de calcul est survenue!" << std::endl;
return(-1);
}
std::cout << std::fixed << std::setprecision(2);
std::cout << "\nMontant mensualites : " << montantMensualites;
std::cout << "\tCout du credit : " << coutDuCredit << std::endl;
return 0;
}

Petits calculs sur le credit

Capital a emprunter ? 200000

Nombre de mensualites ? 24

TEG (%) ? 4

Montant mensualites : 8684.98 Cout du credit : 8439.63

A+,

Message cité 1 fois
Message édité par gilou le 23-05-2016 à 21:35:41

---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --

Reply

Marsh Posté le 24-05-2016 à 07:49:31

gilou a écrit :

Petits calculs sur le credit

Capital a emprunter ? 200000

Nombre de mensualites ? 24

TEG (%) ? 4

Montant mensualites : 8684.98 Cout du credit : 8439.63

A+,

Rembourser 200000 € en 24 mensualités, c'est très HFR-compliant [:clooney3]

Sinon, pas d'avis sur la partie C++ des posts C++, je passais juste comme ça (et je n'ai pas encore eu l'occasion de me mettre à C++1x [:lergo:3] )

---------------
On n'est jamais très fort pour ce calcul !

Reply

Marsh Posté le 24-05-2016 à 17:31:13

> et je n'ai pas encore eu l'occasion de me mettre à C++1x

Faut! C++11 (voire 14 pour le move) c'est vraiment bien foutu.
Même si pour les manipulations de strings, un bon vieux char * à la C est toujours bien plus rapide.

A+,

---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --

Reply

Marsh Posté le 24-05-2016 à 18:17:40

Bon, je crois que je suis passé à un g++ 5.1

Je l'ai donc pris ici (on demand installer) : http://tdm-gcc.tdragon.net/
Suivant un petit tuto sur OpenClassrooms

A debut ça ne fonctionnait pas (j'avais oublié le debugger). Maintenant tout fonctionne comme avant, après installation de tdm-gcc j'ai en fait juste changé les path du compiler et du debugger...

Mes petits programmes fonctionnent, mais ton nouveau code ne passe pas mieux qu'avec mon ancien g++. C'est le même type d'erreur (pour ne pas dire "exactement les même erreurs" )

Voici ce que j'ai dans la zone de notification :

Citation :

||=== Build: Debug in creditgilou (compiler: GNU GCC Compiler) ===|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\c++0x_warning.h|32|error: #error This file requires compiler and library support for the ISO C++ 2011 standard. This support is currently experimental, and must be enabled with the -std=c++11 or -std=gnu++11 compiler options.|
F:\C++\creditgilou\main.cpp||In function 'int main()':|
F:\C++\creditgilou\main.cpp|55|error: 'feclearexcept' is not a member of 'std'|
F:\C++\creditgilou\main.cpp|55|error: 'FE_ALL_EXCEPT' was not declared in this scope|
F:\C++\creditgilou\main.cpp|58|error: 'fetestexcept' is not a member of 'std'|
F:\C++\creditgilou\main.cpp|58|error: 'FE_OVERFLOW' was not declared in this scope|
F:\C++\creditgilou\main.cpp|58|error: 'FE_UNDERFLOW' was not declared in this scope|
F:\C++\creditgilou\main.cpp|58|error: 'FE_INVALID' was not declared in this scope|
F:\C++\creditgilou\main.cpp|58|error: 'FE_DIVBYZERO' was not declared in this scope|
||=== Build failed: 8 error(s), 0 warning(s) (0 minute(s), 0 second(s)) ===|

Chose nouvelle pour moi, cette fois ci un fichier c++0x_warning.h s'ouvre dans une fenêtre :

Citation :

#ifndef _CXX0X_WARNING_H
#define _CXX0X_WARNING_H 1

#if __cplusplus < 201103L
#error This file requires compiler and library support for the \
ISO C++ 2011 standard. This support is currently experimental, and must be \
enabled with the -std=c++11 or -std=gnu++11 compiler options.
#endif

#endif

J'ai aussi l'exercice corrigé du prof de mon exo de jeu de mot à lettres mélangées qui ne fonctionne toujours pas.

Message cité 1 fois
Message édité par Lt Ripley le 24-05-2016 à 18:19:48

Reply

Marsh Posté le 24-05-2016 à 18:37:02

Citation :

and must be enabled with the -std=c++11 or -std=gnu++11 compiler options.

Faut rajouter ça dans les paramètres passés au compilateur. Tu utilises quoi pour compiler? (Code::Blocks, Dev-C++, Eclipse, ligne de commande, ...)

Reply

Marsh Posté le 24-05-2016 à 19:29:21

J'allais le dire.
A priori, compiler avec -std=c++11 et aussi avec -Wall si possible

A+,

---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --

Reply

Marsh Posté le 24-05-2016 à 19:32:04

Lt Ripley a écrit :

Bon, je crois que je suis passé à un g++ 5.1

Je l'ai donc pris ici (on demand installer) : http://tdm-gcc.tdragon.net/

C'est ce que j'ai fait hier

Ensuite, en ligne de commande:
g++ -std=c++11 -Wall -o creditgilou.exe creditgilou.cpp
(perso, j'évite de nommer un fichier main.cpp quand il n'y en a qu'un)

A+,

---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --

Reply

Marsh Posté le 24-05-2016 à 22:29:27

Merci les gars

Ça marche. J'ai ajouté les paramètres ici (j'ai ajouté -Wall aussi entre temps)

J'ai bon ?

J'utilise code::blocks 16.01

Bon l'exo corrigé du prof ne passe pas mais c'est pas la fin du monde

Reply

Marsh Posté le 24-05-2016 à 23:02:35

Ca peut fonctionner mais je regarderais plutôt sous Compiler settings -> Compiler Flags, tu dois pouvoir activer les deux par là.

Si l'exo du prof ne passera toujours pas tu peux toujours le poster (avec les messages d'erreurs).

Reply

Marsh Posté le 25-05-2016 à 09:01:19

Merci rat de combat j'ai pu faire comme tu dis

Voici le programme du prof

Code :

// Main.cpp : Defines the entry point for the console application.
//
#include "stdafx.h"
#include "MotMysterePlus.h"
int _tmain(int argc, _TCHAR* argv[])
{
bool play(true);
srand(time(0));
vector<string> dicoList(initDico());
if(dicoList.size()==0)
{
return 1;
}
cout << "**** BIENVENUE au jeu du Mot Mystere ****" <<endl;
do
{
bool wrongValue(true);
char answer('a');
//0- Proposer un mode Solo ou Duo
bool solo(false);
cout << "Voulez-vous jouer en Solo ou en Duo ? (s/d)"<<endl;
while(wrongValue)
{
cin.get(answer);
switch(answer)
{
case 's':
wrongValue=false;
solo=true;
break;
case 'd':
wrongValue=false;
break;
default:
cout << "'" <<answer<<"' n'est pas une valeur correct : (s/d)"<<endl;
}
cin.ignore(numeric_limits<streamsize>::max(), '\n'); // skip bad input
}
answer='a';
wrongValue=true;
//1- récupérer le mot à trouver
string soluce(getFinalWord(solo, dicoList));
if(soluce.compare("" )==0)
{
return 1;
}
//2- générer le mot Mystère
string mysteryWord(getMysteryWord(soluce));
//3- Trouver le mot Mystère
resolveMysteryWord(mysteryWord, soluce,solo);
cout<<"voulez-vous faire une autre partie ? (o/n)"<<endl;
while(wrongValue)
{
cin.ignore(numeric_limits<streamsize>::max(), '\n'); // skip bad input
cin.get(answer);
switch(answer)
{
case 'o':
wrongValue=false;
cin.ignore(numeric_limits<streamsize>::max(), '\n');
break;
case 'n':
wrongValue=false;
cin.ignore(numeric_limits<streamsize>::max(), '\n');
play=false;
break;
default:
cout << "'" <<answer<<"' n'est pas une valeur correct : (o/n)"<<endl;
}
}
}while(play);
return 0;
}

Code :

// stdafx.h : include file for standard system include files,
// or project specific include files that are used frequently, but
// are changed infrequently
//
#pragma once
#include <stdio.h>
#include <tchar.h>
// TODO: reference additional headers your program requires here

Code :

// stdafx.cpp : source file that includes just the standard includes
// MotMysterePlus.pch will be the pre-compiled header
// stdafx.obj will contain the pre-compiled type information
#include "stdafx.h"
// TODO: reference any additional headers you need in STDAFX.H
// and not in this file

(Celui ci se nomme MotMysterePlus.h)

Code :

// MotMysterePlus.cpp : Defines functions prototype.
//
#include "stdafx.h"
#include <iostream>
#include <string>
#include <ctime>
#include <cstdlib>
#include <fstream>
#include <vector>
using namespace std;
string getFinalWord(bool solo, vector<string> wordList);
string getMysteryWord(string word);
void resolveMysteryWord(string const mixWord, string const finalWord,bool const solo);
vector<string> initDico();

Code :

// MotMysterePlus.cpp : Defines functions source code.
//
#include "stdafx.h"
#include "MotMysterePlus.h"
using namespace std;
string getFinalWord(bool const solo, vector<string> wordList)
{
string word("" );
if(solo)
{
int randLevel (rand() % wordList.size());
word = wordList[randLevel];
}
else
{
cout << "JOUEUR 1 ! Saisissez un mot :" << endl;
cin >> word ;
}
return word;
}
string getMysteryWord(string word)
{
string newWord("" );
while(word.compare("" )!=0)
{
int index = rand() % word.size();
newWord += word[index];
word.erase(index,1);
}
return newWord;
}
void resolveMysteryWord(string const mixWord, string const finalWord, bool const solo)
{
bool finish(false);
int count(5);
string temp("" );
string joueur("JOUEUR 2 : " );
if(solo)
{
joueur="";
}
while(!finish)
{
cout <<joueur<< "Quel est ce mot ? "<< mixWord << endl;
cout << "Il vous reste " <<count<<" coup(s)."<<endl;
cin >> temp;
if(temp.compare(finalWord)==0)
{
cout << "BRAVO !" <<endl;
cout << "Vous avez trouve en "<<5-count+1<<" essai(s)"<<endl;
finish=true;
}
else
{
--count;
if(count!=0)
{
cout << "ce n'est pas le bon mot ! "<<endl;
}
else
{
cout<<joueur<< "PERDU ! le mot mystere etait : "<<finalWord<<endl;
finish=true;
}
}
}
}
vector<string> initDico()
{
cout << "veuillez patienter, nous initialisons le jeu ..." <<endl;;
vector<string> wordList;
string word("" );
ifstream dico("dico.txt" );
if(dico)
{
string temp;
while(getline(dico, temp)&&(temp.compare("" )!=0))
{
wordList.push_back(temp);
}
int randLevel (rand() % wordList.size());
word = wordList[randLevel];
}
else
{
cout << "ERREUR : impossible d'ouvrir le fichier en lecture." <<endl;
}
return wordList;
}

Et les erreurs :

Citation :

||=== Build: Debug in corri mot mystere (compiler: GNU GCC Compiler) ===|
F:\C++\corri mot mystere\main.cpp||In function 'int main(int, _TCHAR**)':|
F:\C++\corri mot mystere\main.cpp|40|error: 'numeric_limits' was not declared in this scope|
F:\C++\corri mot mystere\main.cpp|40|error: expected primary-expression before '>' token|
F:\C++\corri mot mystere\main.cpp|40|error: no matching function for call to 'max()'|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\stl_algobase.h|219|note: candidate: template<class _Tp> const _Tp& std::max(const _Tp&, const _Tp& )|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\stl_algobase.h|219|note: template argument deduction/substitution failed:|
F:\C++\corri mot mystere\main.cpp|40|note: candidate expects 2 arguments, 0 provided|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\stl_algobase.h|265|note: candidate: template<class _Tp, class _Compare> const _Tp& std::max(const _Tp&, const _Tp&, _Compare)|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\stl_algobase.h|265|note: template argument deduction/substitution failed:|
F:\C++\corri mot mystere\main.cpp|40|note: candidate expects 3 arguments, 0 provided|
F:\C++\corri mot mystere\main.cpp|57|error: 'numeric_limits' was not declared in this scope|
F:\C++\corri mot mystere\main.cpp|57|error: expected primary-expression before '>' token|
F:\C++\corri mot mystere\main.cpp|57|error: no matching function for call to 'max()'|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\stl_algobase.h|219|note: candidate: template<class _Tp> const _Tp& std::max(const _Tp&, const _Tp& )|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\stl_algobase.h|219|note: template argument deduction/substitution failed:|
F:\C++\corri mot mystere\main.cpp|57|note: candidate expects 2 arguments, 0 provided|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\stl_algobase.h|265|note: candidate: template<class _Tp, class _Compare> const _Tp& std::max(const _Tp&, const _Tp&, _Compare)|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\stl_algobase.h|265|note: template argument deduction/substitution failed:|
F:\C++\corri mot mystere\main.cpp|57|note: candidate expects 3 arguments, 0 provided|
F:\C++\corri mot mystere\main.cpp|63|error: expected primary-expression before '>' token|
F:\C++\corri mot mystere\main.cpp|63|error: no matching function for call to 'max()'|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\stl_algobase.h|219|note: candidate: template<class _Tp> const _Tp& std::max(const _Tp&, const _Tp& )|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\stl_algobase.h|219|note: template argument deduction/substitution failed:|
F:\C++\corri mot mystere\main.cpp|63|note: candidate expects 2 arguments, 0 provided|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\stl_algobase.h|265|note: candidate: template<class _Tp, class _Compare> const _Tp& std::max(const _Tp&, const _Tp&, _Compare)|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\stl_algobase.h|265|note: template argument deduction/substitution failed:|
F:\C++\corri mot mystere\main.cpp|63|note: candidate expects 3 arguments, 0 provided|
F:\C++\corri mot mystere\main.cpp|67|error: expected primary-expression before '>' token|
F:\C++\corri mot mystere\main.cpp|67|error: no matching function for call to 'max()'|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\stl_algobase.h|219|note: candidate: template<class _Tp> const _Tp& std::max(const _Tp&, const _Tp& )|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\stl_algobase.h|219|note: template argument deduction/substitution failed:|
F:\C++\corri mot mystere\main.cpp|67|note: candidate expects 2 arguments, 0 provided|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\stl_algobase.h|265|note: candidate: template<class _Tp, class _Compare> const _Tp& std::max(const _Tp&, const _Tp&, _Compare)|
C:\TDM-GCC-32\lib\gcc\mingw32\5.1.0\include\c++\bits\stl_algobase.h|265|note: template argument deduction/substitution failed:|
F:\C++\corri mot mystere\main.cpp|67|note: candidate expects 3 arguments, 0 provided|
||=== Build failed: 10 error(s), 0 warning(s) (0 minute(s), 0 second(s)) ===|

Reply

Marsh Posté le 25-05-2016 à 11:48:06

Internet me dit que pour numeric_limits faut rajouter #include <limits>. Toujours selon la même source int _tmain(int argc, _TCHAR* argv[]) est une extension de chez Microsoft, si ça pose problème remplace le par int main(void) (tu n'utilises pas les arguments passés à main ici).

Reply

Marsh Posté le 25-05-2016 à 12:21:30

Il suffit de rajouter
#include <limits>
dans MotMysterePlus.h
et ca compile, je viens de tester.
g++ -o myst.exe stdafx.cpp MotMysterePlus.cpp Main.cpp

Note:
dico.txt vient d’où, histoire que je puisse tester.

perso, j'ai reformaté le code suivant mes conventions:

Code :

// MotMysterePlus.cpp : Defines functions source code.
//
#include "stdafx.h"
#include "MotMysterePlus.h"
using namespace std;
string getFinalWord(bool const solo, vector<string> wordList) {
string word("" );
if (solo) {
int randLevel (rand() % wordList.size());
word = wordList[randLevel];
}
else {
cout << "JOUEUR 1 ! Saisissez un mot :" << endl;
cin >> word ;
}
return word;
}
string getMysteryWord(string word) {
string newWord("" );
while (word.compare("" ) != 0) {
int index = rand() % word.size();
newWord += word[index];
word.erase(index, 1);
}
return newWord;
}
void resolveMysteryWord(string const mixWord, string const finalWord, bool const solo) {
bool finish(false);
int count(5);
string temp("" );
string joueur("JOUEUR 2 : " );
if (solo) {
joueur = "";
}
while (!finish) {
cout << joueur << "Quel est ce mot ? " << mixWord << endl;
cout << "Il vous reste " << count << " coup(s)." << endl;
cin >> temp;
if (temp.compare(finalWord) == 0) {
cout << "BRAVO !" <<endl;
cout << "Vous avez trouve en " << 5-count+1 << " essai(s)" << endl;
finish = true;
}
else {
--count;
if (count != 0 ) {
cout << "ce n'est pas le bon mot ! "<< endl;
}
else {
cout << joueur << "PERDU ! le mot mystere etait : " << finalWord << endl;
finish = true;
}
}
}
}
vector<string> initDico() {
cout << "veuillez patienter, nous initialisons le jeu ..." << endl;
vector<string> wordList;
string word("" );
ifstream dico("dico.txt" );
if (dico) {
string temp;
while (getline(dico, temp) && (temp.compare("" ) != 0)) {
wordList.push_back(temp);
}
int randLevel (rand() % wordList.size());
word = wordList[randLevel];
}
else {
cout << "ERREUR : impossible d'ouvrir le fichier en lecture." << endl;
}
return wordList;
}

Code :

// Main.cpp : Defines the entry point for the console application.
//
#include "stdafx.h"
#include "MotMysterePlus.h"
int _tmain(int argc, _TCHAR* argv[]) {
bool play(true);
srand(time(0));
vector<string> dicoList(initDico());
if(dicoList.size() == 0) {
return 1;
}
cout << "**** BIENVENUE au jeu du Mot Mystere ****" <<endl;
do {
bool wrongValue(true);
char answer('a');
//0- Proposer un mode Solo ou Duo
bool solo(false);
cout << "Voulez-vous jouer en Solo ou en Duo ? (s/d)" << endl;
while (wrongValue) {
cin.get(answer);
switch (answer) {
case 's':
wrongValue = false;
solo = true;
break;
case 'd':
wrongValue = false;
break;
default:
cout << "'" << answer << "' n'est pas une valeur correct : (s/d)" << endl;
}
cin.ignore(numeric_limits<streamsize>::max(), '\n'); // skip bad input
}
answer = 'a';
wrongValue = true;
//1- récupérer le mot à trouver
string soluce(getFinalWord(solo, dicoList));
if (soluce.compare("" ) == 0) {
return 1;
}
//2- générer le mot Mystère
string mysteryWord(getMysteryWord(soluce));
//3- Trouver le mot Mystère
resolveMysteryWord(mysteryWord, soluce, solo);
cout << "voulez-vous faire une autre partie ? (o/n)" << endl;
while (wrongValue) {
cin.ignore(numeric_limits<streamsize>::max(), '\n'); // skip bad input
cin.get(answer);
switch (answer) {
case 'o':
wrongValue = false;
cin.ignore(numeric_limits<streamsize>::max(), '\n');
break;
case 'n':
wrongValue = false;
cin.ignore(numeric_limits<streamsize>::max(), '\n');
play = false;
break;
default:
cout << "'" << answer << "' n'est pas une valeur correct : (o/n)" << endl;
}
}
} while (play);
return 0;
}

A+,

Message cité 1 fois
Message édité par gilou le 25-05-2016 à 12:49:22

---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --

Reply

Marsh Posté le 25-05-2016 à 12:41:21

Merci les gars

Voici le lien cjoint pour dico.txt : http://www.cjoint.com/c/FEzkNw3NfBz

Je vais tester tout ça et je reviens un peu plus tard

Reply

Marsh Posté le 25-05-2016 à 13:51:20

La solution marche.
Par contre, le jeu est assez injouable, il faudrait des niveaux de difficulté.

A+,

Message cité 1 fois

---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --

Reply

Marsh Posté le 25-05-2016 à 14:14:26

Je viens de réussir à faire tourner l'exo du prof, en mettant #include "stdafx.cpp" et MotMysterePlus.cpp au lieu de .h pour les deux, au tout début de main.cpp

Bon mais j'ai pas à rougir, le mien est mieux ! Je pioche partout dans le dico et non que au début et en incrémentant, l'aspect visuel est mieux, la jouabilité aussi, plus rapide...

Edit : je viens de voir ton post

Message édité par Lt Ripley le 25-05-2016 à 14:15:57

Reply

Marsh Posté le 25-05-2016 à 14:15:25

gilou a écrit :

La solution marche.
Par contre, le jeu est assez injouable, il faudrait des niveaux de difficulté.

A+,

Pas faux

Reply

Marsh Posté le 25-05-2016 à 14:23:16

Mais j'ai un peu de mal avec mes boucles je ne sais plus trop ou j'en suis, il faut que je travaille à ce niveau déjà

Voici mon code //proud of me

Code :

#include <iostream>
#include <ctime>
#include <cstdlib>
#include <string>
#include <fstream>
using namespace std;
/* Exercice : Jeu du mot mystere : Merci OpenClassrooms
Mode 1 ou 2 Joueurs
*/
//melange : fonction qui renvoi un melange du mot mystere
string melange(string motAMelanger)
{
srand(time(0));
int nbAleatoire(0);
string motMelange;
do
{
nbAleatoire = rand() %motAMelanger.size();
motMelange += motAMelanger[nbAleatoire];
motAMelanger.erase(nbAleatoire,1);
}while (motAMelanger.size() !=0);
return motMelange;
}
// fonction pour nb joueur = 1
int jeu1Joueur()
{
string nlPartie1J("OUI" );
do
{
cout <<"\n Mode 1 joueur \n";
string motMystere1J, motDevine1J, motADeviner1J, motRecupere;
ifstream monFlux("dico.txt" );
if(monFlux)
{
string ligne;
int nbLignes(0);
while(getline(monFlux,ligne))
{
nbLignes++;
}
monFlux.clear();
monFlux.seekg(0, ios::beg); // renvoie le curseur au début
int positionAleatoire(0), multiplicateurAleatoire;
srand(time(0));
do
{
multiplicateurAleatoire = rand()%50;
positionAleatoire = multiplicateurAleatoire*(rand()%nbLignes);
}while (positionAleatoire>nbLignes);
string motExtrait;
for (int i(0);i<positionAleatoire; i++)
{
getline(monFlux,motExtrait);
}
motMystere1J = melange(motExtrait); // appel la fonction melange
cout << endl << "\t\t\t\t\t " << motMystere1J << endl << endl << endl;
int nbEssais1J(5);
do
{
if (nbEssais1J == 0)
{
cout << "Vous n'avez plus d'essais ! Il s'agissait de : " << motExtrait << "\n\n";
motDevine1J=motExtrait; //pour arrêter la boucle
}
else
{
cout << " Quel est ce mot ? (" << nbEssais1J << " essais) : ";
cin >> motDevine1J;
nbEssais1J-=1;
if (motDevine1J==motExtrait)
{
cout << "\n ============= Bravo vous avez gagne ! =============\n\n";
}
else
{
cout << "\n Ce n'est pas le mot (" << nbEssais1J << " essais) : ";
}
}
}while (motDevine1J!=motExtrait);
}else cout <<"Erreur : impossible d'acceder au fichier dico.txt";
cout << "\n Nouvelle partie ? (tapez OUI ou NON) : ";
cin >> nlPartie1J;
}while (nlPartie1J=="OUI" );
}
// Fonction pour nb joueurs = 2
int jeu2Joueurs()
{
string nlPartie2J("OUI" );
do
{
string motADeviner, motDevine;
cout << "\n Mode 2 joueurs \n";
cout << "\n Joueur 1 : Saisissez un mot mystere : ";
cin >> motADeviner;
string motMystere, motMelange; //sort le mot saisi de l'écran
for (int i(0); i<30; i++)
{
cout << endl;
}
motMystere = melange(motADeviner); // appel la fonction melange
cout << endl << "\t\t\t\t\t voici le mot melange : " << motMystere << endl << endl << endl;
int nbEssais(5);
do
{
if (nbEssais == 0)
{
cout << "Vous n'avez plus d'essais ! Il s'agissait de : " << motADeviner << "\n\n";
motDevine=motADeviner;
}
else
{
cout << " Joueur 2 : Quel est ce mot ? (" << nbEssais << " essais) : ";
cin >> motDevine;
nbEssais-=1;
if (motDevine==motADeviner)
{
cout << "\n ============= Bravo vous avez gagne ! =============\n\n";
}
else
{
cout << "\n Ce n'est pas le mot (" << nbEssais << " essais) : ";
}
}
}while (motDevine!=motADeviner);
cout << "\n Nouvelle partie ? (tapez OUI ou NON) : ";
cin >> nlPartie2J;
}while (nlPartie2J=="OUI" );
}
int main()
{
cout << "\n \t ================== \n \t JEU DU MOT MYSTERE \n \t ================== \n Majuscules obligatoires !" << endl << endl;
cout << "\n Entrez le nombre de joueurs (1 ou 2) : ";
int nbJoueurs;
cin >> nbJoueurs;
switch (nbJoueurs)
{
case 1:
jeu1Joueur();
break;
case 2:
jeu2Joueurs();
break;
}
return 0;
}

Reply

Marsh Posté le 25-05-2016 à 14:40:10

S'agissant de C++ je laisse la parole à gilou, juste deux remarques:
- srand(time(0)); on le fait une seule fois au début du programme (dans le main()).
- C'est bien d'activer les warnings (-Wall) mais faut aussi les prendre en compte.
Dans le cas présent ce n'est rien qui va empêcher le fonctionnement du programme mais les warnings peuvent donner des précieuses infos si il y a un problème et puis - surtout en tant que débutant - autant faire les choses proprement...

Code :

tst.cpp: In function 'int jeu1Joueur()':
tst.cpp:85:1: warning: no return statement in function returning non-void [-Wreturn-type]
tst.cpp: In function 'int jeu2Joueurs()':
tst.cpp:129:1: warning: no return statement in function returning non-void [-Wreturn-type]

int -> void

Message édité par rat de combat le 25-05-2016 à 14:40:34

Reply

Marsh Posté le 25-05-2016 à 14:53:54

Merci rat de combat je modifie de ce pas

Là les gars j'en peux plus : si vous voyiez le foutoir dans les fichiers/projets/dossiers and co :pt1cable: je vais reprendre à zero pour avoir plusieurs projets ouverts dans code::blocks tout bien propre. J'avais commencé comme ça mais ça n'avait pas fonctionné...

Reply

Marsh Posté le 25-05-2016 à 14:58:55

gilou a écrit :

Il suffit de rajouter
#include <limits>
dans MotMysterePlus.h

Il faut que je le mette dans main.cpp pour que ça fonctionne

Reply

Marsh Posté le

Reply

Petits calculs pour les crédits, ne fonctionne pas [noob]

Sujets relatifs:

Leave a Replay