避免内存泄漏

Question

所以我在C++中学习OOP,我认为编写自己的字符串类是个好习惯(当然是为了学习目的).我想出了一个我不知道如何解决的问题.这是一些代码的和平:

class String {
    char *str;
public:
    String(char const *str);
    ~String();
    String operator + (char const *str);
};

String::String(char *str) {
    this->str = _strdup(str);
}

String::~String() {
    free(this->str);
}

String String::operator+(char const *str) {
    char *temp = (char *) malloc(strlen(str) + strlen(this->str) + 1);
    strcpy(temp, this->str);
    strcat(temp, str);
    return temp;
}

这里的问题是,这段代码会导致内存泄漏.从"operator +"返回调用我的构造函数,它通过分配更多的内存来复制temp,我找不到任何方法可以释放它.

Answer 1

您operator +被定义为返回a String但是您返回a char*表示编译器使用构造函数隐式转换它.这会复制字符串,但不会释放您正在泄漏的原件.

正如其他人所建议的那样,你可以做很多事情来改进代码,但要修复实际的泄漏,你可以这样做:

String String::operator+(char const *str) {
    char *temp = (char *) malloc(strlen(str) + strlen(this->str) + 1);
    strcpy(temp, this->str);
    strcat(temp, str);
    String strTmp(temp);
    free(temp);
    return strTmp;
}