c语言指针输出两个数的最大值和最小值，并求和，有些不清楚这样写为什么不行。

#include<stdio.h>
int main()
{
    int *p,*q,*s,a,b;
    s=NULL;p=&a;q=&b;
    scanf("%d%d",p,q);
    if(*p<*q)
    {
        *s=*q;
        *q=*p;
        *p=*s;
    }
    printf("最大值为:%d，最小值为:%d,和为%d",*p,*q,*p+*q);
    return 0;
}

s被初始化成空指针，后面的交换赋值用到了s，这时的s是空指针，赋值操作根本不知道要把数据放到哪里。

s=NULL;

这句话去掉。具体原因我也不是很清楚。

就算s不是空指针，它没有分配空间也是用不了的，会报segmentation fault的错误。你可以改成这样

if (*p < *q)
{
    s = q;
    q = p;
    p = s;
}

效果是一样的。更关键的是，它是正确的。