xiaoxaio..
2021-01-16 17:47
采纳率: 84.2%
浏览 29

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;
}

  • 写回答
  • 关注问题
  • 收藏
  • 邀请回答

4条回答 默认 最新

  • 云中屋 2021-01-16 18:21
    已采纳

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

    打赏 评论
  • 递龟_ 2021-01-16 18:17
    #include<stdio.h>
    int main()
    {
        int *p,*q,*s,a,b;        //建议指针和整型分开定义
        s=NULL;p=&a;q=&b;        
        scanf("%d%d",p,q);        //输入指针用 scanf("%p",&p);
        if(*p<*q)
        {//既然是取值就没必要用 *s
            *s=*q;
            *q=*p;
            *p=*s;
        }
        printf("最大值为:%d,最小值为:%d,和为%d",*p,*q,*p+*q);    //这里建议先赋值给变量,再输出
        return 0;
    }

    一点点建议,可能不全~~

    对指针理解可以参考我的博客~~

    打赏 评论
  • 发狂的蜗牛 2021-01-16 18:18

    s=NULL;

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

    打赏 评论
  • 樱满无门 2021-01-17 19:25

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

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

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

    打赏 评论

相关推荐 更多相似问题