클래스의 동적 메모리 배열 초기화

Question

저는 C++ 프로그래밍을 처음 접했을뿐 아니라 사소한 문제 일 수도 있지만 클래스에 가변 크기의 배열을 만들고 텍스트 파일 데이터를이 클래스로 전송해야합니다 (아래 참조). 내가 통해 메인 프로그램에서 정보에 접근하려고하지만 경우 여기 HISTORYFile >> ClusterCoord[i];는

cout << CoordClassExample.ClusterCoord[1] << "\n"; 

내가 버스 오류, 정보의 벌금이 걸릴 것으로 보인다. 할 수 있으면 도와주세요! 당신은 두 번 함수를 호출하는 경우 당신이 어딘가에 리소스를 해제하지 않는 한

class CoordClass{ 
public: 
    int Entries; 
    double * ClusterCoord; 
    void set_valuesCoord(ifstream &HISTORYFile,int MolAtomNum, int MolNum); 
}; 

void CoordClass::set_valuesCoord(ifstream& HISTORYFile,int MolAtomNum, int MolNum) { 
    Entries=MolAtomNum*MolNum; 
    double *ClusterCoord = new double [Entries]; 

    for (int i=0;i<Entries;i++) { 
     HISTORYFile.ignore(1000,'\n');  
      HISTORYFile >> ClusterCoord[i]; 
     cout << ClusterCoord[i] << "\n"; 
      HISTORYFile.ignore(1000,'\n'); 
    } 
} 

Answer 1

당신은 set_valuesCoord() 기능에 누수가. 그것은 이 아니고 문제이지만 문제입니다. std::vector<>을 사용하십시오. 문제가 무엇인지

class CoordClass { 
    // ... 
    std::vector<double> ClusterCoord; // instead of double *ClusterCoord 
    // ... 
}; 

당신이 double가 제대로 구문 분석 여부를 확인하지 않는다는 것입니다. 그렇지 않은 경우 초기화되지 않은 메모리에 액세스하고 있으며 이로 인해 정의되지 않은 동작이 발생합니다.

또한 변경 항목을 제거하고 ClusterCoord.size()을 사용하는 것입니다 : 다른 것들 사이에 누출되지 않습니다 vector 방법을 보여주는 연습으로

void CoordClass::set_valuesCoord(...) 
{ 
    // ... 
    double cluster_coord = 0; 
    if(HISTORYFile >> cluster_coord) 
     ClusterCoord.push_back(cluster_coord); 
    else 
     std::cerr << "Error parsing cluster coord.\n"; 
    // ... 
} 

Answer 2

.

class CoordClass{ 
public: 
    int Entries; 
    std::vector<double> ClusterCoord; 
    void set_valuesCoord(ifstream &HISTORYFile,int MolAtomNum, int MolNum); 
}; 

void CoordClass::set_valuesCoord(ifstream& HISTORYFile,int MolAtomNum, int MolNum) { 
    Entries=MolAtomNum*MolNum; 
    ClusterCoord.resize(Entries); 

    for (int i=0;i<Entries;i++) { 
     HISTORYFile.ignore(1000,'\n');  
      HISTORYFile >> ClusterCoord[i]; 
     cout << ClusterCoord[i] << "\n"; 
      HISTORYFile.ignore(1000,'\n'); 
    } 
}