Я пытаюсь подключить все утечки памяти (что является массовым). Я новичок в STL. У меня есть библиотека классов, где у меня есть 3 набора. Я также создаю много памяти с новыми в классе библиотеки для добавления информации к наборам ...Утечки памяти - наборы STL
Нужно ли мне освобождать комплекты? Если да, то как?
Вот 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();
};
Я не уверен, что мне нужно сделать для деструктора?
Library::~Library()
{
}
также, я не выделяю stringset правильно?
#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;
}
в классе библиотеки я создаю много памяти, но не деструктор убрать это выдумал? пример:
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);
Примечание: Я не конструктор копирования. Я не уверен, что мне даже нужен один или как добавить его. Я не думаю, что мои деструкторы вызваны либо.
Краткий ответ: найдите каждый экземпляр символа '*' в своем коде и удалите его. Вам вряд ли когда-либо понадобится использовать указатели на C++. Каждый раз, когда вы это делаете, вы просите утечки памяти. – jalf
Существует много раз, когда вам нужно использовать указатели на C++, даже если вы используете стандартные типы. – Xorlev
В этом случае 'Item' является базовым классом для сохраненных объектов, поэтому вам нужно хранить указатели в' ItemSet' и 'ItemMap'. Хранение интеллектуальных указателей устраняет эти утечки. 'ItemSetMap' должен содержать только заданные объекты. –