classe complexe surdéfinition de ==

Bonjour,

Je cherches à surdéfinir l'opérateur == pour la classe complexe.
Le code suivant donne l'erreur:

Code:

---

1 2 3

R2.hpp:76:32: erreur: expected primary-expression before ‘return’ R2.hpp:76:32: erreur: expected ‘:’ before ‘return’

---

Code:

---

1 2 3 4 5 6 7 8

class complex{ public: double x; double y; }   int operator==(complex z){ x == z.x && y == z.y? return 1 : return 0; }

---

Bonsoir,

Le return est à mettre à gauche :

Code:

---

1 2 3

int operator==(complex z){ return x == z.x && y == z.y ? 1 : 0; }

---

J'imagine que c'est une erreur de recopie, mais ce bout de code tel quel est à mettre à l'intérieur de :

Code:

---

1 2 3

class complex{ // ... };

---

Et pourquoi ne pas retourner un bool :question:

Salut,

Pour que ton code fonctionne, il faudrait que l'opérateur == soit une fonction membre (et de préférence constante) de ta classe, or, là, il s'agit d'une fonction libre.

Comme cette fonction libre n'a strictement rien à voir avec ta classe complex, elle ne sait meme pas où aller trouver les variables x et y avec lesquelles tu tente de comparer les valeurs z.x et z.y.

Tu as donc deux solutions :

Soit, tu "remonte" l'opérateur directement dans la classe (avant l'accolade fermante), soit tu transmet deux élément de type complex à ton opérateur == et tu compare les valeurs entre elles ;)

Pour information :

C'est sans doute un oubli de ta part, mais il manque le ";" final après la déclaration de ta classe complex (juste après l'accolade fermante)

Généralement, on essaye de passer les classes et structures par référence (constante, si l'objet ne doit pas être modifié par la fonction appelée), pour éviter les copies inutiles

Les opérateurs de comparaison renvoient classiquement un booléen, qui est un type particulier en C++ (et qui existe d'ailleurs maintenant dans la dernière norme de C, me semble-t-il)

Question bête : La classe <complex> qui fait partie du standard C++, ne répond pas à tes besoins, car dans cette classe tu as déjà ce qu'il faut pour comparer des nombres complexes.

Merci.
ns le meme ordre d'idée j'ai:

Code:

---

return power (w, i)==z ? atan2(imag(z), real(z)) : ;

---

qui me donne l'erreur:

Code:

---

main.cpp:19:78: erreur: expected primary-expression before ‘;’ token

---

PS: Je suis sensé programmer ma propre classe complexe à titre d'exercice.
Sinon la classe complex par défault marche très bien je penses.

Ben oui c'est normal, une clause de ton opérateur ternaire est vide...

Mais j'ai besoin de ne rien retourner justement.
Comme dans :

Code:

---

if (a==1) return 1; else ;

---

Bonjour,

Citation:

---

Envoyé par xavierdestev

Mais j'ai besoin de ne rien retourner justement.
Comme dans :

Code:

---

if (a==1) return 1; else ;

---

---

Ceci n'a aucun sens, imaginons une fonction foo retournant un int :

Code:

---

1 2 3 4 5 6 7

int foo(void) { if( jeSuiSGentilEtDeBonneHumeur) return 1; else // et bien je retourne rien na ! ; }

---

Ce code là compilera mais tu auras un joli warning (c'est trop permissif ça ne devrait même pas compiler), en effet, foo doit retourne quelle valeur si on entre dans le else ? 0 ? 1 ? 2? Il ne peut pas "rien" retourner, il doit retourner un int.
D'ailleurs si tu ne met pas de return, je pense qu'il retourne une valeur indéterminée :aie:

De même pour l'opérateur ternaire, tu dois "retourner" une valeur, peut importe laquelle, tu dois avoir une valeur.


Attention, ceci est tout à fait correct :

Code:

---

1 2 3 4 5 6 7 8 9 10

int foo(void) { // do some stuff if( condition ) return 1; // j'aime bien retourner 1 else ;//do some stuff //do some stuff return 1; // oui, j'aime bien retourner 1 ;) }

---

Sinon, lance une exception (?)

Pourtant quand je fais

Code:

---

if (a==1) return 1;

---

sans rien d'autre ca mache bien.
Je veux juste faire ca avec

Code:

---

a ? c : d;

---

Pourquoi chercher midi à 14h ?
"result = a ? c : ;" => if( a ) result = c;Sinon tu n'a pas compris que l'opérateur ternaire n'est pas équivalent à un if/else mais plutôt :

Code:

---

a ? b : c;

---

équivaut à :

Code:

---

1 2 3 4 5 6 7

type foo(bool a, type b, type c) { if(a) return b; else return c; }

---

Et ta fonction foo a besoin de retourner un type, elle ne peut pas ne rien retourner.

Citation:

---

Envoyé par xavierdestev

Mais j'ai besoin de ne rien retourner justement.
Comme dans :

Code:

---

if (a==1) return 1; else ;

---

---

C'est strictement impossible de ne rien retourner en C++.
Si ta méthode doit retourner quelque chose, elle doit toujours retourner quelque chose. Ou générer une exception.

Merci pour vos réponses.

Citation:

---

Envoyé par Neckara

Pourquoi chercher midi à 14h ?
"result = a ? c : ;" => if( a ) result = c;Sinon tu n'a pas compris que l'opérateur ternaire n'est pas équivalent à un if/else mais plutôt :

Code:

---

a ? b : c;

---

équivaut à :

Code:

---

1 2 3 4 5 6 7

type foo(bool a, type b, type c) { if(a) return b; else return c; }

---

Et ta fonction foo a besoin de retourner un type, elle ne peut pas ne rien retourner.

---

Et encore, avec l'opérateur ternaire, le type de retour doit correspondre (comprends : être, au minimum compatible) entre ce qui est retourné si la condition est évaluée à true et ce qui est retourné si la condition est évaluée à false.

Avec un code proche de

Code:

---

1 2 3 4

std::string str("salut"); int i(153); bool condition(false); return condition ? str : i ;

---

le compilateur t'enverra sur les roses car il n'y a aucune compatibilité entre un entier et une chaine de caractères ;)