回答編集履歴

見直しキャンペーン中

2023/07/22 10:02

投稿

TN8001

スコア9862

test CHANGED Viewed

@@ -1,319 +1,157 @@
 > あるクラス（Mainとは別）において，ある変数の値によって，ピクチャーボックスに適当な図（簡単な長方形で良い）を表示させたいと考えています．
 「あるクラス（Mainとは別）」ここが肝なのかなと思いました。
 具体的な使用イメージがわからないのですが、時間のかかる計算やリアルタイムのデータとかなんでしょうか。
-`.Net`には変数（正確にはプロパティ）の変更を、監視者に通知する仕組みが既にあります。
+.Netには変数（正確にはプロパティ）の変更を、監視者に通知する仕組みが既にあります。
 通知する側が`INotifyPropertyChanged`を実装し、監視側は`PropertyChanged`を購読します。
 難しそうに感じますが定型コードなのであまり気にせず、「プロパティが変更されると、イベントが来る」と思っておけば十分です。
 `Form1`と`Form2`の連携も、入門者の方が必ず引っかかるポイントですね。
 今回は`Class1`をやり取りすることになります。
 文字で説明すると無駄に長くなってしまうので、回答コードを試していただいたほうが早そうです^^;
-Form1.cs
+```cs:Form1.cs
-```C#
 using System;
 using System.Drawing;
 using System.Threading.Tasks;
 using System.Windows.Forms;
+namespace Questions281063
+{
+    public partial class Form1 : Form
+    {
+        private Button button1 = new Button();
+        private Button button2 = new Button();
+        private Class1 class1 = new Class1();
+        public Form1()
+        {
+            InitializeComponent();
+            button1.Text = "show";
+            button1.Click += Button1_Click;
+            Controls.Add(button1);
+            button2.Text = "run";
+            button2.Location = new Point(100, 0);
+            button2.Click += Button2_Click;
+            Controls.Add(button2);
+        }
+        private void Button1_Click(object sender, EventArgs e)
+        {
+            // Class1のインスタンスclass1を渡します
+            new Form2(class1).Show();
+        }
+        private async void Button2_Click(object sender, EventArgs e)
+        {
+            button2.Enabled = false;
+            // 別スレッドでclass1のRunを実行し、非同期で終了を待ちます
+            await Task.Run(() => class1.Run());
+            button2.Enabled = true;
+        }
+    }
+}
+```
+```cs:Form2.cs
+using System.ComponentModel;
+using System.Drawing;
+using System.Windows.Forms;
 namespace Questions281063
