m0_52178025
xiaoxaio..
2021-01-16 17:47
采纳率: 68.4%
浏览 24
已采纳

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条回答 默认 最新

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

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

    点赞 评论
  • qq_42622746
    递龟_ 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;
}

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

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

    点赞 评论
  • digitalkee
    iam笨笨 2021-01-16 18:18

    s=NULL;

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

    点赞 评论
  • weixin_45206746
    樱满无门 2021-01-17 19:25

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

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

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

    点赞 评论
提交

相关推荐