2
#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]

我一无所知。任何帮助将非常感激。

4

3 回答 3

10

代替

char *data = new char(16);

char *data = new char[16];

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

同样,替换

char *res = new char(10);

char *res = new char[10];
于 2013-03-06T08:07:38.847 回答
3

显然,正确的行为是不使用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)但更容易被编译器解析。

于 2013-03-06T08:20:47.857 回答
1

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

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

当你得到未定义的行为时,任何事情都可能发生。它不必是可重现的,所以它看起来很“奇怪”。

于 2013-03-06T08:17:14.287 回答