質問編集履歴

仮想プロパティをオーバーライドする方式追記

2018/03/23 05:04

投稿

sh_akira

スコア380

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -98,19 +98,35 @@
+文字数制限で省略(編集履歴参照)
+```
+プロパティ名をCallerMemberNameでもらって、Dictionary<string, object>にすべての値を入れる方式です。
+取得と設定にはGetterとSetterを使用して、次のような書き方が出来ます。
+```csharp
-public static class IDictionaryExtensions
+public class Sample : ViewModelBase
 {
+    public bool IsExpanded { get => Getter<bool>(); set => Setter(value); }
+    public bool? IsChecked { get => Getter<bool?>(); set => Setter(value); }
-    public static TValue TryGet<TKey, TValue>(this IDictionary<TKey, TValue> source, TKey key)
+    public string Text { get => Getter<string>(); set => Setter(value); }
-    {
+    public object Data { get => Getter<object>(); set => Setter(value); }
-        if (source.ContainsKey(key)) return source[key];
+    public Sample Parent { get => Getter<Sample>(); set => Setter(value); }
-        return default(TValue);
+    public ObservableCollection<Sample> Children { get => Getter<ObservableCollection<Sample>>(); set => Setter(value); }
-    }
 }
@@ -118,29 +134,189 @@
-プロパティ名をCallerMemberNameでもらって、Dictionary<string, object>にすべての値を入れる方式です。
+###質問
-取得と設定にはGetterとSetterを使用して、次のような書き方が出来ます。
+基本的に使用する際は
-```csharp
+```csharp
+public 型 名前 { get => Getter<型>(); set => Setter(value); }
+```
+これだけ書けばよいので短くて気に入っています。
+ただ、同じ型名を2度書かないといけないため、ObservableCollectionだったりすると
+かなり長くなってしまうのが気に入らないので、何か別の方法で、さらに短くできないかという質問です。
+具体的には2度書いている型を1度にできないのか？returnでの型推論は効かせる方法はないのか？です。
+###追記：仮想プロパティをオーバーライドするクラス動的生成
+Zuishinさんの提案を参考に下記のように書き換えました。
+```csharp
-public class Sample : ViewModelBase
+public abstract class ViewModelBase : INotifyPropertyChanged
 {
-    public bool IsExpanded { get => Getter<bool>(); set => Setter(value); }
-    public bool? IsChecked { get => Getter<bool?>(); set => Setter(value); }
+    public event PropertyChangedEventHandler PropertyChanged;
-    public string Text { get => Getter<string>(); set => Setter(value); }
+    protected virtual void RaisePropertyChanged(string propertyName)
+    {
+        PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
+    }
+    //ref:https://qiita.com/Zuishin/items/0b7080e6f7e277d9394b
+    //ref:http://www.gutgames.com/post/Overridding-a-Property-With-ReflectionEmit.aspx
+    private static Dictionary<Type, Type> typeDictionary = new Dictionary<Type, Type>();
-    public object Data { get => Getter<object>(); set => Setter(value); }
+    public static T Create<T>(params object[] parameters) where T : ViewModelBase
+    {
+        if (!typeDictionary.TryGetValue(typeof(T), out Type type))
+        {
+            var name = "ViewModel_" + Guid.NewGuid().ToString("N");
+            var assemblyName = new AssemblyName(name);
+            var assemblyBuilder = AppDomain.CurrentDomain.DefineDynamicAssembly(
+                assemblyName,
+                AssemblyBuilderAccess.RunAndCollect);
+            var moduleBuilder = assemblyBuilder.DefineDynamicModule(name);
+            var typeBuilder = moduleBuilder.DefineType(name, TypeAttributes.Public | TypeAttributes.Class, typeof(T));
+            //プロパティはvirtualではなくアクセサがvirtualになる。
-    public Sample Parent { get => Getter<Sample>(); set => Setter(value); }
+            var virtualProperties = typeof(T).GetProperties().Where(p => p.GetAccessors().Any(a => a.IsVirtual));
+            foreach (var property in virtualProperties)
+            {
+                /*
+                    * プロパティのオーバーライドするときは、Getter/Setterだけオーバーライドしないといけないらしい
+                    * DefinePropertyでプロパティまで作ってしまうと、新クラスとベースクラスに二つのプロパティが生まれて
+                    * 正しく動作しなくなってしまった
+                    */
+                //新しいプロパティ
+                //var propertyBuilder = typeBuilder.DefineProperty(
+                //    property.Name,
+                //    PropertyAttributes.None,
+                //    property.PropertyType,
+                //    new Type[] { property.PropertyType }
+                //    );
+                //フィールド
+                //バッキングフィールドは直接触れなかった
+                //var field = typeof(T).GetField($"<{property.Name}>k__BackingField", BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.Public);
+                //なので新しいフィールド作成
+                var field = typeBuilder.DefineField($"field_{property.PropertyType}", property.PropertyType, FieldAttributes.Private);
-    public ObservableCollection<Sample> Children { get => Getter<ObservableCollection<Sample>>(); set => Setter(value); }
+                var accessorAttr = MethodAttributes.Public | MethodAttributes.SpecialName | MethodAttributes.HideBySig | MethodAttributes.Virtual;
+                //Getter
+                var getMethod = typeBuilder.DefineMethod($"get_{property.Name}", accessorAttr, property.PropertyType, Type.EmptyTypes);
+                var getIL = getMethod.GetILGenerator();
+                getIL.Emit(OpCodes.Ldarg_0);
+                getIL.Emit(OpCodes.Ldfld, field);
+                getIL.Emit(OpCodes.Ret);
+                //propertyBuilder.SetGetMethod(getMethod);
+                //Setter
+                var setMethod = typeBuilder.DefineMethod($"set_{property.Name}", accessorAttr, null, new Type[] { property.PropertyType });
+                var setIL = setMethod.GetILGenerator();
+                setIL.Emit(OpCodes.Ldarg_0);
+                setIL.Emit(OpCodes.Ldarg_1);
+                setIL.Emit(OpCodes.Stfld, field);
+                setIL.Emit(OpCodes.Ldarg_0);
+                setIL.Emit(OpCodes.Ldstr, property.Name);
+                var meth = typeof(T).GetMethod("RaisePropertyChanged", BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.Public, Type.DefaultBinder, new Type[] { typeof(string) }, null);
+                setIL.Emit(OpCodes.Callvirt, meth);
+                setIL.Emit(OpCodes.Ret);
+                //propertyBuilder.SetSetMethod(setMethod);
+            }
+            type = typeBuilder.CreateType();
+            typeDictionary[typeof(T)] = type;
+        }
+        return (T)Activator.CreateInstance(type, parameters);
+    }
 }
