Невозможно распечатать переменную-член класса C++

Question

Я пытаюсь изучить некоторые методы ООП при разработке модели для моих исследований. Кажется, я сталкиваюсь с несколькими проблемами, которые я действительно не уверен в том, что касается той книги, которую я использую, C++ Primer от Stephen Prata.

Прямо сейчас, я пытаюсь разработать объект, называемый StochasticModel. Этот объект SM должен иметь возможность принимать другой объект с именем LengthDistribution и запускать симуляцию на нем. В настоящее время я настраиваю рамки для этого типа взаимодействия со следующим псевдокодом. Как предостережение, я объединил определения классов, поскольку я изначально имел их в двух файлах каждый. Вы также увидите некоторые CudaMallocManaged(), поскольку я намерен использовать эти объекты на графическом процессоре, но CUDA здесь не проблема.

int main() 
{ 

//Read initial conditions, maxlength, maxiterations from input 

//Create a length distribution object from IC's and length 
LengthDistribution* LD = new LengthDistribution(initialconditions, maxlength) 

//Create a model object to operate on LD maxiterations number of times. 
StochasticModel* SM = new StochasticModel(LD, maxiterations) 

//Test that I've created an LD object as I expected by printing its values out. 
LD -> h_printToScreen(); //<- Works fine! 

//Test that model has a correct version of LD by printing LD's information to screen through 
//SM's print to screen function. 
SM->printToScreen(); //<- But when I have an object that contains the object call a function that calls its function (inception), memory access violations occur. 

} 

My Length Distribution class.

class LengthDistribution : public Managed 
{ 
private: 
    int m_maxlength; 
    int* m_lengthDistribution; 

public: 
    //User defined constructor. 
    LengthDistribution(int* initialconditions, int maxlength) 
    { 
    m_maxlength = maxlength; 
    m_lengthDistribution = new int[maxlength]; 
    m_lengthDistribution = initialconditions; 
    } 

    //Default constructor 
    //Default destructor 
    //Unified memory copy constructor allows pass-by-value 
    LengthDistribution::LengthDistribution(const LengthDistribution &LD) 
    { 
    //Copy maxlength to new state. 
    m_maxlength = LD.m_maxlength; 

    //Allocate CUDA Managed memory for lengthDistribution array 
    cudaMallocManaged(&m_lengthDistribution, m_maxlength); 

    //Copy array to new state. 
    memcpy(m_lengthDistribution, LD.m_lengthDistribution, m_maxlength); 

    } 
    __host__ void h_printToScreen() 
    { 
     printf("Host maxlength: "); 
     std::cout<<m_maxlength; 
     std::cout<<"\n"; 
     printf("Host length distribution: "); 
     for (int i = 0; i < m_maxlength; i++) 
      std::cout<<m_lengthDistribution[i]; 
     printf("\n"); 
    } 

} 

Мой Стохастический класс Модель

class StochasticModel : public Managed 
{ 

private : 
    int m_numberOfIterations; 
    LengthDistribution* state; 

public: 
    //User defined constructor 
    StochasticModel(LengthDistribution* LD, int numberOfIterations) 
    { 
     //Copy desired number of iterations. 
     m_numberOfIterations = numberOfIterations; 

     //Assign LD to SM's state variable. I think I'm having an issue here. 
     //Copy LD into SM object's state variable. 
     LengthDistribution* state = new LengthDistribution(*LD); 
    } 

    //User defined copy constructor 
    Stochastic::Model(const StochasticModel &SM) 
    { 
     m_numberOfIterations = SM.m_numberOfIterations; 
     cudaMallocManaged(&state, sizeof(SM.state)); 
     state = new LengthDistribution(*SM.state); 
     //memcpy(state, SM.state, sizeof(state)); 
    } 

    //Print out member length distribution's values. 
    printToScreen() 
    { 
    state->h_printToScreen(); //When I trace the debugger through here, it triggers an access violation when trying to print out the state's array. 
    } 
} 

Это что управляемый класс, что два других класса наследуют от. Его цель - разрешить перенос на графическую карту в будущем.

class Managed { 
public: 
    void *operator new(size_t len) { 
    void *ptr; 
    cudaMallocManaged(&ptr, len); 
    return ptr; 
    } 

    void operator delete(void *ptr) { 
    cudaFree(ptr); 
    } 
}; 

Конечные результаты состоят в том, что приложение после запуска «перестало работать». Он компилируется отлично, но когда он попадает на ошибку доступа к памяти для состояния < -printToScreen, он «перестает работать». Пройдя через отладчик, он говорит, что массив, который он пытается распечатать для объекта LD и целого, не определен. Я чувствую, что мне не хватает чего-то элементарного здесь, но я хотел бы иметь возможность использовать объект SM для манипулирования и отображения информации из объекта LD.

Answer 1

В вашем конструкторе вы создали новую локальную переменную, которая скрывает переменную-член state. Это оставляет переменную-член неопределенной, что приводит к сбою позже, когда вы пытаетесь получить к ней доступ.

Вы должны предпочесть использовать список инициализации вместо:

StochasticModel(LengthDistribution* LD, int numberOfIterations) : 
    m_numberOfIterations(numberOfIterations), 
    state(new LengthDistribution(*LD)) 
{ 
}