C++

Problème objet/classe

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Problème objet/classe

Akumetsu

Bonjour, j'ai quelques soucis à propos d'un programme que je suis en train de faire (pas compliqué du tout, je ne suis encore que débutant) et j'ai encore quelques soucis avec les objets/classes, c'est donc pour cette raison que je fais appel à vous. Mon programme s'apparente à un Pokemon-like avec des personnages et des attaques, bon pour l'instant il n'y a rien de très emballant mais je m'occuperai du reste lorsque les erreurs rencontrées seront corrigés.

Tout d'abord voici mon .h (si celui ci n'est pas à la norme, des petites erreurs, si les attributs sont mal utilisés ou si vous avez des conseils, n'hésitez pas à m'en faire part) :

FAUX (effacé pour ne pas encombrer la page).

Et voici les quelques messages d'erreurs :

43 RPKeik.h new types may not be defined in a return type

43 RPKeik.h two or more data types in declaration of `ChoixJoueur1'

43 RPKeik.h prototype for `RPKeik RPKeik::ChoixJoueur1()' does not match any in class `RPKeik'

Je ne vois pas trop où est le problème étant donné qu'il n'y a pas de return pour le premier message.

Merci d'avance.

Message édité par Akumetsu le 05-01-2009 à 23:18:02

Publicité

alexandre_j

Ta classe doit se terminer par un ;

Code :

class test
{
};

Message cité 1 fois
Message édité par alexandre_j le 03-01-2009 à 15:33:40

Akumetsu

alexandre_j a écrit :

Ta classe doit se terminer par un ;

Code :

class test
{
};

Quel idiot je fais :pfff: ... vivement que ça devienne un automatisme. Merci.

Les erreurs, quand il n'y en a plus, il y en a encore.

189 C:\Users\Keikoku\Desktop\Programmation\Projet BTS 2008\RPKeik\RPKeik.h no match for 'operator>>' in 'std::cin >> ((RPKeik*)this)->RPKeik::AttaqueJoueur1'

On ne peut pas utiliser de cin pour un attribut ?

alexandre_j

Je ne sais pas. Perso je passerais par une variable intermédiaire qui ne soit pas un int, car si l'utilisateur rentre autre chose qu'un int, ça plante. Je passerais par un autre type (char) et je testerais si c'est un int et le balancerais dans le switch. Maintenant vu les experts qu'il y a ici, j'ai bien envie de savoir qu'elle solution adopter face à ce genre de problème.

alexandre_j

A regarder le code de plus prêt... AttaqueJoueur1 est une fonction !

Message cité 1 fois
Message édité par alexandre_j le 03-01-2009 à 17:03:50

Akumetsu

alexandre_j a écrit :

A regarder le code de plus prêt... AttaqueJoueur1 est une fonction !

Exact, j'ai viré les parenthèses et ça va mieux. J'ai honte de faire des erreurs comme celles-ci. Encore merci.

Akumetsu

Encore besoin d'aide... en espérant cette fois ci ne pas avoir fait d'erreurs de débutant.

[Linker error] undefined reference to `RPKeik::RPKeik()'
[Linker error] undefined reference to `RPKeik::~RPKeik()'

Je ne vois absolument pas ce qu'il faut faire, ce n'est pas faute d'avoir mis :

RPKeik::RPKeik()
{
}

RPKeik::~RPKeik()
{
}

Comme dit sur ce lien :
http://www.keil.com/support/docs/3136.htm

Message édité par Akumetsu le 03-01-2009 à 22:41:40

alexandre_j

Est-ce que tu peux remettre ton code mis à jour ?

Message cité 1 fois

Akumetsu

alexandre_j a écrit :

Est-ce que tu peux remettre ton code mis à jour ?

J'ai voulu faire de la POO (bon par contre je ne suis pas sur du résultat) mais j'ai toujours l'ancienne version mise à jour au cas où.

Code :

