Структуры и двойные указатели внутри других структур, многослойная путаница
Таким образом, у меня есть три структуры с функциями, чтобы согласиться с ними...
struct Song{
std::string songTitle;
std::string songLength;
};
Song* createSong(string title, string length);
void displaySong(Song* s);
void destroySong(Song* s);
struct CD{
std::string cdArtist;
std::string cdTitle;
int cdYear;
int cdRate;
int cdNumSongs;
Song** songs;
};
CD* createCD(std::string artist, std::string title, int year, int rate, int numSongs);
void displayCD(CD* c);
void destroyCD(CD* c);
void addSong(CD* cd, std::string title, std::string length);
struct CDs{
CD** cdArray;
int cdMaxSize=1000;
int cdCurrentSize=0;
};
CDs* createCDs (const char* filename);
void displayCDs (CDs* c);
void destoryCDs (CDs* c);
Я создал и протестировал функции struct Song, а также функции struct CD. Однако у меня возникают проблемы с реализацией компакт-дисков * createCD (const char* filename), которые извлекают данные из файла и создают из этих компакт-дисков динамически сохраняемый массив.
Пока это мой код, я проверил на наличие висячих указателей, и когда мой код компилируется, он отлично работает. Это только когда бежит, он останавливается на несколько секунд и выходит. Чтение данных из текстового файла также было проверено, и эта часть работает сама по себе.
Код до сих пор...
CDs* createCDs(const char* filename){
CDs* cds = new CDs;
ifstream inFile(filename);
CD* tempcd;
CD** tempcdArray = new CD*[cds->cdCurrentSize];
CD** tempcdNewArray;
string tempArtist, tempTitle, tempSongTitle, tempSongLength;
int tempYear, tempRate, tempNumSongs;
while(getline(inFile, tempTitle)){ // WHEN IT GRABS NOTHING
cds->cdMaxSize++;
tempcdNewArray = new CD*[cds->cdMaxSize];
for(int i=0; i<cds->cdCurrentSize; i++){
tempcdNewArray[i] = tempcdArray[i];
delete[] tempcdArray;
}
getline(inFile, tempArtist);
inFile>>tempYear;
inFile>>tempRate;
inFile>>tempNumSongs;
inFile.ignore();
tempcd = createCD(tempArtist, tempTitle, tempYear, tempRate, tempNumSongs);
for(int i=0; i<tempNumSongs; i++){
getline(inFile, tempSongLength, ',');
getline(inFile, tempSongTitle);
addSong(tempcd, tempSongTitle, tempSongLength);
}
tempcdNewArray[cds->cdCurrentSize] = tempcd;
tempcdArray = new CD*[cds->cdMaxSize];
for(int i=0; i<cds->cdMaxSize; i++){
tempcdArray[i] = tempcdNewArray[i];
delete tempcdNewArray;
}
delete[] tempcd;
cds->cdCurrentSize++;
}
for(int i=0; i<cds->cdCurrentSize; i++){
cds->cdArray[i] = tempcdArray[i];
}
inFile.close();
return cds;
}
В случае, если вам нужно знать, один компакт-диск в макете текстового файла..
Eternal Tears of Sorrow (cd title)
Saivon Lapsi (cd artist)
2013 (cd year)
7 (cd rating)
13 (number of songs)
1:10,Saivo (length, title)
... (other songs after here)
Я попытался отследить свой код для висячих указателей, я удостоверился, что "новый" все разделы памяти, которые я возвращаю… и т. Д.
1 ответ
Я бы держался подальше от new/delete как можно дольше, и я бы также переместил функции, работающие с вашими структурами, в структуры или, по крайней мере, сделал их дружественными функциями. Используйте стандартный контейнер (например, std::vector) для хранения. Я заменил ваши функции для чтения и отображения объектов на операторы потоков, чтобы упростить чтение / запись с использованием стандартных потоков. Предполагается, что файлы будут прочитаны в указанном вами формате, а затем переданы созданные объекты в std::cout того же формата, что и в исходных файлах.
#include <iostream>
#include <vector>
#include <fstream>
#include <iterator>
class Song {
std::string m_title;
std::string m_length;
public:
Song() : m_title(), m_length() {}
friend std::ostream& operator<<(std::ostream&, const Song&);
friend std::istream& operator>>(std::istream&, Song&);
};
std::ostream& operator<<(std::ostream& os, const Song& song) {
os << song.m_length << "," << song.m_title;
return os;
}
std::istream& operator>>(std::istream& is, Song& song) {
std::string tmp;
std::getline(is, tmp);
std::string::size_type comma = tmp.find(",");
if(comma != std::string::npos) {
std::copy(tmp.cbegin(), tmp.cbegin()+comma, std::inserter(song.m_length, song.m_length.end()));
std::copy(tmp.cbegin()+comma+1, tmp.cend(), std::inserter(song.m_title, song.m_title.end()));
}
return is;
}
class CD {
std::string m_artist;
std::string m_title;
int m_year;
int m_rate;
std::vector<Song> m_songs;
public:
CD() :
m_artist(),
m_title(),
m_year(),
m_rate(),
m_songs()
{}
size_t size() { return m_songs.size(); }
friend std::ostream& operator<<(std::ostream&, const CD&);
friend std::istream& operator>>(std::istream&, CD&);
};
std::ostream& operator<<(std::ostream& os, const CD& cd) {
os << cd.m_title << "\n" << cd.m_artist << "\n" << cd.m_year << "\n"
<< cd.m_rate << "\n" << cd.m_songs.size() << "\n";
for(const Song& song : cd.m_songs) {
os << song << "\n";
}
return os;
}
std::istream& operator>>(std::istream& is, CD& cd) {
int no_songs;
std::getline(is, cd.m_title);
std::getline(is, cd.m_artist);
is >> cd.m_year;
is >> cd.m_rate;
is >> no_songs;
is.ignore(); // discard rest of line
cd.m_songs.reserve(no_songs);
while(no_songs--) {
Song a_song;
is >> a_song;
cd.m_songs.emplace_back(std::move(a_song));
}
return is;
}
class CDs {
std::vector<CD> m_cds;
public:
CDs() : m_cds() {}
CDs(const std::string& Filename) :
m_cds()
{
std::fstream cdfile(Filename);
cdfile >> *this;
}
size_t size() { return m_cds.size(); }
friend std::ostream& operator<<(std::ostream&, const CDs&);
friend std::istream& operator>>(std::istream&, CDs&);
};
std::ostream& operator<<(std::ostream& os, const CDs& cds) {
for(const CD& cd : cds.m_cds) os << cd;
return os;
}
std::istream& operator>>(std::istream& is, CDs& cds) {
while( !is.eof() ) {
CD a_cd;
is >> a_cd;
if( !is.eof() ) cds.m_cds.emplace_back(a_cd);
}
return is;
}
int main(int argc, char* argv[]) {
std::vector<std::string> args(argv+1, argv+argc);
CDs cds;
for(const std::string& file : args) {
std::fstream fs(file);
fs >> cds;
}
std::cerr << "Number of CDs: " << cds.size() << "\n";
std::cout << cds;
return 0;
}