Variable or field 'ecrire' declared void

Version imprimable

Voir 40 message(s) de cette discussion en une page

Je ne comprends pas, j'ai une classe avec des fonctions virtuelles pures, et GCC refuse de me reconnaitre la méthode "ecrire" comme une fonction...
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 #ifndef JETON_H_INCLUS #define JETON_H_INCLUS #include <iostream> class IConteneurAutomate; class IJeton { public: //virtual ~IJeton() {}; virtual void justBeenReadFromInfix( IConteneurAutomate &refPostfix, IConteneurAutomate &refPile) const = 0; virtual void ecrire(ostream &sortie) const = 0; virtual IJeton *clone() const = 0; virtual IJeton *cloneThrow() const = 0; }; #endif// ! JETON_H_INCLUS
(j'utilise Dev-C++ avec GCC 3.4.2)
GCC me sort ces messages d'erreurs:

Citation:

Envoyé par GCC

Jeton.h:14: error: variable or field `ecrire' declared void

Jeton.h:14: error: `ecrire' declared as a `virtual' field
Jeton.h:14: error: expected `;' before '(' token

Comment faire pour que GCC veuille bien reconnaitre ma méthode "écrire" comme une méthode et non une variable ?

Edit; bizarrement, on dirait que c'est le ostream qu'il n'aime pas, car quand je vire le paramètre, je n'ai pas cette erreur...

13/11/2005, 13h22
Médinoc

Finalement, j'ai corrigé l'erreur en rajoutant std:: devant ostream, mais je ne comprend pas pourquoi j'ai eu droit à une erreur pareille au lieu d'avoir un "ostream undeclared" ou un truc du genre...
13/12/2005, 09h20
hansaplast
je post a ta suite, car mon pb est similaire :

j'ai utilisé l'interface de dev c++ 5 beta afin de créer mes classes...
lorsqu'ils compile, il me genere la meme erreure (au niveau de "Setpassword"):
Code:

1 2 3 4 5 6 7 8 9 class communication { public: // class constructor communication(); // class destructor ~communication(); void Setpassword(string x); // sets the value of password private:
est-ce que ca peut venir d'erreures dans le corps de la fonction dans le fichier .cpp?
(dsl : total newbee...)

il me dit :

Citation:

variable or field 'Setpassword' declared void'

Citation:

expected ';' before (

pour cette ligne la...
13/12/2005, 09h23
Fiquet

Tu as essayé de mettre std::string au lieu de string comme Médinoc? Sinon là ça me semble correct.
13/12/2005, 09h31
hansaplast

yep, j'ai esayé, ca ne marche toujours pas...

Code:

void Setpassword(std::ostream x);

j'ai remis le meme type que dans son exemple...
13/12/2005, 09h37
Médinoc

est-ce que <string> (ou <iostream> dans ton second test) est inclus AVANT ton header ?
13/12/2005, 09h37
Fiquet

Citation:

Envoyé par hansaplast

Code:

void Setpassword(std::ostream x);

8O Euh mais je pense que le type de ta fonction est juste non? Pour spécifier un password c'est juste d'utiliser string. Mais pas un flux standard.

Mets

Code:

void Setpassword(std::string x);

Et vérifie que tu a inclu l'en-tête <string> en haut de ton .h.

Pour plus d'infos FAQ : http://c.developpez.com/faq/cpp/?page=strings
13/12/2005, 09h40
Médinoc

Le problème n'est pas que c'est le type de la fonction, mais que le compilo la prend pour une variable au lieu de dire que le type de son paramètre n'est pas déclaré)
13/12/2005, 09h44
Fiquet

Citation:

Envoyé par Médinoc

Le problème n'est pas que c'est le type de la fonction, mais que le compilo la prend pour une variable au lieu de dire que le type de son paramètre n'est pas déclaré)

Oui mais je pense que son type string devait être juste non? Et apparement vous avez eu le même problème que GCC met une erreur inappropriée, c'est pour ça que j'ai dit la même chose avec l'en-tête...

donc voila, mon fichier main :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 #include <winsock2.h> #pragma comment(lib, "ws2_32.lib") #include <stdio.h> /* Autorise l'utilisation de printf et de scanf. */ #include <stdlib.h> /* Autorise l'utilisation de malloc et de free. */ #include <string.h> /* Autorise l'utilisation de strcpy, strlen et de strcmp. */ #include <windows.h> /*API win32, permettant la capture de l'ecran*/ int main(){...
et mon fichier communication.h :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 // Class automatically generated by Dev-C++ New Class wizard #ifndef COMMUNICATION_H #define COMMUNICATION_H /** * implementation de base des methodes de communication reseau * * va etre etendue en plusieurs heritages */ class communication { public: // class constructor communication(); // class destructor ~communication(); void Setpassword(std::string x); // sets the value of password private:...
j'inclut tout dans le main, ca doit suffire non?

(c'est ma premeire compile en C++...)

le compilateur par defaut, c'est migw (un truc dans le genre...) je croit bien.
est-ce que cette erreur de compile peut venir d'une autre erreure dans un autre fichier?
(j'ai enoormement ocdé avant de pouvoir compiler... et... j'avait oublié que les string sont speciaux en C... j'avait utilisé la syntaxe de php;.. :'(... donc c'esty truffé d'erreure... mias plus loin, dans mon .cpp... pas dans le .h...

13/12/2005, 10h02
Fiquet

Et bien déjà là, comme on te l'a dit, inclue le bon en-tête avant la définition de ta classe:

Code:

#include <string>

Ensuite pour tes headers si tu fais du C++ regarde ici : http://c.developpez.com/faq/cpp/?pag...le#SL_iostream (en bas)

Qu'entends-tu par

Citation:

j'inclut tout dans le main, ca doit suffire non?

Tu as bien mis #include "communication.h" ?
13/12/2005, 10h30
hansaplast

Citation:

Tu as bien mis #include "communication.h" ?

sous dev c++ :
non, je ne l'ai pas mis...
si je le met, ca me met une erreure.

j'ai telechargé code::blocks, lui me met une erreure differente :

Citation:

error : 'string' has not been declared

et ne me genere pas d'erreure quand je met

Citation:

#include "communication.h" ?
13/12/2005, 10h48
Fiquet

Il faudra bien l'inclure à une place, si tu utiliser la classe déclarée dedans dans un de tes autres fichiers.

Pour le string, on ta dit de l'inclure en-haut de ton header, et de mettre std:: devant.

Si je t'ai dit de mettre

Code:

#include "communication.h"

C'est pour dans les endroits où tu l'utilise. Là je ne sais même pas si tu utilises ta classe "communication" dans le main.
13/12/2005, 10h56
hansaplast

ok, ca marche pour le moment...
si je n'avait pas mit

Code:

#include <string>

dans mon fichier .h, c'est parceque je pensait que l'inclure dans le main suffisait : dans chaque fichier, il faut tout re-inclure ?
mais ce n'est pas possible ^^ (a dire avec la voix du grand sthroumf... tiens comment ça s'écrit ça stroumph?)
13/12/2005, 11h17
Fiquet

Citation:

Envoyé par hansaplast

c'est parceque je pensait que l'inclure dnas le main suffisait : dans chaque fichier, il faut tout re-inclure ?

Le compilateur va d'abord compiler chaque classe une par une avant de linker le .exe. Donc il lui faut bien savoir d'ou vient cette variable string, il ne connait pas le main avant la compilation de la classe...

Citation:

Envoyé par hansaplast

tiens comment ça s'écrit ça stroumph?)

Ca s'écrit "stroumpf" mais apparement il n'y a pas que ce mot que tu as du mal à écrire :twisted: essaie de soigner un peu...

Sinon regarde le lien de la FAQ que je t'ai mis à propos des en-têtes standards du C. Et en C++ l'allocation dynamique se fait avec new/delete et pas malloc() (encore une fois voir FAQ)... :?
13/12/2005, 11h50
hansaplast

oki, dsl pour les fautes (croit moi, j'en corrige pourtant avant de poster :'( )

donc, faut tout réinclure a chaque fichier...
c'est pas possible de créer un fichier genre includer.debut
et de faire

Citation:

#include <includer.debut>

a chaque debut de fichier?

ce serait plus pratique non?
13/12/2005, 11h54
Fiquet

Tu peux bien faire un fichier style

Code:

#include "inclusions.h"

où tu mets tous tes #include mais je ne sais pas trop si c'est la meilleure solution, tu n'as pas forcément besoin de tout partout. Mettre une inclusion pour string.h dans chaque classe si tu les utilises me semble pas trop difficile :)

j'ai une autre question :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
// crée une connection UDP, return false si echec
bool communication::setConnectionUDP()
{
	// ********************************************************
    // Ouverture d'une Socket
    // ********************************************************
    this.mySocket = socket(AF_INET,SOCK_DGRAM,0);
    /*d'apres les doc : af_inet : permet d'utiliser IPV4
    *SOCK_STREAM => communication tcp, SOCK_DGRAM => UDP...
    */
    if (this.mySocket == INVALID_SOCKET) {
       return false;
    }
    return true;
}

pour acceder aux données membres d'un objet, je fait :

Code:

this.mySocket

en sachant que je l'ai declaré ainsi :

Code:

SOCKET mySocket;

est-ce une erreure? dois-je y acceder snas la prefixzer par this? (ca me genere une erreure)

13/12/2005, 12h26
Fiquet

Ah là là tu devrais voir la :faq: avant de poser tes questions, tout y est!
http://c.developpez.com/faq/cpp/?pag...EFINITION_this
C'est un pointeur donc:

Code:

this->mySocket

et tu n'est pas obligé de l'employer si c'est un membre de la classe tu peux faire simplement:

Code:

mySocket
13/12/2005, 12h34
hansaplast

:marteau: dsl, j'arrete mes question saoulantes de :bebe:

merci en tout cas :)

Voir 40 message(s) de cette discussion en une page