#ifndef RPKEIK_H
#define RPKEIK_H
#include <cstdlib>
#include <iostream>
#include <string>
using namespace std ;
class RPKeik
{
public :
RPKeik() ;
~RPKeik() ;
void ChoixJoueur1() ;
void ChoixJoueur2() ;
void Combat() ;
private :
int Joueur1 ;
int Joueur2 ;
string NomJoueur1 ;
int VieJoueur1 ;
int AttaqueJoueur1 ;
string NomAttaque1Joueur1 ;
int Attaque1Joueur1 ;
string NomAttaque2Joueur1 ;
int Attaque2Joueur1 ;
string NomJoueur2 ;
int VieJoueur2 ;
int AttaqueJoueur2 ;
string NomAttaque1Joueur2 ;
int Attaque1Joueur2 ;
string NomAttaque2Joueur2 ;
int Attaque2Joueur2 ;
} ;
RPKeik::RPKeik()
{
RPKeik Joueur12 ;
system("cls" ) ;
cout << "Joueur 1 choisissez un personnage :" << endl << "1.Sorel" << endl << "2.Pikachu" << endl << "3.Vegeta" << endl << "4.Mario" << endl << "5.Ryu" << endl << "Captain Falcon" << endl << "7.Dark Vador" << endl << "8.Windows" << endl << "9.Joker" << endl << "10.Leonidas" << endl ;
Joueur12.ChoixJoueur1() ;
system("cls" ) ;
cout << "Joueur 2 choisissez un personnage :" << endl << "1.Sorel" << endl << "2.Pikachu" << endl << "3.Vegeta" << endl << "4.Mario" << endl << "5.Ryu" << endl << "Captain Falcon" << endl << "7.Dark Vador" << endl << "8.Windows" << endl << "9.Joker" << endl << "10.Leonidas" << endl ;
Joueur12.ChoixJoueur2() ;
system("cls" ) ;
Joueur12.Combat() ;
system("PAUSE>nul" );
}
RPKeik::~RPKeik()
{
}
void RPKeik::ChoixJoueur1()
{
cin >> Joueur1 ;
switch (Joueur1)
{
case 1 : this->NomJoueur1="Sorel" ;
this->NomAttaque1Joueur1="1, 2, 3 Sorel" ;
this->Attaque1Joueur1=50 ;
this->NomAttaque2Joueur1="Sorel & Hardy" ;
this->Attaque2Joueur1=60 ;
break ;
case 2 : this->NomJoueur1="Pikachu" ;
this->NomAttaque1Joueur1="Tonnerre" ;
this->Attaque1Joueur1=50 ;
this->NomAttaque2Joueur1="Vive-Attaque" ;
this->Attaque2Joueur1=60 ;
break ;
case 3 : this->NomJoueur1="Vegeta" ;
this->NomAttaque1Joueur1="Final Flash" ;
this->Attaque1Joueur1=50 ;
this->NomAttaque2Joueur1="It's over 9000 !!!" ;
this->Attaque2Joueur1=60 ;
break ;
case 4 : this->NomJoueur1="Mario" ;
this->NomAttaque1Joueur1="Saut" ;
this->Attaque1Joueur1=50 ;
this->NomAttaque2Joueur1="Marteau" ;
this->Attaque2Joueur1=60 ;
break ;
case 5 : this->NomJoueur1="Ryu" ;
this->NomAttaque1Joueur1="Hado Ken" ;
this->Attaque1Joueur1=50 ;
this->NomAttaque2Joueur1="Sho Ryu Ken" ;
this->Attaque2Joueur1=60 ;
break ;
case 6 : this->NomJoueur1="Captain Falcon" ;
this->NomAttaque1Joueur1="Falcon Punch" ;
this->Attaque1Joueur1=50 ;
this->NomAttaque2Joueur1="Falcon Kick" ;
this->Attaque2Joueur1=60 ;
break ;
case 7 : this->NomJoueur1="Dark_Vador" ;
this->NomAttaque1Joueur1="Je suis ton père" ;
this->Attaque1Joueur1=50 ;
this->NomAttaque2Joueur1="Sabre Laser" ;
this->Attaque2Joueur1=60 ;
break ;
case 8 : this->NomJoueur1="Windows" ;
this->NomAttaque1Joueur1="BSOD" ;
this->Attaque1Joueur1=50 ;
this->NomAttaque2Joueur1="Bug" ;
this->Attaque2Joueur1=60 ;
break ;
case 9 : this->NomJoueur1="Joker" ;
this->NomAttaque1Joueur1="Pencil Trick" ;
this->Attaque1Joueur1=50 ;
this->NomAttaque2Joueur1="Why so serious ?" ;
this->Attaque2Joueur1=60 ;
break ;
case 10 : this->NomJoueur1="Leonidas" ;
this->NomAttaque1Joueur1="This is Sparta !" ;
this->Attaque1Joueur1=50 ;
this->NomAttaque2Joueur1="We dine on hell !" ;
this->Attaque2Joueur1=60 ;
break ;
default : cout << "Veuillez selectionner un personnage existant !" << endl ;
break ;
}
}
void RPKeik::ChoixJoueur2()
{
cin >> Joueur2 ;
switch (Joueur2)
{
case 1 : this->NomJoueur2="Sorel" ;
this->NomAttaque1Joueur2="1, 2, 3 Sorel" ;
this->Attaque1Joueur2=50 ;
this->NomAttaque2Joueur2="Sorel & Hardy" ;
this->Attaque2Joueur2=60 ;
break ;
case 2 : this->NomJoueur2="Pikachu" ;
this->NomAttaque1Joueur2="Tonnerre" ;
this->Attaque1Joueur2=50 ;
this->NomAttaque2Joueur2="Vive-Attaque" ;
this->Attaque2Joueur2=60 ;
break ;
case 3 : this->NomJoueur2="Vegeta" ;
this->NomAttaque1Joueur2="Final Flash" ;
this->Attaque1Joueur2=50 ;
this->NomAttaque2Joueur2="It's over 9000 !!!" ;
this->Attaque2Joueur2=60 ;
break ;
case 4 : this->NomJoueur2="Mario" ;
this->NomAttaque1Joueur2="Saut" ;
this->Attaque1Joueur2=50 ;
this->NomAttaque2Joueur2="Marteau" ;
this->Attaque2Joueur2=60 ;
break ;
case 5 : this->NomJoueur2="Ryu" ;
this->NomAttaque1Joueur2="Hado Ken" ;
this->Attaque1Joueur2=50 ;
this->NomAttaque2Joueur2="Sho Ryu Ken" ;
this->Attaque2Joueur2=60 ;
break ;
case 6 : this->NomJoueur2="Captain Falcon" ;
this->NomAttaque1Joueur1="Falcon Punch" ;
this->Attaque1Joueur2=50 ;
this->NomAttaque2Joueur2="Falcon Kick" ;
this->Attaque2Joueur2=60 ;
break ;
case 7 : this->NomJoueur2="Dark_Vador" ;
this->NomAttaque1Joueur2="Je suis ton père" ;
this->Attaque1Joueur2=50 ;
this->NomAttaque2Joueur2="Sabre Laser" ;
this->Attaque2Joueur2=60 ;
break ;
case 8 : this->NomJoueur2="Windows" ;
this->NomAttaque1Joueur2="BSOD" ;
this->Attaque1Joueur2=50 ;
this->NomAttaque2Joueur2="Bug" ;
this->Attaque2Joueur2=60 ;
break ;
case 9 : this->NomJoueur2="Joker" ;
this->NomAttaque1Joueur2="Pencil Trick" ;
this->Attaque1Joueur2=50 ;
this->NomAttaque2Joueur2="Why so serious ?" ;
this->Attaque2Joueur2=60 ;
break ;
case 10 : this->NomJoueur2="Leonidas" ;
this->NomAttaque1Joueur2="This is Sparta !" ;
this->Attaque1Joueur2=50 ;
this->NomAttaque2Joueur2="We dine on hell !" ;
this->Attaque2Joueur2=60 ;
break ;
default : cout << "Veuillez selectionner un personnage existant !" << endl ;
break ;
}
}
void RPKeik::Combat()
{
do
{
cout << "Joueur1 : " << NomJoueur1 << "choisissez une attaque : " << endl << "1." << NomAttaque1Joueur1 << endl << "2." << NomAttaque2Joueur1 << endl ;
cin >> this->AttaqueJoueur1 ;
switch (this->AttaqueJoueur1)
{
case 1 : cout << NomJoueur1 << " lance " << NomAttaque1Joueur1 << endl ;
this->VieJoueur2 = this->VieJoueur2 - this->Attaque1Joueur1 ;
break ;
case 2 : cout << NomJoueur1 << " lance " << NomAttaque2Joueur1 << endl ;
this->VieJoueur2 = this->VieJoueur2 - this->Attaque2Joueur1 ;
break ;
}
cout << NomJoueur2 << " a " << this->VieJoueur2 << " de vie " << endl << endl ;
if (this->VieJoueur2 != 0)
{
cout << "Joueur2 : " << NomJoueur2 << "choisissez une attaque : " << endl << "1." << NomAttaque1Joueur2 << endl << "2." << NomAttaque2Joueur2 << endl ;
cin >> this->AttaqueJoueur2 ;
switch (this->AttaqueJoueur2)
{
case 1 : cout << NomJoueur2 << " lance " << NomAttaque1Joueur2 << endl ;
this->VieJoueur1 = this->VieJoueur1 - this->Attaque1Joueur2 ;
break ;
case 2 : cout << NomJoueur2 << " lance " << NomAttaque2Joueur2 << endl ;
this->VieJoueur1 = this->VieJoueur1 - this->Attaque2Joueur2 ;
break ;
}
cout << NomJoueur1 << " a " << this->VieJoueur1 << " de vie " << endl ;
}
}
while (!(this->VieJoueur1 == 0 || this->VieJoueur2 == 0)) ;
if ( this->VieJoueur2 <= 0 )
{
cout << "Joueur 1 a gagné !" << endl ;
}
else
{
cout << "Joueur 2 a gagné !" << endl ;
}
}
#endif

Le constructeur m'a pas l'air bon du tout (ou alors les objets) car quand je compile tout marche correctement, par contre à l'exécution ça plante (j'ai mis un system PAUSE au début du constructeur et ça plante quand même).

Message édité par Akumetsu le 04-01-2009 à 01:05:37

Joel F

Real men use unique_ptr

revois les abses avec un cours propre. La tu fait n'importe quoi ...

ligne 43 : RPKeik Joueur12 ;

pourquoi instancie tu un RPKeik dans le constructeur de RPKeik ?
L'erreur à l'execution est certaienemnt une stack overflow car l'instanciation rappelles le constructeur etc ...

Ensuite, tu melange code des entités, interface utilisateur et gestion du flux du programme.
La, y a genre 4-5 classes à faire : personnage, joueur, partie, etc ...

Des liens :

http://cpp.developpez.com/cours/cppavance/
http://bruce-eckel.developpez.com/ [...] icpp2vol1/

Message édité par Joel F le 04-01-2009 à 09:52:36

Publicité

Akumetsu

Merci pour les liens, ils sont très complet et j'ai enfin (je l'espère) saisis l'utilité des classes et des objets. J'ai donc fait la class Personnage comme tu le préconisais :

Code :

#ifndef PERSONNAGE_H
#define PERSONNAGE_H
#include <string>
using namespace std ;
class Personnage
{
public :
Personnage() ;
~Personnage() ;
void Sorel() ;
void Pikachu() ;
void Vegeta() ;
void Mario() ;
void Ryu () ;
void CaptainFalcon() ;
void DarkVador() ;
void Windows() ;
void Joker() ;
void Leonidas() ;
void PersoCree() ;
private :
string Nom ;
string NomAttaque1 ;
int Attaque1 ;
int EnergieAttaque1 ;
string NomAttaque2 ;
int Attaque2 ;
int EnergieAttaque2 ;
} ;
Personnage Joueur1 ;
Personnage Joueur2 ;
void Personnage::Sorel()
{
this->Nom="Sorel" ;
this->NomAttaque1="1, 2, 3 Sorel" ;
this->Attaque1=25 ;
this->EnergieAttaque1=45 ;
this->NomAttaque2="Sorel & Hardy" ;
this->Attaque2=60 ;
this->EnergieAttaque2=70 ;
}
void Personnage::Pikachu()
{
this->Nom="Pikachu" ;
this->NomAttaque1="Tonnerre" ;
this->Attaque1=50 ;
this->EnergieAttaque1=60 ;
this->NomAttaque2="Vive-Attaque" ;
this->Attaque2=15 ;
this->EnergieAttaque2=20 ;
}
void Personnage::Vegeta()
{
this->Nom="Vegeta" ;
this->NomAttaque1="Final Flash" ;
this->Attaque1=30 ;
this->EnergieAttaque1=50 ;
this->NomAttaque2="It's over 9000 !!!" ;
this->Attaque2=80 ;
this->EnergieAttaque2=100 ;
}
void Personnage::Mario()
{
this->Nom="Mario" ;
this->NomAttaque1="Saut" ;
this->Attaque1=35 ;
this->EnergieAttaque1=55 ;
this->NomAttaque2="Marteau" ;
this->Attaque2=60 ;
this->EnergieAttaque2=70 ;
}
void Personnage::Ryu()
{
this->Nom="Ryu" ;
this->NomAttaque1="Hado Ken" ;
this->Attaque1=15 ;
this->EnergieAttaque1=20 ;
this->NomAttaque2="Sho Ryu Ken" ;
this->Attaque2=30 ;
this->EnergieAttaque2=50 ;
}
void Personnage::CaptainFalcon()
{
this->Nom="Captain Falcon" ;
this->NomAttaque1="Falcon Punch" ;
this->Attaque1=80 ;
this->EnergieAttaque1=100 ;
this->NomAttaque2="Falcon Kick" ;
this->Attaque2=25 ;
this->EnergieAttaque2=45 ;
}
void Personnage::DarkVador()
{
this->Nom="Dark Vador" ;
this->NomAttaque1="Je suis ton père" ;
this->Attaque1=10 ;
this->EnergieAttaque1=15 ;
this->NomAttaque2="Sabre Laser" ;
this->Attaque2=30 ;
this->EnergieAttaque2=50 ;
}
void Personnage::Windows()
{
this->Nom="Windows" ;
this->NomAttaque1="BSOD" ;
this->Attaque1=50 ;
this->EnergieAttaque1=60 ;
this->NomAttaque2="Bug" ;
this->Attaque2=50 ;
this->EnergieAttaque2=60 ;
}
void Personnage::Joker()
{
this->Nom="Joker" ;
this->NomAttaque1="Pencil Trick" ;
this->Attaque1=80 ;
this->EnergieAttaque1=100 ;
this->NomAttaque2="Why so serious ?" ;
this->Attaque2=20 ;
this->EnergieAttaque2=30 ;
}
void Personnage::Leonidas()
{
this->Nom="Leonidas" ;
this->NomAttaque1="This is Sparta !" ;
this->Attaque1=50 ;
this->EnergieAttaque1=60 ;
this->NomAttaque2="We dine on hell !" ;
this->Attaque2=10 ;
this->EnergieAttaque2=15 ;
}
#endif

