各位大神，这种情况怎么解决？

VC小学生

要实现的功能是从键盘输入字母，然后接收后以大写字母的形式在控制台窗口打印显示
代码如下：

1.         char* p_buf=new char[20];
   
2.         gets(p_buf);
   
3.         static int idx=0;
   
4.         for (;*p_buf!='\0';p_buf++)
   
5.         {
   
6.                 if (*p_buf>=97 && *p_buf<=122)
   
7.                 {
   
8.                         *p_buf=*p_buf-32;
   
9.                        
   
10.                 }
    
11.                 idx++;
    
12.         }
    
13.         p_buf-=idx;
    
14.         for (;*p_buf!='\0';p_buf++)
    
15.         {
    
16.                 cout<<*p_buf<<" ";
    
17.         }
    
18. 
    
19.         delete[] p_buf;
    

复制代码

结果是这样：

VC小学生

1.         char* p_buf=new char[20];
   
2.         gets(p_buf);
   
3.         static int idx=0;
   
4.         for (;*p_buf!='\0';p_buf++)
   
5.         {
   
6.                 if (*p_buf>=97 && *p_buf<=122)
   
7.                 {
   
8.                         *p_buf=*p_buf-32;
   
9.                 }
   
10.                 idx++;
    
11.         }
    
12.         p_buf-=idx;
    
13.         int len=strlen(p_buf);
    
14.         for (;*p_buf!='\0';p_buf++)
    
15.         {
    
16.                 cout<<*p_buf<<" ";
    
17.         }
    
18.         p_buf-=len;
    
19.         delete[] p_buf;
    

复制代码

修改成这样就好了，应该是指针指向的问题，虽然问题解决了，但还是不太理解，有谁能解释一下

netforfun

好久没看到这么诡异的代码了p_buf-=idx和p_buf-=len碉堡了，充分践行了无法接手一看就想重构的超难维护代码的可行性。估计假如这个作为大型软件的一部分，真心不敢随意改动，动一下就泄漏或者动一下就报错编译0w0e也不行。

寻丶

数组啊，p_buf[idx].你上面代码我看着好诡异...

zh_lily

第一个代码第二次for循环之后指针没有回到原来的位置，使用delete释放指针指针时，指针指向错误。第二个代码for循环之后指针回到了原来的位置。建议不要去动指针，使用p_buf[0]之类的会更好点

vc666666

VC小学生 发表于 2022-10-25 13:29
修改成这样就好了，应该是指针指向的问题，虽然问题解决了，但还是不太理解，有谁能解释一下

因为你第二次打印的时候，已经把指针往后移动了，如果每日有还原到首位地址，那么此时的地址是其他内存地址了，delete的时候就非法释放内存地址了。就跟delete野指针一个道理了