既に同様に回答を頂いているようですが、せっかく作成しましたので投稿します。
表示しているForm2を閉じるには、Closeメソッドを使用します。
this.Close();
従って、Closeメソッドを呼び出す前にチェックを実行し、
チェックの結果がNGなのであれば、Windowを閉じないようするのであれば、
エラーメッセージを表示後、Closeメソッドを呼び出さなければ画面は閉じられません。
そうなると、OKボタン押下時かつ必須条件を満たす場合と、
キャンセルボタン押下時にCloseメソッドが呼び出されることになるのですが、
Form2を呼び出したForm1において、Form2がどちらの状態で画面を閉じたかを判定する必要があります。
現在は、
if (form.ShowDialog() == DialogResult.OK)
というように、ShowDialogメソッドの戻り値であるDialogResultで結果を見ていますので、
OKボタン押下字は、Closeメソッドを呼び出す前に、
DialogResult = DialogResult.OK;
を実装して、戻り値にDialogResult.OKを設定します。
これにより、OKボタンを押下して画面を閉じた場合は、DialogResult.OKが返り、
キャンセルボタン押下で画面を閉じた場合は、DialogResult.Cancelが返ります。
補足ですが、×ボタンやAlt+F4で閉じた場合も画面が閉じられ、Form1にはDialogResult.Cancelが返ります。
このあたりの動作は実際にデバッグしながら確認すればわかると思います。
private void okButton_Click(object sender, EventArgs e)
{
list = new List<string>();
if (findBox.Text == "")
{
MessageBox.Show("入力の不備があります。", "エラー!", MessageBoxButtons.OKCancel, MessageBoxIcon.Error);
}
else
{
// 画面が条件を満たして終了したことを設定
DialogResult = DialogResult.OK;
// 画面を閉じる
this.Close();
}
}
private void cancelButton_Click(object sender, EventArgs e)
{
// 画面を閉じる
this.Close();
}
また、Form2には、WindowクラスのShowDialogメソッドと同じ名前、引数を持つShowDialogメソッドが実装されています。
public DialogResult ShowDialog()
このままでは、継承されたメンバを暗黙的に非表示にする旨の警告が表示されていると思います。
意図的にWindowクラスのShowDialogメソッドを隠したいのであれば、newをつけて
public new DialogResult ShowDialog()
とした方が良いです。
ただし、今回のケースでは隠す必要がありませんので、別名のメソッドを用意するかした方が良いと思います。
(ShowDialogの引数はもう不要になったのですね？)
