Tính toán cộng trừ nhân chia trên phân số viết bằng C++ Class

[child2606]

bài này là chương trình em viết để + - * / và so sánh 2 phân số, khi chạy chương trình với các số nhỏ và tử khác 0 thì không có vấn đề gì. Nhưng khi nhập tử là 0 hoặc các số lớn( 12/17 và 23/21 chẳng hạn) thì nó lại không chạy. Em chẳng biết là sao nữa. giúp dùm em với...

Đây là code mà em viết:

#include "iostream.h"#include "conio.h" class PhanSo{private:    int iTu;    int iMau;public:    void Nhap();    void Xuat();    int KiemTra ();    PhanSo Cong (const PhanSo &);    PhanSo Tru (const PhanSo &);    PhanSo Nhan (const PhanSo & );    PhanSo Chia (const PhanSo &);    int SoSanh(const PhanSo &);}; void PhanSo::Nhap(){    cout<<"+Nhap tu: ";    cin>>this->iTu;     cout<<"+Nhap mau: ";    cin>>this->iMau;     if(this->iTu == this->iMau)        this->iTu=this->iMau=1;} void PhanSo::Xuat(){    if(this->iTu == 0)        cout<<"0\n";    else    {        if((this->iTu != 0) && (this->iMau == 1))            cout<<this->iTu<<"\n";        else        {            if((this->iTu != 0) && (this->iMau != 0))                cout<<this->iTu<<"/"<<this->iMau<<"\n";        }    }} int RutGon(int a,int b){    while((a!=0)&&(b!=0))    {        if(a>=b)        a-=b;        else        b-=a;    }    if(a==0)            return b;    else        return a; } int PhanSo::KiemTra(){    return(this->iMau != 0)?1:0;} PhanSo PhanSo::Cong(const PhanSo &a){    PhanSo c;     c.iTu = (this->iTu * a.iMau) + (this->iMau * a.iTu);    c.iMau = this->iMau * a.iMau;     int k =RutGon(c.iTu, c.iMau);    c.iTu=c.iTu/k;    c.iMau=c.iMau/k;     return c;} PhanSo PhanSo::Tru(const PhanSo &a){    PhanSo c;     c.iTu = (this->iTu * a.iMau) - (this->iMau * a.iTu);    c.iMau = this->iMau * a.iMau;     int k =RutGon(c.iTu, c.iMau);    c.iTu/=k;    c.iMau/=k;     return c;} PhanSo PhanSo::Nhan(const PhanSo &a){    PhanSo c;     c.iTu = this->iTu * a.iTu;    c.iMau = this->iMau * a.iMau;     int k =RutGon(c.iTu, c.iMau);    c.iTu=c.iTu/k;    c.iMau=c.iMau/k;     return c;} PhanSo PhanSo::Chia(const PhanSo &a){    PhanSo c;     c.iTu = this->iTu * a.iMau;    c.iMau = this->iMau * a.iTu;     int k =RutGon(c.iTu, c.iMau);    c.iTu/=k;    c.iMau/=k;     return c;} int PhanSo::SoSanh(const PhanSo &a){    int tong;     tong = (this->iTu * a.iMau) - (this->iMau * a.iTu);     if(tong <0)        return -1;    if(tong == 0)        return 0;    if(tong > 0)        return 1;} void main(){    PhanSo a, b, c, d ,e, f;lap_lai:    cout<<"\tNHAP PHAN SO THU 1: \n";    a.Nhap();    a.KiemTra();    int i = a.KiemTra();    if(i==1)    {        cout<<"\tNHAP PHAN SO THU 2: \n";        b.Nhap();        b.KiemTra();         int j = b.KiemTra();        if(j == 1)        {            cout<<"-------------KET QUA-------------\n";            cout<<"Tong hai phan so la: ";            c = a.Cong(b);            c.Xuat();             cout<<"Hieu hai phan so la: ";            d = a.Tru(b);            d.Xuat();             cout<<"Tich hai phan so la: ";            e = a.Nhan(b);            e.Xuat();             cout<<"Thuong hai phan so la: ";            f = a.Chia(b);            f.Xuat();             a.SoSanh(b);            int i = a.SoSanh(b);            if(i == -1 )                cout<<"Phan so thu nhat nho hon phan so thu hai"<<endl;            else            {                if(i == 0)                    cout<<"Hai phan so bang nhau"<<endl;                else                    cout<<"Phan so thu nhat lon hon phan so thu hai"<<endl;            }        }        else         {            cout<<"Ban nhap mau so sai, nhap lai\n";            goto lap_lai;        }    }    else    {        cout<<"Ban nhap mau so sai, nhap lai\n";        goto lap_lai;    }}

[nhc1987]

Mình có vài ý về bài làm của bạn như sau:

+ Đặt tên hàm ko đúng ý nghĩa (hàm Rutgon theo mình nghĩ là bạn đang tìm Bội số chung nhỏ nhất và hàm này bạn code sai)
+ Tất cả các toán tử trên phải để là hàm friend, ko nên để trong lớp, rất tối nghĩa. Và nếu đã là toán tử thì ai lại dùng hàm như thế, phải dùng operator chứ
+ Bạn mất công tạo đối tượng, khởi tạo đối tượng để rồi sau đó kiểm tra đối tượng có lỗi hay ko. Do ở đây chỉ là lớp PhanSo còn nhỏ, lỡ bạn code một cái lớp lớn thì ... ~~~> chuyển thành kiểm tra khi dữ liệu được nhập
+ Có nhiều lỗi chưa bắt, kiểm tra chưa chặt ...

Thân.

[ABC_HT]

bool operator > (CPhanSo p)    {        return tu*p.mau > mau*p.tu;    };class CPhanSo  {public:        CPhanSo(int t = 0, int m = 1):tu(t) ,mau(m){RutGon();};     void InRa() { cout <<tu<<"/"<<mau; };        CPhanSo operator ++()    {        tu = tu + mau ;    return CPhanSo (tu,mau);    };     CPhanSo operator ++(int)    {            int tam = tu;    tu = tu + mau;    return CPhanSo (tam,mau);    };    bool operator> (CPhanSo p)    {        return tu*p.mau > mau*p.tu;    }; private:    int tu;    int mau;    CPhanSo RutGon()    {        int a = tu , b = mau;        while (b>0)        {            int r = a % b;            a = b;            b = r;        }        tu = tu / a;        mau = mau / a;        return *this;    }; };