回答編集履歴

見直しキャンペーン中

2023/07/23 06:43

投稿

TN8001

スコア9326

test CHANGED Viewed

@@ -1,475 +1,235 @@
 `Application.Run(Form1.GetInstance());`
 こうしてください。
 ---
 今の状況というか問題点は、
 1. メインフォームとサブフォーム1-13は既にできていて、動作自体に問題はない
 2. サブフォームを排他的（どれか一つだけしか開けないよう）に改造しようとしている
 3. その際地道にやれば（コメントアウトのコード）できることはわかっている
 4. しかし１３個はだるいし、もっといい方法がありそうに思える
 ってことですよね？（実際2個目くらいでもうだるかったです^^;
 自分だったらこうするかなぁ？ってコードです。
 `button1_Click`もまとめてもよさそうですが、行数も減りそうにないのでそのまま。
-Form1.cs
+```cs:Form1.cs
-```C#
 using System;
 using System.Drawing;
 using System.Windows.Forms;
 namespace Questions293390
 {
     public partial class Form1 : Form
-    {
+    {
         // シングルトンの必要はないかもしれないが、どうせSubFormからForm1をいじりたくなりますよね？
         public static Form1 Instance { get; } = new Form1();
         private Form1() => InitializeComponent();
         private void button1_Click(object sender, EventArgs e)
             => ShowSubForm(new SubForm1(), panel1);
         private void button2_Click(object sender, EventArgs e)
             => ShowSubForm(new SubForm2(), panel1);
         private void button3_Click(object sender, EventArgs e)
             => ShowSubForm(new SubForm3(), panel1);
         // class Commonにあっても別に構わない
         private static void ShowSubForm(Form subForm, Panel group)
-        {
+        {
             // Panel配下のボタンを不活性
             foreach(var control in group.Controls)
             {
                 if(control is Button button)
                     button.Enabled = false;
             }
             // SubFormのクローズを購読
             subForm.FormClosed += SubForm_FormClosed;
             // PanelをTagに入れて覚えておく
             subForm.Tag = group;
             subForm.Show();
-        }
+        }
         // クローズ時にPanel配下のボタンを戻す
         private static void SubForm_FormClosed(object sender, FormClosedEventArgs e)
-        {
+        {
             // senderはSubForm自身
             if(sender is Form subForm)
             {
                 // PanelをTagから取り出す
                 if(subForm.Tag is Panel group)
                 {
                     foreach(var control in group.Controls)
                     {
                         if(control is Button button)
                             button.Enabled = true;
                     }
                 }
             }
-        }
+        }
-    }
+    }
     // SubFormではFormClosed等のイベントを書く必要がなくて楽々という意味
     // 実際は普通に作る
     public class SubForm1 : Form
-    {
+    {
         public SubForm1() { Text = "SubForm1"; ClientSize = new Size(300, 200); }
-    }
+    }
     public class SubForm2 : Form
-    {
+    {
         public SubForm2() { Text = "SubForm2"; ClientSize = new Size(300, 200); }
-    }
+    }
     public class SubForm3 : Form
-    {
+    {
         public SubForm3() { Text = "SubForm3"; ClientSize = new Size(300, 200); }
-    }
+    }
 }
 ```
-Form1.Designer.cs
+```cs:Form1.Designer.cs
-```C#
 namespace Questions293390
 {
     partial class Form1
-    {
+    {
         /// <summary>
         /// 必要なデザイナー変数です。
         /// </summary>
         private System.ComponentModel.IContainer components = null;
         /// <summary>
         /// 使用中のリソースをすべてクリーンアップします。
         /// </summary>
         /// <param name="disposing">マネージド リソースを破棄する場合は true を指定し、その他の場合は false を指定します。</param>
         protected override void Dispose(bool disposing)
-        {
+        {
             if(disposing && (components != null))
             {
                 components.Dispose();
             }
             base.Dispose(disposing);
-        }
+        }
         #region Windows フォーム デザイナーで生成されたコード
         /// <summary>
         /// デザイナー サポートに必要なメソッドです。このメソッドの内容を
         /// コード エディターで変更しないでください。
         /// </summary>
         private void InitializeComponent()
-        {
+        {
             this.button1 = new System.Windows.Forms.Button();
             this.button2 = new System.Windows.Forms.Button();
             this.button3 = new System.Windows.Forms.Button();
             this.panel1 = new System.Windows.Forms.Panel();
             this.otherButton = new System.Windows.Forms.Button();
             this.label1 = new System.Windows.Forms.Label();
             this.panel1.SuspendLayout();
             this.SuspendLayout();
-            //
+            //
             // button1
-            //
+            //
             this.button1.Location = new System.Drawing.Point(5, 34);
             this.button1.Name = "button1";
             this.button1.Size = new System.Drawing.Size(75, 23);
             this.button1.TabIndex = 0;
             this.button1.Text = "button1";
             this.button1.UseVisualStyleBackColor = true;
             this.button1.Click += new System.EventHandler(this.button1_Click);
-            //
+            //
             // button2
-            //
+            //
             this.button2.Location = new System.Drawing.Point(5, 63);
             this.button2.Name = "button2";
             this.button2.Size = new System.Drawing.Size(75, 23);
             this.button2.TabIndex = 1;
             this.button2.Text = "button2";
             this.button2.UseVisualStyleBackColor = true;
             this.button2.Click += new System.EventHandler(this.button2_Click);
-            //
+            //
             // button3
-            //
+            //
             this.button3.Location = new System.Drawing.Point(5, 92);
             this.button3.Name = "button3";
             this.button3.Size = new System.Drawing.Size(75, 23);
             this.button3.TabIndex = 2;
             this.button3.Text = "button3";
             this.button3.UseVisualStyleBackColor = true;
             this.button3.Click += new System.EventHandler(this.button3_Click);
-            //
+            //
             // panel1
-            //
+            //
             this.panel1.Controls.Add(this.label1);
             this.panel1.Controls.Add(this.button3);
             this.panel1.Controls.Add(this.button1);
             this.panel1.Controls.Add(this.button2);
             this.panel1.Location = new System.Drawing.Point(153, 136);
             this.panel1.Name = "panel1";
             this.panel1.Size = new System.Drawing.Size(112, 125);
             this.panel1.TabIndex = 3;
-            //
+            //
             // otherButton
-            //
+            //
             this.otherButton.Location = new System.Drawing.Point(538, 54);
             this.otherButton.Name = "otherButton";
             this.otherButton.Size = new System.Drawing.Size(75, 23);
             this.otherButton.TabIndex = 4;
             this.otherButton.Text = "otherButton";
             this.otherButton.UseVisualStyleBackColor = true;
-            //
+            //
             // label1
-            //
+            //
             this.label1.AutoSize = true;
             this.label1.Location = new System.Drawing.Point(3, 10);
             this.label1.Name = "label1";
             this.label1.Size = new System.Drawing.Size(35, 12);
             this.label1.TabIndex = 3;
             this.label1.Text = "label1";
-            //
+            //
             // Form1
-            //
+            //
             this.AutoScaleDimensions = new System.Drawing.SizeF(6F, 12F);
             this.AutoScaleMode = System.Windows.Forms.AutoScaleMode.Font;
             this.ClientSize = new System.Drawing.Size(800, 450);
             this.Controls.Add(this.otherButton);
             this.Controls.Add(this.panel1);
             this.Name = "Form1";
             this.Text = "Form1";
             this.panel1.ResumeLayout(false);
             this.panel1.PerformLayout();
             this.ResumeLayout(false);
-        }
+        }
         #endregion
         private System.Windows.Forms.Label label1;
         private System.Windows.Forms.Button otherButton;
         private System.Windows.Forms.Button button1;
         private System.Windows.Forms.Button button2;
         private System.Windows.Forms.Button button3;
         private System.Windows.Forms.Panel panel1;
-    }
+    }
 }
 ```
