Estoy tratando de tapar todas las fugas de memoria (que es masivo). Soy nuevo en STL. Tengo una biblioteca de clase donde tengo 3 juegos. También estoy creando mucha memoria con los nuevos en la clase de la biblioteca para agregar información a los conjuntos ...Fugas de memoria - conjuntos de STL
¿Debo desasignar los juegos? ¿Si es así, cómo?
Aquí es el library.h
#pragma once
#include <ostream>
#include <map>
#include <set>
#include <string>
#include "Item.h"
using namespace std;
typedef set<Item*> ItemSet;
typedef map<string,Item*> ItemMap;
typedef map<string,ItemSet*> ItemSetMap;
class Library
{
public:
// general functions
void addKeywordForItem(const Item* const item, const string& keyword);
const ItemSet* itemsForKeyword(const string& keyword) const;
void printItem(ostream& out, const Item* const item) const;
// book-related functions
const Item* addBook(const string& title, const string& author, int const nPages);
const ItemSet* booksByAuthor(const string& author) const;
const ItemSet* books() const;
// music-related functions
const Item* addMusicCD(const string& title, const string& band, const int nSongs);
void addBandMember(const Item* const musicCD, const string& member);
const ItemSet* musicByBand(const string& band) const;
const ItemSet* musicByMusician(const string& musician) const;
const ItemSet* musicCDs() const;
// movie-related functions
const Item* addMovieDVD(const string& title, const string& director, const int nScenes);
void addCastMember(const Item* const movie, const string& member);
const ItemSet* moviesByDirector(const string& director) const;
const ItemSet* moviesByActor(const string& actor) const;
const ItemSet* movies() const;
~Library();
};
No estoy seguro de lo que tengo que hacer por el destructor?
Library::~Library()
{
}
también, ¿estoy de allocating the stringset right?
#ifndef CD_H
#define CD_H
#pragma once
#include "item.h"
#include <set>
typedef set<string> StringSet;
class CD : public Item
{
public:
CD(const string& theTitle, const string& theBand, const int snumber);
void addBandMember(const string& member);
const int getNumber() const;
const StringSet* getMusician() const;
const string getBand() const;
virtual void print(ostream& out) const;
string printmusicians(const StringSet* musicians) const;
~CD();
private:
string band;
StringSet* music;
string title;
int number;
};
ostream& operator<<(ostream& out, const CD* cd);
#endif
cd.cpp
#include "CD.h"
using namespace std;
CD::CD(const string& theTitle, const string& theBand, const int snumber)
: Item(theTitle), band(theBand),number(snumber), music(new StringSet)
{
}
CD::~CD()
{
delete []music;
}
en la biblioteca de clases que estoy creando una gran cantidad de memoria, pero no te el destructor limpiar eso? ejemplo:
const Item* Library::addBook(const string& title, const string& author, const int nPages)
{
ItemSet* obj = new ItemSet();
Book* item = new Book(title,author,nPages);
allBooks.insert(item); // add to set of all books
obj->insert(item);
Nota: No tengo un constructor de copia. No estoy seguro si incluso necesito uno o cómo agregar uno. No creo que mis destructores sean llamados tampoco ...
Respuesta breve: Busque todas las instancias del carácter '*' en su código y bórrelo. Casi nunca necesita utilizar punteros en C++. Cada vez que lo haces, estás pidiendo pérdidas de memoria. – jalf
Hay muchas veces que necesita utilizar punteros en C++, incluso si usa tipos estándar. – Xorlev
En este caso, 'Item' es una clase base para los objetos almacenados, por lo que debe almacenar los punteros en' ItemSet' y 'ItemMap'. El almacenamiento de punteros inteligentes solucionará esas filtraciones. 'ItemSetMap' solo debería contener objetos establecidos. –