+{
+    public partial class Form2 : Form
+    {
+        private PictureBox pictureBox1 = new PictureBox();
+        private Class1 class1;
-{
-    public partial class Form1 : Form
+        public Form2(Class1 class1)
-    {
-        private Button button1 = new Button();
-        private Button button2 = new Button();
-        private Class1 class1 = new Class1();
-        public Form1()
         {
+            // Class1のインスタンスを受け取り、class1変数に入れておきます
+            this.class1 = class1;
             InitializeComponent();
+            pictureBox1.Dock = DockStyle.Fill;
+            pictureBox1.Paint += PictureBox1_Paint;
+            Controls.Add(pictureBox1);
+            // 変更通知イベントを購読します
-            button1.Text = "show";
+            class1.PropertyChanged += Class1_PropertyChanged;
-            button1.Click += Button1_Click;
+            FormClosed += Form2_FormClosed;
-            Controls.Add(button1);
-            button2.Text = "run";
-            button2.Location = new Point(100, 0);
-            button2.Click += Button2_Click;
-            Controls.Add(button2);
         }
-        private void Button1_Click(object sender, EventArgs e)
+        private void PictureBox1_Paint(object sender, PaintEventArgs e)
         {
-            // Class1のインスタンスclass1を渡します
+            // class1の状態によって何か描きます
+            if(class1.A == 1) e.Graphics.DrawRectangle(Pens.Black, 5, 5, 400, 400);
+            if(class1.B == 1) e.Graphics.FillRectangle(Brushes.Blue, 50, 50, 200, 100);
-            new Form2(class1).Show();
+            if(class1.C == 1) e.Graphics.FillEllipse(Brushes.Pink, 200, 300, 50, 50);
         }
-        private async void Button2_Click(object sender, EventArgs e)
+        private void Class1_PropertyChanged(object sender, PropertyChangedEventArgs e)
         {
-            button2.Enabled = false;
-            // 別スレッドでclass1のRunを実行し、非同期で終了を待ちます
+            // なにか変更されたのでpictureBox1を描き直します
-            await Task.Run(() => class1.Run());
-            button2.Enabled = true;
+            pictureBox1.Invalidate();
         }
+        private void Form2_FormClosed(object sender, FormClosedEventArgs e)
+        {
+            // 変更通知イベントを購読解除します（忘れるとメモリーリークします）
+            class1.PropertyChanged -= Class1_PropertyChanged;
+        }
     }
 }
 ```
-Form2.cs
+```cs:Class1.cs
-```C#
 using System.ComponentModel;
-using System.Drawing;
-using System.Windows.Forms;
+using System.Runtime.CompilerServices;
 namespace Questions281063
 {
-    public partial class Form2 : Form
+    public class Class1 : INotifyPropertyChanged
     {
-        private PictureBox pictureBox1 = new PictureBox();
-        private Class1 class1;
-        public Form2(Class1 class1)
-        {
-            // Class1のインスタンスを受け取り、class1変数に入れておきます
-            this.class1 = class1;
-            InitializeComponent();
-            pictureBox1.Dock = DockStyle.Fill;
-            pictureBox1.Paint += PictureBox1_Paint;
-            Controls.Add(pictureBox1);
-            // 変更通知イベントを購読します
-            class1.PropertyChanged += Class1_PropertyChanged;
-            FormClosed += Form2_FormClosed;
-        }
-        private void PictureBox1_Paint(object sender, PaintEventArgs e)
-        {
-            // class1の状態によって何か描きます
-            if(class1.A == 1) e.Graphics.DrawRectangle(Pens.Black, 5, 5, 400, 400);
-            if(class1.B == 1) e.Graphics.FillRectangle(Brushes.Blue, 50, 50, 200, 100);
-            if(class1.C == 1) e.Graphics.FillEllipse(Brushes.Pink, 200, 300, 50, 50);
-        }
-        private void Class1_PropertyChanged(object sender, PropertyChangedEventArgs e)
-        {
-            // なにか変更されたのでpictureBox1を描き直します
-            pictureBox1.Invalidate();
-        }
-        private void Form2_FormClosed(object sender, FormClosedEventArgs e)
-        {
-            // 変更通知イベントを購読解除します（忘れるとメモリーリークします）
-            class1.PropertyChanged -= Class1_PropertyChanged;
-        }
-    }
-}
-```
-Class1.cs
-```C#
-using System.ComponentModel;
-using System.Runtime.CompilerServices;
-namespace Questions281063
-{
-    public class Class1 : INotifyPropertyChanged
-    {
         // ちょっと冗長なのですが、通知する変数はこのようにする必要があります
         public int A { get => _A; set => Set(ref _A, value); }
         private int _A;
         public int B { get => _B; set => Set(ref _B, value); }
         private int _B;
         public int C { get => _C; set => Set(ref _C, value); }
         private int _C;
         public void Run()
         {
             A = 1; // AをON
             System.Threading.Thread.Sleep(1000);
             A = 0; // AをOFF
             B = 1;
             C = 1;
             System.Threading.Thread.Sleep(1000);
             B = 0;
             System.Threading.Thread.Sleep(1000);
             C = 0;
         }
         // 定型コードなので気にしないで結構です
         #region INotifyPropertyChanged
         public event PropertyChangedEventHandler PropertyChanged;
         protected void Set<T>(ref T storage, T value, [CallerMemberName] string propertyName = null)
         {
             if(Equals(storage, value)) return;
             storage = value;
             OnPropertyChanged(propertyName);
         }
         protected void OnPropertyChanged(string propertyName) => PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
         #endregion
     }
 }
 ```
 `Designer.cs`が行数をとるのでコードで`Button`等を作っていますが、もちろんデザイナでやってもらっても結構です。