@@ -148,26 +324,196 @@
-###質問
-基本的に使用する際は
-```csharp
-public 型 名前 { get => Getter<型>(); set => Setter(value); }
-```
-これだけ書けばよいので短くて気に入っています。
-ただ、同じ型名を2度書かないといけないため、ObservableCollectionだったりすると
-かなり長くなってしまうのが気に入らないので、何か別の方法で、さらに短くできないかという質問です。
-具体的には2度書いている型を1度にできないのか？returnでの型推論は効かせる方法はないのか？です。
-そもそも現状のコードに何か問題がある場合はご指摘お願いします。
-WPFとXamarin.Formsで使用しています。C#のバージョンの指定はありません。
+```csharp
+public class CheckTreeSource : ViewModelBase
+{
+    public virtual bool IsExpanded { get; set; }
+    public virtual bool? IsChecked { get; set; }
+    public virtual string Text { get; set; }
+    public virtual object Data { get; set; }
+    public virtual CheckTreeSource Parent { get; set; }
+    public virtual ObservableCollection<CheckTreeSource> Children { get; set; }
+}
+```
+使用方法は
+```csharp
+var item = ViewModelBase.Create<CheckTreeSource>();
+```
+動作確認に使用したコードは
+```XAML
+<Window x:Class="ViewModelSample.MainWindow"
+        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
+        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
+        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
+        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
+        xmlns:local="clr-namespace:ViewModelSample"
+        mc:Ignorable="d"
+        Title="MainWindow" Height="450" Width="800">
+    <DockPanel>
+        <Button DockPanel.Dock="Top" Content="CheckAllItems" Click="Button_Click"/>
+        <TreeView Name="CheckTreeView">
+            <TreeView.Resources>
+                <Style TargetType="TreeViewItem">
+                    <Setter Property="IsExpanded" Value="{Binding Path=IsExpanded,Mode=TwoWay}"/>
+                </Style>
+            </TreeView.Resources>
+            <TreeView.ItemTemplate>
+                <HierarchicalDataTemplate DataType="local:CheckTreeSource" ItemsSource="{Binding Children}">
+                    <CheckBox Margin="1" IsChecked="{Binding IsChecked}">
+                        <TextBlock Text="{Binding Text}"/>
+                    </CheckBox>
+                </HierarchicalDataTemplate>
+            </TreeView.ItemTemplate>
+        </TreeView>
+    </DockPanel>
+</Window>
+```
+```csharp
+public partial class MainWindow : Window
+{
+    private ObservableCollection<CheckTreeSource> checkItems = new ObservableCollection<CheckTreeSource>();
+    public MainWindow()
+    {
+        InitializeComponent();
+        var random = new Random();
+        foreach (var i in Enumerable.Range(0, 20))
+        {
+            var item = ViewModelBase.Create<CheckTreeSource>();
+            item.Text = $"Parent{i}";
+            item.IsChecked = random.Next(2) == 0;
+            item.IsExpanded = true;
+            checkItems.Add(item);
+        }
+        foreach (var item in checkItems)
+        {
+            item.Children = new ObservableCollection<CheckTreeSource>();
+            foreach (var i in Enumerable.Range(0, 3))
+            {
+                var childitem = ViewModelBase.Create<CheckTreeSource>();
+                childitem.Parent = item;
+                childitem.Text = $"Child{i}";
+                childitem.IsChecked = item.IsChecked;
+                item.Children.Add(childitem);
+            }
+        }
+        CheckTreeView.ItemsSource = checkItems;
+    }
+    private void Button_Click(object sender, RoutedEventArgs e)
+    {
+        foreach(var item in checkItems)
+        {
+            item.IsChecked = true;
+            foreach(var childitem in item.Children)
+            {
+                childitem.IsChecked = true;
+            }
+        }
+    }
+}
+```
+初めてILに触ったのでとても勉強になりました。
+プロパティのオーバーライドもgetter/setterだけオーバーライドでいいと思わず、新発見でした。
+値を入れてたDictionaryも無くなってとってもいい感じです。