Класс "Дроби"

[JeyKip]

Всем доброе время суток!!! Помогите, пожалуйста, исправить код программы, которая задает класс "Дроби" и включает в себя операции сложения, вычитания, деления, умножения и поиск наименьшего общего знаменателя при сложении и вычитании...исходник работает, но препод говорит, что не тот принцип построения...

#include <iostream>
#include <conio.h>
using namespace std;

class Drobi {
private:
	int x1,y1,x2,y2;
public:
	Drobi(int a, int b, int e, int f)
	{
	x1=a;
	y1=b;
	x2=e;
	y2=f;
	}

float summa (float c, float d, float k)
{
	return (c+d)/k;
}

float raznost (float c, float d, float k)
{
	return (c-d)/k;
}

float proizv (float x1, float y1, float x2, float y2)
{
	return ((x1/y1)*(x2/y2));
}

float chastnoe (float x1, float y1, float x2, float y2)
{
	return ((x1/y1)/(x2/y2));
}
};
void main ()
{
	setlocale (0,"");
	int a,b,e,f;
	cout<<"Введите числитель первой дроби: ";
	cin>>a;
	cout<<endl;
	cout<<"Введите знаменатель первой дроби: ";
	cin>>b;
	cout<<endl;

	cout<<"Введите числитель второй дроби: ";
	cin>>e;
	cout<<endl;
	cout<<"Введите знаменатель второй дроби: ";
	cin>>f;
	Drobi A(a,b,e,f);
	cout<<endl;
int c=0;
int s1,s2;
	for (int i=1;c!=1; i++)
	{
		if ((i%b==0)&&(i%f==0))
		{
			c=1;
			s1=a*(i/b);
			s2=e*(i/f);
			cout<<"\nПервая дробь: "<<a<<"/"<<b<<"\n";
		cout<<"\nВторая дробь: "<<e<<"/"<<f<<"\n";
		cout<<"\nНаименьший общий знаменатель = "<<i<<endl;
		cout<<"\nСумма дробей = ("<<s1<<"+"<<s2<<")/"<<i<<" = "<<A.summa(s1,s2,i)<<endl; 
		cout<<"\nРазность дробей равна = ("<<s1<<"-"<<s2<<")/"<<i<<" = "<<A.raznost (s1,s2,i)<<endl;
		}
	}
	
	cout<<"\nПроизведение дробей равно = "<<A.proizv (a,b,e,f)<<endl;
	cout<<"\nЧастное дробей равно = "<<A.chastnoe (a,b,e,f)<<endl;
	getch();
}

[netrino]

Вероятнее всего имелось ввиду, что класс должен представляет одну единственную дробь, в то время как у вас он представляет сразу две дроби. Операции сложения, вычитания, умножения и деления скорее всего надо было делать перегрузкой соответствующих операторов(если вы такое уже проходили, естественно). То есть примерно так:

class Drobi {
    int chislitel;
    unsigned int znamenatel;
public:
    Drobi();
    Drobi(int);
    Drobi(int, unsigned int);
//  Drobi(double); // Это в общем не обязательно, но если реализуете, то, вероятно, положительно скажется на оценке :)

