为什么这个非常简单的构造函数会导致seg错误?

时间:2013-09-22 16:53:35

标签: c++ constructor segmentation-fault template-classes

这很令人尴尬,但我的C ++上有点生疏,对于我的生活,我看不出为什么这段代码会导致分段错误。奇怪的是,这几次迭代之前似乎工作得很好。我现在甚至都不知道自己在做什么。

它基本上是模板类对象的构造函数,用于存放三个对象。我在构造函数的实现的第一行得到了一个seg错误(我将其分配给x *):

VecXd.hpp:

#ifndef JF_VecXd
#define JF_VecXd

#include <iostream>

template <class T>
class VecXd{
    public: 
        VecXd(T a, T b, T c);   

        VecXd(const VecXd & vector);

        T getElement(int n)const;
        void setElements(T a, T b, T c);
        void display();
        void VecXd<T>::clearElements();
        VecXd<T>& VecXd<T>::operator=(const VecXd& vector);

/*
        VecXd& VecXd::operator<<(const VecXd & vector);
        VecXd& VecXd::operator>>(const VecXd & vector);
        VecXd&  VecXd::operator+(const VecXd & vector);
        VecXd& VecXd::operator+=(const VecXd & vector);

        ~VecXd();
*/              

        private:
            T * x, T * y, T * z; 

};

template <class T> 
VecXd<T>::VecXd(T a, T b, T c){
    x = new T(a); 
    y = new T(b);
    z = new T(c);
}
//CVector& CVector::operator= (const CVector& param)


template <class T> 
VecXd<T>::VecXd(const VecXd & vector){
    x = new T(vector.getElement(0));
    y = new T(vector.getElement(1));
    z = new T(vector.getElement(2));
}

template <class T> 
VecXd<T>& VecXd<T>::operator=(const VecXd& vector){
    if(this != &vector){
        *x = vector.getElement(0);
        *y = vector.getElement(1);
        *z = vector.getElement(2);
    }
    return *this;
}

template <class T> 
T VecXd<T>::getElement(int n) const{
    n = n%3;
    T result;  
    switch(n){
        case 0: 
            result = *x;
            break;
        case 1:
            result = *y;
            break;
        case 2:
            result = *z;
            break;  
    }    
    return result;    
}

template <class T> 
void VecXd<T>::clearElements(){
    delete x; 
    delete y;
    delete z;    
}

template <class T>
void VecXd<T>::setElements(T a, T b, T c){
    clearElements(); 
    *x = a;
    *y = b;
    *z = c;    
}

template <class T>
void VecXd<T>::display(){
    std::cout << "x: " << x << "\n";
    std::cout << "y: " << y << "\n";
    std::cout << "z: " << z << "\n\n\n";
}

#endif

TEST.CPP:

#include "vecxd.hpp"
#include <iostream>

int main(){
    std::cout << "Creating vector 1..." << std::endl;
    VecXd<int> v1(1,2,3);
    std::cout << "Vector 1:" << std::endl; 
    v1.display();

    std::cout << "Vector 2 (copy-constructed from v1):" << std::endl;
    VecXd<int> v2(v1);
    v2.display();

    std::cout << "V1 set to 3,4,5:" << std::endl;
    v1.setElements(3,4,5);
    v1.display();

    std::cout << "V2 = V1, display V2" << std::endl;
 //   v2 = v1; 
    v2.display();


    system("pause");

    return 0;
}

我尝试了一些变体,包括

x* = a;

x = new(a);

我尝试过这样的功能:

VecXd(T & a, T & b, T & c);

但是它不会让我通过使用:

来调用它

VecXd<int>(1,2,3);

非常感谢您的时间,非常感谢!

1 个答案:

答案 0 :(得分:2)

您在分配前取消引用您的成员变量。直接将新指针分配给它们:

template <class T> 
VecXd<T>::VecXd(T a, T b, T c){
    x = new T(a); 
    y = new T(b);
    z = new T(c);
}

当然,除非你有一个析构函数,否则你会泄漏这个记忆。