结构数组的动态分配



我正在尝试制作一个仿冒字符串结构,它将为我的代码提供所需的基本内容(我不需要所有内容,并希望使我的代码尽可能快和小(。 因此,除了获取strcpystrcmp的来源之外(我可以这样做吗?我已经做了一个struct hstring来帮助我的代码。 到目前为止,我有以下struct

struct hstring{
private:
    char *s; // pointer to what holds the string
    int size; // size of the string
public:
    hstring(){
        s=(char *)malloc(0);
        size=0;
    }
    void set(const char* str){ // set the string
        size=0;
        while(str[size]!='')
            size++;
        s=(char*)realloc((void *)s,size*sizeof(*s)); // reallocate memory to hold just enough for the character array
        for(int i=0;i<size;i++)
            s[i]=str[i];
        s[size]='';
    }
    bool is(const char* str){ // check if something is equal to the string
        int i=0;
        while((s[i]==str[i])&&(str[i]!=''))
            i++;
        if((i==size)&&(str[i]==''))
            return true;
        return false;
    }
    inline char* get(){ // return the string
        return s;
    }
    inline int length(){ // return the size of the string
        return size;
    }
};

我注意到 set() 函数工作的唯一方法是我在那里放一个显式字符串或没有数组。例如。

// This works
printf("nTest1n");
hstring test;
char tmp_c[50];
scanf("%s",tmp_c);
test.set(tmp_c);
printf("%sn",test.get());
// This works
printf("nTest2n");
hstring test2[2];
test2[0].set("Hello ");
test2[1].set("world!");
printf("%s %sn",test2[0].get(),test2[1].get());
// This works
printf("nTest3n");
hstring test3[2];
scanf("%s",tmp_c);
test3[0].set(tmp_c);
scanf("%s",tmp_c);
test3[1].set(tmp_c);
printf("%s %sn",test3[0].get(),test3[1].get());
// This, what I want to do, does NOT work
printf("nTest4n");
hstring *test4 = (hstring *)malloc(2*sizeof(hstring));
for(int i=0;i<2;i++){
    scanf("%s",tmp_c);
    test4[i].set(tmp_c);
}
printf("%s %s",test4[0],test4[1]);
free(test4);

我很困惑为什么第四个测试没有正常运行。它可以编译,但在到达 test4 并尝试在 .set() 函数中重新分配内存时崩溃。我收到"访问冲突读取位置"错误,这让我假设我正在写/读我不应该写的地方;但是,我无法确定确切原因(尽管我可以在尝试重新分配字符数组的大小时告诉导致错误的行s=(char*)realloc((void *)s,size*sizeof(*s));。有人注意到我忽略的问题吗?

对于test4,您可以使用malloc()为两个对象分配内存:

hstring *test4 = (hstring *)malloc(2*sizeof(hstring));

但是,不为它们中的任何一个调用构造函数。因此,test4[0]test4[1]都没有正确初始化。您的类方法可能假定引用的对象初始化,因此您将获得非确定性行为。

解决此问题的方法是不要使用 malloc() 来分配test4,而是使用 std::vector

std::vector<hstring> test4(2);

然后,您可以删除对 free() 的调用,因为当对象超出范围时test4将被正确销毁。

hstring类本身正在管理指向已分配内存的指针。因此,您需要为其定义一个析构函数来释放该内存。

struct hstring {
    //...
    ~hstring () { free(s); }
    //...
};

但是,由于析构函数已成为必需的,因此现在还需要为 hstring 类定义复制构造函数和赋值运算符。这被称为三法则。

您可以通过利用对象为您管理内存来避免这种复杂性。对于您的hstring类,最简单的方法是将内部char *设为std::vector<char>。这样就不需要析构函数、复制构造函数和赋值运算符了。

0 传递给 malloc 要么返回 NULL 或可用于调用 free 的特殊指针,最好让它简单地使用 NULL 或更好的带有"\0"字符的空字符串进行初始化。

您正在打印结构本身printf("%s %s",test4[0],test4[1]);打印。要打印字符串,您应该printf("%s %s",test4[0].get (),test4[1].get ());

此外,程序中存在内存泄漏。您在hstring结构对象中mallocrealloc编辑了s指针,但从未释放它们。

我注意到当你做一个malloc时没有调用构造函数。您需要使用 new/delete 而不是 malloc/free

对于您需要做的最小更改:分配时hstring *test4 = new hstring[2];,释放时delete [] test4

另请参阅jxh的建议。

相关内容

  • 没有找到相关文章

最新更新