一个c++小程序 求2个数的最大公约数 和最小公倍数的问题

Question

#include<iostream>
using namespace std;
int main()
{
double a,b,c=1,d;//a,b接受数值，c最小公倍数，d最大公约数
cout<<"请输入两个数值：";
cin>>a;
cin>>b;
asd:
if(a*c!=b*c)
{
c++;
goto asd;
}
if(a>=b)
d=a;
else
d=b;
qwe:
if(a/d!=b/d)
{
d--;
goto qwe;
}
cout<<"这两个数的最大公约数和最小公倍数分别是："<<d<<" "<<c<<endl;
return 0;
}
这个程序有什么错误么？
为什么没有输出呢？

Answer 1

1、2个 if 语句判断条件是错的，
if(a*c!=b*c)只要输入a!=b 你的判断永远true 后面那个也是错的
2、goto语句一般都不用了，基本被淘汰了，因为绝大多数的goto 都可以用while或for解决
3、double型没有公约数公倍数之说 只有int 或者 long才有
4、你在 if 里面用 c++ d-- 会即使你判断条件正确也会在最后在执行一次运算 比如 3 和6你会算出 6然后+1输出 7； 这也是goto语句的一个问题
5、#include<iostream>
using namespace std;
int main()
{
int a,b,c=1,d;//a,b接受数值，c最小公倍数，d最大公约数
cout<<"请输入两个数值：";
cin>>a>>b;
for(c=(a>b?a:b);c<=a*b;c++)// c=(a>b?a:b) 取ab较大的数字 三目运算符 " ?: "应该知道
if(c%a==0&&c%b==0)
{
break;
}
for(d=(a<b?a:b);d<=a*b;d--)
if(a%d==0&&b%d==0)
{
break;
}
cout<<c<<"\t"<<d<<endl;
return 0;
}

Answer 2

求最大公约数的方法:两个数求最大公约数，可以用辗转相除法。始终用较大数除以较小数，然后用余数代替较大数。整除时的除数就是最大公约数。
#include "stdio.h"
void main()
{
int a,b,num1,num2,temp;
printf("Please input two numbers:\n");
scanf("%d,%d",&num1,&num2);
if(num1<num2)
{
temp = num1;
num1 = num2;
num2 = temp;
}
a = num1;
b = num2;
while(b!=0)
{
temp = a%b; //取模,即取余数
a=b;
b=temp;
}
printf("最大公约数为:%d\n",a);
printf("最小公倍数为:%d\n",num1*num2/a);
}

Answer 3

你的算法本来就是错的，一开始就死循环了，按下面这个，辗转相除法。

#include<iostream>
using namespace std;
int main()
{
int a,b,c=1,d;//a,b接受数值，c最小公倍数，d最大公约数
cout<<"请输入两个数值：";
cin>>a;
cin>>b;
d = a % b;
while(d != 0) {
a = b;
b = d;
d = a % b;
}
d = b;
c = a / d * b;
cout<<"这两个数的最大公约数和最小公倍数分别是："<<d<<" "<<c<<endl;
return 0;
}

Answer 4

你这算法是怎么生出来的，莫名其妙。
而且还用double这种数据类型，是int吧

用辗转相除法：
void main()
{
int a,b;
int temp;
int sum=a*b;
scanf("%d%d",&a,&b);
if(a<b)
{
temp=a;
a=b;
b=temp;
}
while((temp=a%b)!=0)//辗转相除，当a能被b整除时的b的值就是最大公约数。
{
a=b;//被除数作为被除数
b=temp;//余数作为除数
}
printf("最大公约数：%d\n",b);
printf("最小公倍数：%d\n",sum/b);
}

Answer 5

if(a*c!=b*c)
{
c++;
goto asd;
}
这是死循环，如果 输入的 a 和 b 不相等，那么他们乘以一个相同的数也不会相等的。应该改为if (（c%a==0)&&(c%b==0))才对。
另外：不建议用 goto 语句，这是在C++中被弃用的，是一个不好的习惯，因为大程序里 goto 语句太多，会很混乱很难修改。建议你改为 循环实现。
还有一点，你的算法效率也不高啊。