2

私は現在、ノードを使用してリンクリストプログラムを作成しています（他の方法はわかりません）。ディープコピーを作成し、〜List（）を使用してすべてのノードとセンチネルを削除する際に問題が発生しました。ノードの削除は問題ではありませんが、最初のノードにはインデックス値が割り当てられていないため、センチネルは問題になります。

List::~List()
{
  for(size_t i=0; i<size; i++)
    {
      _setCurrentIndex(i);
      if(current && curent->next == NULL)
    {
      Node *temp = current->next;
      delete temp;
      delete current;
    }
      else
    {
      Node *old = current;
      current = current->next;
      delete old;
    }
    }
}

List::List(const List & orig)
{
for(size_t i=0; i<size; i++)
{
 if(i==0)
  {
   Node *copyFront = new Node; //the first sentinel
   copyFront->data = orig.front->data; //front is defined in private in list.h
   copyFront->prev = NULL; // it is defined as a Node (same for rear)
  }
 else if(0<=i && i<size) //put in i<size b/c 0<=i would always be true
  {
   _setCurrentIndex(i) //sets what current is and currentIndex which pts to diff Nodes
   Node *copy = new Node;
   copy->data = current->data; 
   copy->next = current->next;
   current = current->next;
  }
 else if(i+1 == size)
  {
   Node *copyRear = new Node; //making the last sentinel, but it has to be
   copyRear->data = orig.rear->data; //after data Node
   copyRear->next = NULL;
  }
 }
}

次に進む方法や、何かがひどく間違っている場合に何を変更するかについて、このコードに関するアドバイスとコメントを求めています！

4

1 に答える 1

0

リンクリストは、あらゆるタイプの変数をリストに含めることができるテンプレートです。私の正直な意見では、ヘッダーファイルstd::listを必要とするを使用するのが最善でしょう。#include <list>

もちろん、リンクリストクラスを自分で作成する体験が本当に必要な場合は、次のコードでリストのディープコピーを作成します。

List::List( const List& other) {
    if( other.head_ != nullptr) {
        head_ = new Node( other.head_->item_);  // copy first node

        assert( head_ != nullptr);  // ensure that the memory was allocated correctly

        // copy the rest of the list
        Node* pnew = head_;
        // loop through the list until you reach the end (i.e. a node that's nullptr)
        for( Node* porig( other.head_->next_); porig != nullptr; porig = porig->next_) {
            // assign the next node in the destination list to the next node in the paramter's list
            pnew->next_ = new Node( porig->item_);
            assert( pnew->next_ != nullptr);  // ensure that the memory was allocated correctly
            pnew = pnew->next_;  // move onto the newly created node in the destination list
        }
    }
    else
        // if the parameter is empty then the destination list will be empty as well
        head_ = nullptr;
}

デストラクタについては、リストを実行してノードを削除する必要があります。

List::~List() {
    while( head_ != nullptr) {  // keep looping until the list gets to the end
        // make a second pointer to the node you are about to delete (so you don't lose track of it)
        Node* pn( head_);
        // move the head_ onto the next node essentially "removing" the first node from your list
        head_ = head_->next_;
        // delete the node that you've just "removed" from your list
        delete pn;
    }
}

私はコメントが不明確であるかもしれない何かを明らかにするように試みました。

于 2012-10-26T18:01:33.533 に答える