二重リンク リストのディープ コピーに問題があります。これは宿題なので、理解できないコードを動作させるよりも、コードが動作しない理由を知りたいです。
これが私のクラスです：
#include "textAnalyzer.h"
#include <string>
#include <iostream>
#include <sstream>
TextAnalyzer::Node* TextAnalyzer::createNode(const string& word, const int wordFrequency,
Node* const previous, Node* const next)
{
return new Node(word, wordFrequency, previous, next);
}
void TextAnalyzer::releaseNodes()
{
Node* del = tail;
while(tail != NULL)
{
tail = tail->previous;
tail->next = del;
delete del;
del = tail;
}
delete [] head;
delete [] tail;
head = tail = del = NULL;
}
void TextAnalyzer::copyNodes(Node* const copyHead)
{
head = new Node(*copyHead);
Node* iter = head->next;
for(Node* np = copyHead->next; np != NULL; np = np->next)
{
iter->next = new Node(*np);
iter = iter->next;
}
iter = NULL;
}
TextAnalyzer::TextAnalyzer():head(createNode("0",0,NULL,NULL)),tail(head)
{}
TextAnalyzer::TextAnalyzer(const TextAnalyzer& copyObject)
{
copyNodes(copyObject.head);
}
TextAnalyzer::~TextAnalyzer()
{
releaseNodes();
}
TextAnalyzer TextAnalyzer::operator=(const TextAnalyzer& assignObject)
{
return TextAnalyzer(assignObject);
}
void TextAnalyzer::insertWord(const string& word)
{
Node* iter = head->next;
while(iter != NULL)
{
if(iter->word == word)
iter->wordFrequency++;
else if(iter->word[0] == word[0] && iter->next != NULL)
{
Node* temp = iter->next;
iter->next = createNode(word, 1, iter, temp);
iter = iter->next;
temp->previous = iter;
temp = NULL;
}
else if(iter->word[0] == word[0] && iter->next == NULL)
{
iter = createNode(word, 1, tail, NULL);
tail = iter;
}
else
iter = iter->next;
}
iter = NULL;
}
int TextAnalyzer::wordCount() const
{
Node* iter = head->next;
int count = 0;
while(iter != NULL)
count++;
return count;
}
int TextAnalyzer::wordCountWithInitialCharacter(const char startsWith)
{
Node* iter = head->next;
int count = 0;
for(int i = 0; i < wordCount(); i++)
{
if(startsWith == iter->word[0])
count++;
iter->previous = iter;
iter = iter->next;
}
iter = NULL;
return count;
}
string TextAnalyzer::toString() const
{
Node* iter = head->next;
string desc = "List of words: \n";
ostringstream convert;
for(int i = 0; i < wordCount(); i++)
{
convert << iter->word[0] << " words:\n"
<< iter->word << "("
<< iter->wordFrequency
<< ")\n";
iter->previous = iter;
iter = iter->next;
}
iter = NULL;
return desc + convert.str();
}
インターフェースは次のとおりです。
#ifndef TEXTANALYZER_H
#define TEXTANALYZER_H
#include <iostream>
#include <string>
using namespace std;
class TextAnalyzer {
private:
/*
* Class: Node
*
* This class represents a node in a sorted doubly linked list that stores a
* list of words and their frequency of occurrence.
*/
class Node {
public:
string word;
int wordFrequency;
Node* previous;
Node* next;
Node(const string& word,
const int wordFrequency,
Node* const previous,
Node* const next)
: word(word),
wordFrequency(wordFrequency),
previous(previous),
next(next)
{}
}; // end ListNode
/*********************************************************************/
Node* head;
Node* tail;
/*
* Releases all the memory allocated to the list.
*/
void releaseNodes();
/*
* Makes a deep copy of the object.
*/
void copyNodes(Node* const copyHead);
/*
* Returns a populated Node.
* Throws a bad_alloc exception if memory is not allocated.
*/
Node* createNode(const string& word,
const int wordFrequency,
Node* const previous,
Node* const next);
public:
/*
* Initializes head and tail, each to a dymmy node.
*/
TextAnalyzer();
/*
* Makes a deep copy of the object passed in.
* Calls copyNodes() to do the actual work.
*/
TextAnalyzer(const TextAnalyzer& copyObject);
/*
* Releases all the memory allocated to the object.
* Calls the releaseNodes() method to do the actual work.
*/
~TextAnalyzer();
/*
* Makes a deep copy of the rhs object.
*/
TextAnalyzer operator =(const TextAnalyzer& assignObject);
/*
* Inserts the word in a sorted order into the list.
*
* If no Node exists with that initial character, one is added in
* sorted order. If one does exist (same word), then the word frequency
* of that word is incremented by one.
*/
void insertWord(const string& word);
/*
* Returns a count of all the words in the list.
*/
int wordCount() const;
/*
* Returns a count of all the words with the initial character.
*/
int wordCountWithInitialCharacter(const char startsWith);
/*
* Returns a description of the object. The string is formatted as:
* [A words:]
* [<word>(<count>)]
* [<word>(<count>)]
* ...
*
* [B words:]
* [<word>(<count>)]
* [<word>(<count>)]
* ...
*
*...
*/
string toString() const;
};
#endif
上記のインターフェースを使用する必要があります。私の問題は、コピーコンストラクターで「オブジェクトには互換性のない修飾子があります」などのエラーが表示されることです。これはcopyObjectが定数であるためだと思います。しかし、それ以外の方法でこれを行う方法について途方に暮れています。誰かがここで何が欠けているか教えてもらえますか? 私は C++ にはかなり慣れていませんが、Java の経験が豊富なので、混乱している可能性があります。
編集：
回答ありがとうございます。ディープコピーを成功させる方法を理解しようとしていたと思います。これまでに完了したことを示すためにコードを更新しました。コードをコンパイルしたので、新しいエラーが発生しました。実行するたびに「未処理の例外0xc0000005」。私はそれをグーグルで検索し、null ポインターを逆参照しようとしたために発生したエラーであると信じています。デバッガーは、それが私のreleaseNodes()メソッドでスローされたことを示しています。
void TextAnalyzer::releaseNodes()
{
Node* del = tail;
while(tail != NULL)
{
tail = tail->previous; //error on this line
tail->next = del;
delete del;
del = tail;
}
delete [] head;
delete [] tail;
head = tail = del = NULL;
}
上記は単なる私のreleaseNodes()方法であり、デバッガーがエラーの発生源を示している場所を示すコメントが付いています。私は C++ を初めて使用するので、コードの残りの部分が機能するかどうかを確認したいと思います。残念ながら、このエラーが解決されるまで、何もテストできません。何が原因であるかを見つけようとして、コードをまだトレースしています。誰かが私を正しい方向に向けることができれば、それはありがたいです。