Variable or field 'ecrire' declared void

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...

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...

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...

Tu as essayé de mettre std::string au lieu de string comme Médinoc? Sinon là ça me semble correct.

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...

est-ce que <string> (ou <iostream> dans ton second test) est inclus AVANT ton header ?

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

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é)

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...

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" ?

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" ?

---

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.

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?)

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)... :?

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?

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)

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

---

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

merci en tout cas :)

heu...
en fait, j'en ai encore une...
ce genre de message d'erreure :

Citation:

---

Linking console executable: E:\Mes projets C++\MyVNC\MyVNC.exe
.objs\communication.o:communication.cpp:(.text+0x1ac): undefined reference to `socket@12'
.objs\communication.o:communication.cpp:(.text+0x1fa): undefined reference to `socket@12'
.objs\communication.o:communication.cpp:(.text+0x22f): undefined reference to `htons@4'
.objs\communication.o:communication.cpp:(.text+0x253): undefined reference to `bind@12'
.objs\communication.o:communication.cpp:(.text+0x292): undefined reference to `listen@8'
.objs\communication.o:communication.cpp:(.text+0x2c7): undefined reference to `accept@12'
.objs\communication.o:communication.cpp:(.text+0x307): undefined reference to `recv@16'
.objs\communication.o:communication.cpp:(.text+0x340): undefined reference to `inet_addr@4'
.objs\communication.o:communication.cpp:(.text+0x356): undefined reference to `htons@4'
.objs\communication.o:communication.cpp:(.text+0x37a): undefined reference to `connect@12'
.objs\Main.o:Main.cpp:(.text+0x145): undefined reference to `WSAStartup@8'
.objs\Main.o:Main.cpp:(.text+0x14d): undefined reference to `WSACleanup@0'
collect2: ld returned 1 exit status

---

ca vient d'une erreure de ma part?
(j'utilise winsock) :
pourtant, je fait bien :

Citation:

---

#include <winsock2.h>
#pragma comment(lib, "ws2_32.lib")

---

...

j'ai tiré cet exemple de :http://c.developpez.com/WalrusSock/ (le tuto de walrus)

Si tu ne compiles pas avec VC++, le #pragma comment ne te servira à rien.

Puisqu'apparemment tu compiles avec Dev-C++, il faut aller dans les options du projet et ajouter "-lws2_32" dans la ligne d'édition de liens.

le :boulet: vous dit MER##...
heu...
:merci: quoi ;p

MERCI! (ca marche)

Au fait: si tu es en C++, inclus autant que possible les headers C++. Normalement, tous les headers standard peuvent être remplacés par leur équivalent C++ (windows.h et winsock2.h n'étant pas standard, eux, tu les laisseras en windows.h et winsock2.h)

PS: ça s'écrit Schtroumpf.

Citation:

---

Envoyé par Médinoc

PS: ça s'écrit Schtroumpf.

---

Ah oui juste je l'ai même pas écrit juste :oops: je n'ai pas du assez en lire de ceux-là :lol:

:arrow: