クリフノート:
あるケースではメソッドチェーンが期待どおりに動作するようになりましたが、別のケースでは何かおかしなことが起こっています。
これら 2 つの例の出力はまったく同じであると
予想
されます。
私は Javascript を使用して多くの連鎖を行ってきたので、C++ でも同様の方法でクラス メソッドを連鎖できることを知ったとき、試してみたいと思いました。しかし、いくつかの問題に遭遇しました。チェーンが問題を引き起こしているのか、それとも何か他のものなのかはわかりません。
への参照を返すことで連鎖を行っていますthis
。例えば:
Class LLL
{
public:
LLL & addNode( ... );
そして add Node メソッドは次のように終了します。
...
return *this;
}
2 つの関連するクラスと 3 つの関連するメソッドがあります。
線形連結リスト クラスLLL
とノードクラスがありNode
ます。これをできるだけ単純にするために、ノードは単純にint
(と呼ばれるguid
) とnext
andprev
ポインターを保持します。線形連結リストは、多数のノードを LLL に配置します。これはすべて正常に機能しますが、チェーンを使用してノードを追加すると、時々奇妙な結果が得られます。
期待どおりに動作する例:
// Create 3 node objects & change their guids
Node node1; node1.change(1); // change(n) outputs "-n-"
Node node2; node2.change(2);
Node node3; node3.change(3);
// Create a linear linked list object
LLL lll;
// Add the nodes to the LLL and show the LLL
lll.addNode(node1).addNode(node2).addNode(node3).show();
// Expected and real output:
-1--2--3-123
このコードパッドで試してみてください
ここで、ノード オブジェクトを 1 つだけ使用してみました。
何が起こっているのか理解できない例:
Node nod;
LLL lll;
lll.addNode(nod.change(1)).addNode(nod.change(2)).addNode(nod.change(3)).show();
// Expected output:
-1--2--3-123
// Output:
-3--2--1-111
// Why are the changes being made in reverse order? And why do all three
// nodes have a guid of 1?
このコードパッドで試してみてください
上記の 2 つの例は同じ結果になるはずです。
2 番目の例で単一ノード オブジェクトに何が起こるかについて、私は何かを誤解しているに違いないと思います。
上記のコードパッドのいずれかでコードを確認できます。以下の 2 番目の例の完全なコードを含めます。コードパッドに配置できるように、コードを 1 つの巨大なファイルに設定しましたがnode.h
、node.cpp
、 、lll.h
、lll.cpp
、およびmain.cpp
:
#include <cstdlib>
#include <iostream>
#include <cstring>
#include <ctime>
using namespace std;
// NODE.H ======================================================================*/
// This class holds the data
class Node
{
public:
Node();
Node(int guidIn);
Node(const Node & nodeIn);
Node & change(int guidIn);
void commonConstructor();
// Get the next node
Node * next();
// Set the next node
void next(Node * nodeIn);
// Get the previous node
Node * prev();
// Set the previous node
void prev(Node * nodeIn);
Node & show();
private:
int guid;
Node * nextNode;
Node * prevNode;
};
/* EOF
=============================================================================*/
// LLL.H =======================================================================*/
// This is a LLL to hold nodes
class LLL
{
public:
LLL();
~LLL();
LLL & addNode(const Node & nodeIn);
LLL & show();
private:
Node * head;
};
/* EOF
=============================================================================*/
// NODE.CPP ====================================================================*/
Node::Node()
{
guid = 0;
commonConstructor();
}
Node::Node(int guidIn)
{
guid = guidIn;
commonConstructor();
}
Node::Node(const Node & nodeIn)
{
guid = nodeIn.guid;
commonConstructor();
}
Node & Node::change(int guidIn)
{
guid = guidIn;
cout << "-" << guidIn << "-";
return *this;
}
void Node::commonConstructor()
{
nextNode = NULL;
prevNode = NULL;
}
Node * Node::next()
{
return nextNode;
}
void Node::next(Node * nodeIn)
{
nextNode = nodeIn;
}
Node * Node::prev()
{
return prevNode;
}
void Node::prev(Node * nodeIn)
{
prevNode = nodeIn;
}
Node & Node::show()
{
cout << guid;
return *this;
}
/* EOF
=============================================================================*/
// LLL.CPP =====================================================================*/
LLL::LLL()
{
head = NULL;
}
LLL::~LLL()
{
Node * temp = head;
while(head)
{
temp = head;
head = head->next();
delete temp;
}
}
LLL & LLL::addNode(const Node & nodeIn)
{
Node * tempNode = new Node(nodeIn);
if(!head)
{
head = tempNode;
} else
{
Node * temp = head;
while(temp->next())
{
temp = temp->next();
}
temp->next(tempNode);
}
return *this;
}
LLL & LLL::show()
{
Node * temp = head;
while(temp)
{
temp->show();
temp = temp->next();
}
return *this;
}
/* EOF
=============================================================================*/
// MAIN.CPP ====================================================================*/
int main()
{
Node node;
LLL lll;
lll.addNode(node.change(1)).addNode(node.change(2))
.addNode(node.change(3)).show();
cout << "\n";
return 0;
}
/* EOF
=============================================================================*/