Bonne façon d'initialiser une carte et de supprimer, en C++

Je suis en train de créer une carte statique déclarées dans le constructeur de ma classe.
Cette carte doit être initialisé et rempli avec des données dans une méthode et gratuit d une autre méthode.
Est-ce la bonne façon de le faire?

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;
 }
  • Dans votre code, le myMap n'est pas static. Que voulez-vous dire par "statique"?
  • Pourquoi pensez-vous que vous avez besoin à nouveau d'une carte et de le supprimer par la suite? Il suffit de tenir une carte en valeur et à l'appel clear() sur celui-ci au moment approprié.
  • Je doute que vous vouliez inclure l' [tsl] balise. J'ai édité ton post pour y inclure l' [tsl] tag à la place.
InformationsquelleAutor snihalani | 2012-06-06
  1. 9
      map<string, a> myMap;
  ....
  myMap = new map<string, a>;

    Ici myMap n'est pas un pointeur, donc l'initialisation avec new est incorrect. Peut-être que vous êtes à la recherche pour:

      myMap = map<string,a>();

    à copier dans myMap un initialisé par défaut carte.

    Notez que vous n'avez pas besoin (et, en fait, ne peuvent pas) delete myMap, que ce n'est pas un pointeur. C'est une variable membre, et le compilateur va prendre en charge automatiquement de le détruire lors de votre classe est détruite.

    • Comment puis-je gratuit il?
    • Vous n'avez pas gratuit, il est, ce n'est pas un pointeur... peut-être tout ce que vous voulez est de changer myMap être map<string,a>*?
    • vous n'avez pas gratuit, il est, C++ va le libérer lorsque le contenant aClass est détruit
    • Ce sera utilisée plusieurs fois, 200 fois, il n'y a pas de problèmes avec la bonne ness de ce code, non?
    • Il n'y a pas de questions, cependant à la réinitialisation de la carte dans method semble inutile, car il sans doute a déjà été nettoyé.
    InformationsquelleAutor K-ballo
  2. 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
}

    L'objet myMap existe déjà à l'intérieur d'une instance de aClass et est construit lors de son contenant instance est construit. Vous n'avez pas besoin d'utiliser new pour la créer, c'est une Java et C#) où les variables sont juste des références à quelques exemple sur le tas et tout est le garbage collector. En C++ il est plus facile de rendre les données membres d'une valeur plutôt qu'un pointeur ou une référence à un autre objet.

    InformationsquelleAutor Jonathan Wakely