画面操作に伴う処理なのですが、ボタン押下の度にメソッドは呼ばれます。
変数は特に引数もないので初期化自体は一度で大丈夫です。
ほかのインスタンスはメソッド内で定義してます。
それだけは頻繁に呼ばれるので一度実体化しておいた方が実行効率としてはいいのかなと思っていました。
if文で囲んでnullの場合だけ生成するようにするのだといかがでしょうか。
一応メソッドの最後でdisposeはします。