从类成员函数中访问字符串数组

我正在使用基本类并尝试访问成员函数中的成员字符串数组。我很困惑，因为字符串数组初始化并没有任何内存错误（博士存储器）被释放的时候，如果我只能调用set_words，print_words，并del_words功能：从类成员函数中访问字符串数组

class XWORD { 
    public: 
    int vdir; 
    int len; 
    int wid; 
    int nWords; 
    char ** arr; 
    char ** words;  
    ... 
}; 

void XWORD::set_words(int istart, int inWords, char* iwords[]) { 
    int w = istart; 
    int k = 0; 
    this->nWords = inWords; 

    this->words = new char* [this->nWords]; 
    for (w=istart; w<(istart+this->nWords); w++) { 
     this->words[k] = new char [1+strlen(iwords[w])]; 
     this->words[k] = strcpy(this->words[k], iwords[w]); 
     k++; 
    } 
} 

void XWORD::del_words() { 
    int w = 0; 
    for (w=0; w<(this->nWords); w++) { 
     delete[] this->words[w]; 
    } 
    delete[] this->words; 
} 

void XWORD::print_words() { 
    int w = 0; 
    for (w=0; w<(this->nWords); w++) { 
     printf("\n%s",this->words[w]); 
    } 
    printf("\n"); 
}

然而，当我调用下面的函数，我得到如下所示的博士内存错误（init_arr和del_arr单独不给错误其一）：

void XWORD::add_word_to_arr(int iw, int iy, int ix) { 
    int k = 0; 
    int y = iy; 
    int x = ix; 

    for (k=0; k<(int)strlen(this->words[iw]); k++) { 
     this->arr[y][x] = this->words[iw][k]; 
     if (this->vdir) { 
      y++; 
     } else { 
      x++; 
     } 
    } 
} 

int main(int argc, char * argv[]) { 
    XWORD x; 
    x.set_words(1, argc-1, argv); 
    x.init_arr(10,10); 

    x.add_word_to_arr(0, x.len/2, x.wid/2); // WITHOUT THIS, NO ERRORS 
    x.print_words(); 

    x.del_words(); 
    x.del_arr(); 
}

内存博士错误：

c:\MinGW\WORKSPACE\cpp\xword>make runmem 
drmemory -brief -batch bin/test.exe 
~~Dr.M~~ Dr. Memory version 1.8.0 
~~Dr.M~~ Running "bin/test.exe" 
~~Dr.M~~ 
~~Dr.M~~ Error #1: UNADDRESSABLE ACCESS beyond heap bounds: reading 4 byte(s) 
~~Dr.M~~ # 0 XWORD::add_word_to_arr    [../mingwrt-4.0.3-1-mingw32-src/src/libcrt/crt/crt1.c:301] 
~~Dr.M~~ # 1 main         [../mingwrt-4.0.3-1-mingw32-src/src/libcrt/crt/crt1.c:302] 
~~Dr.M~~ Note: refers to 0 byte(s) beyond last valid byte in prior malloc 
~~Dr.M~~ 
~~Dr.M~~ Error #2: UNADDRESSABLE ACCESS: reading 1 byte(s) 
~~Dr.M~~ # 0 replace_strlen      [d:\drmemory_package\drmemory\replace.c:375] 
~~Dr.M~~ # 1 XWORD::add_word_to_arr    [../mingwrt-4.0.3-1-mingw32-src/src/libcrt/crt/crt1.c:301] 
~~Dr.M~~ # 2 main         [../mingwrt-4.0.3-1-mingw32-src/src/libcrt/crt/crt1.c:302] 
~~Dr.M~~ 
~~Dr.M~~ Error #3: LEAK 0 bytes 
~~Dr.M~~ # 0 replace_operator_new_array    [d:\drmemory_package\common\alloc_replace.c:2638] 
~~Dr.M~~ # 1 XWORD::set_words       [../mingwrt-4.0.3-1-mingw32-src/src/libcrt/crt/crt1.c:301] 
~~Dr.M~~ # 2 main          [../mingwrt-4.0.3-1-mingw32-src/src/libcrt/crt/crt1.c:302] 
~~Dr.M~~ 
~~Dr.M~~ Error #4: LEAK 40 direct bytes + 110 indirect bytes 
~~Dr.M~~ # 0 replace_operator_new_array    [d:\drmemory_package\common\alloc_replace.c:2638] 
~~Dr.M~~ # 1 XWORD::init_arr       [../mingwrt-4.0.3-1-mingw32-src/src/libcrt/crt/crt1.c:301] 
~~Dr.M~~ # 2 main          [../mingwrt-4.0.3-1-mingw32-src/src/libcrt/crt/crt1.c:302] 
~~Dr.M~~ 
~~Dr.M~~ ERRORS FOUND: 
~~Dr.M~~  2 unique,  2 total unaddressable access(es) 
~~Dr.M~~  0 unique,  0 total uninitialized access(es) 
~~Dr.M~~  0 unique,  0 total invalid heap argument(s) 
~~Dr.M~~  0 unique,  0 total GDI usage error(s) 
~~Dr.M~~  0 unique,  0 total handle leak(s) 
~~Dr.M~~  0 unique,  0 total warning(s) 
~~Dr.M~~  2 unique,  2 total, 150 byte(s) of leak(s) 
~~Dr.M~~  0 unique,  0 total,  0 byte(s) of possible leak(s) 
~~Dr.M~~ Details: C:\Users\jesse\AppData\Roaming\Dr. Memory\DrMemory-test.exe.4512.000\results.txt 
~~Dr.M~~ WARNING: application exited with abnormal code 0xc0000005 
make: *** [runmem] Error 5

