c++分数类 ，下面代码哪里有问题呢？求高手解释

Question

#include <iostream>
using namespace std;
class fenshu{
public:
int a,b;
fenshu(int a,int b) {
this->a=a;
this->b=b;
}

void print(){
if(a==0) cout<<"此分数为0"<<endl;

else cout<<a<<"/"<<b<<endl;}
};
int yuefeng(int a,int b){ //a,b的最大公约数
int c,min;
min=(a<b?a:b);
for(int i=min;i>1;i--)
{
if (b%i==0&&a%i==0)
{
b=b/i ;a=a/i;
}
}
c=(b/c);
return c;
}
void fenshujia(fenshu& a,fenshu& b){ //此函数为2个分数相加
int j=(a.b)*(b.b);
int k=(a.a*b.b)+(b.a*a.b);
int l=yuefeng(k,j);
cout<<(k/l)<<'/'<<(j/l)<<endl;

}
int main()
{

fenshu a(2,7);
fenshu b(3,14);
fenshujia(a,b);

system("pause");
}

Answer 1

两处错误:

1)c没有赋初值
将约分函数里第一句改成
int c =1,min;
即可
2)c的赋值不对,此处应该是//c=(b/c);不对,
改为

c*=i;并放在循环的判断里边
更正后的程序

#include <iostream>
using namespace std;
class fenshu{
public:
int a,b;
fenshu(int a,int b) {
this->a=a;
this->b=b;
}

void print(){
if(a==0) cout<<"此分数为0"<<endl;

else cout<<a<<"/"<<b<<endl;}
};
int yuefeng(int a,int b){ //a,b的最大公约数
int c =1,min;
min=(a<b?a:b);
for(int i=min;i>1;i--)
{
if (b%i==0&&a%i==0)
{
b=b/i ;a=a/i;
c*=i;
}
}
//c=(b/c);
return c;
}
void fenshujia(fenshu& a,fenshu& b){ //此函数为2个分数相加
int j=(a.b)*(b.b);
int k=(a.a*b.b)+(b.a*a.b);
int l=yuefeng(k,j);
cout<<(k/l)<<'/'<<(j/l)<<endl;

}
int main()
{

fenshu a(2,7);
fenshu b(3,14);
fenshujia(a,b);
system("pause");
}

运行结果

1/2
请按任意键继续. . .

追问

c*=i;并放在循环的判断里边
那这句话是c=c*i么？那就是C=1，那就是说c=i？i是最小的那个公约数啊？应该是最大公约数呀。。

Answer 2

#include <iostream>
using namespace std;
class fenshu
{
public:
int a,b;
fenshu(int a,int b)
{
this->a=a;
this->b=b;
}

void print()
{
if(a==0)
cout<<"此分数为0"<<endl;
else
cout<<a<<"/"<<b<<endl;
}
};
//a,b的最大公约数
int yuefeng(int a,int b)
{
int temp = 0;
int n1, n2;
if (a < b)/*交换两个数，使大数放在num1上*/
{
temp = a;
a = b;
b = temp;
}
n1 = a; n2 = b;
while (n2 != 0)/*利用辗除法，直到b为0为止*/
{
temp = n1 % n2;
n1 = n2;
n2 = temp;
}
return n1;
}
//此函数为2个分数相加
void fenshujia(fenshu& a,fenshu& b)
{
int j = (a.b)*(b.b);
int k = (a.a*b.b) + (b.a*a.b);
int l = yuefeng(k, j);
cout<<(k/l)<<'/'<<(j/l)<<endl;
}
int main()
{
fenshu a(2,7);
fenshu b(3,14);
fenshujia(a,b);

system("pause");
return 0;
}

Answer 3

你以下的函数有问题:
int yuefeng(int a,int b){ //a,b的最大公约数
int c,min;
min=(a<b?a:b);
for(int i=min;i>1;i--)
{
if (b%i==0&&a%i==0)
{
b=b/i ;a=a/i;
}
}
c=(b/c);//执行以上循环以后这里的c = 0;导致你除以一个为0的数,运行时当然出错
return c;
}
还有这个函数有待优化,它执行好多次没有用的循环.
我的修改是:
void yuefeng(int &a,int &b){ //a,b的最大公约数
int c,min;
min=(a<b?a:b);
for(int i=min;i>1;i--)
{
if (b%i==0&&a%i==0)
{
b=b/i ;a=a/i;
}
}
}
此处的修改并没有优化你的多次无用循环,还不懂的Q我:234347828

追问

是说我这个是形参的改变跟实参的没关系么？我只想返回这个c，跟原来的那个应该没关系把？

追答

你返回的c已经是0了,所以用这个方法(引用)直接改变
yuefeng(k, j);中的k和j的值,然后输出.你通过调试看得到c = 0,还有可以看到好多次无用循环;
c为什么会等于0呢?因为执行第一次循环后,a=1,b=2,往后的循环都没有用了,然后到了这句:
c=(b/c);这个为什么是0,应该是0.5啊,因为c是int型,(int) 0.5 = 0;

记得再把你的
void fenshujia(fenshu& a,fenshu& b)

函数里面的
cout<<(k/l)<<'/'<<(j/l)<<endl;
改为:
cout<<k<<'/'<<j<<endl;
就行了

Answer 4

c=(b/c);

//c没有赋初值

Answer 5

看看先