Dado & operator+(Dado const & d){
int dado2; set(dado2);
d.get()+dado2.get();
return *this;
}
Este operador no va a cumplir con su función por tres motivos:
- Te piden que devuelvas el resultado en un entero y tu estás devolviendo un dado
- El operdor
+
no debe modificar el estado de ninguno de los dos dados. Para esto está el operador +=
- El operador, tal y como está declarado, no es una función miembro de
Dado
, por lo que tendrás que pasarle los dos Dados.
Los dos primeros puntos creo que se entienden bastante bien, si te piden un entero y tu devuelves un objeto de tipo Dado
, la interfaz de la función no es la correcta y sobre modificar objetos, creo que también está claro.
Algunos operadores en C++ pueden (o no) ser miembros de una clase. Las diferencias básicas son las siguientes:
- Si se implementa como función miembro te puedes ahorrar un parámetro.
- Si se implementa como función miembro puedes acceder a las partes protegidas y privadsa.
- Si se implementa como función libre ganas libertad y reduces el acoplamiento.
- Hay casos en los que no se puede implementar el operador como función miembro.
Un ejemplo práctico:
struct POO
{
int offset;
// Operador miembro
int operator+(int n) const
{ return offset + n; }
};
// Función libre, nota que tiene 2 parámetros
int operator-(POO p, int n)
{ return p.offset - n; }
// Función libre
// No se podría implementar como función miembro
int operator+(int n, POO p)
{ std::cout << "operador libre: " << n + p.offset << '\n'; }
int main()
{
POO p;
p.offset = 10;
std::cout << p + 5 << '\n'; // imprime "15"
std::cout << p - 5 << '\n'; // imprime "5"
std::cout << 5 + p << '\n'; // imprime "operador libre: 15"
}
Así pues tu sobrecarga del operador + podrías implementarla de dos formas diferentes:
Función miembro
class Dado{
public:
int operator+(Dado const& d2) const;
};
int Dado::operator+(Dado const& d2) const{
return _valor + d2._valor;
}
Nota que la función termina en const
, esta coletilla fuerza al compilador a mostrar un error en tiempo de compilación si intentas modificar el estado del objeto, por ejemplo haciendo this->_valor = 10;
.
Función libre
int operator+(Dado const& d1, Dado const& d2)
{
return d1.getValor() + d2.getValor();
}
En este caso la función no puede ser const
ya que no pertenece a ninguna clase (por algo es una función libre). Sin embargo ambos parámetros sí que son const
, por lo que no podrás modificarlos.
El resultado en ambos casos es el mismo, puedes elegir el que más te guste.