我正在尝试制作一个仿冒字符串结构,它将为我的代码提供所需的基本内容(我不需要所有内容,并希望使我的代码尽可能快和小(。 因此,除了获取strcpy
和strcmp
的来源之外(我可以这样做吗?我已经做了一个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
结构对象中malloc
和realloc
编辑了s
指针,但从未释放它们。
我注意到当你做一个malloc时没有调用构造函数。您需要使用 new
/delete
而不是 malloc
/free
对于您需要做的最小更改:分配时hstring *test4 = new hstring[2];
,释放时delete [] test4
。
另请参阅jxh的建议。