[已解决]消除警告

巴巴鲁

/*
        寻找三个字符串的最长前缀，如没有返回空字符""
        eg:输入"flower", "flow","flight"
                输出"fl"
                 
                输入"dog", "car", "mouse"
                输出""
*/
#include <stdio.h>
#include <string.h>
#define MAX 1024
char *search(char *s1,char *s2, char *s3, int min);
int main(void)
{
                int  a, b, c, min;
                void *ptr;
                char s1[MAX], s2[MAX], s3[MAX];
                printf("请输入s1字符串：");
                gets(s1);
                printf("请输入s2字符串：");
                gets(s2);
                printf("请输入s3字符串：");
                gets(s3);
               
                a = strlen(s1);
                b = strlen(s2);
                c = strlen(s3);
               
                min = a < b ? a : b;
                min = min  < c ? min : c;
               
                ptr = search(s1, s2, s3, min);
                printf("%s",ptr);
               
                return 0;
}
char *search(char *s1,char *s2, char *s3, int min)
{
                char s[MAX];
                int i = 0;
               
                for(i = 0; i < min; i++)
                {
                        if((s1[i] == s2[i]) && (s2[i] == s3[i]))
                        {
                                s[i] = s1[i];
                        }
                        else
                        {
                                break;
                        }
                }
                s[i] = '\0';
               
                if(s[0] == '\0')
                {
                        return "";
                }
                else
                {
                        return s;
                }
}

程序能运行，结果也正确，但是会出现警告：[Warning] function returns address of local variable [enabled by default]
怎么消除呢？

最佳答案

月排行榜 / 总排行榜

livcui

2020-7-30 10:05:07

你这里 return s 的 s 是数组地址，而这个数组会在函数结束时被销毁，所以返回了一个被销毁内存的地址，没有意义，也就报警了，

C 我不熟，编译器也没有 gets() ，你自己改改吧

这样？

1. /*
   
2.         寻找三个字符串的最长前缀，如没有返回空字符""
   
3.         eg:输入"flower", "flow","flight"
   
4.                 输出"fl"
   
5. 
   
6.                 输入"dog", "car", "mouse"
   
7.                 输出""
   
8. */
   
9. #include <stdio.h>
   
10. #include <string.h>
    
11. #define MAX 1024
    
12. char* search(char* s1, char* s2, char* s3, int min, char*);
    
13. char s[MAX];
    
14. int main(void)
    
15. {
    
16.     int  a, b, c, min;
    
17.     void* ptr;
    
18.     char s1[MAX], s2[MAX], s3[MAX];
    
19.     printf("请输入s1字符串：");
    
20.     gets(s1);
    
21.     printf("请输入s2字符串：");
    
22.     gets(s2);
    
23.     printf("请输入s3字符串：");
    
24.     gets(s3);
    
25. 
    
26.     a = strlen(s1);
    
27.     b = strlen(s2);
    
28.     c = strlen(s3);
    
29. 
    
30.     min = a < b ? a : b;
    
31.     min = min < c ? min : c;
    
32. 
    
33.     ptr = search(s1, s2, s3, min, s);
    
34.     printf("%s", ptr);
    
35. 
    
36.     return 0;
    
37. }
    
38. char* search(char* s1, char* s2, char* s3, int min, char* s)
    
39. {
    
40.     //char s[MAX];
    
41.     int i = 0;
    
42. 
    
43.     for (i = 0; i < min; i++)
    
44.     {
    
45.         if ((s1[i] == s2[i]) && (s2[i] == s3[i]))
    
46.         {
    
47.             s[i] = s1[i];
    
48.         }
    
49.         else
    
50.         {
    
51.             break;
    
52.         }
    
53.     }
    
54.     s[i] = '\0';
    
55. 
    
56.     if (s[0] == '\0')
    
57.     {
    
58.         return '\0';
    
59.     }
    
60.     else
    
61.     {
    
62.         return s;
    
63.     }
    
64. }

复制代码

跳转到最佳答案楼层

livcui

你这里 return s 的 s 是数组地址，而这个数组会在函数结束时被销毁，所以返回了一个被销毁内存的地址，没有意义，也就报警了，

C 我不熟，编译器也没有 gets() ，你自己改改吧

这样？

1. /*
   
2.         寻找三个字符串的最长前缀，如没有返回空字符""
   
3.         eg:输入"flower", "flow","flight"
   
4.                 输出"fl"
   
5. 
   
6.                 输入"dog", "car", "mouse"
   
7.                 输出""
   
8. */
   
9. #include <stdio.h>
   
10. #include <string.h>
    
11. #define MAX 1024
    
