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

Question

前提

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

実現したいこと

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

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

閉じられずに増えていく

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

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

        public static int i = 0;

        private void Button1_Click(object sender, EventArgs e)
        {
            Form2 view = new Form2();
            if (i == 0)
            {
                i++;  
                view.Show();
            }
            else
            {
                i--;
                view.Close();
            }
        }

試したこと

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

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

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

Answer 1

        public static int i = 0;
        Form2 view;
        private void Button1_Click(object sender, EventArgs e)
        {
            if (i == 0)
            {
                view = new Form2();
                i++;  
                view.Show();
            }
            else
            {
                i--;
                view.Close();
            }
        }

Comments

[Reindeer_Csharp]

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

[ozwk]

静的(static)フィールドにはしていません。

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

[退会済みユーザー]

質問者さん>

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

[Reindeer_Csharp]

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

[Reindeer_Csharp]

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

[KOZ6.0]

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

[Zuishin]

bool もどきの使い方しかしていないのに int なのが気になります。

view クローズで null にするのであれば、それで判定すれば i は要らないように思います。

new Close の代わりに Show Hide でも良さそうに思います。その場合は Visible = !Visible に短縮化できます。