    Drobi operator+ (Drobi);
    Drobi operator+ (int);
//    Drobi operator+ (double);
    Drobi operator- (Drobi);
    Drobi operator- (int);
//... то же самое для *, /

Реализацию сами опишите. Должна быть возможность использовать так:

int main()
{
    Drobi n(2, 3);
    Drobi m(3, 2);

    Drobi l = m + n;
    l = l / 2;

    Drobi d(10);
    d = d * l;

    return 0;
}

Желательно также реализовать операторы *=, /=, +=, -=, а также перегрузить хотя бы оператор вывода для ostream( << ). Нахождение наименьшего общего знаменателя должно быть также методом класса, имхо.

[mishutka999]

Мы писали такую прогу. Вот она:

class Rational
	{int a,b; //a-числитель b-знаменатель//
public: 
		Rational (int=0, int=1);
		Rational Add (const Rational&) const;
		Rational Sub (const Rational&) const;
		Rational Mult (const Rational&) const;
		Rational Div (const Rational&) const;
		void Print () const;
		void Change (int,int);
private:
	int Nod (int, int) const;
};

Rational::Rational (int chislit, int znamenat)
	{a=chislit;
	if(znamenat)
		b=znamenat;
	else
		{cout<<"Ne mogu podelit na ZERO"<<endl;
		exit(1);}
	}
Rational Rational::Add (const Rational&s) const
	{
	int c;
	Rational Sum;
	Sum.a=a*s.b+b*s.a;
	Sum.b=b*s.b;
	if (Sum.a>Sum.b)
		c=Nod (Sum.b, Sum.a);
	else
	c=Nod (Sum.b, Sum.a);
	Sum.a=Sum.a/c;
	Sum.b=Sum.b/c;
	return Sum;
	}

Rational Rational::Sub (const Rational&s) const
	{
	int c;
	Rational Sum;
	Sum.a=a*s.b-b*s.a;
	Sum.b=b*s.b;
	if (Sum.a>Sum.b)
		c=Nod (Sum.b, Sum.a);
	else
	c=Nod (Sum.b, Sum.a);
	Sum.a=Sum.a/c;
	Sum.b=Sum.b/c;
	return Sum;
	}

Rational Rational::Mult (const Rational&s) const
	{
	int c;
	Rational Sum;
	Sum.a=a*s.a;
	Sum.b=b*s.b;
		if (Sum.a>Sum.b)
		c=Nod (Sum.b, Sum.a);
	else
	c=Nod (Sum.b, Sum.a);
	Sum.a=Sum.a/c;
	Sum.b=Sum.b/c;
	return Sum;
	}

Rational Rational::Div (const Rational&s) const
	{
	int c;
	Rational Sum;
	Sum.a=a*s.b;
	Sum.b=b*s.a;
	if  (s.a==0)
		{cout<<"nelzia delit na Zero"<<endl;
		exit (1);};
	if (Sum.a>Sum.b)
		c=Nod (Sum.b, Sum.a);
	else
	c=Nod (Sum.b, Sum.a);
	Sum.a=Sum.a/c;
	Sum.b=Sum.b/c;
	return Sum;
	}
void Rational::Print () const
	{cout<<a<<"/"<<b<<endl;}
void Rational::Change (int d, int e)
	{int f;
	a=d;
	if (e)
		b=e;
	else
	{cout<<"ne mogu delit na zero"<<endl;
	exit (1);};
	if (a>b)
		f=Nod(a,b);
	else 
		f=Nod(b,a);
	a=a/f;
	b=b/f;
	}

int Rational::Nod (int c, int d) const
	{
	if (d==0)
		return c;
	if (c%d==0)
	return 1;
	return Nod (d, c%d);
	}

int main ()
{
	Rational R1 (7,11), R2 (0,7), R3;
R3=R1.Add(R2);
R1.Print();
R2.Print();
R3.Print();

R3=R1.Sub(R2);
R3.Print();
R3=R2.Sub(R1);
R3.Print();
R3=R1.Mult(R2);
R3.Print();
R3=R1.Div(R2);
R3.Print();



return 0;
}

[ahab]

во всех функциях-членах в роли параметра

const Rational&s

обьясните пож. почему именно в таком виде? где об этом можно поподробней почитать?
мы в универе тоже делали класс дробь, перегружали операторы, и тоже параметр такого вида, но что-то не пойму почему такого? что и как
спасибо

[netrino]

& указывает, что нам нужна ссылка на объект, а не копия объекта(чтобы не тратить времени на копирование), а const говорит, что объект по этой ссылке менять нельзя. Если не будет const, то действия над объектом по ссылке смогут изменить его состояние(поля) и эту ошибку трудно будет отловить

[ahab]

?
в общем как следствие выполнения функции-члена, эта самая функция возращает константную сслыку (допустим) &s типа Rational (наш класс дробь)
???
спасибо

[netrino]

Сообщение от ahab

?
в общем как следствие выполнения функции-члена, эта самая функция возращает константную сслыку (допустим) &s типа Rational (наш класс дробь)
???
спасибо

Нет, константную ссылку метод принимает в качестве параметра, возвращает он новый объект.
Принимает ссылкой, чтобы работало быстрее, константной, чтобы нигде случайно не изменить объект, что по ссылке.

[ahab]

точно )
тормознул, спасибо огромное