C++

Erreur de compilation

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Erreur de compilation

Giz

Je dispose d'1 membre protege dans une classe de base fichier CShared_buffer.h)

Code :

typedef struct _DataBuffer
{
unsigned long *bufAddr;
int bufSize;
const char *bufName;
}DataBuffer;
class CShared_buffer
{
private:...
public:...
protected:
DataBuffer buffer;
};

soit la classe derivee suivante (CFile.h):

Code :

#include "CShared_buffer.h"
class CFile : public CShared_buffer
{
private: ...
public :
void WriteLogFile(CShared_buffer *bufToWrite);
};

definition de la fonction dans CFile.cpp :

Code :

void CFile::WriteLogFile(CShared_buffer *bufToWrite)
{
SYSTEMTIME a_date;
/* switch according to the first letter */
if (nb_data_columns) {
for (int i = 0; i < (bufToWrite->buffer).bufSize; i++) {
GetSystemTime(&a_date);
fprintf(f, "%ld:%ld:%ld:%ld\t", a_date.wHour, a_date.wMinute, a_date.wSecond, a_date.wMilliseconds);
for (int j = 0; j < nb_data_columns; j++)
fprintf(f, "%ld\t", (bufToWrite->buffer).bufAddr[j]);
fprintf(f, "\n" );
}
}
/* It is in the destructor */
//fclose(f);
}

la ligne en gras provoque l'erreur suivante a la compilation :

Code :

d:\amap2\sources\cfile.cpp(62) : error C2248: 'buffer' : cannot access protected member declared in class 'CShared_buffer'

aidez moi :sweat:
PS : ne me dites pas d'allez voir les cours, je suis passer sous google et le cours http://www.ann.jussieu.fr/courscpp [...] 08-B0.html dit que mon erreur ne devrait pas avoir lieu

Publicité

Giz

je galere je galere je galere ! tous les cours du net me disent que c bon mon truc :cry: :cry: :cry:

Vko

- Le roi des patates -

a priori ca me semble correct.

vu que le membre est en protected j'vois pas ce qui cloche.

Pour tester, met le en public et recompile.

Willyzekid

Je crois que tu as mal lu ton cours

Tu peux pas accéder à un membre protected (ici buffer) de l'exterieur...Et avec ton bufToWrite->buffer, ben, c'est exactement ce que tu fais!!!

Oriente toi vers le mot clef friend ou surtout, revois la structure de ton prg! Peut être que le WriteLogFile() devrait etre appelé par le bufToWrite que tu lui passe??? 'nfin bref, la structure de ton prg, ca me parait n'importe quoi....

ACut

Je crois que c'est la notion <<extérieur à la classe>> qui pose un pb à Giz. Vu que bufToWrite->buffer est invoqué DANS le corps d'une méthode de la classe dérivée, il considère que c'est un accès "intérieur" au membre protégé, et donc ça le scandalise que le compilateur lui refuse cet accès.

Moi qui suis assez profane aussi en POO, j'ai du mal à déjouer facilement ce type de trompe-l'oeil et je comprends la réaction de Giz.

---------------
NOUVEAU! Le guide de l'édition en version ebook : http://marcautret.free.fr/autret/150q-ebook/

Taz

bisounours-codeur

amis du C bonsoir. des printf, des pointeurs, pas d'encapsulation ... je déplace?

ACut

Taz a écrit :

amis du C bonsoir. des printf, des pointeurs, pas d'encapsulation ... je déplace?

Pas compris. On parle d'accès aux membres protégés, c'est mal?

Message édité par ACut le 13-08-2003 à 22:20:18

---------------
NOUVEAU! Le guide de l'édition en version ebook : http://marcautret.free.fr/autret/150q-ebook/

ACut

Pour info, un transtypage de bufToWrite vers le type CFile* (=pointeur sur classe dérivée) offre p-ê une piste. En effet, le code suivant se compile:

Code :

class Base
{
protected:
int x;
};
class Derivee : public Base
{
void Test(Base* pBase)
{
Derivee* clone;
clone = (Derivee*) pBase;
x = clone->x; // alors que pBase->x aurait été rejeté
}
};

