지도 C 반복을 통해

Question

다항식의 구성 요소를 저장하는지도가 있습니다. 키는 지수이고 값은 계수입니다. 정수 키와 정수 값을 갖는 맵. 내 인쇄 기능에서 iterating 오전, 내 반복 할 때 프로그램이 충돌합니다. 키를 배열에 집어 넣는 동안 모든 것이 체크 아웃 된 것처럼 보였습니다. 입력 파일이 각 쌍은 형식 -1 0 5 1 20 -2 3 -3 2 1 1 2 3 1 -2 9 (계수 지수) .`

//header 
#ifndef poly_h 
#define poly_h 
#include <map> 
class poly { 

private: 
    std::map<int, int>* pol; 

public: 
    poly(char* filename); 
    poly& operator+(poly& rhs); 
    poly& operator-(poly& rhs); 
    poly& operator*(poly& rhs); 
    void print(); 
    ~poly(); 
}; 
#endif 

//cpp file 
#include "poly.h" 
#include <iostream> 
#include <fstream> 
using namespace std; 
poly::poly(char* filename) 
{ 

    map<int, int>* pol = new map<int, int>; 
    ifstream input_file; 
    input_file.open("input.txt"); 
    int coe; 
    int exp; 

    while (input_file >> coe) { 
     input_file >> exp; 
     cout << coe << " ^" << exp << " "; 
     map<int, int>::iterator it = pol->find(exp); 
     if (it == pol->end()) { 
      (*pol)[exp] = coe; 
     } 
     else { 
      (*pol)[exp] += coe; 
      if ((*pol)[exp] == 0) { 
       pol->erase(it); 
      } 
     } 
     cout << (*pol)[exp]; 
     //print(); 
     cout << endl; 
    } 
    input_file.close(); 
} 
void poly::print() 
{ 
    cout << "inside print<<endl;                 
     for (map<int, int>::iterator outer_iter = pol->begin(); outer_iter != pol->end(); ++outer_iter); 
    cout << "done"; 
} 
poly::~poly() 
{ 
    delete pol; 
} 

을 갖는다

Answer 1

라인

map<int, int>* pol = new map<int, int>; 

가 범인입니다. 함수 로컬 변수를 만듭니다. 멤버 변수의 값을 설정하지 않습니다. 결과적으로 동일한 이름의 멤버 변수는 초기화되지 않은 상태로 유지됩니다. 다른 함수에서 해당 변수를 역 참조하면 정의되지 않은 동작이 발생합니다. 에 라인

변경 :

pol = new map<int, int>; 

---

으로는 난 강력 객체에 대한 포인터에서 해당 멤버 변수를 변경 권합니다, 의견 중 하나에 제안했다. 자동 메모리 관리는 객체를 사용하지 못하게합니다. 메모리를 할당하기 위해 new을 사용하고 그것에 사용되는 메모리를 할당 해제하기 위해 delete을 사용하는 것에 대해 걱정할 필요가 없습니다. 뿐만 아니라 학급에서 메모리를 할당하는 일을 맡는다면 The Rule of Three에 대해 알아야하며 수업에서 해당 규칙을 확인해야합니다.

C++ 11 컴파일러를 사용하는 경우 The Rule of Three은 The Rule of Five이됩니다.