[已解决]这是越界访问了吗？

redHmian

题目描述：编写一个程序，将两个字符串连接起来，不要用 strcat 函数。

代码如下：

1. #include <stdio.h>
   
2. 
   
3. void stringContact(char *str1, char *str);
   
4. 
   
5. void stringContact(char *str1, char *str)
   
6. {
   
7.     int i;
   
8.     for(i=0;*(str1+i)!='\0';i++)
   
9.     {
   
10.         ;
    
11.     }
    
12.     for(int j=0;*(str+j)!='\0';j++,i++)
    
13.     {
    
14.         *(str1+i)=*(str+j);
    
15.     }
    
16.     *(str+i)='\0';
    
17.     printf("%s",str1);
    
18. }
    
19. 
    
20. void main()
    
21. {
    
22.     char *str1="ABCD",*str="EFG";
    
23. 
    
24.     stringContact(str1,str);
    
25. }
    

复制代码

最佳答案

月排行榜 / 总排行榜

大马强

2021-10-31 23:53:45

1. #include <stdio.h>
   
2. #include<stdlib.h>
   
3. 
   
4. void stringContact(char *str1, char *str);
   
5. 
   
6. void stringContact(char *str1, char *str)
   
7. {
   
8.        
   
9.     int i;
   
10.     char *p;
    
11.     for(i=0;*(str1+i)!='\0';i++)
    
12.     {
    
13.         *(p+i) = *(str1+i) ;
    
14.     }
    
15.     for(int j=0;*(str+j)!='\0';j++,i++)
    
16.     {
    
17.         *(p+i) =*(str+j);
    
18.     }
    
19.     *(p+i)='\0';
    
20.     printf("%s",p);
    
21. }
    
22. 
    
23. int main()
    
24. {
    
25.     char *str1="ABCD",*str="EFG";
    
26. 
    
27.     stringContact(str1,str);
    
28.     return 0;
    
29. }

复制代码

跳转到最佳答案楼层

大马强

是的，因为char *str1="ABCD" 定义时就已经将长度固定了，要不然就申请新空间，要不用一个临时变量来存放

大马强

1. #include <stdio.h>
   
2. #include<stdlib.h>
   
3. 
   
4. void stringContact(char *str1, char *str);
   
5. 
   
6. void stringContact(char *str1, char *str)
   
7. {
   
8.        
   
9.     int i;
   
10.     char *p;
    
11.     for(i=0;*(str1+i)!='\0';i++)
    
12.     {
    
13.         *(p+i) = *(str1+i) ;
    
14.     }
    
15.     for(int j=0;*(str+j)!='\0';j++,i++)
    
16.     {
    
17.         *(p+i) =*(str+j);
    
18.     }
    
19.     *(p+i)='\0';
    
20.     printf("%s",p);
    
21. }
    
22. 
    
23. int main()
    
24. {
    
25.     char *str1="ABCD",*str="EFG";
    
26. 
    
27.     stringContact(str1,str);
    
28.     return 0;
    
29. }

复制代码

jackz007

        是的

1. char *str1="ABCD"

复制代码

        这样定义的 str1[] 只有 5 个字符的存储空间，如果把 str 的 "EFG" 拼接进来，实际需要的存储空间为 8 个字符，显然，这样使用 str1 必然是会越界的。

1. #include <stdio.h>
   
2. 
   
3. char * stringContact(char s1[] , char s2[])
   
4. {
   
5.         int i , j                                   ;
   
6.         for(i = 0 ; s1[i] ; i ++)                   ;
   
7.         for(j = 0 ; s2[j] ; j ++) s1[i + j] = s2[j] ;
   
8.         s1[i + j] = '\0'                            ;
   
9.         return s1                                   ;
   
10. }
    
11. 
    
12. int main(void)
    
13. {
    
14.     char s1[64] = "ABCD" , s2[] = "EFG"             ; // s1 必须定义足够大的存储空间，以便可以容纳拼接后的字符串；
    
15.     printf("%s\n" , stringContact(s1 , s2))         ;
    
16. }

复制代码

zhangmeng525

刚开始的指针变量没有初始化吧

svsv

这么多vip