[C++] 単方向リストのプログラム

Question

単方向リストをC++で書いています。
恐らくfor文の書き方が理解できていないためprint関数がうまく実装できないのですが、
詳しい方がいらっしゃいましたら教えていただきたいです。

ソースコード

#include <iostream>

using namespace std;

class Node {
private:
    // BEGIN
	int value;
	Node* next;
    // END
public:
    Node(int nodeValue, Node* list) {
        // BEGIN
    	this->value=nodeValue;
    	this->next = list;
        // END
    }
    void print() const {
        // BEGIN
    	for(Node* n = this->next; n != NULL; n = n->next){
			cout << this->value << endl;
    	}
        // END
    }
};

int main() {
    Node* list = new Node(1, new Node(2, new Node(3, new Node(4, NULL))));
    list->print();
    // deleteは省略
    return 0;
}

実行結果

1
1
1

Answer 1

cout << this->value << endl;
これだとthis->valueばかり表示するから，
cout << n->value << endl;
にする．

print()を呼ばれたインスタンスのvalueも表示するなら，ループの開始を
this->next ではなく this からにすれば良いかと．

Comments

[amber_snob]

fanaさん、回答ありがとうございます。

thisだとエラーが出ていたので困ってたのですが、
下のようにconst Node* n = thisと宣言すれば回りました????

void print() const {
// BEGIN
for(const Node* n = this; n != NULL; n = n->next){
cout << n->value << endl;
}
// END
}

Answer 2

        cout << this->value << endl;

thisは不動なので、n->valueが正しいでしょう。

Comments

[amber_snob]

asmさん回答ありがとうございます！
n->valueに変更したところ、
2 3 4
と表示されました。
このプログラムではリストのすべての要素を出力したいので、
1 2 3 4
と表示されるべきなのですが、やはりfor文の書き方が問題なのでしょうか？
個人的には、Node* n = this->next;の部分があやしいと思います。。