Question

我正在尝试制作模板类。它似乎没问题，但是当我尝试创建一个指向myclass'对象的指针时，会发生核心转储错误。

#include <iostream>
#include <stdlib.h>

template <typename T>
class vec {
private:
    T* m_pnt;
    int m_dim;

public:
    vec();
    vec(int);
    ~vec();

    void prnt();
    void reduce(int);
    void null();

    void set(int, T);
    void print();
};

template <typename T>
vec<T>::vec() {
    m_dim = 100;
    m_pnt = (T*)calloc(100, sizeof(T));
    std::cout << "building without par" << std::endl;
}

template <typename T>
vec<T>::vec(int n) {
    m_dim = n;
    m_pnt = (T*)calloc(m_dim, sizeof(T));
    std::cout << "building" << std::endl;
}

template <typename T>
vec<T>::~vec() {
    free(m_pnt);
    std::cout << "killed" << std::endl;
}

template <typename T>
void vec<T>::reduce(int dim) {
    std::cout << "reduce" << std::endl;
    if (dim <= m_dim) {
        m_dim = dim;
        T* p = (T*)calloc(m_dim, sizeof(T));
        for (int i = 0; i != m_dim; i++)
            *(p + i) = *(m_pnt + i);

        free(m_pnt);

        m_pnt = p;
    } else {
        std::cout << "error: rowvector not valid dimension" << std::endl;
        throw;
    }
}

template <typename T>
void vec<T>::set(int idx, T val) {
    if (idx >= 1 && idx <= m_dim)
        *(m_pnt + idx - 1) = val;
    else {
        std::cout << "error: rowvector not valid index" << std::endl;
        throw;
    }
}

template <typename T>
void vec<T>::prnt() {
    for (int i = 0; i != m_dim; i++)
        std::cout << i + 1 << "\t" << m_pnt + i << "\t" << *(m_pnt + i)
                  << std::endl;
}

template <typename T>
void vec<T>::null() {
    for (int i = 1; i <= m_dim; i++)
        *(m_pnt + i - 1) = T(0);
}

int main() {
    vec<int>* d;
    std::cout << "d: " << d << std::endl;
    d->prnt();

    return 0;
}

编译时没有错误，但最终输出类似于

    d: 0
    segmentation fault (core dumped)

Answer 1

你从未分配过你的指针：

vec<int>* d = new vec<int>;

您当前的代码：

vec<int>* d;

只需定义一个变量d，其中包含指向vec<int>对象的指针。但它无处可指。要使其指向某些内容，您必须使用vec<int>在内存中构建new vec<int>

Answer 2

你应该使用C ++样式分配和解除分配（new和 delete）代替calloc和free。这是为了确保适当的对齐。
您不应使用int来保持向量的大小，而应使用无符号类型。 C ++标准是使用std::size_t。
你不能指望一个未初始化的指针（d）指向什么有用的。因此使用它会导致错误（或更糟：无意义的行为）。
查看std::vector<>的任何实现，了解事情是如何以专业方式完成的。
您的代码不是例外安全：如果calloc的分配失败，m_dim仍会显示初始维度。

指向类的指针创建错误核心转储

2 个答案: