Problème de Toad...

Chargement d'image...

17 décembre 2007 à 16:24:58

Bonjour!

J'ai un petit problème. Voilà : je programme un petit jeu qui va se nommé Toad Adventureet je suis coincer au tout début! Mon problème, c'est que quand je compile, il me donne toute ces erreurs :

1 C:\Documents and Settings\Famille\Mes documents\Mes Projets\main.cpp In file included from main.cpp
C:\Documents and Settings\Famille\Mes documents\Mes Projets\cToad.cpp In member function `void Toad::SetStop(const char*, const char*, const char*, const char*)':
24 C:\Documents and Settings\Famille\Mes documents\Mes Projets\cToad.cpp no match for 'operator=' in '((Toad*)this)->Toad::mToadStop[0] = SDL_LoadBMP_RW(SDL_RWFromFile(bas, ((const char*)"rb")), 1)'
note C:\Dev-Cpp\include\SDL\SDL_video.h:88 candidates are: SDL_Surface& SDL_Surface::operator=(const SDL_Surface&)
25 C:\Documents and Settings\Famille\Mes documents\Mes Projets\cToad.cpp base operand of `->' has non-pointer type `SDL_Surface'
27 C:\Documents and Settings\Famille\Mes documents\Mes Projets\cToad.cpp no match for 'operator=' in '((Toad*)this)->Toad::mToadStop[2] = SDL_LoadBMP_RW(SDL_RWFromFile(droite, ((const char*)"rb")), 1)'
note C:\Dev-Cpp\include\SDL\SDL_video.h:88 candidates are: SDL_Surface& SDL_Surface::operator=(const SDL_Surface&)
28 C:\Documents and Settings\Famille\Mes documents\Mes Projets\cToad.cpp base operand of `->' has non-pointer type `SDL_Surface'
30 C:\Documents and Settings\Famille\Mes documents\Mes Projets\cToad.cpp no match for 'operator=' in '((Toad*)this)->Toad::mToadStop[1] = SDL_LoadBMP_RW(SDL_RWFromFile(haut, ((const char*)"rb")), 1)'
note C:\Dev-Cpp\include\SDL\SDL_video.h:88 candidates are: SDL_Surface& SDL_Surface::operator=(const SDL_Surface&)
31 C:\Documents and Settings\Famille\Mes documents\Mes Projets\cToad.cpp base operand of `->' has non-pointer type `SDL_Surface'
33 C:\Documents and Settings\Famille\Mes documents\Mes Projets\cToad.cpp no match for 'operator=' in '((Toad*)this)->Toad::mToadStop[3] = SDL_LoadBMP_RW(SDL_RWFromFile(gauche, ((const char*)"rb")), 1)'
note C:\Dev-Cpp\include\SDL\SDL_video.h:88 candidates are: SDL_Surface& SDL_Surface::operator=(const SDL_Surface&)
34 C:\Documents and Settings\Famille\Mes documents\Mes Projets\cToad.cpp base operand of `->' has non-pointer type `SDL_Surface'
C:\Documents and Settings\Famille\Mes documents\Mes Projets\Makefile.win [Build Error] [main.o] Error 1

Je pense (D'après ce que je vois

) que ça viens de mes '='.
Voici mes codes:

main.cpp:

#include "cToad.cpp"
#include "system.cpp"
int main(int argc, char *argv[])
{
SDL_Init(SDL_INIT_VIDEO);
SDL_Quit();
return 0;
}

cToad.cpp:

#include "system.cpp"
class Toad
{
private:
SDL_Surface mToadStop[4];
int mPoads;
int mNiveau, mSante, mAttaque, mDefense;
SDL_Rect mPosition;
public:
Toad()
{
mPoads = 10;
mNiveau = 1;
mSante = 12;
mAttaque = 2;
mDefense = 1;
}
void SetStop(const char *bas, const char *droite, const char *haut, const char *gauche)
{
mToadStop[BAS] = SDL_LoadBMP(bas);
SDL_SetColorKey(mToadStop[BAS],SDL_SRCCOLORKEY,SDL_MapRGB(mToadStop[BAS]->format,128,255,255));
mToadStop[DROITE] = SDL_LoadBMP(droite);
SDL_SetColorKey(mToadStop[DROITE],SDL_SRCCOLORKEY,SDL_MapRGB(mToadStop[DROITE]->format,128,255,255));
mToadStop[HAUT] = SDL_LoadBMP(haut);
SDL_SetColorKey(mToadStop[HAUT],SDL_SRCCOLORKEY,SDL_MapRGB(mToadStop[HAUT]->format,128,255,255));
mToadStop[GAUCHE] = SDL_LoadBMP(gauche);
SDL_SetColorKey(mToadStop[GAUCHE],SDL_SRCCOLORKEY,SDL_MapRGB(mToadStop[GAUCHE]->format,128,255,255));
}
};

