微信公众号搜"智元新知"关注
微信扫一扫可直接关注哦!

模板中的错误 std::bad_array_new_length

如何解决模板中的错误 std::bad_array_new_length

我试图定义自己的类模板 Array<T> 来练习模板的使用。
生成代码构建正确,但执行时出现以下错误

terminate called after throwing an instance of 'std::bad_array_new_length'
  what():  std::bad_array_new_length

我想我已经找到了该问题的解决方,但我很想知道之前的代码中是否存在潜在错误,如果存在,是哪个错误

这是我之前写的代码

#include <iostream>

class Empty{
private:
    char error;
public:
    Empty(char e) : error(e) { std::cout << "Azione non disponibile,lista vuota" << std::endl;}
};

template <class T>
class Array;

template <class T>
std::ostream& operator<<(std::ostream&,const Array<T>&);

template <class T>
class Array{
    friend std::ostream& operator<< <T> (std::ostream&,const Array<T>&);
private:
    T* arr;
    unsigned int size;
    unsigned int capacity;

    static T* copia(T* a,unsigned int s,unsigned int c){
        if(c > 0) {
            T* app = new T[c];
            for (int i = 0; i<s; ++i) {
                app[i] = a[i];
            }
            return app;
        }else return nullptr;
    }

public:
    Array(int k = 0,const T& t = T()) : size(k > 0 ? k : 0),capacity(size),arr(k > 0 ? new T[size] : nullptr){
        for (int i = 0; i < k ; ++i) arr[i] = t;
    }

    Array(const Array& a) : size(a.size),capacity(a.capacity),arr(copia(a.arr,a.size,a.capacity)){}

    Array& operator=(const Array& a){
        if(this != &a){
            delete[] arr;
            capacity = size = a.size;
            arr = copia(a.arr,a.capacity);
        }
        return *this;
    }

    ~Array(){delete[] arr;}

    void pushBack(const T& t) {
        if(size == capacity){
            capacity > 0 ? capacity *= 2 : capacity = 1;
            T* app = copia(arr,size,capacity);
            delete[] arr;
            arr = app;
        }
        ++size;
        arr[size-1] = t;
    }

    T popBack() {
        if (size != 0) {
            T temp = arr[size - 1];
            --size;
            return temp;
        } else throw Empty('e');
    }

};

template <class T>
std::ostream& operator<<(std::ostream& os,const Array<T>& a){
    for (int i = 0; i < a.size; ++i) {
        os << a.arr[i] << ' ';
    }
    std::cout << std::endl;
    return os;
}

int main(){

    Array<int> a(5,5),e;
    std::cout << a << std::endl;

    a.pushBack(16);
    a.pushBack(17);
    a.pushBack(18);

    std::cout << a << std::endl;

    return 0;
}

如果我在没有 a.pushBack(x) 函数调用的情况下运行此代码,它会起作用。
只要我插入一个函数调用,我就会在输出中收到该错误

在调试时,我发现我写的 T* arr 行不正确。
知道构造函数遵循其自身子对象的初始化顺序,首先要构造的元素是指针。 由于我正在尝试创建维度为 Tsize 元素向量,因此正确地给了我错误,因为我尚未初始化整数大小。
所以我通过换行解决了这个问题。

template <class T>
class Array{
    friend std::ostream& operator<< <T> (std::ostream&,const Array<T>&);
private:
    unsigned int size;
    unsigned int capacity;
    T* arr;
    ...
};

但此时我想知道:为什么,如果我不进行函数调用,我就不会得到同样的错误,知道构造时的大小是未定义的?

从逻辑上讲,在这种情况下也应该出现问题,但是似乎一切正常

enter image description here

PS:不要指望我没有处理抛出的异常,代码还没有完全完成,但目前我热衷于至少实现三规则。

解决方法

在 jdoodle.com 中使用 GCC 9.1.0 进行编译,我的原始代码始终出现 bad_alloc 运行时异常。

我添加了一个具有不同签名的新构造函数,以便我可以看到它用于分配数组的 size 的值 注意:即使这个新 ctor 的存在也阻止了 bad_alloc 错误,无论它是否被调用。

Array(char c,int k = 0,const T& t = T()) : 
    size(k > 0 ? k : 0),capacity(size),arr(DebugInit(size)){
    for (int i = 0; i < k ; ++i) arr[i] = t;
}

T* DebugInit( unsigned long size_init )
{
    std::cout << "DebugInit size_init=" << size_init << " cap=" << capacity << std::endl;
    return size_init > 0 ? new T[size_init] : nullptr;
}

使用这个新 ctor 时,结果似乎是随机的。 size_init 每次都不同,这与使用尚未初始化的成员字段一致。

在原始代码中,可能是 size 碰巧始终包含 0。 通过添加更多代码,即使从未调用过,编译后的版本也会访问一个随机但现在非零的 size 值。

这似乎是经典的“未定义行为”。如果在初始化之前使用 size,则无法保证其中的内容。 如果幸运的话,size 会在初始化之前始终返回 0,并且您会收到分配错误。 但是对代码的一个小改动可能会开始为 size 返回随机值。 如果未初始化的 size 的值比您预期的值大得多,您将不会在稍后或可能永远不会看到问题。

在对代码进行了更多操作之后,std::bad_alloc 异常又回来了!所以,绝对不能保证!

版权声明:本文内容由互联网用户自发贡献,该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 dio@foxmail.com 举报,一经查实,本站将立刻删除。