Ca compile mais est-ce que ça fonctionne...? A priori je ne vois comment une conversion de (Base*) vers (Derivee*) pourrait poser un pb...

---------------
NOUVEAU! Le guide de l'édition en version ebook : http://marcautret.free.fr/autret/150q-ebook/

Taz

bisounours-codeur

les cast C, c'est mal (surtout surtout ici)

Message édité par Taz le 13-08-2003 à 22:23:34

ACut

Je sais que c'est pas orthodoxe, mais je tâtonne, là. Je le trouve intéressant ce pb, pour moi c'est pédagogique.

Et entre nous, je ne trouve pas beaucoup plus séduisante la solution consistant à passer par des "friend"...

---------------
NOUVEAU! Le guide de l'édition en version ebook : http://marcautret.free.fr/autret/150q-ebook/

Publicité

Taz

bisounours-codeur

il me semble qu'en fait Base est bien la base, mais ça ne donne pas de droit comme friend.

la solution:

Code :

class Foo
{
public:
virtual ~Foo()
{}
protected:
int x;
};
class Bar
: public Foo
{
void f(Foo *b)
{
Bar *other=dynamic_cast<Bar*>(b);
if(other)
{
this->x = other->x;
Foo::x = other->Foo::x;
}
}
};

Message édité par Taz le 13-08-2003 à 22:32:29

Taz

bisounours-codeur

alors ils sont où les [:prosterne] / [:roi] ?

Message édité par Taz le 13-08-2003 à 22:38:09

ACut

Ouais, c'est blindé, mais

Code :

Bar *other=dynamic_cast<Bar*>(b);

ça reconsomme de l'allocation mémoire, non?

Ca me fait pas bander ton truc, sauf ton respect. Giz cherche certainement une solution plus naturelle à son pb.

---------------
NOUVEAU! Le guide de l'édition en version ebook : http://marcautret.free.fr/autret/150q-ebook/

Taz

bisounours-codeur

1) non
2) c'est la solution la plus naturelle

ACut

Je me suis rendu compte que ça "reconsommait" pas après avoir posté!! Pff, et voilà je suis grillé comme un newb!

Bon, ben, il ne me reste plus qu'à te présenter toutes mes génuflexions les plus sincères.

---------------
NOUVEAU! Le guide de l'édition en version ebook : http://marcautret.free.fr/autret/150q-ebook/

Vko

- Le roi des patates -

Précision ... le dynamic_cast est plus lent qu'un "bon petit vieux cast C" car il y a un "data type check", alors que le cast c est totalement bourrin et dénué d'intelligence.

mais c'est sûr que c'est plus propres.

Taz

bisounours-codeur

plus lent, vous avez que ça comme mot à la bouche... si je jetais un oeil a vos codes, moi je vous en trouve de l'optimisation et de la belle!

le dynamic_cast il est sur et efficace. surtout que si le type peut etre déterminer au moment de la compilation, ça a le meme effet qu'un static_cast. donc flipper pas betement. le dynamic_cast etst pas plus propre, c'estla seule manière de procéder. si quelqu'un a une autre solution pour savoir si on est en présence d'un objet apparenté à tel type, je demande à voir. le cast C ici serait un bug très très grave aux conséquences difficiles.

Willyzekid

Willyzekid a écrit :

Peut être que le WriteLogFile() devrait etre appelé par le bufToWrite que tu lui passe??? 'nfin bref, la structure de ton prg, ca me parait n'importe quoi....

Je persiste et signe! Cette fonction WriteLogFile() est ridicule...A aucun moment dans l'implémentation tu ne fais référence à un membre de la classe appellante. Cette fonction pourrait très bien ne pas faire partie de la classe! Ou elle pourrait être appellé par la classe que tu lui passe en paramètre, tout simplement!

C'est l'architecture qui est à revoir!

Giz

Taz a écrit :

il me semble qu'en fait Base est bien la base, mais ça ne donne pas de droit comme friend.

