关于最大公约数的问题，请大家看下我的代码

在编译的时候可以，但是在运行的时候却出现了问题，请大家帮我看一下，另外有没有关于最大公约数问题的比较简单的算法？

main()
{
int a,b,c;
scanf("%d,%d",&a,&b);
c=division(a,b);
printf("the number is %d",c);
}
#include <stdio.h>
int division(int x,int y)
{
int i,j,c;
char str1[81];
char str2[81];
for(i=1;i<=x;i++)
if(x%i==0) gets(str1);
for(j=1;j<=y;j++)
if(y%j==0)get(str2);
strcpy(str1,str2);
c=max(str1);
return c;
}
max(int a[])
{
int i,t=a[0];
for(i=1;i<'\0';i++)
if(t<a[i])t=a[i];
return t;
}
----------------解决方案--------------------------------------------------------
#include <stdio.h>

int fun(int m, int n)
{
    int t;
    while(m%n)
    {
        t = m%n;
        m = n;
        n = t;
    }
    return n;
}

int main()
{
    printf("%d\n", fun(8,42));
    return 0;
}
----------------解决方案--------------------------------------------------------

----------------解决方案--------------------------------------------------------
很感谢楼上两位朋友对我提出的算法作出的解答，你们的算法很简单，我采取了。但是可以帮我解释一下我的错在哪里么？
----------------解决方案--------------------------------------------------------
真的没人可为这个问题作出解答么？？
----------------解决方案--------------------------------------------------------

1、我觉得#include <stdio.h>最好放在顶上
2、自定义函数放在main下面的话最好能在main之上声明一下
3、max函数应该定义个类型，不然怎么return t;
4、gets函数用的不太妥当，还有个gets掉了s
5、strcpy用的更是不知所谓
6、这个for(i=1;i<'\0';i++)就更无语了
7、还有很多问题，我就不一一列举了

[[it] 本帖最后由 sxn0508 于 2008-1-29 08:54 编辑 [/it]]
----------------解决方案--------------------------------------------------------

我改了改你的程序，估计能用了，不过有待改进


----------------解决方案--------------------------------------------------------
谢谢上面内位朋友，我对你的帮助真的不胜感激，我现在改进了算法，与你分享一下
main()
{
int a,b,c;
scanf("%d,%d",&a,&b);
c=division(a,b);
printf("the number is %d",c);
}
#include <stdio.h>
int division(int x,int y)
{
    int i,j,c;
    int str1[81];
    if(x>y)c=y;
    else c=x;
    for(i=1,j=0;i<=c;i++)
    {
        if(x%i==0&&y%i==0)
        {
            str1[j++]=i;
            
        }
    }
    c=str1[j-1];
return c;
}
----------------解决方案--------------------------------------------------------

我觉得不用定义这两个数组吧：  int str1[81]={};
   int str2[81]={};
我觉得直接
int str1[81];
int str2[82];
就行了吧
----------------解决方案--------------------------------------------------------

   int str2[81]={};这样是为了顺便把值全置为0
----------------解决方案--------------------------------------------------------