回答編集履歴

見直しキャンペーン中

2023/07/29 14:31

投稿

TN8001

スコア10013

test CHANGED Viewed

@@ -21,7 +21,7 @@
 MainWindow
-```xaml
+```xml
 <Window
   x:Class="Q2x9yv8s160mb6u.MainWindow"
   xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
@@ -82,7 +82,7 @@
   </UniformGrid>
 </Window>
 ```
-```C#
+```cs
 using System.Windows;
 using CommunityToolkit.Mvvm.ComponentModel;
 using CommunityToolkit.Mvvm.Input;
@@ -117,7 +117,7 @@
 ```
 UserControl1（部分ビュー的）
-```xaml
+```xml
 <UserControl
   x:Class="Q2x9yv8s160mb6u.UserControl1"
   xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
@@ -129,7 +129,7 @@
   </StackPanel>
 </UserControl>
 ```
-```C#
+```cs
 using System.Windows.Controls;
 namespace Q2x9yv8s160mb6u
@@ -142,7 +142,7 @@
 ```
 UserControl2（カスタムコントロール的）
-```xaml
+```xml
 <UserControl
   x:Class="Q2x9yv8s160mb6u.UserControl2"
   xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
@@ -155,7 +155,7 @@
   </StackPanel>
 </UserControl>
 ```
-```C#
+```cs
 using System.Windows;
 using System.Windows.Controls;
@@ -175,7 +175,7 @@
 ```
 UserControl3（オリジナル）
-```xaml
+```xml
 <UserControl
   x:Class="Q2x9yv8s160mb6u.UserControl3"
   xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
@@ -187,7 +187,7 @@
   </StackPanel>
 </UserControl>
 ```
-```C#
+```cs
 using System.Windows;
 using System.Windows.Controls;
 using System.Windows.Data;
@@ -213,7 +213,7 @@
 ```
 UserControl4（継承）
-```xaml
+```xml
 <local:UserControl4Base
   x:Class="Q2x9yv8s160mb6u.UserControl4"
   xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
@@ -227,7 +227,7 @@
   </StackPanel>
 </local:UserControl4Base>
 ```
-```C#
+```cs
 using System.Windows;
 using System.Windows.Controls;
@@ -253,7 +253,7 @@
 ```
 UserControl5（添付プロパティ）
-```xaml
+```xml
 <UserControl
   x:Class="Q2x9yv8s160mb6u.UserControl5"
   xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
@@ -267,7 +267,7 @@
   </StackPanel>
 </UserControl>
 ```
-```C#
+```cs
 using System.Windows;
 using System.Windows.Controls;

無理やりMVVM

2022/01/24 11:14

投稿

TN8001

スコア10013

test CHANGED Viewed

@@ -1,13 +1,24 @@
 回答しないといいましたが、コメントだけでは伝えられそうにないので回答します^^;
 文字数制限があるので可能な限りシンプルに、`NumericUpDown`を題材にさせていただきました（超雑なのは目をつぶってください^^;
+６パターンあります。
+* ベタ置き
+* 部分ビュー的
-４パターン（ベタ置き・部分ビュー的・カスタムコントロール的・得体の知れないもの、が）あります。
+* カスタムコントロール的
+* オリジナル
+* 継承
+* 添付プロパティ
 イメージとしてはベタ置きで書いていたがゴチャついてきたので、部分ビュー的に切り出した。
 その後複数個使いたくなったので、カスタムコントロール的に作り替えたといった感じです。
 もちろん`NumericUpDown`ならカスタムコントロールにするのがベストですが、使いまわす気がないなら手前で留めておくのもありです。
+> MVVMで実装する前提で回答を頂けるとありがたいです。
-得体の知れないものは、部分ビュー的・カスタムコントロール的を合体させたような感じになっています。
+とのことですので後半３つは無理やり作りましたが、個人的には意味が分かんないですね。
-両方の面倒なところが合わさったというか無駄にまどろっこしいというか、「それ何の意味があるの？」感が伝わりますかね？^^;
+部分ビュー的・カスタムコントロール的の面倒なところをわざわざ合わせたというか無駄にまどろっこしいというか、「それ何の意味があるの？」感が伝わりますかね？^^;
 MainWindow
 ```xaml
@@ -21,7 +32,7 @@
   <Window.DataContext>
     <local:MainViewModel />
   </Window.DataContext>
-  <UniformGrid Columns="2">
+  <UniformGrid Columns="3">
     <GroupBox Header="ベタ置き">
       <StackPanel>
         <TextBlock Text="{Binding Value}" />
