コミット: 2f5623d38805ae9d31bae63d2acf87afabe48407 - star (git)

コミットメタ情報

• リビジョン: 2f5623d38805ae9d31bae63d2acf87afabe48407 (tree)
• 日時: 2020-11-22 21:09:30
• 作者: dhrname <dhrname@user...>
• コミッター: dhrname

ログメッセージ

Modify the removeChild method

変更サマリ

• modified: joestar/cpp/main.cpp
• modified: source_code/ntree.c

差分

--- a/joestar/cpp/main.cpp
+++ b/joestar/cpp/main.cpp
@@ -163,20 +163,25 @@ void BaseNode::setParent(Node* node)
 
 void BaseNode::setNext(Node* node)
 {
+  this->throwNULLArgumentError(node, "setNext");
   this->nextSibling = node;
 }
 
 void BaseNode::setPrev(Node* node)
 {
+  this->throwNULLArgumentError(node, "setPrev");
   this->previousSibling = node;
 }
 
 void BaseNode::setFirstChild(Node* node)
 {
+  this->throwNULLArgumentError(node, "setFirstChild");
   this->firstChild = node;
 }
 
-void BaseNode::setLastChild(Node* node){
+void BaseNode::setLastChild(Node* node)
+{
+  this->throwNULLArgumentError(node, "setLastChild");
   this->lastChild = node;
 }
 
@@ -207,6 +212,17 @@ Node* BaseNode::removeChild(Node* const child)
   
   child->setParent(emptynode);
   
+  if (this->lastChild == child)
+  {
+      /*末子ノードがchildである場合は自分のlastChildメンバを書きかえておく*/
+      this->setLastChild(child->getPrev());
+  }
+  if (this->firstChild == child)
+  {
+      /*長子ノードがchildである場合は自分のfirstChildメンバを書きかえておく*/
+      this->setFirstChild(child->getNext());
+  }
+  
   /*nodeが抜けた後、隣接ノードに関するメンバは書き換えておく*/
   if (child->getPrev()->isNode())
   {
@@ -216,12 +232,6 @@ Node* BaseNode::removeChild(Node* const child)
   {
       child->getNext()->setPrev(child->getPrev());
   }
- 
-  if (this->getLastChild() == child)
-  {
-      /*末尾ノードがchildである場合は自分のlastChildメンバを書きかえておく*/
-      this->lastChild = child->getPrev();
-  }
       
   child->setNext(emptynode);
   child->setPrev(emptynode);

--- a/source_code/ntree.c
+++ b/source_code/ntree.c
@@ -156,7 +156,7 @@ ST_Node ST_removeChild(ST_Node parent, ST_Node child)
    
     if (ST_getLastChild(parent) == child)
     {
-        /*末尾ノードがnodeである場合はparentノードのメンバを書きかえておく*/
+        /*末尾ノードがchildである場合はparentノードのメンバを書きかえておく*/
         parent->lastChild = ST_getPreviousNode(child);
     }