#include<stdio.h>
void main( )
{
    int a,b,c;
    printf("输入需要找最大公约数的两个数\n");
    scanf("%d,%d",&a,&b);
    c=(a>b)?a:b;
    for(c=a;c>0;c--)
    {
        while(a%c==0&&b%c==0)
            printf("%d",c);
    }
}

大约这样行不

int main (void)
{
    int a,b,c;
    printf("输入需要找最大公约数的两个数\n");
    scanf("%d%d",&a,&b);
    int s=1;
    for(c=1;c<=((a>b)?a:b);c++)
    {
       if(a%c==0 && b%c==0) s=c;
    }
    printf("%d",s);
    return 0;
}

有个叫辗转相除的专业求最大公约数的
大约如下代码吧

int f(int a,int b)
{
    int t;
    if(a<b) 
    {
        t=a;
        a=b;
        b=t;
    }
    int c,d;
    while(1)
    {
        c=a/b;
        d=a%b;
        if(d==0) return b;
        a=b;
        b=d;

    }
}

哦 还有递归

int gcd(int x,int y)   
//欧几里得辗转相除法求两数的最大的公约数
{
    if(x<y)    return gcd(y,x);
    if(x%y!=0) return gcd(y,x%y);
    else return y;
}

还可以这样

int foo(int e,int f)
{
    if(e<f) foo(f,e); 
    int b,c,d;
    for(b = e, c = f; d = b % c; b = c, c = d);
    return c;
}

楼主主要有两处错误：
1、c=(a>b)?a:b;后，for语句里c=a就矛盾了。
2、while(a%c==0&&b%c==0)肯定用错，死循环在这里发生。
结合你的代码修改如下：

#include<stdio.h>
void main( )
{
    int a,b,c;
    printf("输入需要找最大公约数的两个数\n");
    scanf("%d,%d",&a,&b);
    for(c=a<b?a:b;c&&(a%c||b%c);c--);
    printf("%d\n",c);
}

另：为提高循环效率，在for语句的循环条件里固定值的判断最好不要用算式，因为每次循环判断时都做运算，极大降低效率，如下几例均红色部分均不可取：
for(i=0;i<a*b;i++)  //a、b值已经固定，并且在循环里不再改变，类似的二楼w版主的c<=((a>b)?a:b)固定值的二目运算也是不可取的
for(i=0;i<=sqrt(n);i++) //这个循环再找素数里常用，貌似找到sqrt(n)位置减少了循环次数，但其实每次都调用开方函数和进行类型转换，相反降低循环效率

我并没有追求效率  而是追求代码行数少 嘿嘿

在合理规则下追求代码精炼是艺术，否则容易误导初学者。
*代码精炼不等于执行效率高，也不等于实际机器代码就少！

[此贴子已经被作者于2015-12-31 11:21编辑过]

恩恩  
我去看了这个辗转相除法，明白了
多谢提示。。