Variable enum et unqualified-id

Version imprimable

Bonjour,
Voici la portion de code qui concerne ce probleme:

Intervalle.cxx
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 Intervalle::Intervalle (double a, double b, char* e) { if (e!="EMPTY" && e!="NO_EMPTY") { cout<<"Erreur sur le 3e parametre : sa valeur doit etre EMPTY ou NO_EMPTY.\n"<<endl; } else { if (min2(a,b)<-1000000 || max2(a,b)>1000000) { cout<<"Erreur sur les bornes : doivent etre comprises entre -1000000 et +1000000"<<endl; } else { inf=min2(a,b); sup=max2(a,b); IntervalleType=e; } } }
Intervalle.hxx
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 #ifndef INTERVALLE #define INTERVALLE class Intervalle { private: double inf; double sup; enum IntervalleType { EMPTY = 0, NO_EMPTY = 1 }; public: Intervalle(double, double, char*); Intervalle operator + (const Intervalle&); Intervalle operator - (const Intervalle&); Intervalle operator * (const Intervalle&); Intervalle operator / (const Intervalle&); void AfficheIntervalle (void); void IntervalleInitialise (double, double, char*); }; #endif
Probleme:

Le compilateur me dit pour la ligne en rouge:

Code:

Intervalle.cxx:49: error: expected unqualified-id before ‘=’ token

Il y a aussi d'autres erreurs de compilation, mais que je prefere chercher tout seul pour l'instant.

C'est a travers Internet que j'ai appris a utiliser les enum, mais apparemment j'ai zappe quelque chose. Pouvez-vous eclairer ma lanterne ?

Merci d'avance

21/03/2011, 16h55
AF_2.8

Concernant le fait de montrer qu'une seule erreur de compil tu peux mais seulement si c'est la première erreur. Car des fois certaines erreurs disparaissent après avoir corrigé la première.

Concernant ton histoire d'enum, tu affectes un char* à un enum de type int. Pour moi c'est là que ce situe le problème.
De même concernant tes tests entre un char* et une chaine de caractères, j'avoue que celà me parait bizarre (tu n'as pas une erreur de compil la dessus ).
21/03/2011, 17h38
User Name

En fait, le prof me dit dans son sujet: "IntervalleType est de type enum et determine si l'intervalle est vide ou non en prenant les valeurs EMPTY et NO_EMPTY".

Si l'enum est un enum de int, comment l'associer a EMPTY et NO_EMPTY?

Et pour la comparaison de char* et string, j'ai effectivement des "Warnings" mais pas des erreurs. Ayant compris que "techniquement" c'est la meme chose, je ne vois pas non-plus pourquoi. Cela dit, je n'arrive pas a utiliser "string" : meme avec #include <string>, il ne connait pas...

PS : Je suis sous Ubuntu avec g++ et desole pour l'absence d'accents dans mes posts, mais je suis sous Linux avec un Qwerty, donc...
21/03/2011, 20h19
AF_2.8

Les strings et les char ne sont pas la même chose, les string c'est du C++ les char c'est un reste du C.
Pour pouvoir utiliser les string il faut que tu déclare leur espace de nimage c'est à dire :

Code:

std::string

si tu ne veux pas utiliser la résolution de l'espace de nomage std:: alors il faut que tu mettes dans ton code (dans ton .cxx par exemple :

Code:

using namespace std;

Un enum te permet d'utiliser des noms de variables plutôt que des numéros. Tu dois donc utiliser directement NO_EMPTY et EMPTY dans ton code sans les ". Par contre tu dois les comparer à integer et non un char.
L'intérêt des enum est que plutoyt d'avoir a se souvenir que 0 signifie EMPTY, on l'appel directemet EMPTY, le compilateur remplacera EMPTY par 0 .
Jette un oeil ici

Salut,

Ton énumération est de type... IntervalleType. La signature correcte de ta fonction devrait donc être proche de

Code:

Intervalle(double, double, IntervalleType);

ou, sous la forme "pleinement qualifiée", proche de

Code:

Intervalle::Intervalle(double, double, Intervalle::IntervalleType);

ce qui te permettra, lors de la déclaration d'une variable, d'écrire quelque chose proche de
Code:

1 2 3 4 int main() { Intervalle i(2.3, 3.1415926, Intervalle::NO_EMPTY); }
Les valeurs énumérées sont, quant à elle, des numériques entières, ce qui fait qu'il ne faut pas les comparer avec des chaines de caractères, mais bel et bien avec... des valeurs numériques entières :P)
La première partie de ta fonction devrait donc prendre la forme de
Code:

1 2 3 4 if (e!=EMPTY && e!=NO_EMPTY) { cout<<"Erreur sur le 3e parametre : sa valeur doit etre EMPTY ou NO_EMPTY.\n"<<endl; }
(Penses, en outre, au fait que, si tu écris "EMPTY" ou "NO_EMPTY", tu te trouve face à des chaines de caractères "C style", et qu'il faudrait donc les comparer avec les fonctions strcmp et similaires, sous peine de comparer... la valeur de l'adresse à laquelle se trouve le premier caractère de chaque chaine :P)

Enfin, même si ce n'est qu'un détail, il serait sans doute préférable de lancer une exception qui permettrait de récupérer l'erreur survenue plutôt que de demander directement au constructeur d'envoyer quelque chose vers la sortie standard : tu n'es en effet meme pas sur, en créant ta classe, que "ce qui va l'utiliser" dispose, effectivement, d'une sortie standard :P

En outre, l'envoi de données vers la sortie standard (ou vers toute autre sortie d'ailleurs) est une responsabilité qui n'a absolument pas à échoir ni au constructeur (de manière particulière) ni même à l'objet (de manière générale) :P

Et, comme, de plus, si une erreur survient, il s'agit d'éviter la création de l'objet, cela te permettrait de simplifier ta logique sous une forme proche de
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 //l'exception lancée si on s'est trompé dans l'intervalle struct BadType: public std::exception { const char * what() const throw(){return "Erreur dans le type d'intervale";} ~BadType() throw(){} }; // l'erreur lancée si on s'est trompé dans les valeurs struct BadValues : public std::sexception { const char * what() const throw(){return "Erreur sur les bornes : doivent etre comprises entre -1000000 et +1000000";} ~BadValues () throw(){} } //le constructeur de intervale Intervalle::Intervalle(double v1, double v2, Intervalle::IntervalleType t) { if(t!=EMPTY && t!= NO_EMPTY) throw BadType(); if (min2(a,b)<-1000000 || max2(a,b)>1000000) trhow BadValues(); inf=min2(a,b); sup=max2(a,b); IntervalleType=e; }
et de l'utiliser sous une forme proche de
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 int main() { Intervalle::IntervalleType t= /* détermination de l'intervale type */ double v1 = /* détermination de la première valeur */ double v2 = /* détermination de la seconde valeur */ try { Intervalle i(v1, v2, t ); /* ... */ } catch( std::exception & e) { /* c'est ici qu'il faut, si nécessaire, provoquer l'affichage */ std::cout<<e.what(); } }

26/03/2011, 21h00
User Name

Merci pour ces explications. Effectivement, j'ai declare l'enum en tant que variable globale publique, puis je rentre en parametre de mon constructeur un int. Et ca marche !

Pour tes conseils sur les exceptions et compagnie, je ne suis malheureusement pas encore assez a l'aise avec le langage pour les suivre. Mais je les garde sous la main pour une utilisation ulterieure.

Merci beaucoup.