Je pense que jusque là, tout est à peu près bon (j'espère que je ne mélange plus rien cette fois ).

Malheureusement j'ai la droit aux messages d'erreur suivant :

[Linker error] undefined reference to `Personnage::Personnage()'
[Linker error] undefined reference to `Personnage::Personnage()'
[Linker error] undefined reference to `Personnage::~Personnage()'
[Linker error] undefined reference to `Personnage::~Personnage()'
ld returned 1 exit status

J'ai donc rajouté, après quelques recherches, ces lignes :

Personnage::Personnage()
{
}

et

Personnage::~Personnage()
{
}

sans grand succès, malheureusement.

Peut être faut-il implémenter

Joel F

Real men use unique_ptr

* Vegetta etc devrait etre des INSTANCES de personnage
* Pas de code dnas un .h mais dans un .cpp
* Pas de pollution de namespace ne ouvrant std dnas un .h

Message édité par Joel F le 05-01-2009 à 21:54:09

Akumetsu

*Vegeta etc... en instance (objet si j'ai bien compris) ? En fait, je dois écrire dans le Personnage.cpp :

Code :

Vegeta.Attaque1 = 30 ;
Vegeta.EnergieAttaque1=50 ;
etc...

Puis dans ma class Joueur je ferai des méthodes du style :

Code :

void Joueur::Vegeta
{
Attaque1 = Vegeta.Attaque1 ;
EnergieAttaque1 = Vegeta.EnergieAttaque1 ;
}

Cette explication me parait assez tordu (je fais toujours compliqué), j'en conclus qu'elle est entièrement fausse.

*Effectivement, les méthodes se mettent dans un cpp, je viens de corriger.

*Pour le namespace, j'ai mis ça à cause du string. Mais je viens de remplacer par std::string pour virer le namespace.

Message cité 1 fois
Message édité par Akumetsu le 05-01-2009 à 23:08:18

Joel F

Real men use unique_ptr

Akumetsu a écrit :

du caca...

Revois tes bases.

Ta classe personnage devrait avoir un constructeur non trivial qui permet de construire un personnage ) partir de ces caractéristiques :

Code :

Personnage::Personnage(std::string const& nom,std::string const& na1, int a1, int ea1,std::string const& na2, int a2, int ea2)
: Nom(nom),NomAttaque1(a1),Attaque1(a1),EnergieAttaque1(ea1),NomAttaque2(a2),Attaque2(a2),EnergieAttaque2(ea2)
{}

Ensuite, tu crée un personnage pré-défini ainsi :

Code :

Personnage vegeta("Vegeta", "Final Flash",30,50,"It's over 9000 !!!",80,100);

Le mieux serait neanmoins de les ranger dans un tableau de personnage, de les remplir au début et d'associser à chacun un index.

Code :

Personnage persos[] = { {"bob","truc",1,2,"chose",3,50}
, {"Mario","Saut",10,20,"Fleur",50,20}
// etc ...
};

Le choix du joueur est alors directement utilisé pr accéder au peros n° X.

C'est bancal et très moche, mais la solution propre te dépasse je pense. Tu es encore en train d'écrire des trucs sans savoir ce que ça veut dire.
Reprends les cours de 0 et valide morceaux par morceaux ton code.

Akumetsu

Ah oui, j'avais omis l'utilisation des parenthèses...

Pour la deuxième méthode, ne faut-il pas un tableau à 2 dimensions par hasard ?

Et encore une fois merci.

Joel F

Real men use unique_ptr

non, un tableau 1D suffit. Pourquoi ?

Akumetsu

Car tu mets plusieurs accolades dans une accolade : { {} {} } (Jamais vu ça)
Je pensais que c'était uniquement pour les tableaux 2D (D'ailleurs sur ce site il utilise cette méthode seulement pour le tableau 2D) où chaque accolade correspond à une colonne.

