Question

#include <iostream>

using namespace std;

int main (int args, char **argv) {
    char *data = new char(16);
    for (int i = 0; i < 16; ++i) {
        data[i] = i; // works fine when commented out, also fails when data[i] = 0
    }

    char *res = new char (10);
    delete[] res;
    return 0;
}

给出错误，请亲自看看：http://ideone.com/AgZhZB

*** glibc detected *** ./a.out: free(): invalid next size (fast): 0x09377018 ***
======= Backtrace: =========
/lib/i386-linux-gnu/libc.so.6(+0x75ee2)[0xb7519ee2]
/usr/lib/i386-linux-gnu/libstdc++.so.6(_ZdlPv+0x1f)[0xb76f751f]
/usr/lib/i386-linux-gnu/libstdc++.so.6(_ZdaPv+0x1b)[0xb76f757b]
./a.out[0x80485af]
/lib/i386-linux-gnu/libc.so.6(__libc_start_main+0xf3)[0xb74bd4d3]
./a.out[0x80484b1]

我没有线索。任何帮助将不胜感激。

Answer 1

替换

char *data = new char(16);

与

char *data = new char[16];

您现在拥有的内容会分配一个char并将其初始化为16。要分配数组，必须使用如上所示的方括号。

同样，替换

char *res = new char(10);

与

char *res = new char[10];

Answer 2

显然，正确的行为不使用new 。

#include <vector>

int main () {
    std::vector<char> data{16};
    for (int i = 0; i < 16; ++i) {
        data.at(i) = i;
    }

    std::vector<char> res{10};
    return 0;
}

注意：data{10}表示法是C ++ 11初始化语法，它大部分等同于data(10)，但更容易被编译器解析。

Answer 3

您的代码中唯一可能“奇怪”的是您正在调用未定义的行为，首先是通过越界访问：

char *data = new char(16); // pointer to single char, value 16
for (int i = 0; i < 16; ++i) {
    data[i] = i; // out of bounds access: UNDEFINED BEHAVIOUR
}

然后，当您需要delete[]时调用delete：

char *res = new char (10); // pointer to single char, value 10
delete[] res; // Oops, UNDEFINED BEHAVIOUR

当你得到未定义的行为时，任何事情都可能发生。它不必具有可重复性，因此看起来很“奇怪”。

奇怪的C ++指针新/删除问题

3 个答案: