#include <iostream> #include <string> #include <cstring> #include <ctype.h> std::string solution(std::string words, std::string msg){ std::string result; // TODO: int tStrLenW = words.length(); int tStrLenM = msg.length(); int tWordsFlag = 0; int tMsgFlag = 0; char* pCharText = new char[tStrLenW + 1]; memset(pCharText, 0, tStrLenW + 1); while(tStrLenW > tWordsFlag && tStrLenM > tMsgFlag) { if( words[tWordsFlag] == msg[tMsgFlag] ) { pCharText[tMsgFlag] = msg[tMsgFlag]; //strcat(pCharText, &msg[tMsgFlag]); tMsgFlag++; } else { tWordsFlag++; tMsgFlag = 0; } } if(strcasecmp(pCharText, msg.c_str()) == 0) result = "Yes"; else result = "No"; return result; } int main() { std::string words; std::string msg; getline(std::cin, words);; getline(std::cin, msg);; std::string result = solution(words, msg); std::cout<<result<<std::endl; return 0; } 程序缺陷,程序修改
时间: 2024-03-06 22:49:04 浏览: 67
此程序的主要缺陷是在使用 `strcasecmp` 比较字符串时,会出现未初始化的情况,因为 `pCharText` 在某些情况下可能没有被完全填充。此外,`pCharText` 的大小只为 `tStrLenW+1`,但是在循环中我们使用 `tMsgFlag` 来填充 `pCharText`,而 `tMsgFlag` 的大小应该是 `tStrLenM+1`。
以下是对程序的修改建议:
1. 在循环开始前,将 `pCharText` 数组全部初始化为字符 `' '`,这样可以确保所有未匹配的字符都被填充为一个空格字符。
2. 在循环中,将 `pCharText` 数组的下标改为 `tWordsFlag`,这样可以确保只填充 `tWordsFlag` 对应的位置,而不是 `tMsgFlag` 对应的位置。
3. 在循环结束后,将 `pCharText` 数组的最后一个元素设为 `'\0'`,以确保字符串以 `'\0'` 结尾。
4. 在使用 `strcasecmp` 比较字符串前,先将 `pCharText` 数组中的所有字符转换为小写字母,以确保比较的是同一种大小写形式的字符串。
5. 在使用完 `pCharText` 数组后,需要将其释放掉,避免内存泄漏。
修改后的程序如下所示:
```
#include <iostream>
#include <string>
#include <cstring>
#include <ctype.h>
std::string solution(std::string words, std::string msg){
std::string result;
int tStrLenW = words.length();
int tStrLenM = msg.length();
int tWordsFlag = 0;
int tMsgFlag = 0;
char* pCharText = new char[tStrLenM + 1];
memset(pCharText, ' ', tStrLenM);
pCharText[tStrLenM] = '\0';
while(tStrLenW > tWordsFlag && tStrLenM > tMsgFlag) {
if( words[tWordsFlag] == msg[tMsgFlag] ) {
pCharText[tWordsFlag] = msg[tMsgFlag];
tWordsFlag++;
tMsgFlag++;
} else {
tWordsFlag++;
tMsgFlag = 0;
}
}
for (int i = 0; i < tStrLenM; i++) {
pCharText[i] = tolower(pCharText[i]);
}
if(strcasecmp(pCharText, msg.c_str()) == 0) result = "Yes";
else result = "No";
delete[] pCharText;
return result;
}
int main() {
std::string words;
std::string msg;
getline(std::cin, words);
getline(std::cin, msg);
std::string result = solution(words, msg);
std::cout<<result<<std::endl;
return 0;
}
```
阅读全文