C++/thisキーワードの危険性 のバックアップソース(No.2)

• バックアップ一覧
• 差分 を表示
• 現在との差分 を表示
• 現在との差分 - Visual を表示
• バックアップ を表示
• C++/thisキーワードの危険性 へ行く。
  • 1 (2007-11-18 (日) 22:24:08)
  • 2 (2007-11-18 (日) 23:36:44)

* thisキーワードの危険性 [#i9971c47]
#contents

** メンバ初期化子にthisキーワードを使うのはよくない [#ub72cb79]
MessageButtonクラスに注目。
先にbutton_を初期化してからmessage_を初期化しています。
そのため、もしButtonクラスのコンストラクタの中でIButtonListener::onButtonPushed()が
呼ばれるような実装の変更があった場合、未初期化のmessage_を出力することになってしまいます。

まず
 message_のコンストラクタを先にすればいい
という意見も出てくるかと思います。

今回はそれでいいのですが、
 thisなものをメンバ変数に渡すときは、その時点でリスナに対する呼び出しの準備が完了している必要がある
ということを言いたかったのです。

message_を先にコンストラクトしたとしても、
MessageButtonクラスのコンストラクタは完了していない状態です。
その状態で他のオブジェクトからメソッドを呼ばれるのはものすごく嫌な感じです。

今回のようなシンプルな例だとそんなに問題ないのですが
オブジェクト間でリスナクラスを受け渡ししているような場合だと
複雑になってきてそこまで考慮できなくなってくるんですよね。

VC8.0では警告に扱っている

:コード１|
#code(c,){{
#include <iostream>
//

/// ボタンのイベントを受け取るクラス。
class IButtonListener
{
public:
	virtual ~IButtonListener(){}
	
	/// ボタンが押されたときに呼ばれる。
	virtual void onButtonPushed()=0;
};

/// ボタンクラス。
class Button
{
public:
	Button( IButtonListener& aListener )
	: listener_( aListener )
	{
	}

	/// 押しボタンイベントを発行する。
	void push()
	{
		listener_.onButtonPushed();
	};
	
private:
	IButtonListener& listener_;
};

/// ボタンが押されたらメッセージを出力するクラス。
class MessageButton : public IButtonListener
{
public:
	MessageButton( const char* aMessage )
	: button_( *this )
	, message_( aMessage )
	{
	}
	
	// IButtonListener
	virtual void onButtonPushed()
	{
		std::cout << message_ << "\n";
	}
	
	/// ボタンを押す。
	void push()
	{
		button_.push();
	}
	
private:
	Button button_;
	const std::string message_;
};

/// メイン関数。
int main (int argc, char * const argv[]) {
	MessageButton button( "Button Pushed Message" );
	button.push();
    return 0;
}

// EOF

}}

:コード2|
#code(c,){{
/// メッセージを返すインターフェースクラス。
class IMessage
{
public:
	virtual ~IMessage(){}
	virtual std::string createMessage()const=0;
};

/// HelloWorldを出力し、その後任意のメッセージを出力するMessageButton。
class HelloWorldButton : public MessageButton
{
public:
	HelloWorldButton( const IMessage& aIMesssage )
	: MessageButton( "Hello World" )
	, message_( aIMesssage )
	{
	}
	
	// MessageButton
	virtual void onButtonPushed()
	{
		MessageButton::onButtonPushed();
		std::cout << message_.createMessage();
	}
	
private:
	const IMessage& message_;
};


}}
** リスナーとしてthisキーワードを渡す場合 [#a320108d]


** 実装としてthisキーワードを渡す場合 [#n5b19c69]