Mais en continuant de chercher ( http://forum.hardware.fr/hfr/Progr [...] 5716_1.htm ) j'ai vu que ça marchait de cette façon :

Code :

Toto Tab[] = { Toto(1, 2), Toto(2, 3) }

Donc pour mon cas, cela donnerait ceci (tout du moins si je suis le message de BifaceMcLeOD :

Code :

Personnage persos[] = { Personnage("bob", "truc", 1, 2...), Personnage("Mario"...) }

Bien sur ta manière de faire est bien plus courte car pas besoin de réécrire le nom de la classe (le constructeur donc) à chaque fois.

J'espère ne pas avoir dit trop de bêtises (pour une fois).

Message édité par Akumetsu le 06-01-2009 à 21:03:48

Joel F

Real men use unique_ptr

A posteriori ca marche pas avec {}, car ta classe Personnage n'est pas uen classe Plain Old Data. Reprends la syntaxe avec le nom de la classe.

Akumetsu

Ah d'accord, c'est donc de là que venait l'erreur :
brace-enclosed initializer used to initialize `Personnage'
J'étais justement en train de chercher une solution. Après modification, il n'y a plus d'erreur hormis un LinkerError pour le destructeur.

Akumetsu

Afin de faire en sorte que le joueur1 et le joueur2 ait des nom, a1 etc... distinctes, je ne vois pas du tout que faire car c'est l'objet Joueur1 qui "utilise" l'objet oPersos.

Joel F

Real men use unique_ptr

le joueur c'est pas le personnage ... Un joueur "utilise" un personnage.donc ta classe joueur contient une instance de personnage.

Revois les principes de modélisation objet de base v_v

Publicité

FORUM HardWare.fr

Programmation

C++

Problème objet/classe

Sujets relatifs
Problème algorithme de Dijkstra	Problème d'attribution de valeur à une structure
[C - Gros débutant] Probleme de variable ...	probleme image 4d, PHP, SQL
impossible de rejouer....probleme de scanf :(	Problème avec condition dans script batch
petit probleme avec un script css	probleme sur formulaire
Problème de connexion avec SQL serveur 2005	[résolu] Probleme d'appel d'un objet dans ma classe
Plus de sujets relatifs à : Problème objet/classe

Page générée en 0.233 secondes