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

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       

#include<stdio.h>
#include<string.h>
#define m 41

char str[m];
int main() {        
        char order[5], judge, a, b;
        gets(str);      // 输入字符串
        gets(order);       // 输入指令
        judge = order[0];

        if (judge == 'D') {
                a = order[2];   // 删除只需一个字符
        }
        else {                                // 其他命令需要两个
                a = order[2];
                b = order[4];
        }

        int i = 0;
        if (judge == 'D') {     // 删除字符
                while (str[i] != a)
                        i++;
                for (i; str[i + 1] != '\0'; i++) {
                        str[i] = str[i + 1];
                }
                str[i] = '\0';
        }
        
        else if (judge == 'I') {    // 把字符放在另一个字符前
                int len = strlen(str);
                int k = len - 1;
                while (str[k] != a)
                        k--;
                for (len; len != k; len--) {
                        str[len] = str[len - 1];
                }
                str[k] = b;
        }
        
        else if (judge == 'R') {   // 替换字符
                int j;
                for (j = 0; str[j] != '\0'; j++) {
                        if (str[j] == a)
                                str[j] = b;
                }
        }

        printf("%s", str);
        return 0;
}

最佳答案

月排行榜 / 总排行榜

hrpzcf

2021-10-30 02:21:13

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

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

跳转到最佳答案楼层

hrpzcf

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

fgets(str, m, stdin);   // 输入字符串
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'}
...

学习了