C# と VB.NET の質問掲示板

≪言語：VB2008≫

こんにちは。いつもここを参考にさせて頂いております。

さて、フォームのShowDialogの際に、色々と処理を追加したいと思い、
このフォームのShowDialogのオーバーライドをしたいと考えましたが、
ShowDialogはOverridableではないと怒られます。

そこで、色々と試した結果、Shadowsというキーワードをつけると、それらしいことができるようですが、
以下の方法で良いのでしょうか？
（MyBaseって継承元を表すという理解でしたが、これでいいのか分かりません。）

    Public Shadows Function ShowDialog(ByVal owner As IWin32Window) As DialogResult
        'ここに処理を追加
             ・
             ・
             ・
        Return MyBase.ShowDialog(owner)
    End Function

よろしくお願いいたします。

ダイアログを表示すると同時に何かをしたいだけであれば、ShowDialogをオーバー
ライドするより以下のイベントで処理する方が一般的かと思います。
  Initialized
  Loaded

private void FormDialog1_Load(object sender, EventArgs e)
{
    this.DialogResult = DialogResult.Yes;
    this.Close();
}

としておくと
using (FormDialog1 dlg=new FormDialog1())
{
    DialogResult r=dlg.ShowModal();
    System.Diagnostics.Debug.WriteLine(r.ToString());
}
で Yes が得られるわけだけど、これぢゃダメかな？
DialogResult.Cancel や Ignore なんかも返却できたけど。

> あと、フォームのLoadイベントでフォームのCloseはすべきでないと聞きました。
この辺でも議論されてますね。
http://blogs.wankuma.com/jeanne/archive/2006/02/12/21205.aspx

開きたくないならそもそも開かなきゃいい、に１票。
using (FormDialog1 dlg = new FormDialog1())
{
    if (dlg.SomeThingToDo()) r=dlg.ShowDialog();
}

開きたくない判定を Form 自体が持っているのがおかしい、にもう１票。
データベースに接続できないとか、そういう判定が当該 Form の中にあるのは設計がヘン。
だったらそもそも new FormDialog1() 自体しなくていいぢゃん。
if (SomethingToDo())
{
    using 以下略
}

既存コードをできるだけ触りたくないなどの理由により FormDialog1 自体を変更する量を最小限にしたい
ならオイラも Shown や Idle にて処理すべし、に意見修正しておきます。
一瞬 Form が見えちゃいますけど。