重载operator new /分配超过对象大小

时间:2013-03-05 22:54:53

标签: c++ operator-overloading new-operator

我试图重载我班级的新操作员,我想为物品分配空间'属性以及未在类中定义的字符向量。

但是当我实例化一个新对象时,先前对象的字符正在改变。 我做错了吗?

class StringData
{       
public:
    friend class String;
    int refCount;
    int len;
    int maxLen;
    ~StringData()
    {

    }
    StringData()
    {
        maxLen = 1000;
        len = 0;
        refCount = 1;
    }
    StringData(const char * string)
    {
        maxLen = 1000;
        char * data = buffer();

        len = 0;
        while (string[len] != '\0' && len < (maxLen - 1))
        {
            data[len] = string[len];
            len++;
        }
        data[len] = '\0';
        refCount = 1;
    }
    char* buffer()
    {
        return reinterpret_cast<char*>(this + 12);
    }

public:
    void* operator new(size_t size, int maxLen)
    {

        return ::operator new(size + round4(maxLen + 1));

    }
    static int round4(int len)
    {
        return ((((int) (len / 4)) + 1) * 4);
    }

    void operator delete(void* obj)
    {
        ::operator delete(obj); 
    }
    void operator delete(void* obj, int size) throw ()
    {
        ::operator delete(obj); 
    }
};

我在其他课程中使用它:

class String{
public:
    StringData * data;

    String(StringData * data){
        this->data = data;
    }
public:
    String(const char*);
    String(const String&);
    ~String();
    String& operator =(const String);
    String& operator =(const char *);
    int length() const;
    bool operator ==(const String&) const;
    int compare(const String&) const;
    bool operator ==(const char*) const;
    int compare(const char*) const;
    String& operator +(const String&) const;
    String operator +(const char*) const;
    String operator +=(const String&);
    String operator +=(const char*);
    String toLower();
    String toUpper();
    char operator [ ](int) const;
    char& operator [ ](int);
    void print() const;

};

String::String(const char * string){
int stringLen = 0;

while (string[stringLen] != '\0')
    stringLen++;

data = new (stringLen + 1) StringData(string);

}

String::String(const String& string){

data = string.data;
string.data->refCount++;

}

String::~String(){
this->data->refCount--;
if (this->data->refCount == 0)
    delete data;

}

String& String::operator=(const String string){
data->refCount--;
if (this->data->refCount == 0)
    delete data;
data = string.data;
string.data->refCount++;
return *this;

}

void String::print() const{ printf("%s\n", data->buffer()); }

我的主要功能是:

int main(){
String *a = new String("boisahzashdacaraverdeepretaeazuleamarelaecinzaevermelha");
a->print();
String * s = new String("freestepehnoisquevoaashashashhasshashhasssasassadasd");
String * b = new String("kkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkk");

a->print();
s->print();
return 0;

}

当我执行时,字符串被破坏了:

boisahzashdacaraverdeepretaeazuleamarelaecinzaevermelha

boisahzashdacaraverd

freestepehnoisquevoa

1 个答案:

答案 0 :(得分:1)

这不是一个好主意:

return reinterpret_cast<char*>(this + 12);

如果您添加/删除某些成员函数[或创建虚函数],它不仅会失败,而且还取决于int的大小。

使用

return reinterpret_cast<char*>(this) + sizeof(*this);

[注意reinterpret_cast结束的新位置&lt;&gt;()!!] 第一个版本跳过12 * sizeof(*this)转发,而不是12个字节。 [当然,在我的机器上,当我使用maxLen的4 *乘数时,它确实有效 - 它只是在我删除它时才开始失败]

这可以简化:

((((int) (len / 4)) + 1) * 4);

这应该比maxlen大4倍:

(4 * maxLen)

为:

4 * (len / 4 + 1);

一旦我知道关于它如何被测试的评论的答案,我将回来编辑这个答案。