[Débutant]Erreur de compilation inconprehensible

**@r$£~%[** · 11/08/2005, 22h30

Bonjours à tous,

Je suis entrain de créer quelque classes pour manipuler plus simplement la bibliothéque SDL.

Mon premier objectif est de créer une petite image qui suivent la souris. Le clic fait sortir de l' application.

J' ai crée une classe SDL::application, une classe SDL::surface et une classe SDL::Cpoint .

J' ai une erreur bisarre lors que j' essaie de compiler ce code : http://iceteagreen.ifrance.com/main.cpp.htm

Note : J' ai gardé la même arborescence. J' ai juste rajouté des .htm pour qu' il soit reconnu par le navigateur. La bibliothéque UserSDl se trouve donc dans http://iceteagreen.ifrance.com/inclu.../SDLmain.h.htm

Dans le constructeur par copie de la classe surface, je calcule les dimensions pour l' allocations de la surfaces. Pour çela, j' utilise les deux points (couple x, y) CoinSupGauche et CoinInfDroit. Mais voilà : le compilateur m' indique que x et y ne sont pas des fonctions valide. Tout se passe comme si x et y n' avait été declaré

.

Merci de m'aider. C' est sans doute un truc tout bête (Dans ce cas, j' aurais l' air redicule(?)

). Mais ça fait deux jours que je cherche.

**Laurent Gomila** · 11/08/2005, 23h14

Je n'arrive pas à visualiser Csurface.cpp, je ne sais pas si j'ai loupé quelque chose ou si tu ne l'as pas uploadé. Quoiqu'il en soit si l'erreur se trouve dans ce fichier eh bien... c'est lui qu'il faut voir

A part ça, "#include "Csurface.cpp"

Quelle idée saugrenue !!

Et puis pour pinailler, le constructeur prenant en paramètre un surface* n'est pas le constructeur par copie. Ce qui veut dire que ce dernier sera toujours généré par le compilo, et pas forcément de la manière dont tu le souhaites. Si tu ne comptes jamais t'en servir tu devrais au moins le déclarer privé.

**@r$£~%[** · 11/08/2005, 23h26

A part ça, "#include "Csurface.cpp"
Quelle idée saugrenue !!

Csurface parce que C comme classe... Ca m' évite à confondre avec les .h où je met les fonctions que j' utilise.

Et puis pour pinailler, le constructeur prenant en paramètre un surface* n'est pas le constructeur par copie. Ce qui veut dire que ce dernier sera toujours généré par le compilo, et pas forcément de la manière dont tu le souhaites. Si tu ne comptes jamais t'en servir tu devrais au moins le déclarer privé.

Mais que prend comme paramétre un constructeurs par copie, alors? Un argument de type de la classe passé par valeurs?

Je n'arrive pas à visualiser Csurface.cpp, je ne sais pas si j'ai loupé quelque chose ou si tu ne l'as pas uploadé. Quoiqu'il en soit si l'erreur se trouve dans ce fichier eh bien... c'est lui qu'il faut voir Smile

Je confirme tu a loupé quelque chose

:

Le .H :
http://iceteagreen.ifrance.com/inclu...Csurface.h.htm
Le .CPP
http://iceteagreen.ifrance.com/inclu...urface.cpp.htm

**Laurent Gomila** · 11/08/2005, 23h59

Envoyé par @r$£~%[

A part ça, "#include "Csurface.cpp"
Quelle idée saugrenue !!

Csurface parce que C comme classe... Ca m' évite à confondre avec les .h où je met les fonctions que j' utilise.

Je ne parlais pas du nom, mais de l'inclusion d'un .cpp...

Envoyé par @r$£~%[

Mais que prend comme paramétre un constructeurs par copie, alors? Un argument de type de la classe passé par valeurs?

Un constructeur par copie prend toujours comme paramètre une référence (constante ou non) sur le type de la classe. Car quand tu fais "surface a = b", ce n'est pas un pointeur (sur b) qui est passé en paramètre au constructeur

. De plus, situ prenais le paramètre par valeur, cela créerait une récursion sans fin (puisque le passage par valeur appel justement le constructeur par copie).

Envoyé par @r$£~%[

Je confirme tu a loupé quelque chose

Comment on est censé savoir que le fichier s'appelle cpp/surface.cpp.htm, et non Csurface.cpp.htm ?

Enfin pour ton problème... Et bien où sont déclarés x et y

:

**@r$£~%[** · 12/08/2005, 00h14

@r$£~%[ a écrit:
Je confirme tu a loupé quelque chose Wink

Comment on est censé savoir que le fichier s'appelle cpp/surface.cpp.htm, et non Csurface.cpp.htm ? Pleure

Ben, oui, puisque le CPP est inclus dans le .H :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
...
 
};
 
[b]#include "cpp/surface.cpp"[/b]
 
/* Fin d' utilisation du namespace SDL */}

@r$£~%[ a écrit:
Mais que prend comme paramétre un constructeurs par copie, alors? Un argument de type de la classe passé par valeurs?

Un constructeur par copie prend toujours comme paramètre une référence (constante ou non) sur le type de la classe. Car quand tu fais "surface a = b", ce n'est pas un pointeur (sur b) qui est passé en paramètre au constructeur Smile. De plus, situ prenais le paramètre par valeur, cela créerait une récursion sans fin (puisque le passage par valeur appel justement le constructeur par copie).

Là, je ne comprend plus :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

surface( const surface *Surface);                      //Constructeur par copie

Ce n' est pas un constructeur qui prend une réference constante sur la classe?

Et je ne voix pas pourquoi c' est saugrenu : on définit la classes, puis on inclus les implémentations? Ce n' est pas comme ça qu' il faut faire?

Et x et y sont définit dans le fichier Cpoint.h qui est inclus dans Csurface.h :

http://iceteagreen.ifrance.com/inclu...t/Cpoint.h.htm

**Elendil_BzH** · 12/08/2005, 00h24

Ton constructeur par copie doit être déclaré comme ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

surface( const surface& Surface);

Et pour ton problème d'inclusion, c'est l'inverse, ton fichier .h doit être inclus dans le .cpp

Et rajoute des include guards dans tes .h sinon tu risques d'avoir des erreurs de définitions multiples :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
#ifndef _CSURFACE_H
#define _CSURFACE_H
 
class structure {
  // Le reste de la déclaration
};
 
#endif

**Laurent Gomila** · 12/08/2005, 00h24

Ben, oui, puisque le CPP est inclus dans le .H :

Autant pour moi

Ce n' est pas un constructeur qui prend une réference constante sur la classe?

Euh... non, il s'agit d'un pointeur. La référence constante c'est const surface &.

Et je ne voix pas pourquoi c' est saugrenu : on définit la classes, puis on inclus les implémentations? Ce n' est pas comme ça qu' il faut faire?

Surtout pas. On inclut la définition (le .h) partout où l'on a besoin de la classe, et l'éditeur de lien fait le reste tout seul. Mais surtout, ne jamais inclure de .cpp en C++, et de manière générale ne jamais inclure de code !

**@r$£~%[** · 12/08/2005, 00h29

Donc pour finir, je doit mettre mon *.H dans mon *.CPP et non le contraire.

J' ai édité le précedent message.

Ils faudrat que je revoie les pointeurs, les réferences et tout ça

Merci à vous deux

**Laurent Gomila** · 12/08/2005, 00h36

Et tu as résolu le problème concernant x et y ?

**@r$£~%[** · 12/08/2005, 11h06

Non.

Et x et y sont définit dans le fichier Cpoint.h qui est inclus dans Csurface.h :

http://iceteagreen.ifrance.com/inclu...t/Cpoint.h.htm

Mais je ne comprend pas :

J' inclus le .H dans mon main.cpp...
J' inclus le .H dans mon .CPP

Mais le .CPP ne serat pas inclus dans mon main.cpp

**Laurent Gomila** · 12/08/2005, 11h29

x et y sont membres de ta classe point, ce ne sont pas des variables globales. Donc à moins qu'il s'agisse des (x, y) d'un point particulier, il faudra les déclarer (comme tu l'as d'ailleurs fait dans le premier constructeur).

Mais le .CPP ne serat pas inclus dans mon main.cpp

Non, en aucun cas il ne doit l'être. C'est l'éditeur de lien qui va savoir quel code se trouve dans quel fichier compilé, et réussir à créer comme un grand le binaire final.

**@r$£~%[** · 12/08/2005, 11h39

Mon pseudo-constructeur par copie acceuile un pointeur vers une surface :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
surface::surface( const surface* Surface)
{
...

La surface deux variables membres de type Cpoint nommé CoinSupGauche et CoinInfDroit, qui représente les deux coins opposé.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
class surface
{
 
   protected : 
        //L' image
        string imageBMP;
 
        //L' emplacement
        Cpoint CoinSupGauche;
        Cpoint CoinInfDroit;
 
...

La classe Cpoint comporte deux entier membre : x et y...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
class Cpoint
{
    public : 
      int x;
      int y;

Alors, où est l' erreur ?

:

**@r$£~%[** · 12/08/2005, 19h03

UP

**Laurent Gomila** · 13/08/2005, 14h33

C'est CoinSupGauche.x ou CoinSupDroit.y ou que sais-je, mais pas juste x ou y... Ce sont des membres de la class Cpoint, il faut donc bien leur coller une instance de Cpoint derrière non ?

**@r$£~%[** · 13/08/2005, 18h17

C'est CoinSupGauche.x ou CoinSupDroit.y ou que sais-je, mais pas juste x ou y... Ce sont des membres de la class Cpoint, il faut donc bien leur coller une instance de Cpoint derrière non?

Je ne comprend pas trop ce que tu veux dire :

CoinSupGauche et CoinInfDroit sont des points... Les deux doivent donc avoir des membres nommés x et y...

**Laurent Gomila** · 14/08/2005, 20h04

Ben oui... C'est justement ce que je voulais dire. Toi tu écris juste "x = ..." et "y = ...". Ce sont les x et y de quel Cpoint ?

[Débutant]Erreur de compilation inconprehensible

C++

Discussions similaires

Partager

Partager