Discussion :

[xavierdestev]

Bonjour,

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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;
		}

[Invité]

Bonsoir,

Le return est à mettre à gauche :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
class complex{
    // ...
};

Et pourquoi ne pas retourner un bool

[koala01]

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)

[AF_2.8]

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.

[xavierdestev]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

qui me donne l'erreur:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[Médinoc]

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

[xavierdestev]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[Neckara]

Bonjour,

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ;)
}

[Ehonn]

Sinon, lance une exception (?)

[xavierdestev]

Pourtant quand je fais

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if (a==1) return 1;

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

a ? c : d;

[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 :

Sélectionner tout - Visualiser dans une fenêtre à part

a ? b : c;

équivaut à :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[Bousk]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[xavierdestev]

Merci pour vos réponses.

[koala01]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

a ? b : c;

équivaut à :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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