Q&A
特に問題が起きてないのでしたらこれでいいような。
どのようなアドバイスを期待されているのでしょうか。（回答者も魔法使いではないのであくまで言語仕様と要件に沿ったものしかできません）
C#のFormアプリケーションで、Form1からForm2をモードレスで1つだけ表示させたいです。
上のようにするとボタンクリックの度に表示されるので、下のようにしたら所望の動作をしましたが、なんとなくスマートな方法ではないと感じています。
他の良い方法は無いでしょうか？それとも下の方法で充分でしょうか。
C#
1 // Form1のイベントハンドラ 2 private void button1_Click(object sender, EventArgs e) 3 { 4 Form2 form2 = new Form2(); 5 form2.Show(); 6 }
C#
1 // Form1のメンバ変数 2 Form2 form2 = new Form2(); 3 4 // Form1のイベントハンドラ 5 private void button1_Click(object sender, EventArgs e) 6 { 7 form2.Show(); 8 }
以下、追記。
＜スマートで無いと感じる理由＞
Show()によりフォームが開くかどうかをShow()の仕様に任せているように感じるため。
下のコードで1つしか開かないことをShow()で保障しているかがわからない。
Show()呼び出し前に明示的に確認必要かどうかがわからない。
＜知りたいこと＞
システム全体としてのスマートさやデータ受渡し方法などは考慮せず、
「Form1からForm2をモードレスで1つだけ表示させたい」という点のみ
に絞ったときに下のコードが適切か。他によい方法があるか。
回答3件
0
この内容だけで何がスマートかを判断するのは不可能です。
現状問題が起きていないのならそれでOKとして進むべきではないでしょうか。
もし、もっと期待されているアドバイスがあるのでしたら作ろうとしているアプリケーションの全体の概要とか仕様とかそのあたり詳しく書かれるべきかと思います。
投稿2019/03/29 00:50
総合スコア80888
すみません、質問が簡略過ぎました。
データ・イベントの渡し方なども含めた全体としてのスマートさではなく、
「Form1からForm2をモードレスで1つだけ表示させる」の1点に絞ったつもりでした。
form2.Show()実行のとき表示されるかされないかがform2側で常に適切に処理してくれるかがわからず、明示的に記述する方法があればと思い質問させていただきました。
ご存知かもしれませんが、質問は編集できますので要件は具体的にお願いします。エスパー回答期待は逆に回答者を遠ざけますし、ざっくり質問にはざっくり回答しかつきません。
質問を追記しておきます。
アドバイスありがとうございます。
0
ベストアンサー
なぜスマートな方法でないのでしょう？
他の条件を無視して質問に書いてある条件のみを材料に考えれば、下がベストです。
投稿2019/03/29 00:42
総合スコア28673
Show()の仕様として既に開かれている場合は再度開かれないことがはっきりしているかどうかの自信が無かったもので、スマートで無いと感じました。
探しても見つからなかったのですが、下記のような方法があるかと思い質問させていただきました。
if(!form2.IsOpened)
{
form2.Show();
}
Show は Visible を true にするので Visible で条件分岐できますが、その分岐は Show 内部に含まれているので不要です。
そういうことでしたか！
すごくスッキリしました。
0
表示するだけなら上の方法でOKです
が、繰り返しクリックされると、その分だけフォームが表示されることになりますんで、そこらへん考える必要はありますね
投稿2019/03/29 00:47
総合スコア88163
下の方法だと繰り返しクリックでも１つしか表示されなかったのですが、他によい方法があるのではと感じたので質問させていただきました。
ならそういうふうに質門に書きましょう。
上ではなぜクリックした分フォームが表示されるのか、
下ではなぜ一つだけなのか、を考えてみればどうでしょうか
そういうふうに質問に書いたつもりです。
上の理由はわかります。
下の理由はわかりませんでした。Form2.Show()がうまく判断してくれてるからと思いました。
Show()メソッドはたんにフォームのVisibleプロパティをTrueにするだけ、
と考えればわかるでしょうか。
下の場合はForm2のインスタンスが一つだけで固定されてますんで、
繰り返しクリックしたところで同じフォームのVisibleをTrueにするだけ、ってことですね
試してみると、Show()の換わりに.Visible = trueでも開くことができました。
下が動作する理由も納得できました。
一応追加確認で、
.Visible = trueで開く ⇒ Form閉じる ⇒ .Visible = trueで開く
をすると「破棄されたオブジェクトにアクセスできません。」例外が出たので、Show()はインスタンス作成とVisible trueをやっているようでした。
いや、インスタンス作成はnewです。
フォームをCloseするとフォームのインスタンスは破棄されます
なので、Closeして再度開けるためには、
・FormClosingイベントで、フォーム閉じないでVisibleをFalseにするだけにする
・再度開くときにはnewをやり直す
のいずれかの対処が必要です。
https://docs.microsoft.com/ja-jp/dotnet/api/system.windows.forms.form.close?view=netframework-4.7.2
> When the Close method is called on a Form displayed as a modeless window, you cannot call the Show method to make the form visible, because the form's resources have already been released. To hide a form and then make it visible, use the Control.Hide method.
×ボタンや Close を呼んだ場合には破棄されるのでそれ以降は使えません。
×ボタンを表示しないようにして「閉じる」ボタンを作り、そこで Hide を呼ぶか、Closing イベントハンドラを使って破棄されないようにする必要があります。
mts10806 さんのおっしゃる通り、聞きたいことだけ書かれていて詳細なユースケースがわからないので回答の方も詳細に書くことができません。私の回答に「他の条件を無視して」と但し書きがついているのもそのためです。次回の質問は具体的な使用法がわかるようにお書きください。
ご指摘の通りです。反省します。
あなたの回答
tips
プレビュー
