Question

请学过c++的人帮忙看看。这道题我这么编哪里错了？

题目：对给定的两组数（每组包括2个整数），分别计算出两组数的最大公约数和最小公倍数的和。如：20、12为第一组，28、7为第二组。第一组数的最大公约数为4，最小公倍数为60；第二组数的最大公约数为7，最小公倍数为28；所以两组数的最大公约数之和为11，两组数的最小公倍数之和为88。
#include <stdio.h>
int main()
{int first (int u,int v);
int second (int a,int b);
int a,b,u,v,z;
scanf("%d%d",&u,&v);
printf("\n");
scanf("%d%d",a,b);
z=first(u,v)+second(a,b);
printf("%d",z);
return (0);}

int first (int u,int v)
{int r,t; if (v>u)
{t=v;v=u;u=t;}
while ((r=u%v)!=0);
{r=u%v;u=v;v=r;}
return (v);}

int second(int a,int b)
{int r,t;if (b>a)
{t=a;b=a;a=t;}
while ((r=a%b)!=0);
{r=a%b;a=b;b=r;}
return (b);}
请再告诉我一下应该怎样改？谢谢

Answer 1

/*
题目：对给定的两组数（每组包括2个整数），
分别计算出两组数的最大公约数和最小公倍数的和。
如：
20、12为第一组，
28、7为第二组。
第一组数的最大公约数为4，最小公倍数为60；
第二组数的最大公约数为7，最小公倍数为28；
所以两组数的最大公约数之和为11，两组数的最小公倍数之和为88。
*/
#include <stdio.h>
int first(int u,int v);
int second(int a,int b);
//----在main外声明全局函数,在VC6测试通过---------
int main()
{

int a,b,u,v,z;
printf("输入两个整数做为第一组：");//加一个提示，让用户明白要做什么。
scanf("%d%d",&u,&v);
printf("\n");
printf("输入两个整数做为第二组：");//加一个提示，让用户明白要做什么。
scanf("%d%d",&a,&b);//修改此处，&a,&b.
printf("\n两组数的最大公约数之和为：");//加一个提示，让用户明白要做什么。
z=first(u,v)+second(a,b);
printf("%d\n",z);

return 0;
}

int first (int u,int v)
{
int r,t;
if (v>u)
{
t=v;
v=u;
u=t;
}
while ((r=u%v)!=0)//不要此处的;
{
r=u%v;
u=v;
v=r;
}
return v;
}

int second(int a,int b)
{
int r,t;
if (b>a)
{
t=a;
b=a;
a=t;
}
while ((r=a%b)!=0)
{
r=a%b;
a=b;
b=r;
}
return b;
}

提醒:
(1):你应该多加注释.
(2):应该有提示用户操作的语句.
(3):多看看别人的程序,多练习.
(4):此问题,你还没有实现最小公倍数的问题.

Answer 2

int first (int u,int v)
{int r,t; if (v>u)
{t=v;v=u;u=t;}
首先这个函数你的目的是想交换u和v的值,但实际上根本不会交换,这是一个常识性的问题.你代入函数的形参并不是两个变量,而是个实际的数值,除非你代入的是变量的地址才会发挥作用.这在讲函数形参的那一章有讲到