[已解决]有两个警告，帮我看下怎么优化下代码

a327904410

问题描述
　　从键盘输入一个字符串（长度<=40个字符），并以字符 ’.’ 结束。编辑功能有：
　　1 D：删除一个字符，命令的方式为： D a 其中a为被删除的字符，例如：D s 表示删除字符 ’s’ ，若字符串中有多个 ‘s’，则删除第一次出现的。
　　2 I：插入一个字符，命令的格式为：I a1 a2 其中a1表示插入到指定字符前面，a2表示将要插入的字符。例如：I s d 表示在指定字符 ’s’ 的前面插入字符 ‘d’ ，若原串中有多个 ‘s’ ，则插入在最后一个字符的前面。
　　3 R：替换一个字符，命令格式为：R a1 a2 其中a1为被替换的字符，a2为替换的字符，若在原串中有多个a1则应全部替换。
输入格式
　　输入共两行，第一行为原串(以’.’结束)，第二行为命令（输入方式参见“问题描述”)。
输出格式
　　输出共一行，为修改后的字符串或输出指定字符不存在的提示信息。
样例输入
This is a book.
D s
样例输出
Thi is a book.
输入输出样例解释
　　命令为删去s，第一个在字符中出现的s在This中，即得到结果。

虽然代码能运行输出结果，但会有警告，不知道是哪里出了问题。警告如下
严重性        代码        说明        项目        文件        行        禁止显示状态
警告        C6001        使用未初始化的内存“order”。        蓝桥杯试题集        D:\Mycode\蓝桥杯试题集\蓝桥杯试题集\字符串编辑.c        10       
警告        C4013        “gets”未定义；假设外部返回 int        蓝桥杯试题集        D:\Mycode\蓝桥杯试题集\蓝桥杯试题集\字符串编辑.c        8       

1. #include<stdio.h>
   
2. #include<string.h>
   
3. #define m 41
   
4. 
   
5. char str[m];
   
6. int main() {       
   
7.         char order[5], judge, a, b;
   
8.         gets(str);      // 输入字符串
   
9.         gets(order);       // 输入指令
   
10.         judge = order[0];
    
11. 
    
12.         if (judge == 'D') {
    
13.                 a = order[2];   // 删除只需一个字符
    
14.         }
    
15.         else {                                // 其他命令需要两个
    
16.                 a = order[2];
    
17.                 b = order[4];
    
18.         }
    
19. 
    
20.         int i = 0;
    
21.         if (judge == 'D') {     // 删除字符
    
22.                 while (str[i] != a)
    
23.                         i++;
    
24.                 for (i; str[i + 1] != '\0'; i++) {
    
25.                         str[i] = str[i + 1];
    
26.                 }
    
27.                 str[i] = '\0';
    
28.         }
    
29.        
    
30.         else if (judge == 'I') {    // 把字符放在另一个字符前
    
31.                 int len = strlen(str);
    
32.                 int k = len - 1;
    
33.                 while (str[k] != a)
    
34.                         k--;
    
35.                 for (len; len != k; len--) {
    
36.                         str[len] = str[len - 1];
    
37.                 }
    
38.                 str[k] = b;
    
39.         }
    
40.        
    
41.         else if (judge == 'R') {   // 替换字符
    
42.                 int j;
    
43.                 for (j = 0; str[j] != '\0'; j++) {
    
44.                         if (str[j] == a)
    
45.                                 str[j] = b;
    
46.                 }
    
47.         }
    
48. 
    
49.         printf("%s", str);
    
50.         return 0;
    
51. }

复制代码

最佳答案

月排行榜 / 总排行榜

hrpzcf

2021-10-30 02:21:13

gets不安全，你的编译器可能已经弃用了，用fgets代替

1. fgets(str, m, stdin);   // 输入字符串
   
2. fgets(order, 5, stdin); // 输入指令

复制代码

跳转到最佳答案楼层

hrpzcf

gets不安全，你的编译器可能已经弃用了，用fgets代替

1. fgets(str, m, stdin);   // 输入字符串
   
2. fgets(order, 5, stdin); // 输入指令

复制代码

a327904410

hrpzcf 发表于 2021-10-30 02:21
gets不安全，你的编译器可能已经弃用了，用fgets代替

真的欸，但为什么之前写gets好像没见警告，就这次有

a327904410

hrpzcf 发表于 2021-10-30 02:21
gets不安全，你的编译器可能已经弃用了，用fgets代替

大佬，我又发现了个问题，用fgets有些指令不能执行，比如：
输入：.
         R . %
输出就是空白了，输出应该是%

hrpzcf

a327904410 发表于 2021-10-30 13:46
大佬，我又发现了个问题，用fgets有些指令不能执行，比如：
输入：.
         R . %

因为你定义的order不够长， "R . %"有6个字节长度(包括\0)，所以只读到了{'R', ' ', '.', ' ', '\0'}
把char order[5]改成char order[6]或者更长点， fgets(order, 5, stdin);改成fgets(order, sizeof(order), stdin);

a327904410

hrpzcf 发表于 2021-10-30 14:08
因为你定义的order不够长， "R . %"有6个字节长度(包括\0)，所以只读到了{'R', ' ', '.', ' ', '\0'}
...

学习了