回答編集履歴

見直しキャンペーン中

2023/07/29 10:35

投稿

TN8001

スコア9862

test CHANGED Viewed

@@ -1,231 +1,116 @@
 `image_[x] = new Image();`こうしたところで、元の`ButtonImage_0`等には何の影響もありません。
 `Image`はすでにxamlで配置しているわけですから新たに作る必要はなく、`Source`を変更するだけでいいのです^^
-```xaml
+```xml
 <Window
   x:Class="Questions372108.MainWindow"
   xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
   xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
   Width="800"
   Height="450">
   <StackPanel>
+    <UniformGrid Rows="1">
+      <Button x:Name="Button_0" Click="Button_Click" Content="Button_0" />
+      <Button x:Name="Button_1" Click="Button_Click" Content="Button_1" />
+      <Button x:Name="Button_2" Click="Button_Click" Content="Button_2" />
+      <Button x:Name="Button_3" Click="Button_Click" Content="Button_3" />
+      <Button x:Name="Button_4" Click="Button_Click" Content="Button_4" />
+    </UniformGrid>
     <UniformGrid Rows="1">
-      <Button x:Name="Button_0" Click="Button_Click" Content="Button_0" />
+      <Image x:Name="ButtonImage_0" IsHitTestVisible="False" />
-      <Button x:Name="Button_1" Click="Button_Click" Content="Button_1" />
+      <Image x:Name="ButtonImage_1" IsHitTestVisible="False" />
-      <Button x:Name="Button_2" Click="Button_Click" Content="Button_2" />
+      <Image x:Name="ButtonImage_2" IsHitTestVisible="False" />
-      <Button x:Name="Button_3" Click="Button_Click" Content="Button_3" />
+      <Image x:Name="ButtonImage_3" IsHitTestVisible="False" />
-      <Button x:Name="Button_4" Click="Button_Click" Content="Button_4" />
+      <Image x:Name="ButtonImage_4" IsHitTestVisible="False" />
     </UniformGrid>
-    <UniformGrid Rows="1">
-      <Image x:Name="ButtonImage_0" IsHitTestVisible="False" />
-      <Image x:Name="ButtonImage_1" IsHitTestVisible="False" />
-      <Image x:Name="ButtonImage_2" IsHitTestVisible="False" />
-      <Image x:Name="ButtonImage_3" IsHitTestVisible="False" />
-      <Image x:Name="ButtonImage_4" IsHitTestVisible="False" />
-    </UniformGrid>
   </StackPanel>
 </Window>
 ```
-```C#
+```cs
 using System;
 using System.Windows;
 using System.Windows.Controls;
 using System.Windows.Media.Imaging;
 namespace Questions372108
 {
     public partial class MainWindow : Window
     {
         private readonly int[] counts = new int[5];
         private readonly Button[] buttons = new Button[5];
         private readonly Image[] images = new Image[5];
         private Button Button_0_Test;
         private Button currentButton;
         public MainWindow()
         {
             InitializeComponent();
+            for (var i = 0; i < 5; i++)
+            {
+                buttons[i] = FindName($"Button_{i}") as Button;
+                images[i] = FindName($"ButtonImage_{i}") as Image;
+            }
+            Button_0_Test = Button_0;
+        }
+        private void Button_Click(object sender, RoutedEventArgs e)
+        {
-            for (var i = 0; i < 5; i++)
+            var x = Array.IndexOf(buttons, sender);
+            if (counts[x] == 0)
             {
+                var bi = new BitmapImage();
+                bi.BeginInit();
+                //bi.UriSource = new Uri("/Resources/AAA.png", UriKind.Relative);
+                bi.UriSource = new Uri("https://teratail-v2.storage.googleapis.com/uploads/avatars/u13/132786/KnkDDC5A_thumbnail.jpg");
+                bi.EndInit();
+                images[x].Source = bi;
+                counts[x]++;
+            }
+            else if (counts[x] == 1)
+            {
+                var bi = new BitmapImage();
+                bi.BeginInit();
-                buttons[i] = FindName($"Button_{i}") as Button;
+                //bi.UriSource = new Uri("/Resources/BBB.png", UriKind.Relative);
+                bi.UriSource = new Uri("https://teratail-v2.storage.googleapis.com/uploads/avatars/u20/201856/m2cobCxI_thumbnail.jpg");
+                bi.EndInit();
+                images[x].Source = bi;
+                counts[x]++;
+            }
+            else if (counts[x] == 2)
+            {
-                images[i] = FindName($"ButtonImage_{i}") as Image;
+                images[x].Source = null;
+                counts[x] = 0;
             }
-            Button_0_Test = Button_0;
-        }
-        private void Button_Click(object sender, RoutedEventArgs e)
-        {
-            var x = Array.IndexOf(buttons, sender);
-            if (counts[x] == 0)
-            {
-                var bi = new BitmapImage();
-                bi.BeginInit();
-                //bi.UriSource = new Uri("/Resources/AAA.png", UriKind.Relative);
-                bi.UriSource = new Uri("https://teratail-v2.storage.googleapis.com/uploads/avatars/u13/132786/KnkDDC5A_thumbnail.jpg");
-                bi.EndInit();
-                images[x].Source = bi;
-                counts[x]++;
-            }
-            else if (counts[x] == 1)
-            {
-                var bi = new BitmapImage();
-                bi.BeginInit();
-                //bi.UriSource = new Uri("/Resources/BBB.png", UriKind.Relative);
-                bi.UriSource = new Uri("https://teratail-v2.storage.googleapis.com/uploads/avatars/u20/201856/m2cobCxI_thumbnail.jpg");
-                bi.EndInit();
-                images[x].Source = bi;
-                counts[x]++;
-            }
-            else if (counts[x] == 2)
-            {
-                images[x].Source = null;
-                counts[x] = 0;
-            }
             // これが何の意味もない理由
             //image_[x] = new Image();
             // 例えばこうしたとして、表示されているButton_0が入れ替わるわけではない！
             Button_0_Test = new Button { Content = "Button_0がこうは変わらない！", };
             // 単に変数Button_0_Testの指す先が、新しいボタンに変わっただけ
             // Button_0は依然表示されたまま
             // 新たに作ったボタンはウィンドウに追加されていないので、
             // 表示されずにただメモリ内にあるだけになっている
             // こういった使い方なら意味はあるが
             currentButton = sender as Button;
             currentButton.Content = $"Button_{x}_{counts[x]}";
             //buttons[x].Content = $"Button_{x}_{counts[x]}";
         }
     }
 }
 ```
 ![アプリ画像](aebe654b5e9dc7d1809fcd7a84d35622.png)