指定フォームの閉じ方（C#）

前提

C#でソフト制作しています。
別フォームを消したくて、コードを組んだのですが、出たら消えずに増えていく一方になっています。何が良くないのかアドバイスしてくれると嬉しいです。

実現したいこと

トグルの実装（親フォームから指定フォームを出したり閉まったり）

発生している問題・エラーメッセージ

閉じられずに増えていく

該当のソースコード（Form1側）

フォームは2つ、form2には特にコードなし

C#
1        public static int i = 0;
2
3        private void Button1_Click(object sender, EventArgs e)
4        {
5            Form2 view = new Form2();
6            if (i == 0)
7            {
8                i++;  
9                view.Show();
10            }
11            else
12            {
13                i--;
14                view.Close();
15            }
16        }

試したこと

Closeなどを試してみました。

補足情報（FW/ツールのバージョンなど）

VisualStudio Community　v17.2.5
コード：C#
フォームアプリケーションの形で制作しています。

回答1件

ベストアンサー

C#
1        public static int i = 0;
2        Form2 view;
3        private void Button1_Click(object sender, EventArgs e)
4        {
5            if (i == 0)
6            {
7                view = new Form2();
8                i++;  
9                view.Show();
10            }
11            else
12            {
13                i--;
14                view.Close();
15            }
16        }

投稿2022/06/24 00:39

ozwk

総合スコア13364

回答へのコメント

Reindeer_Csharp

2022/06/24 00:45

なるほど、静的の方で命名して中で初期化したらいいんですね。勉強になりました。

ozwk

2022/06/24 00:58 編集

静的(static)フィールドにはしていません。そういえば変数 i について、このコード片だけだとイマイチ判断できませんが、静的フィールドにする意味がいまいち感じられません。

SurferOnWww

2022/06/24 01:02 編集

質問者さん> 横レス失礼します。「静的」とは何を言っているのか不明ですが、変数のスコープと値型/参照型の違いあたりの問題なのでその辺を調べてみることをお勧めします。

Reindeer_Csharp

2022/06/24 01:03

変数iは仮置きの感覚で置いています。置く場所がいまいちわからなくてとりあえず一番上に置こうかなと思いました。

Reindeer_Csharp

2022/06/24 01:04

進められたとおりに値型/参照型について調べていきます。ありがとうございます。

KOZ6.0

2022/06/24 02:02 編集

３点ほど (1) i は他からアクセスされることを想定していますか？でなければ private にすべきです。 (2) i と view は対になるものですから、両方静的メンバーにするか、両方インスタンスメンバーにするか統一したほうが良いです。 Form1 はひとつしか開かないというのであれば大丈夫ですが、複数開いたとき異常終了する可能性があります。両方静的メンバーだと、Form1 が何枚開いても Form2 はひとつしか存在しません。両方インスタンスメンバーだと、Form1 が複数開いた場合、それぞれの Form1 に対し、Form2 がひとつずつ存在することになります。 (3) インスタンスを保持する明確な理由がない限り、Form2 を閉じたあと、view2 に null を代入してください。GC が回収できません。

Zuishin

2022/06/24 03:54 編集

bool もどきの使い方しかしていないのに int なのが気になります。 view クローズで null にするのであれば、それで判定すれば i は要らないように思います。 new Close の代わりに Show Hide でも良さそうに思います。その場合は Visible = !Visible に短縮化できます。