以下代码有什么问题#include "iostream" using namespace std; class STRING { char* str; //字符指针指向字符串 int ct; //字符串长度 public: STRING(const char*getstr) { ct = strlen(getstr); str = new char[ct + 1]; strcpy(str, getstr); } STRING() { ct =0; str =NULL; }//此处补充构造函数 ~STRING() { if (str != NULL) { delete []str; str=NULL; } }//此处补充析构函数 void show(int i=-1) { if (i <= ct) cout << str << endl; else { int j; for (j = 0;j<=i-1;j++) cout << str[j] << endl; } };//注意设置show的参数 }; int main() { STRING str; char str_1[] = "Hello world!"; STRING str1(str_1); STRING str2 = str1; STRING str3("finished!"); str1.show(); //输出str1字符串 str2.show(3); //输出str2的前三个字符 str2.show(); //输出str3字符串 str3.show(); //输出str3字符串 return 0; }
时间: 2024-01-10 11:02:19 浏览: 77
使用include <iostream>时一定要加using namespace std
在这段代码中,存在两个问题:
1. 缺少拷贝构造函数
因为类中有动态分配内存的成员变量,所以需要实现拷贝构造函数,以确保在进行对象复制时,动态分配的内存空间可以正确地被复制。在这段代码中,缺少了拷贝构造函数的实现,因此会出现对象复制时内存泄漏的问题。
2. show函数的实现不正确
在show函数中,如果参数i小于等于字符串长度ct,应该输出整个字符串;如果i大于字符串长度ct,应该输出整个字符串。但是在这段代码中,当i等于ct时,只输出了字符串的第一个字符。正确的实现应该是:
```c++
void show(int i=-1) {
if (i < 0) {
cout << str << endl;
} else if (i <= ct) {
for (int j = 0; j < i; j++) {
cout << str[j];
}
cout << endl;
} else {
cout << str << endl;
}
}
```
阅读全文