[已解决]指针，一个不容易发现的错误

boot

小甲鱼老师讲过，下面这个是错误的，许多人的答案是在第二个for语句进行指针*p的初始化。

1. #include "stdio.h"
   
2. 
   
3. int main()
   
4. {
   
5.         int *p, i, a[10] = {1, 2, 3, 4, 5, 6, 7, 8, 9, 10};
   
6. 
   
7.         for(i = 0; i < 10; i++)
   
8.         {
   
9.                 p = &a[i];
   
10.                 p++;
    
11.         }
    
12. 
    
13.         for(i = 0; i < 10; i++, p++)
    
14.         {
    
15.                 printf("a[%d] = %d\n", i, *p);
    
16.         }
    
17. 
    
18.         return 0;
    
19. }
    

复制代码

但我又发现这个：我在第二个//for(i = 0; i < 10; i++)//这里我把p++去掉了，结果最后正常输出了数组里的元素

1. #include "stdio.h"
   
2. 
   
3. int main()
   
4. {
   
5.         int *p, i, a[10] = {1, 2, 3, 4, 5, 6, 7, 8, 9, 10};
   
6. 
   
7.         for(i = 0; i < 10; i++)
   
8.         {
   
9.                 p = &a[i];
   
10.                 p++;
    
11.         }
    
12. 
    
13.         for(i = 0; i < 10; i++)//这里我把p++去掉了，结果最后正常输出了数组里的元素
    
14.         {
    
15.                 printf("a[%d] = %d\n", i, *p);
    
16.         }
    
17. 
    
18.         return 0;
    
19. }

复制代码

最佳答案

月排行榜 / 总排行榜

ravenhu13

2017-10-3 06:57:47

楼主代码可以运行0至9？？不明觉厉，第一眼看代码就觉得有问题，因为P的值是固定的，无论怎么打印*P都必须是同一个值，也就是你的循环打印*P这里不可能打印出不同的值，楼主你确定你没剪切错代码？？

1. #include "stdio.h"
   
2. 
   
3. int main()
   
4. {
   
5.         int *p, i, a[10] = {1, 2, 3, 4, 5, 6, 7, 8, 9, 10};
   
6. 
   
7.         for(i = 0; i < 10; i++)
   
8.         {
   
9.                 p = &a[i];
   
10.                 p++;
    
11.         }
    
12. /////////////////下面的这个P就没变过，怎么可能打印出不同的值？？///////////////
    
13.         for(i = 0; i < 10; i++)//这里我把p++去掉了，结果最后正常输出了数组里的元素
    
14.         {
    
15.                 printf("a[%d] = %d\n", i, *p);
    
16.         }
    
17. 
    
18.         return 0;
    
19. }

复制代码

正确的代码应该如下：

1. int *p, i, a[10] = { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 };
   
2. //下面是个没有意义的循环，最终*P等于a[10]（题目里的数组下标最多只到a[9]），这就是经典的数组越界，这个时候引用*p将会产生不可预计后果。
   
3.                 for (i = 0; i < 10; i++)
   
4.                 {
   
5.                         p = &a[i];
   
6.                         p++;
   
7.                 }
   
8.                 //重新纠正p的地址。
   
9.                 p = &a[0];
   
10. 
    
11.                 for (i = 0; i < 10; i++,p++)
    
12.                 {
    
13.                         printf("a[%d] = %d\n", i, *p);
    
14.                 }

复制代码

跳转到最佳答案楼层

boot

求解，这是为什么？？？

ba21

boot

ba21 发表于 2017-10-1 20:02

嗯，谢谢你。
但我还想知道，就是为什么我那个会输出从0到9的数呢？？？

ba21

boot 发表于 2017-10-1 21:57
嗯，谢谢你。
但我还想知道，就是为什么我那个会输出从0到9的数呢？？？

拿下面代码运行分析下。至于什么问题，我只能说是编译器问题

1. #include "stdio.h"
   
2. 
   
3. int main()
   
4. {
   
5.         int *p, i, a[10] = {1, 2, 3, 4, 5, 6, 7, 8, 9, 10};
   
6.                 printf("%p\n\n", p);
   
7. 
   
8.         for(i = 0; i < 10; i++)
   
9.         {
   
10.                 p = &a[i];
    
11.                
    
12.                                 printf("%p\n", p);
    
13. 
    
14.                                 p++;
    
15.         }
    
16. 
    
17.                 printf("\n");
    
18. 
    
19.                  printf("%p\n", p);
    
20. 
    
21.         for(i = 0; i < 10; i++)//这里我把p++去掉了，结果最后正常输出了数组里的元素
    
22.         {
    
23.                 printf("a[%d] = %d\n", i, *p);
    
24.                                 printf("%p\n", p);
    
25.         }
    
26. 
    
27.         return 0;
    
28. }

复制代码

ravenhu13

楼主代码可以运行0至9？？不明觉厉，第一眼看代码就觉得有问题，因为P的值是固定的，无论怎么打印*P都必须是同一个值，也就是你的循环打印*P这里不可能打印出不同的值，楼主你确定你没剪切错代码？？

1. #include "stdio.h"
   
2. 
   
3. int main()
   
4. {
   
5.         int *p, i, a[10] = {1, 2, 3, 4, 5, 6, 7, 8, 9, 10};
   
6. 
   
7.         for(i = 0; i < 10; i++)
   
8.         {
   
9.                 p = &a[i];
   
10.                 p++;
    
11.         }
    
12. /////////////////下面的这个P就没变过，怎么可能打印出不同的值？？///////////////
    
13.         for(i = 0; i < 10; i++)//这里我把p++去掉了，结果最后正常输出了数组里的元素
    
14.         {
    
15.                 printf("a[%d] = %d\n", i, *p);
    
16.         }
    
17. 
    
18.         return 0;
    
19. }

复制代码

正确的代码应该如下：

1. int *p, i, a[10] = { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 };
   
2. //下面是个没有意义的循环，最终*P等于a[10]（题目里的数组下标最多只到a[9]），这就是经典的数组越界，这个时候引用*p将会产生不可预计后果。
   
3.                 for (i = 0; i < 10; i++)
   
4.                 {
   
5.                         p = &a[i];
   
6.                         p++;
   
7.                 }
   
8.                 //重新纠正p的地址。
   
9.                 p = &a[0];
   
10. 
    
11.                 for (i = 0; i < 10; i++,p++)
    
12.                 {
    
13.                         printf("a[%d] = %d\n", i, *p);
    
14.                 }

复制代码

boot

ravenhu13 发表于 2017-10-3 06:57
楼主代码可以运行0至9？？不明觉厉，第一眼看代码就觉得有问题，因为P的值是固定的，无论怎么打印*P都必须 ...

谢谢了，我之前那个代码运行之后就是那样的，可能是编译器的问题吧。我当时也是这么想的已经越界了，没想到会是这个样子呢

林01

对啊，为什么啊，当i等于不同值的时候p不都时会有对应的地址么，为什么第一个循环结束后p的地址一定是&a[9]+4

Mr.鹿

林01 发表于 2020-3-29 19:01
对啊，为什么啊，当i等于不同值的时候p不都时会有对应的地址么，为什么第一个循环结束后p的地址一定是&a[9] ...

因为越界了啊。因为你a[10]实际上从a[0]开始只到a[9]啊，他第一个for循环，直接给干到a[10]了，最多是a[9]，那a[10]自然是最大位数a[9]往后4个字节啊。而且应该不出意外是一串奇怪的数字。