Forma correcta de inicializar un mapa y eliminar en C++

Question

Estoy intentando crear un mapa estático declarado en el constructor de mi clase. Este mapa se debe inicializar y completar con los datos en un método y se debe liberar en otro método. ¿Es esta la forma correcta de hacerlo?

using namespace std; 
#include <map> 

struct a { 
    string b; 
    string c; 
} 

class aClass:public myClass 
{ 
public: 
     aClass(); 
     virtual ~aClass(); 

private: 
     map<string, a> myMap; 
     void method(int a); 
     void amethod(int b); 
} 
void aClass::method(int a) 
{ 
     myMap = new map<string, a>; 
     // Addition of elements; 
} 
void aClass::amethod(int b) 
{ 
     // retrival of elements 
     myMap.clear(); 
     delete myMap; 
} 

Answer 1

map<string, a> myMap; 
    .... 
    myMap = new map<string, a>; 

Aquí myMap no es un puntero, por lo que la inicialización con new es incorrecto. Tal vez está buscando:

myMap = map<string,a>(); 

copiar en myMap un defecto inicializado mapa.

Tenga en cuenta que no necesita (y de hecho no puede) delete myMap, ya que no es un puntero. Es una variable miembro, y el compilador se encargará de destruirla automáticamente cuando se destruya tu clase.

Answer 2

void aClass::method(int a) 
{ 
    myMap.clear(); // ensure it starts off empty 
    // Addition of elements; 
} 
void aClass::amethod(int b) 
{ 
    // retrival of elements 
    myMap.clear(); // maybe not necessary 
} 

El objeto myMap ya existe dentro de una instancia de aClass y se construye cuando se construye su instancia que contiene. No es necesario que use new para crearlo, es una función de Java y C#, donde las variables son solo referencias a alguna instancia en el montón y todo está recolectado. En C++ es más fácil hacer que los miembros de datos sean un valor en lugar de un puntero o referencia a algún otro objeto.