Ayuda con sobrecarga de operadores

[orlowski]

Hola, hice el siguiente programita:


#include <iostream>
#include <cstdlib>
using namespace std;
class fracciones{
int de,no;
public:
fracciones();
fracciones(int a, int b){
de=a;
no=b;
}
void salida(){
cout<<de<<"/"<<no;
}
fracciones operator *(fracciones c){
return((de*c.de),(no*c.no));
}
};
int main(int argc, char *argv[]) {
int a,b,c,d;
fracciones x,y,g;
cin>>a>>b>>c>>d;
system("CLS");
cout<<a<<"/"<<b<<endl<<c<<"/"<<d;
x=fracciones(a,b);
y=fracciones(c,d);
g=x*y;
g.salida();
return 0;
}


y me da el siguiente error: conversion from 'int' to non-scalar type 'fracciones' requested


Me podrian decir porque? Se que hay otras formas de hacer el ejercicio pero me interesa de esta forma en particular porque quiero aprender otras formas... Gracias por su ayuda!

[margarito04]

el error es sencillo de resolver solo tienes que

fracciones &operator *(fracciones c){
return fracciones((de*c.de),(no*c.no));
}

ademas

fracciones();

lo deberías de poner así
fracciones():de(0),no(0){}

[orlowski]

Iniciado por margarito04

el error es sencillo de resolver solo tienes que

fracciones &operator *(fracciones c){
return fracciones((de*c.de),(no*c.no));
}

ademas

fracciones();

lo deberías de poner así
fracciones():de(0),no(0){}

Che, hice lo que me dijiste pero me sigue dando error...
Que puede ser?
invalid initialization of non-const reference of type 'fracciones&' from a temporary of type 'fracciones'

[fightmx]

Si estás definiendo un número racional toma en cuenta que el denominador debe ser diferente de cero, podrías utilizar un solo constructor:

Código C++:

fracciones(int a = 1, int b = 0):de(a), no(b){
}
 

Con esto tendrías por default al número 0/1 (cero) y no el 0/0 que sería un error matemáticamente hablando.

Me parece que si la variable "de" es el denominador y "no" el númerador deberias invertir el orden de las variables en el código para que sea más claro.

Por otro lado para definir el producto es suficiente con que retornes por valor el resultado de la operación, ya que por referencia al final no obtendrás el comportamiento habitual al hacer las operaciones:

Código C++:

const fracciones operator*(const fracciones& c){
        return fracciones(de * c.de, no * c.no);
}