[英]C++: SIGTRAP error in custrom String class
在過去的兩天里,我在互聯網上搜索,找不到發生這種情況的原因。
我的字符串類以人們期望的方式工作:它從堆上的一個初始連續內存塊(128 字節)開始,然后根據需要調整大小。
我對new
的所有調用都與在相應指針上對delete[]
的單個調用相匹配。
SIGTRAP 錯誤僅在調用析構函數時發生(最后一次使用delete[]
)。
為了產生一個最小的可重現示例,我將代碼(大約 1400 行)減少到如下所示(以及似乎導致錯誤的原因)。
#include <string>
#include <iostream>
#include <cstring>
static int MIN_SIZE = 128;
char *setchr_c(char *str, const char ch, size_t pos) {
if (str == nullptr) {
return nullptr;
}
*(str + pos) = ch;
return str;
}
class String {
private:
char *data = nullptr;
char *string = nullptr;
size_t length_w_null = 0;
size_t size = 0;
size_t space_front = 0;
size_t space_back = 0;
bool is_empty = true;
void set_size(bool def_do = true) {
if (def_do) {
do {
size *= 2;
} while (size < length_w_null);
return;
}
while (size < length_w_null) {
size *= 2;
}
}
void constructor(const char *str, bool after_empty = false) {
size = MIN_SIZE;
length_w_null = strlen(str) + 1;
set_size(false);
if (after_empty) { // this is to avoid re-assigning the memory if not necessary
if (size != MIN_SIZE) {
// free(data);
printf("CTOR AFTER EMPTY DELETE, DELETING: %x\n", data);
delete[] data;
// data = (char *) malloc(size);
printf("CTOR AFTER EMPTY NEW, SIZE: %llu, ", size);
data = new char[size];
printf("ADDRESS: %x\n", data);
}
} else {
// data = (char *) malloc(size);
printf("CTOR ELSE NEW, SIZE: %llu, ", size);
data = new char[size];
printf("ADDRESS: %x\n", data);
}
memset(data, '\0', size);
string = data + get_first_pos();
setchr_c(string, '\0', length_w_null - 1);
strcpy(string, str);
space_front = get_first_pos();
space_back = is_even(length_w_null) ? space_front : space_front + 1;
is_empty = false;
}
void empty_constructor() {
// data = (char *) malloc(MIN_SIZE);
printf("EMPTY CTOR NEW, MINSIZE: %llu, ", MIN_SIZE);
data = new char[MIN_SIZE];
printf("ADDRESS: %x\n", data);
memset(data, '\0', size);
string = nullptr;
length_w_null = 0;
size = MIN_SIZE;
space_front = MIN_SIZE / 2;
space_back = MIN_SIZE / 2;
is_empty = true;
}
static bool is_even(size_t num) {
return num % 2 == 0;
}
[[nodiscard]] unsigned long get_first_pos() const {
size_t new_len = length_w_null;
if (!is_even(new_len)) {
new_len++;
}
unsigned long pos = size / 2 - (new_len) / 2;
return pos;
}
public:
static const size_t nopos = -1;
String() {
empty_constructor();
}
String(char ch) {
if (ch == '\0') {
empty_constructor();
} else {
const char str[2]{ch, '\0'};
constructor(str);
}
}
String(const char *str) {
if (str == nullptr) {
return;
}
if (strlen(str) == 0) {
empty_constructor();
} else {
constructor(str);
}
}
void append_back(const char *str) {
if (str == nullptr) {
return;
}
if (is_empty) {
constructor(str, true);
return;
}
size_t l = strlen(str);
size_t old_l = length_w_null - 1;
length_w_null += l;
if (l + 1 > space_back) {
set_size();
char *old_data = data;
char *old_str = string;
// data = (char *) malloc(size);
printf("APPEND BACK NEW, SIZE: %llu, ", size);
data = new char[size];
printf("ADDRESS: %x\n", data);
memset(data, '\0', size);
string = data + get_first_pos();
strcpy(string, old_str);
// free(old_data);
printf("APPEND BACK DELETE OLD_DATA, DELETING: %x\n", old_data);
delete[] old_data;
space_front = get_first_pos();
space_back = is_even(length_w_null) ? space_front : space_front + 1;
}
strcpy(string + old_l, str);
space_back -= l;
}
void append_front(const char *str) {
if (str == nullptr) {
return;
}
if (is_empty) {
constructor(str, true);
return;
}
size_t l = strlen(str);
length_w_null += l;
char gone = *string;
if (l > space_front) {
set_size();
char *old_data = data;
char *old_str = string;
// data = (char *) malloc(size);
printf("APPEND FRONT NEW, SIZE: %llu, ", size);
data = new char[size];
printf("ADDRESS: %x\n", data);
memset(data, '\0', size);
string = data + get_first_pos();
strcpy(string + l, old_str);
// free(old_data);
printf("APPEND FRONT DELETE OLD_DATA, DELETING: %x\n", old_data);
delete[] old_data;
strcpy(string, str);
space_front = get_first_pos();
space_back = is_even(length_w_null) ? space_front : space_front + 1;
} else {
strcpy(string - l, str);
string -= l;
space_front -= l;
}
setchr_c(string, gone, l);
}
void push_back(char ch) {
if (ch == 0) {
return;
}
const char str[2] = {ch, '\0'};
append_back(str);
}
void push_front(char ch) {
if (ch == 0) {
return;
}
const char str[2] = {ch, '\0'};
append_front(str);
}
void clear() noexcept {
// free(data);
printf("CLEAR DELETE, DELETING: %x\n", data);
delete[] data;
empty_constructor();
}
size_t get_size() const {
return size;
}
size_t get_length() const {
return length_w_null == 0 ? 0 : length_w_null - 1;
}
const char *c_str() const noexcept {
return string;
}
~String() {
// free(data);
printf("DTOR DELETE, DELETING: %x\n", data);
delete[] data;
}
friend std::ostream& operator<<(std::ostream& os, const String& str);
};
std::ostream& operator<<(std::ostream& os, const String& str) {
os << str.string;
return os;
}
using namespace std;
int main() {
String bro;
char array[] = "eeee;;dlfkjas;j;a;lAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAsdjfk";
bro.clear();
for (const char &ch : array) {bro.push_front(ch);}
bro.clear();
bro.append_back("Let us see!");
cout << bro << endl;
return 0;
}
我添加了一些 printf() 語句來檢查地址,沒有明顯錯誤。
我得到的示例輸出是:
EMPTY CTOR NEW, MINSIZE: 128, ADDRESS: 9b651920
CLEAR DELETE, DELETING: 9b651920
EMPTY CTOR NEW, MINSIZE: 128, ADDRESS: 9b651920
APPEND FRONT NEW, SIZE: 256, ADDRESS: 9b6519b0
APPEND FRONT DELETE OLD_DATA, DELETING: 9b651920
CLEAR DELETE, DELETING: 9b6519b0
EMPTY CTOR NEW, MINSIZE: 128, ADDRESS: 9b651920
Let us see!
DTOR DELETE, DELETING: 9b651920
...在程序崩潰之前。
編譯器:Windows 上的 MinGW。
user4581301 非常友好地檢查了我的代碼,並指出了我的empty_constructor()
成員函數中的明顯錯誤:我使用memset()
) 在我的data
指針中設置size
個字符(可以大於 128)的值,它(最初)只分配了 128 個字節。
通過將size
更改為MIN_SIZE
作為memset()
參數,一切都已解決。
非常感謝 user4581301。
我會盡快接受這個答案來結束這個問題。
聲明:本站的技術帖子網頁,遵循CC BY-SA 4.0協議,如果您需要轉載,請注明本站網址或者原文地址。任何問題請咨詢:yoyou2525@163.com.