另外，当我在add_word_to_arr函数内打印this->words[0]的“％s”时，它是null。但它可以从print_words();打印。

的exe呼叫是：

bin\test.exe a123 b123

的init_arr()是：

void XWORD::init_arr(int ilen, int iwid) { 
    int y = 0; 
    int x = 0; 
    this->len = ilen; 
    this->wid = iwid; 

    this->arr = new char* [this->len]; 
    for (y=0; y<(this->len); y++) { 
     this->arr[y] = new char [1+this->wid]; 
     for (x=0; x<(this->wid); x++) { 
      this->arr[y][x] = BLANK; 
     } 
     this->arr[y][x] = (char) NULL; 
    } 
}

来源

2015-08-14 Jesse Knight

我建议你使用'std :: vector '，并且只留下手动内存管理。 – NathanOliver

谢谢，我最初会这么做，但我对2d向量表示法感到困惑 –

您不需要2d向量。 'std :: string'将代替'char *'，并将它包装在'std :: vector'中去掉一个'char *'数组。 – NathanOliver

确定init_arr(10,10)初始化尺寸11的10个字符数组的数组（10个字符+ 1个终止空）。

然后，您调用add_word_to_arr(0, 5, 5)，以便将第一个单词（命令行的第一个参数）复制到从位置5,5开始的数组中。所以如果这个词至少有7个字符与vdir == 0或6与vdir！= 0，你写过保留数组。我可以用params“foo”和“bar”成功运行，但不能与“feefoobar”运行。

您应该计算最长的单词的长度，并用它在调用int_arr

不管怎么说，它看起来您测试vdir值，而不会有任何初始化。这会导致不确定的行为，因为你不知道哪个分支（水平或垂直）将紧随其后，你真的应该重写add_word_to_arr这样：

void XWORD::add_word_to_arr(int iw, int iy, int ix) { 
    int k = 0; 
    int y = iy; 
    int x = ix; 

    for (k=0; k<(int)strlen(this->words[iw]); k++) { 
     this->arr[y][x] = this->words[iw][k]; 
     if (this->vdir) { 
      if (++y >= this.len) return; // refuse to write past allocated mem 
     } else { 
      if (++x > this.wid) return; // refuse to write past allocated mem 
     } 
    } 
}

我强迫VDIR值一次为0后执行它在调试器下和一次为1，并且从未访问过去分配的内存。但我不得不想象一些代码，因为你没有给一个Minimal Compilable Verifiable example

来源

2015-08-14 20:52:59

我希望它是那么简单...！上面我显示了makefile中的exe：'bin \ test.exe a123 b123' - 所以最长的单词只有'4 + null = 5'。事实上，当不使用Dr. Memory时程序运行时没有错误。但是，当我在Dr.M下运行时，会出现错误。我怀疑这与公私伙伴有关，甚至可能是Dr.M的错误......？ –

@JesseKnight在消除剩余的未定义行为原因后，我无法复制。看我的编辑 –

原来我是从我makefile运行我的记忆博士工具，忘了包括$(ARGS)在我make runmem目标：

FLAGS = -Wall -std=c++11 -pedantic 
LIBS = -Iinclude 
SRC = src/xword.cpp 
TGT = bin/test.exe 
ARGS = human prothean vorcha krogan asari salarian turian batarian quarian 

all: 
    g++ $(SRC) -o $(TGT) $(FLAGS) $(LIBS) 

run: 
    $(TGT) $(ARGS) 

runmem: 
    drmemory -batch $(TGT) **$(ARGS)** # was missing

这是为什么argv在Dr.Memory封装的执行过程中为空，以及为什么代码只会在这些过程中崩溃。

在此处将$(ARGS)添加到makefile之后，没有发生任何碰撞，也没有报告错误/泄漏。

抱歉鹅追逐，并感谢您的帮助 - 请让我知道如果这是多余的职位应该被删除。

来源

2015-08-18 02:10:37

从类成员函数中访问字符串数组

回答

相关问题