@@ -42,18 +53,30 @@
       <StackPanel>
         <local:UserControl2 x:Name="userControl2" />
         <TextBlock Text="{Binding Value, ElementName=userControl2, StringFormat=今の値は {0} です}" />
-        <!--  もちろんこうなっていてもいい  -->
+        <!--  もちろんこうなっていてもいい（以下3・4・5も同様）  -->
         <!--<local:UserControl2 Value="{Binding Hoge}" />
         <TextBlock Text="{Binding Hoge, StringFormat=今の値は {0} です}" />-->
       </StackPanel>
     </GroupBox>
-    <GroupBox Header="得体の知れないもの">
+    <GroupBox Header="オリジナル">
       <StackPanel>
         <local:UserControl3 x:Name="userControl3" />
         <TextBlock Text="{Binding Value, ElementName=userControl3, StringFormat=今の値は {0} です}" />
+      </StackPanel>
+    </GroupBox>
+    <GroupBox Header="継承">
+      <StackPanel>
-        <!--<local:UserControl3 Value="{Binding Fuga}" />
+        <local:UserControl4 x:Name="userControl4" />
-        <TextBlock Text="{Binding Fuga, StringFormat=今の値は {0} です}" />-->
+        <TextBlock Text="{Binding Value, ElementName=userControl4, StringFormat=今の値は {0} です}" />
+      </StackPanel>
+    </GroupBox>
+    <GroupBox Header="添付プロパティ">
+      <StackPanel>
+        <local:UserControl5 x:Name="userControl5" />
+        <TextBlock Text="{Binding (local:UserControl5.Value), ElementName=userControl5, StringFormat=今の値は {0} です}" />
       </StackPanel>
     </GroupBox>
   </UniformGrid>
