- 追加された行はこの色です。
- 削除された行はこの色です。
* thisキーワードの危険性 [#i9971c47]
#contents
** 問題提起 [#i8249ae9]
** メンバ初期化子にthisキーワードを使うのはよくない [#ub72cb79]
MessageButtonクラスに注目。
先にbutton_を初期化してからmessage_を初期化しています。
そのため、もしButtonクラスのコンストラクタの中でIButtonListener::onButtonPushed()が
呼ばれるような実装の変更があった場合、未初期化のmessage_を出力することになってしまいます。
まず
message_のコンストラクタを先にすればいい
という意見も出てくるかと思います。
今回はそれでいいのですが、
thisなものをメンバ変数に渡すときは、その時点でリスナに対する呼び出しの準備が完了している必要がある
ということを言いたかったのです。
message_を先にコンストラクトしたとしても、
MessageButtonクラスのコンストラクタは完了していない状態です。
その状態で他のオブジェクトからメソッドを呼ばれるのはものすごく嫌な感じです。
今回のようなシンプルな例だとそんなに問題ないのですが
オブジェクト間でリスナクラスを受け渡ししているような場合だと
複雑になってきてそこまで考慮できなくなってくるんですよね。
VC8.0では警告に扱っている
:コード1|
#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]