-Program.cs
+```cs:Program.cs
-```C#
 using System;
 using System.Windows.Forms;
 namespace Questions293390
 {
     static class Program
-    {
+    {
         [STAThread]
         static void Main()
-        {
+        {
             Application.EnableVisualStyles();
             Application.SetCompatibleTextRenderingDefault(false);
             Application.Run(Form1.Instance);
-        }
+        }
-    }
+    }
 }
 ```

メソッド化

2020/09/27 08:30

投稿

TN8001

スコア9326

test CHANGED Viewed

@@ -84,7 +84,7 @@
         {
-            // group配下のボタンを不活性
+            // Panel配下のボタンを不活性
             foreach(var control in group.Controls)
@@ -98,29 +98,47 @@
+            // SubFormのクローズを購読
-            subForm.FormClosed += Closed;
+            subForm.FormClosed += SubForm_FormClosed;
+            // PanelをTagに入れて覚えておく
+            subForm.Tag = group;
             subForm.Show();
+        }
-            // クローズ時にgroup配下のボタンを活性に戻す
+        // クローズ時にPanel配下のボタンを戻す
-            void Closed(object sender, FormClosedEventArgs e)
+        private static void SubForm_FormClosed(object sender, FormClosedEventArgs e)
+        {
+            // senderはSubForm自身
+            if(sender is Form subForm)
             {
+                // PanelをTagから取り出す
-                subForm.FormClosed -= Closed;
+                if(subForm.Tag is Panel group)
-                foreach(var control in group.Controls)
                 {
+                    foreach(var control in group.Controls)
+                    {
-                    if(control is Button button)
+                        if(control is Button button)
-                        button.Enabled = true;
+                            button.Enabled = true;
+                    }
                 }

コード追記

2020/09/27 08:30

投稿

TN8001

スコア9326

test CHANGED Viewed

@@ -1,3 +1,457 @@
 `Application.Run(Form1.GetInstance());`
 こうしてください。
+---
+今の状況というか問題点は、
+1. メインフォームとサブフォーム1-13は既にできていて、動作自体に問題はない
+2. サブフォームを排他的（どれか一つだけしか開けないよう）に改造しようとしている
+3. その際地道にやれば（コメントアウトのコード）できることはわかっている
+4. しかし１３個はだるいし、もっといい方法がありそうに思える
+ってことですよね？（実際2個目くらいでもうだるかったです^^;
+自分だったらこうするかなぁ？ってコードです。
+`button1_Click`もまとめてもよさそうですが、行数も減りそうにないのでそのまま。
+Form1.cs
+```C#
+using System;
+using System.Drawing;
+using System.Windows.Forms;
+namespace Questions293390
+{
+    public partial class Form1 : Form
+    {
+        // シングルトンの必要はないかもしれないが、どうせSubFormからForm1をいじりたくなりますよね？
+        public static Form1 Instance { get; } = new Form1();
+        private Form1() => InitializeComponent();
+        private void button1_Click(object sender, EventArgs e)
+            => ShowSubForm(new SubForm1(), panel1);
+        private void button2_Click(object sender, EventArgs e)
+            => ShowSubForm(new SubForm2(), panel1);
+        private void button3_Click(object sender, EventArgs e)
+            => ShowSubForm(new SubForm3(), panel1);
+        // class Commonにあっても別に構わない
+        private static void ShowSubForm(Form subForm, Panel group)
+        {
+            // group配下のボタンを不活性
+            foreach(var control in group.Controls)
+            {
+                if(control is Button button)
+                    button.Enabled = false;
+            }
+            subForm.FormClosed += Closed;
+            subForm.Show();
+            // クローズ時にgroup配下のボタンを活性に戻す
+            void Closed(object sender, FormClosedEventArgs e)
+            {
+                subForm.FormClosed -= Closed;
+                foreach(var control in group.Controls)
+                {
+                    if(control is Button button)
+                        button.Enabled = true;
+                }
+            }
+        }
+    }
+    // SubFormではFormClosed等のイベントを書く必要がなくて楽々という意味
+    // 実際は普通に作る
+    public class SubForm1 : Form
+    {
+        public SubForm1() { Text = "SubForm1"; ClientSize = new Size(300, 200); }
+    }
+    public class SubForm2 : Form
+    {
+        public SubForm2() { Text = "SubForm2"; ClientSize = new Size(300, 200); }
+    }
+    public class SubForm3 : Form
+    {
+        public SubForm3() { Text = "SubForm3"; ClientSize = new Size(300, 200); }
+    }
+}
+```
+Form1.Designer.cs
+```C#
+namespace Questions293390
+{
+    partial class Form1
+    {
+        /// <summary>
+        /// 必要なデザイナー変数です。
+        /// </summary>
+        private System.ComponentModel.IContainer components = null;
+        /// <summary>
+        /// 使用中のリソースをすべてクリーンアップします。
+        /// </summary>
+        /// <param name="disposing">マネージド リソースを破棄する場合は true を指定し、その他の場合は false を指定します。</param>
+        protected override void Dispose(bool disposing)
+        {
+            if(disposing && (components != null))
+            {
+                components.Dispose();
+            }
+            base.Dispose(disposing);
+        }
+        #region Windows フォーム デザイナーで生成されたコード
+        /// <summary>
+        /// デザイナー サポートに必要なメソッドです。このメソッドの内容を
+        /// コード エディターで変更しないでください。
+        /// </summary>
+        private void InitializeComponent()
+        {
+            this.button1 = new System.Windows.Forms.Button();
+            this.button2 = new System.Windows.Forms.Button();
+            this.button3 = new System.Windows.Forms.Button();
+            this.panel1 = new System.Windows.Forms.Panel();
+            this.otherButton = new System.Windows.Forms.Button();
+            this.label1 = new System.Windows.Forms.Label();
+            this.panel1.SuspendLayout();
+            this.SuspendLayout();
+            //
+            // button1
+            //
+            this.button1.Location = new System.Drawing.Point(5, 34);
+            this.button1.Name = "button1";
+            this.button1.Size = new System.Drawing.Size(75, 23);
+            this.button1.TabIndex = 0;
+            this.button1.Text = "button1";
+            this.button1.UseVisualStyleBackColor = true;
+            this.button1.Click += new System.EventHandler(this.button1_Click);
+            //
+            // button2
+            //
+            this.button2.Location = new System.Drawing.Point(5, 63);
+            this.button2.Name = "button2";
+            this.button2.Size = new System.Drawing.Size(75, 23);
+            this.button2.TabIndex = 1;
+            this.button2.Text = "button2";
+            this.button2.UseVisualStyleBackColor = true;
+            this.button2.Click += new System.EventHandler(this.button2_Click);
+            //
+            // button3
+            //
+            this.button3.Location = new System.Drawing.Point(5, 92);
+            this.button3.Name = "button3";
+            this.button3.Size = new System.Drawing.Size(75, 23);
+            this.button3.TabIndex = 2;
+            this.button3.Text = "button3";
+            this.button3.UseVisualStyleBackColor = true;
+            this.button3.Click += new System.EventHandler(this.button3_Click);
+            //
+            // panel1
+            //
+            this.panel1.Controls.Add(this.label1);
+            this.panel1.Controls.Add(this.button3);
+            this.panel1.Controls.Add(this.button1);
+            this.panel1.Controls.Add(this.button2);
+            this.panel1.Location = new System.Drawing.Point(153, 136);
+            this.panel1.Name = "panel1";
+            this.panel1.Size = new System.Drawing.Size(112, 125);
+            this.panel1.TabIndex = 3;
+            //
+            // otherButton
+            //
+            this.otherButton.Location = new System.Drawing.Point(538, 54);
+            this.otherButton.Name = "otherButton";
+            this.otherButton.Size = new System.Drawing.Size(75, 23);
+            this.otherButton.TabIndex = 4;
+            this.otherButton.Text = "otherButton";
+            this.otherButton.UseVisualStyleBackColor = true;
+            //
+            // label1
+            //
+            this.label1.AutoSize = true;
+            this.label1.Location = new System.Drawing.Point(3, 10);
+            this.label1.Name = "label1";
+            this.label1.Size = new System.Drawing.Size(35, 12);
+            this.label1.TabIndex = 3;
+            this.label1.Text = "label1";
+            //
+            // Form1
+            //
+            this.AutoScaleDimensions = new System.Drawing.SizeF(6F, 12F);
+            this.AutoScaleMode = System.Windows.Forms.AutoScaleMode.Font;
+            this.ClientSize = new System.Drawing.Size(800, 450);
+            this.Controls.Add(this.otherButton);
+            this.Controls.Add(this.panel1);
+            this.Name = "Form1";
+            this.Text = "Form1";
+            this.panel1.ResumeLayout(false);
+            this.panel1.PerformLayout();
+            this.ResumeLayout(false);
+        }
+        #endregion
+        private System.Windows.Forms.Label label1;
+        private System.Windows.Forms.Button otherButton;
+        private System.Windows.Forms.Button button1;
+        private System.Windows.Forms.Button button2;
+        private System.Windows.Forms.Button button3;
+        private System.Windows.Forms.Panel panel1;
+    }
+}
+```
+Program.cs
+```C#
+using System;
+using System.Windows.Forms;
+namespace Questions293390
+{
+    static class Program
+    {
+        [STAThread]
+        static void Main()
+        {
+            Application.EnableVisualStyles();
+            Application.SetCompatibleTextRenderingDefault(false);
+            Application.Run(Form1.Instance);
+        }
+    }
+}
+```