林鹿可爱 发表于 2019-1-19 21:48:16

关于指针逆序输出数组

问题:
原题是把数组逆序存放并输出,我给改成用指针指向数组最后一个元素,p--让指针向前输出
结果执行却出现了一连串的0,不知道问题出在哪里
补充一句,调用reserve之前全部正常

代码如下
#include<stdio.h>

int main()
{
        void reverse(int a[],int n);//交换 初始化+定义形参

        int i,a={2,3,6,4,7};
        printf("调换前:\n");
        for(i=0;i<5;i++)
        {
                printf("%d ",a);
        }
        printf("\n");


        printf("调换后:\n");
        reverse(a,5); //格式 函数名(数组,数值)
        return 0;
}
void reverse(int a[],int n)//交换
{

        int *p;

        for(p=a+10;p>=0;p--)
        {
                printf("%d",*p);
        }

}

Croper 发表于 2019-1-19 23:13:42

本帖最后由 Croper 于 2019-1-19 23:15 编辑

      for(p=a+10;p>=0;p--)
      {
                printf("%d",*p);
      }
首先,a+10已经越界。。这是不安全的行为
其次,p>=0,你是要干哪样- -,

void reverse(int a[],int n)//½»»»
{

      int *p;

      for(p=a+n-1;p>=a;p--)
      {
                printf("%d",*p);
      }

}

行客 发表于 2019-1-19 23:50:27

很正常,估计你是故意这么玩的:
      for(p=a+10;p>=0;p--)
      {
                printf("%d",*p);
      }
按照你的循环要求,p从&a后的40个字节的地址,一直不停的减,要减到p=0为止(当然减不到为0就内存溢出了)。

一开始输出的时候不是0,是一个随机数,然后也会输出你的逆向数。然后还继续减,减着减着就到输出0了。你可以改下代码自己观察下,把void reverse(int a[],int n);改成如下定义:
void reverse(int a[],int n)//交换
{
       
        int *p;
       
        int m=0;        //为了后面限制遇到多少次0后break
        for(p=a+10;p>=0;p--)
        {
                printf("%d\n",*p);
                if(*p==0)
                {
                        m=m+1;
                }
                if(m>10)        //定义遇到多少次0后跳出for
                {
                        break;
                }
                       
        }
       
}

如果你想正常定义,可以修改为:
void reverse(int a[],int n)
{

      int *p;

      for(p=a+n-1;p>=a;p--)
      {
                printf("%d",*p);
      }

}

林鹿可爱 发表于 2019-1-20 02:14:53

Croper 发表于 2019-1-19 23:13
首先,a+10已经越界。。这是不安全的行为
其次,p>=0,你是要干哪样- -,

谢谢你 我发现自己错在哪里了,改正之后输出正常 (^-^)V

林鹿可爱 发表于 2019-1-20 02:15:42

行客 发表于 2019-1-19 23:50
很正常,估计你是故意这么玩的:
      for(p=a+10;p>=0;p--)
      {


谢谢多种方式解答~
页: [1]
查看完整版本: 关于指针逆序输出数组