@@ -66,7 +89,7 @@
 namespace Q2x9yv8s160mb6u
 {
-    public class MainViewModel : ObservableObject
+    public class UserControlViewModel : ObservableObject
     {
         public int Value { get => _Value; set => SetProperty(ref _Value, value); }
         private int _Value;
@@ -74,15 +97,18 @@
         public RelayCommand IncrementCommand { get; }
         public RelayCommand DecrementCommand { get; }
-        public UserControl1ViewModel UserControl1ViewModel { get; } = new();
-        public MainViewModel()
+        public UserControlViewModel()
         {
             IncrementCommand = new(() => Value++);
             DecrementCommand = new(() => Value--);
         }
     }
+    public class MainViewModel : UserControlViewModel // 同じものを書くのが無駄なので継承で済ませた^^;
+    {
+        public UserControlViewModel UserControl1ViewModel { get; } = new();
+    }
     public partial class MainWindow : Window
     {
         public MainWindow() => InitializeComponent();
@@ -105,26 +131,9 @@
 ```
 ```C#
 using System.Windows.Controls;
-using CommunityToolkit.Mvvm.ComponentModel;
-using CommunityToolkit.Mvvm.Input;
-namespace Q2x9yv8s160mb6u
+namespace Q2x9yv8s160mb6u
-{
+{
-    public class UserControl1ViewModel : ObservableObject
-    {
-        public int Value { get => _Value; set => SetProperty(ref _Value, value); }
-        private int _Value;
-        public RelayCommand IncrementCommand { get; }
-        public RelayCommand DecrementCommand { get; }
-        public UserControl1ViewModel()
-        {
-            IncrementCommand = new(() => Value++);
-            DecrementCommand = new(() => Value--);
-        }
-    }
     public partial class UserControl1 : UserControl
     {
         public UserControl1() => InitializeComponent();
@@ -155,9 +164,7 @@
     public partial class UserControl2 : UserControl
     {
         public int Value { get => (int)GetValue(ValueProperty); set => SetValue(ValueProperty, value); }
-        public static readonly DependencyProperty ValueProperty
-            = DependencyProperty.Register(nameof(Value), typeof(int), typeof(UserControl2),
+        public static readonly DependencyProperty ValueProperty = DependencyProperty.Register(nameof(Value), typeof(int), typeof(UserControl2), new PropertyMetadata(0));
-                new PropertyMetadata(0));
         public UserControl2() => InitializeComponent();
@@ -167,7 +174,7 @@
 }
 ```
-UserControl3（得体の知れないもの）
+UserControl3（オリジナル）
 ```xaml
 <UserControl
   x:Class="Q2x9yv8s160mb6u.UserControl3"
@@ -183,42 +190,101 @@
 ```C#
 using System.Windows;
 using System.Windows.Controls;
-using CommunityToolkit.Mvvm.ComponentModel;
-using CommunityToolkit.Mvvm.Input;
+using System.Windows.Data;
-namespace Q2x9yv8s160mb6u
+namespace Q2x9yv8s160mb6u
-{
+{
-    public class UserControl3ViewModel : ObservableObject
-    {
-        public int Value { get => _Value; set => SetProperty(ref _Value, value); }
-        private int _Value;
-        public RelayCommand IncrementCommand { get; }
-        public RelayCommand DecrementCommand { get; }
-        public UserControl3ViewModel()
-        {
-            IncrementCommand = new(() => Value++);
-            DecrementCommand = new(() => Value--);
-        }
-    }
     public partial class UserControl3 : UserControl
     {
         public int Value { get => (int)GetValue(ValueProperty); set => SetValue(ValueProperty, value); }
-        public static readonly DependencyProperty ValueProperty
-            = DependencyProperty.Register(nameof(Value), typeof(int), typeof(UserControl3),
+        public static readonly DependencyProperty ValueProperty = DependencyProperty.Register(nameof(Value), typeof(int), typeof(UserControl3), new PropertyMetadata(0));
-                new PropertyMetadata(0));
-        private readonly UserControl3ViewModel vm = new();
         public UserControl3()
         {
             InitializeComponent();
+            var vm = new UserControlViewModel();
+            //vm.PropertyChanged += (s, e) => Value = vm.Value;
+            // コードでバインドする？
+            SetBinding(ValueProperty, new Binding(nameof(vm.Value)));
             DataContext = vm;
-            vm.PropertyChanged += (s, e) => Value = vm.Value;
-        }
+        }
-    }
+    }
-}
+}
-```
+```
+UserControl4（継承）
+```xaml
+<local:UserControl4Base
+  x:Class="Q2x9yv8s160mb6u.UserControl4"
+  xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
+  xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
+  xmlns:local="clr-namespace:Q2x9yv8s160mb6u"
+  Value="{Binding Value}">
+  <StackPanel>
+    <TextBlock Text="{Binding Value}" />
+    <RepeatButton Command="{Binding IncrementCommand}" Content="Up" />
+    <RepeatButton Command="{Binding DecrementCommand}" Content="Down" />
+  </StackPanel>
+</local:UserControl4Base>
+```
+```C#
+using System.Windows;
+using System.Windows.Controls;
+namespace Q2x9yv8s160mb6u
+{
+    // <local:UserControl4Base x:Class="Q2x9yv8s160mb6u.UserControl4" Value="{Binding Value}">
+    // こう書けるようにするため、仕方なく間にクラスを挟む
+    public class UserControl4Base : UserControl
+    {
+        public int Value { get => (int)GetValue(ValueProperty); set => SetValue(ValueProperty, value); }
+        public static readonly DependencyProperty ValueProperty = DependencyProperty.Register(nameof(Value), typeof(int), typeof(UserControl4Base), new PropertyMetadata(0));
+    }
+    public partial class UserControl4 : UserControl4Base
+    {
+        public UserControl4()
+        {
+            InitializeComponent();
+            DataContext = new UserControlViewModel();
+        }
+    }
+}
+```
+UserControl5（添付プロパティ）
+```xaml
+<UserControl
+  x:Class="Q2x9yv8s160mb6u.UserControl5"
+  xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
+  xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
+  xmlns:local="clr-namespace:Q2x9yv8s160mb6u"
+  local:UserControl5.Value="{Binding Value}">
+  <StackPanel>
+    <TextBlock Text="{Binding Value}" />
+    <RepeatButton Command="{Binding IncrementCommand}" Content="Up" />
+    <RepeatButton Command="{Binding DecrementCommand}" Content="Down" />
+  </StackPanel>
+</UserControl>
+```
+```C#
+using System.Windows;
+using System.Windows.Controls;
+namespace Q2x9yv8s160mb6u
+{
+    public partial class UserControl5 : UserControl
+    {
+        public static readonly DependencyProperty ValueProperty = DependencyProperty.RegisterAttached("Value", typeof(int), typeof(UserControl5), new PropertyMetadata(0));
+        public static int GetValue(DependencyObject obj) => (int)obj.GetValue(ValueProperty);
+        public static void SetValue(DependencyObject obj, int value) => obj.SetValue(ValueProperty, value);
+        public UserControl5()
+        {
+            InitializeComponent();
+            DataContext = new UserControlViewModel();
+        }
+    }
+}
+```
+![アプリ画像](https://ddjkaamml8q8x.cloudfront.net/questions/2022-01-24/7af7cd21-d841-4266-aa6f-2c6939c63acb.png)