la solution:

Code :

class Foo
{
public:
virtual ~Foo()
{}
protected:
int x;
};
class Bar
: public Foo
{
void f(Foo *b)
{
Bar *other=dynamic_cast<Bar*>(b);
if(other)
{
this->x = other->x;
Foo::x = other->Foo::x;
}
}
};

Bon ben merci a tous, je viens d'apprendre kkchose :hello:
Sachez que les cours sont CARRES malheuresement c a dire qu'il ne se limite qu'a des cas simple clair et precis (autrement dit neuneu). Je vois difficilement comment j'aurais pu savoir ce pb sans votre aide

Taz :
1-Je vois pas trop la difference avec le code de ACut
2-qu'est ce que tu appeles un "dynamic cast" (par rapport a un "static cast" )
3-La ligne en gras, le this->x ... ca reviens a mon erreur non ? :sweat: (tu accedes indirectement au membre protege)

Willyzekid :
1-Je veux bien admettre que ma structure est incoherente (pourtant ds ma fonction WriteLogFile, j'ai un appel a la classe de base dc c peut etre pas completement debile non ?) :cry:
2-Je ne SAIS pas comment bien structurer un programme mais je voudrais vraiment apprendre :cry:
3-Je ne rentre qu'en license...bref je pense avoir beaucoup a apprendre encore, d'autant plus que la conception de prog ca touche a l'uml je pense...
4-Si tu trouves cette fonction mal placee, me conseille tu de detruire ma classe et de faire un bon ptit fichier (comme j'ai deja commence) "fonctions.cpp" ou j'y met toutes mes fonctions independantes ?
5-Je voulais faire une sorte d'encapsulation/structuration, car avec les classe on s'y retrouve bien mieux [:spamafote]
6-c que mon 2eme stage, aucun programmeur ou je bosse, je dois me demerder, eux, ils attendent un resultat, kkchose qui marche. Ils s'en branlent de l'optimisation/beaute du code, il regarde que le resultat (le gars m'a dit "Vite il reste que pe de tps, fais la methode VITE maintenant) dc je dois tracer

Merci a tous :jap:

Message édité par Giz le 14-08-2003 à 11:17:42

ACut

giz a écrit :

1-Je vois pas trop la difference avec le code de ACut
2-qu'est ce que tu appeles un "dynamic cast" (par rapport a un "static cast" )
3-La ligne en gras, le this->x ... ca reviens a mon erreur non ? :sweat: (tu accedes indirectement au membre protege)

1) Y a une différence majeure: celui de Taz est blindé

2) dynamic vs static cast: ça peut pas ne pas être dans ton cours (même s'il est lapidaire)

3) this->x ne revient pas à ton erreur, car this est de type (Derivee*) -- par la force des choses!! -- et non de type (Base*)
En fait, les deux lignes:

Code :

this->x = other->x;
Foo::x = other->Foo::x;

conjurent toute espèce d'ambiguïté (y compris si un x homonyme est redéclaré, etc.).

Taz

bisounours-codeur

1) 2) tout est dans le cast. à toi de te trouver un cours. là je taffe, j'ai pas le temps de le faire. le code de Acut est mauvais et buggé
3) voir mon message précédent. protected != friend. tu hérites d'une classe mais ça ne te rends pas ami avec elle.

Publicité

FORUM HardWare.fr

Programmation

C++

Erreur de compilation

Sujets relatifs
VB6: Erreur de compilation incompréhensible....pour moi	Erreur lors de la compilation sous borland....
erreur de compilation	Problème GCC : pas d'éxécutable, ni d'erreur en fin de compilation
Erreur de compilation avec un makefile	[C] message d erreur a la compilation (linking)
Erreur de compilation / comment on appelle une methode d'une class?	[JAVA] erreur de compilation que je capte pas ... [resolu]
Erreur de compilation AIDE SVP	[c++] Erreur incomprehensible a la compilation: help me!
Plus de sujets relatifs à : Erreur de compilation

Page générée en 0.082 secondes