[C++] Overload dell'operatore "==" in una classe

[giuseppe500]

ciao.
ho provato a fare un oveerload dell' operatore == di una mia classe cosi:

class Vertex
{
public:
bool operator ==(Vertex other);
}


questo funziona ad es se faccio:

Vertex v1,v2;
if(v1==v2).....


ma non se ho:
Vertex * v1,v2
if (v1==v2)....


come faccio a fare l'overload di == su due instanze della stessa classe?
Grazie.

[XWolverineX]

Non torna il discorso
V1 è un puntatore, V2 no, dunque non può funzionare.

[MacApp]

Originariamente inviato da giuseppe500

ma non se ho:
Vertex * v1,v2
if (v1==v2)....


come faccio a fare l'overload di == su due instanze della stessa classe?
Grazie.

sostitusci con:

Vertex * v1,v2
if ((*v1)==v2)....

[giuseppe500]

grazie.
Ma non ho capito perchè mcapp,e non c 'è un modo di fare l'overload dell' operatore == tra due instanze della stessa classe.
poi ,ho scritto male,

Vertex* v = new Vertex();
Vertex* v1 = new Vertex();

if(v1==v2)
.
.
.
non

Vertex * v1,v2 e
if(V1==V2)

[MacApp]

quindi:

if ((*v1)==(*v2))....

Cosa intendi per:

fare l'overload dell' operatore == tra due instanze della stessa classe

?

[giuseppe500]

Sto utilizzando le collezioni dell'mfc.
In particolare mi è stato consigliato l' std::set.
Devo inserire un oggetto nella collezione set solo se non è gia' presente un oggetto con gli stessi parametri.
sto cercando di fare l'overload dell'operatore == di una struttura(che è il tipo della collezione) per stabilire qual ' è il criterio di ugualianza utilizzato dal set.find in modo da rendere uguali due oggetti se hanno 3 parametri uguali:
Normalx,
Normaly e
Normalz,.
Per questo devo creare un overload cosi':

bool Vertex:perator==(Vertex* other)
{
if(this->_nx == other._nx && this->_ny == other._ny && this->_nz == other._nz)
return true;
return false;
}

Almeno spero di aver capito bene.
Ciao

[MItaly]

Dovrebbe essere

bool Vertex::operator==(Vertex& other)

.
P.S.: std::set non è un container MFC, ma STL.

[MacApp]

prova così (da notare che ho aggiunto i "const"):

bool Vertex::operator==(const Vertex &other) const{
	return (this->_nx == other._nx) && (this->_ny == other._ny) && (this->_nz == other._nz);
}

p.s: ma perché non usi i tag code?
p.s2: evita di definire identificatori che iniziano con l'underscore. Tali identificatori sono riservati all'implementazione sia del linguaggio che del compilatore.

[giuseppe500]

ti posto il codice,non riesco a farlo funzionare:

#include "stdafx.h"
#include <iostream>
#include <set>
#include "struct2s.h"
using namespace std;

int main ()
{
  set<Vertex*> myset;
 
  // set some initial values:
  Vertex vz,vy;

  Vertex* v = new Vertex();
  Vertex* v1 = new Vertex();


  v->_nx =1;
  v1->_nx =1;

  myset.insert(v);
  
  if(*v==*v1)
	  cout<<" uguale";
  else
	  cout<<" diverso";
  
  if(myset.find(v1)== myset.end())
	  cout<<" non presente";
  else
	  cout<<" gia presente";
  int r;
  cin >> r;
  return 0;
}

struct Vertex
{
	Vertex(){}
	Vertex(float x, float y, float z, 
		float nx, float ny, float nz, float u, float v)
	{
		 _x = x;   _y = y;   _z = z;
		_nx = nx; _ny = ny; _nz = nz;
		 _u = u;   _v = v;
	}

	float _x, _y, _z, _nx, _ny, _nz, _u, _v;
	bool operator ==(Vertex& other);

	//static const DWORD FVF;
};

bool Vertex::operator==(Vertex& other)
{
	if(this->_nx == other._nx)
		return true;
	return false;
}

if(*v==*v1)
entra nell overload ritorna true ed è giusto , ma..

if(myset.find(v1)== myset.end())
cout<<" non presente";
else
cout<<" gia presente";
int r;

non entra nella funzione di overload,ritorna true e non dovrebbe , mi aspetterei un false in quanto è già presente un oggetto con le stesse caratteristiche:
v->_nx =1;
v1->_nx =1;
myset.insert(v);//inserito

ciao.

[MacApp]

il contenuto del tuo "set" sono puntatori. Quindi il confronto in set::find viene fatto sul valore numerico del puntatore, non sull'oggetto puntato.
Devi fornire pertanto al "set" un funtore adatto:

ad esempio:

//
// Compiled and tested with:
// i686-apple-darwin8-gcc-4.0.1 (GCC) 4.0.1 (Apple Computer, Inc. build 5367)
//
// Compilation command line (zero errors, zero warnings):
// $ g++ -Wall -ansi -pedantic main.cpp
// 

#include <iostream>
#include <set>
using namespace std;

struct Vertex
{
    Vertex(){}
    Vertex(float x, float y, float z, 
        float nx, float ny, float nz, float u, float v)
    {
         _x = x;   _y = y;   _z = z;
        _nx = nx; _ny = ny; _nz = nz;
         _u = u;   _v = v;
    }

    float _x, _y, _z, _nx, _ny, _nz, _u, _v;
    bool operator ==(const Vertex& other) const;
    bool operator < (const Vertex& other) const;

    //static const DWORD FVF;
};

bool Vertex::operator==(const Vertex& other) const{
    return this->_nx == other._nx;
}

bool Vertex::operator<(const Vertex& other) const{
    return this->_nx < other._nx;
}

namespace ig{
    template <typename TClass>
    struct SLessPointerFunctor{
        bool operator ()(const TClass * v1,  const TClass * v2) const{
            return *v1 < *v2;
        }    
    };
}

int main ()
{
  set <Vertex *, ig::SLessPointerFunctor <Vertex> > myset;
 
  // set some initial values:
  // Vertex vz,vy;

  Vertex* v = new Vertex();
  Vertex* v1 = new Vertex();


  v->_nx =1;
  v1->_nx =1;

  myset.insert(v);
  
  if(*v==*v1)
      cout<<" uguale";
  else
      cout<<" diverso";
  
  if(myset.find(v1)== myset.end())
      cout<<" non presente";
  else
      cout<<" gia presente";
  int r;
  cin >> r;
  return 0;
} 

osserva che le due parentesi acute prima di myset DEVONO essere separate da uno spazio, altrimenti vengono interpretate come operatore di shift.
Il funtore ig::SLessPointerFunctor come vedi è generico, e probabilmente è già presente nella libreria std in zona std::less, ma non ho voglia di cercarlo ;-)

EDIT: generalizzando ancor di più:

#include <functional>
...
namespace ig{
    template <typename TBinaryPredicate, typename TClass>
    struct SPointerFunctor{
        bool operator ()( const TClass * v1,  const TClass * v2) const{
            TBinaryPredicate a;
            return a (*v1, *v2);
        }    
    };
}
...
  set <Vertex *, ig::SPointerFunctor <less <Vertex>, Vertex> > myset;