12. char* search(char* s1, char* s2, char* s3, int min, char*);
    
13. char s[MAX];
    
14. int main(void)
    
15. {
    
16.     int  a, b, c, min;
    
17.     void* ptr;
    
18.     char s1[MAX], s2[MAX], s3[MAX];
    
19.     printf("请输入s1字符串：");
    
20.     gets(s1);
    
21.     printf("请输入s2字符串：");
    
22.     gets(s2);
    
23.     printf("请输入s3字符串：");
    
24.     gets(s3);
    
25. 
    
26.     a = strlen(s1);
    
27.     b = strlen(s2);
    
28.     c = strlen(s3);
    
29. 
    
30.     min = a < b ? a : b;
    
31.     min = min < c ? min : c;
    
32. 
    
33.     ptr = search(s1, s2, s3, min, s);
    
34.     printf("%s", ptr);
    
35. 
    
36.     return 0;
    
37. }
    
38. char* search(char* s1, char* s2, char* s3, int min, char* s)
    
39. {
    
40.     //char s[MAX];
    
41.     int i = 0;
    
42. 
    
43.     for (i = 0; i < min; i++)
    
44.     {
    
45.         if ((s1[i] == s2[i]) && (s2[i] == s3[i]))
    
46.         {
    
47.             s[i] = s1[i];
    
48.         }
    
49.         else
    
50.         {
    
51.             break;
    
52.         }
    
53.     }
    
54.     s[i] = '\0';
    
55. 
    
56.     if (s[0] == '\0')
    
57.     {
    
58.         return '\0';
    
59.     }
    
60.     else
    
61.     {
    
62.         return s;
    
63.     }
    
64. }

复制代码

Unicorn#

把s的声明放到主函数里边
search改为无返回值，向search传入s

Cool_Breeze

1. 
   
2. /*
   
3.         寻找三个字符串的最长前缀，如没有返回空字符""
   
4.         eg:输入"flower", "flow","flight"
   
5.                 输出"fl"
   
6.                  
   
7.                 输入"dog", "car", "mouse"
   
8.                 输出""
   
9. */
   
10. #include <stdio.h>
    
11. #include <string.h>
    
12. #define MAX 1024
    
13. char *search(char *s1,char *s2, char *s3, int min, char *s);
    
14. int main(void)
    
15. {
    
16.                 int  a, b, c, min;
    
17.                 void *ptr;
    
18.                 char s1[MAX], s2[MAX], s3[MAX], s[MAX];
    
19.                 printf("请输入s1字符串：");
    
20.                 gets(s1);
    
21.                 printf("请输入s2字符串：");
    
22.                 gets(s2);
    
23.                 printf("请输入s3字符串：");
    
24.                 gets(s3);
    
25.                
    
26.                 a = strlen(s1);
    
27.                 b = strlen(s2);
    
28.                 c = strlen(s3);
    
29.                
    
30.                 min = a < b ? a : b;
    
31.                 min = min  < c ? min : c;
    
32.                
    
33.                 ptr = search(s1, s2, s3, min, s);
    
34.                 printf("%s",ptr);
    
35.                
    
36.                 return 0;
    
37. }
    
38. char *search(char *s1,char *s2, char *s3, int min, char *s)
    
39. {
    
40.                 //char s[MAX];
    
41.                 int i = 0;
    
42.                
    
43.                 for(i = 0; i < min; i++)
    
44.                 {
    
45.                         if((s1[i] == s2[i]) && (s2[i] == s3[i]))
    
46.                         {
    
47.                                 s[i] = s1[i];
    
48.                         }
    
49.                         else
    
50.                         {
    
51.                                 break;
    
52.                         }
    
53.                 }
    
54.                 s[i] = '\0';
    
55.                
    
56.                 if(s[0] == '\0')
    
57.                 {
    
58.                         return "";
    
59.                 }
    
60.                 else
    
61.                 {
    
62.                         return s;
    
63.                 }
    
64. }
    

复制代码

巴巴鲁

livcui 发表于 2020-7-30 10:39
你这里 return s 的 s 是数组地址，而这个数组会在函数结束时被销毁，所以返回了一个被销毁内存的地址，没 ...

对的，不过没有公共前缀会返回NULL,58行改成return “”就好了

livcui

巴巴鲁 发表于 2020-7-30 11:30
对的，不过没有公共前缀会返回NULL,58行改成return “”就好了

我用的C++，会报错，所以才改了



设置个最佳吧

巴巴鲁

Unicorn# 发表于 2020-7-30 10:48
把s的声明放到主函数里边
search改为无返回值，向search传入s

感谢感谢，让我又知道了无返回值函数