et finalement system.cpp:

#ifndef DEF_SYSTEM
#define DEF_SYSTEM
#include <iostream>
#include <string>
#include <SDL/SDL.h>
using namespace std;
enum{BAS,HAUT,DROITE,GAUCHE};
#endif

Merci d'avance!

neuneutrinos

17 décembre 2007 à 17:25:32

//je ne suis pas convaincu que inclure un.cpp ca marche ...

void SetStop(const char *bas, const char *droite, const char *haut, const char *gauche)
{
mToadStop[BAS] = SDL_LoadBMP(bas);
SDL_SetColorKey(mToadStop[BAS],SDL_SRCCOLORKEY,SDL_MapRGB(mToadStop[BAS]->format,128,255,255));
mToadStop[DROITE] = SDL_LoadBMP(droite);
SDL_SetColorKey(mToadStop[DROITE],SDL_SRCCOLORKEY,SDL_MapRGB(mToadStop[DROITE]->format,128,255,255));
mToadStop[HAUT] = SDL_LoadBMP(haut);
SDL_SetColorKey(mToadStop[HAUT],SDL_SRCCOLORKEY,SDL_MapRGB(mToadStop[HAUT]->format,128,255,255));
mToadStop[GAUCHE] = SDL_LoadBMP(gauche);
SDL_SetColorKey(mToadStop[GAUCHE],SDL_SRCCOLORKEY,SDL_MapRGB(mToadStop[GAUCHE]->format,128,255,255));
}

remplace ces cpp par des hpp ou des h (change les noms des fichier en consequence et je pense que ca marchera.
(met ce 'using namespace std' la ou tu en as besoins !!)

Anonyme

17 décembre 2007 à 17:36:20

-> Pas d'include de .cpp ! On inclut les .h (ou .hpp). Ici, system.cpp devrait plutôt s'appeler system.hpp, idem pour cToad.cpp : cToad.hpp.
-> Dans les constructeurs on préfère utiliser les listes d'initialisation. CF le cour sur les classes.
-> '->' s'utilise pour les pointeurs. Quand on crée un tableau de SDL_Surface (sans *) en faisant 'tab[i]' on n'a pas accès à un pointeur, mais à une variable "normale".

gargayan

17 décembre 2007 à 19:14:09

Doh!

...
J'ai trouver l'erreur : un simple oubli de '*' durant l'initialisation de mon tableau... :-°

S'cusez-moi : c'est reglé!

neuneutrinos

18 décembre 2007 à 10:35:10

et inclure des .cpp ca ne fait pas buger? o_O

Iodenriel

18 décembre 2007 à 13:49:27

#include se fout de l'extension du fichier que tu lui donnes à manger, tant que le fichier est du texte.
.hpp, .cpp, c'est juste une convention, pour se répérer.

MatteX

18 décembre 2007 à 16:13:33

les .cpp ne devrait être que des fichier à compiler individuellement et lier ensemble ensuite (dans un projet, ou un makefile)

Et les #define sont fonctionnel l'espace de 1 fichier à compiler. Alors imaginez que 2 fichiers .cpp soit compilés et liés et que l'un inclus l'autre (Les protections anti inclusions multiples ne faisant pas effet) : erreur de linkage!

Ici j'imagine que par paresse, les fichiers ne sont pas tous compilés. C'est bien pour des tests ou pour des projets perso qui resteront cachés mais ce n'est pas bon à mettre en pratique et surtout pas à présenter sur un forum! Il y a des gens qui apprennent par ici!

De plus il y a un instable "using namespace std" dans un fichier à inclure et pas de protection anti inclusions multiples dans un autre. Ce n'est pas grave immédiatement dans ce projet mais c'est plutôt maladroit. Et je peux t'assurer que ça ne compileras sur tout les compilateurs.