[已解决]strcpy函数不安全问题

巨兔12138

1. #include<iostream>
   
2. #include<string>
   
3. 
   
4. using namespace std;
   
5. 
   
6. char select(char a[], char b[], char c[]);
   
7. 
   
8. int main()
   
9. {
   
10.         char a[20];
    
11.         char b[20];
    
12.         char c[20];
    
13.         cout << "输入三个国家的英文名：";
    
14.         cin >> a >> b >> c;
    
15. 
    
16.         cout << "按首字母排序最靠前的国家是：" << select(a, b, c);
    
17. }
    
18. char select(char a[], char b[], char c[])
    
19. {
    
20.         char x[20];
    
21.         if (strcmp(a, b) > 0)
    
22.         {
    
23.                 if (strcmp(b, c) > 0)
    
24.                         strcpy(x, c);
    
25.                 else
    
26.                         strcpy(x, b);
    
27.         }
    
28.         else
    
29.         {
    
30.                 if (strcmp(a, c) > 0)
    
31.                         strcpy(x, c);
    
32.                 else
    
33.                         strcpy(x, a);
    
34.         }
    
35.         return x[20];
    
36. }

复制代码

调试的时候编译器报错strcpy不安全，无法运行，但是课本上就是这么教的...
我用的编译器是vs2019
麻烦各位大神帮忙看下该怎么改啊

最佳答案

月排行榜 / 总排行榜

superbe

2020-7-4 22:09:49

select()函数，如果本意是要返回国家的首字母，应该是return x[0]，不是x[20]。x下标范围是0到19，实际国家名可能还不到19，x[20]是一个未知的字符。

如果是要返回完整的国家名，可以参考下面改法。这时不能只返回 x 的地址，因为 x 数组是局部变量，函数结束后 x 内容不存在了，main()输出就不正确。

改法一，在main里添加一个数组用来保存select()比较的结果，把这个数组做为参数传给select()

1. #include<iostream>
   
2. #include <cstring>
   
3. //#include<string>
   
4. 
   
5. using namespace std;
   
6. 
   
7. void select(char a[], char b[], char c[], char result[]);
   
8. 
   
9. int main()
   
10. {
    
11.     char a[20];
    
12.     char b[20];
    
13.     char c[20];
    
14.     char result[20];  //result[]保存比较结果
    
15.     cout << "输入三个国家的英文名：";
    
16.     cin >> a >> b >> c;
    
17. 
    
18.     select(a, b, c, result);
    
19.     cout << "按首字母排序最靠前的国家是：" << result << endl;
    
20. }
    
21. 
    
22. void select(char a[], char b[], char c[], char result[])
    
23. {
    
24.     if (strcmp(a, b) > 0)
    
25.     {
    
26.         if (strcmp(b, c) > 0)
    
27.             strcpy(result, c);
    
28.         else
    
29.             strcpy(result, b);
    
30.     }
    
31.     else
    
32.     {
    
33.         if (strcmp(a, c) > 0)
    
34.             strcpy(result, c);
    
35.         else
    
36.             strcpy(result, a);
    
37.     }
    
38. }
    

复制代码

其中select()函数也可以改成返回result[]的地址：
char * select(char a[], char b[], char c[], char result[])
{
    ......
    return result;
}
这样的好处是cout可以直接 cout << select(a, b, c, result)

跳转到最佳答案楼层

永恒的蓝色梦想

第一行加上：

1. #define _CRT_SECURE_NO_WARNINGS

复制代码

巨兔12138

永恒的蓝色梦想 发表于 2020-7-4 15:40
第一行加上：

你好请问为什么运行结果会出现，cout的结果是一个“？”的情况呢

superbe

select()函数，如果本意是要返回国家的首字母，应该是return x[0]，不是x[20]。x下标范围是0到19，实际国家名可能还不到19，x[20]是一个未知的字符。

如果是要返回完整的国家名，可以参考下面改法。这时不能只返回 x 的地址，因为 x 数组是局部变量，函数结束后 x 内容不存在了，main()输出就不正确。

改法一，在main里添加一个数组用来保存select()比较的结果，把这个数组做为参数传给select()

1. #include<iostream>
   
2. #include <cstring>
   
3. //#include<string>
   
4. 
   
5. using namespace std;
   
6. 
   
7. void select(char a[], char b[], char c[], char result[]);
   
8. 
   
9. int main()
   
10. {
    
11.     char a[20];
    
12.     char b[20];
    
13.     char c[20];
    
14.     char result[20];  //result[]保存比较结果
    
15.     cout << "输入三个国家的英文名：";
    
16.     cin >> a >> b >> c;
    
17. 
    
18.     select(a, b, c, result);
    
19.     cout << "按首字母排序最靠前的国家是：" << result << endl;
    
20. }
    
21. 
    
22. void select(char a[], char b[], char c[], char result[])
    
23. {
    
24.     if (strcmp(a, b) > 0)
    
25.     {
    
26.         if (strcmp(b, c) > 0)
    
27.             strcpy(result, c);
    
28.         else
    
29.             strcpy(result, b);
    
30.     }
    
31.     else
    
32.     {
    
33.         if (strcmp(a, c) > 0)
    
34.             strcpy(result, c);
    
35.         else
    
36.             strcpy(result, a);
    
37.     }
    
38. }
    

复制代码

其中select()函数也可以改成返回result[]的地址：
char * select(char a[], char b[], char c[], char result[])
{
    ......
    return result;
}
这样的好处是cout可以直接 cout << select(a, b, c, result)

superbe

这是改法二，也就是select()比较后返回相应数组的地址，这样就不用添加result数组了。

1. #include<iostream>
   
2. #include <cstring>
   
3. 
   
4. using namespace std;
   
5. 
   
6. char * select(char a[], char b[], char c[]);
   
7. 
   
8. int main()
   
9. {
   
10.     char a[20];
    
11.     char b[20];
    
12.     char c[20];
    
13.     cout << "输入三个国家的英文名：";
    
14.     cin >> a >> b >> c;
    
15. 
    
16.     cout << "按首字母排序最靠前的国家是：" << select(a, b, c) << endl;
    
17. 
    
18. }
    
19. 
    
20. char * select(char a[], char b[], char c[])
    
21. {
    
22.     char * temp;
    
23.     if (strcmp(a, b) > 0)
    
24.     {
    
25.         if (strcmp(b, c) > 0)
    
26.             temp = c;
    
27.         else
    
28.             temp = b;
    
29.     }
    
30.     else
    
31.     {
    
32.         if (strcmp(a, c) > 0)
    
33.             temp = c;
    
34.         else
    
35.             temp = a;
    
36.     }
    
37.     return temp;
    
38. }

复制代码