#include<stdio.h>
void main()
{
 int max(int x,int y,int z);
 int a,b,c,d;
 scanf("%d,%d,%d",&a,&b,&c);
 d=max(a,b,c);
 printf("max=%d\n",d);
}
 int max(int x,int y,int z)
 {
  int p,q;
  if(x>y)p=x;if(y>z)q=p;else if(x>z)q=x;else q=z;if(x>z)q=x;else q=z;
  else if(y>z)q=y;else q=z;if(x>z)q=y;else if(y>z)q=y;else q=z;
  return(q);
 }

什么乱七八糟的
if(x>y)p=x;
if(y>z)q=p;
else if(x>z)q=x;
else q=z;
if(x>z)q=x;
else q=z;
else if(y>z)q=y;《〈〈这句前面的else在语法上是多余的。但即使删了它，你的逻辑也不对。哪有这么用的，自己把自己绕进去了。
else q=z;
if(x>z)q=y;
else if(y>z)q=y;
else q=z;

程序好难看明，如果x<y，则执行下个if，那q=p这个怎么赋值？

大哥你也不用写成那样子啊  没有缩进 而且还写成一坨

int p,q;
  if(x>y)p=x;if(y>z)q=p;else if(x>z)q=x;else q=z;if(x>z)q=x;else q=z;
  else // 这个 多了 if(y>z)q=y;else q=z;if(x>z)q=y;else if(y>z)q=y;else q=z;
  return(q);

逻辑是没问题的  只要去掉了else就可以了  但是我不明白  我是菜鸟  见笑了！

#include<stdio.h>
void main()
{
 int max(int x,int y,int z)
 int a,b,c,d;
 scanf("%d,%d,%d",&a,&b,&c);
 d=max(a,b,c);
 printf("max=%d\n",d);
}
 int max(int x,int y,int z)
 {
  intp,q;
  if(x>y)p=x;
   if(y>z)q=p;
   else if(x>2)q=x;
        elseq=z;
   if(x>z)q=x;
   else q=z;
 标记 if(y>z)q=y;
       elseq=z;
       if(x>z)q=y;
       else if(y>z)q=y;
            elseq=z
   return(q);
 }
--------------------------------------------------------------------------------------
这个就可以了  不同是在标记的地方加个else就报错  我不能理解

乱糟糟的

if ()
    ;
else if ()
    ;
else if ()
    ;
.
.
.
else
    ;

有点乱啊  建议按规范书写  要不一个if配一个else 要不就嵌套 像楼上的：
程序代码：
if ()
    ;
else if ()
    ;
else if ()
    ;
.
.
.
else
    ;
：