refactor and add unit test for AssetProperties_Item

NeVeSpl · NeVeSpl · commit c611dbb2cb8c · 2024-05-12T16:13:08.000+02:00
diff --git a/sources/RevitDBExplorer/Domain/DataModel/Accessors/IAccessorForDefaultPresenter.cs b/sources/RevitDBExplorer/Domain/DataModel/Accessors/IAccessorForDefaultPresenter.cs
@@ -1,13 +1,15 @@
 ﻿using System.Collections.Generic;
 using RevitDBExplorer.Domain.DataModel.ValueContainers.Base;
+using RevitExplorer.Visualizations.DrawingVisuals;
 
 // (c) Revit Database Explorer https://github.com/NeVeSpl/RevitDBExplorer/blob/main/license.md
 
 namespace RevitDBExplorer.Domain.DataModel.Accessors
 {
-    internal interface IAccessorWithReadAndSnoop : IAccessor
+    internal interface IAccessorForDefaultPresenter : IAccessor
     {
         ReadResult Read(SnoopableContext context, object @object);
         IEnumerable<SnoopableObject> Snoop(SnoopableContext context, object @object, IValueContainer state);
+        IEnumerable<DrawingVisual> GetVisualization(SnoopableContext context, object @object, IValueContainer state);
     }
 }
diff --git a/sources/RevitDBExplorer/Domain/DataModel/Accessors/ReadResult.cs b/sources/RevitDBExplorer/Domain/DataModel/Accessors/ReadResult.cs
@@ -1,22 +1,41 @@
-﻿using RevitDBExplorer.Domain.DataModel.ValueContainers.Base;
+﻿using System.Diagnostics.CodeAnalysis;
+using RevitDBExplorer.Domain.DataModel.ValueContainers.Base;
 
 // (c) Revit Database Explorer https://github.com/NeVeSpl/RevitDBExplorer/blob/main/license.md
 
 namespace RevitDBExplorer.Domain.DataModel.Accessors
 {
     internal readonly ref struct ReadResult
     {
-        public string Label { get; init; }
+        public required string Label { get; init; }
         public string AccessorName { get; init; }
-        public bool CanBeSnooped { get; init; }
+        public required bool CanBeSnooped { get; init; } = false;
+        public bool CanBeVisualized { get; init; } = false;
         public IValueContainer State { get; init; }
 
 
-        public ReadResult(string value, string accessorName, bool canBeSnooped, IValueContainer state = null)
+        public ReadResult()
         {
-            Label = value;
+            
+        }
+
+
+        [SetsRequiredMembers]
+        public ReadResult(string label, string accessorName, bool canBeSnooped, IValueContainer state = null)
+        {
+            Label = label;
+            AccessorName = accessorName;
+            CanBeSnooped = canBeSnooped;
+            State = state;
+        }
+
+        [SetsRequiredMembers]
+        public ReadResult(string label, string accessorName, bool canBeSnooped = false, bool canBeVisualized = false, IValueContainer state = null)
+        {
+            Label = label;
             AccessorName = accessorName;
             CanBeSnooped = canBeSnooped;
+            CanBeVisualized = canBeVisualized;
             State = state;
         }
     }
diff --git a/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/AssetProperties/AssetProperties_Item.cs b/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/AssetProperties/AssetProperties_Item.cs
@@ -1,24 +1,29 @@
 ﻿using System.Collections.Generic;
 using System.Linq.Expressions;
-using Autodesk.Revit.DB;
 using Autodesk.Revit.DB.Visual;
+using RevitDBExplorer.Domain.DataModel.Accessors;
+using RevitDBExplorer.Domain.DataModel.ValueContainers.Base;
 
 // (c) Revit Database Explorer https://github.com/NeVeSpl/RevitDBExplorer/blob/main/license.md
 
 namespace RevitDBExplorer.Domain.DataModel.MemberAccessors
 {
-    internal class AssetProperties_Item : MemberAccessorByType<AssetProperties>, ICanCreateMemberAccessor
+    internal class AssetProperties_Item : MemberAccessorTypedWithDefaultPresenter<AssetProperties>, ICanCreateMemberAccessor
     { 
-        IEnumerable<LambdaExpression> ICanCreateMemberAccessor.GetHandledMembers() { yield return (AssetProperties x) => x[0]; }     
+        IEnumerable<LambdaExpression> ICanCreateMemberAccessor.GetHandledMembers() { yield return (AssetProperties x) => x[0]; }
 
 
-        protected override bool CanBeSnoooped(Document document, AssetProperties assetProperties) => assetProperties.Size > 0;
-        protected override string GetLabel(Document document, AssetProperties assetProperties) => $"[{nameof(AssetProperty)} : {assetProperties.Size}]";
-        protected override IEnumerable<SnoopableObject> Snooop(Document document, AssetProperties assetProperties)
+        public override ReadResult Read(SnoopableContext context, AssetProperties assetProperties) => new()
+        {
+            Label = Labeler.GetLabelForCollection(nameof(AssetProperty), assetProperties.Size),
+            CanBeSnooped = assetProperties.Size > 0
+        };
+
+        public override IEnumerable<SnoopableObject> Snoop(SnoopableContext context, AssetProperties assetProperties, IValueContainer state)
         {            
             for (int i = 0; i < assetProperties.Size; i++)
             {
-                yield return new SnoopableObject(document, assetProperties[i]);
+                yield return new SnoopableObject(context.Document, assetProperties[i]);
             }
         }
     }
diff --git a/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/MemberAccessorByFunc.cs b/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/MemberAccessorByFunc.cs
@@ -8,7 +8,7 @@
 
 namespace RevitDBExplorer.Domain.DataModel.MemberAccessors
 {
-    internal class MemberAccessorByFunc<TSnoopedObjectType, TReturnType> : MemberAccessorTypedWithReadAndSnoop<TSnoopedObjectType>
+    internal class MemberAccessorByFunc<TSnoopedObjectType, TReturnType> : MemberAccessorTypedWithDefaultPresenter<TSnoopedObjectType>
     {     
         private readonly Func<Document, TSnoopedObjectType, TReturnType> get;
         private readonly Func<Document, TSnoopedObjectType, IEnumerable<SnoopableObject>> snoop;
@@ -25,7 +25,7 @@ public override ReadResult Read(SnoopableContext context, TSnoopedObjectType @ob
             var value = new ValueContainer<TReturnType>();
             var result = get(context.Document, @object);
             value.SetValueTyped(context, result);
-            return new ReadResult(value.ValueAsString, "[ByFunc] " + value.TypeHandlerName, value.CanBeSnooped, value);
+            return new ReadResult(value.ValueAsString, "[ByFunc] " + value.TypeHandlerName, value.CanBeSnooped, value.CanBeVisualized, value);
         }
         public override IEnumerable<SnoopableObject> Snoop(SnoopableContext context, TSnoopedObjectType @object, IValueContainer state)
         {
diff --git a/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/MemberAccessorByIteration.cs b/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/MemberAccessorByIteration.cs
@@ -10,7 +10,7 @@
 
 namespace RevitDBExplorer.Domain.DataModel.MemberAccessors
 {
-    internal sealed class MemberAccessorByIteration<TSnoopedObjectType, TReturnType> : MemberAccessorTypedWithReadAndSnoop<TSnoopedObjectType>
+    internal sealed class MemberAccessorByIteration<TSnoopedObjectType, TReturnType> : MemberAccessorTypedWithDefaultPresenter<TSnoopedObjectType>
     {
         private readonly string getMethodReturnTypeName;
         private readonly ParameterInfo getMethodParameter;
@@ -30,7 +30,7 @@ public MemberAccessorByIteration(MethodInfo getMethod)
         public override ReadResult Read(SnoopableContext context, TSnoopedObjectType @object)
         {
             var count = CountValues(context, getMethodParameter.ParameterType);
-            return new ReadResult(Labeler.GetLabelForCollection(getMethodReturnTypeName, count), "[ByIteration]", true);
+            return new ReadResult(Labeler.GetLabelForCollection(getMethodReturnTypeName, count), "[ByIteration]", true, false);
         }
         public override IEnumerable<SnoopableObject> Snoop(SnoopableContext context, TSnoopedObjectType @object, IValueContainer state)
         {            
diff --git a/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/MemberAccessorByRef.cs b/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/MemberAccessorByRef.cs
@@ -10,7 +10,7 @@
 
 namespace RevitDBExplorer.Domain.DataModel.MemberAccessors
 {
-    internal sealed class MemberAccessorByRef : MemberAccessorTypedWithReadAndSnoop<object>, IAccessorWithCodeGeneration
+    internal sealed class MemberAccessorByRef : MemberAccessorTypedWithDefaultPresenter<object>, IAccessorWithCodeGeneration
     {
         private readonly MethodInfo getMethod;          
 
@@ -28,12 +28,9 @@ public override ReadResult Read(SnoopableContext context, object @object)
             var resolvedArgs = ResolveArguments(paramsDef, context.Document, @object);            
             var result = getMethod.Invoke(@object, resolvedArgs);
             value.SetValue(context, result);
-            return new ReadResult(value.ValueAsString, "[ByRef] " + value.TypeHandlerName, value.CanBeSnooped, value);
-        }
-        public override IEnumerable<SnoopableObject> Snoop(SnoopableContext context, object @object, IValueContainer state)
-        {            
-            return state.Snoop();
+            return new ReadResult(value.ValueAsString, "[ByRef] " + value.TypeHandlerName, value.CanBeSnooped, value.CanBeVisualized, value);
         }
+       
 
         public static Type[] HandledParameterTypes = new[] { typeof(Document), typeof(Options), typeof(View), typeof(SpatialElementBoundaryOptions) };
 
diff --git a/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/MemberAccessorByRefCompiled.cs b/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/MemberAccessorByRefCompiled.cs
@@ -9,7 +9,7 @@
 
 namespace RevitDBExplorer.Domain.DataModel.MemberAccessors
 {
-    internal sealed class MemberAccessorByRefCompiled<TSnoopedObjectType, TReturnType> : MemberAccessorTypedWithReadAndSnoop<TSnoopedObjectType>, IAccessorWithCodeGeneration
+    internal sealed class MemberAccessorByRefCompiled<TSnoopedObjectType, TReturnType> : MemberAccessorTypedWithDefaultPresenter<TSnoopedObjectType>, IAccessorWithCodeGeneration
     {
         private readonly MethodInfo getMethod;
         private readonly Func<TSnoopedObjectType, TReturnType> func;
@@ -25,13 +25,8 @@ public override ReadResult Read(SnoopableContext context, TSnoopedObjectType typ
             var value = new ValueContainer<TReturnType>();        
             var result = func(typedObject);
             value.SetValueTyped(context, result);
-            return new ReadResult(value.ValueAsString, "[ByRefComp] " + value.TypeHandlerName, value.CanBeSnooped, value);
-        }
-
-        public override IEnumerable<SnoopableObject> Snoop(SnoopableContext context, TSnoopedObjectType typedObject, IValueContainer state)
-        {
-            return state.Snoop();
-        }
+            return new ReadResult(value.ValueAsString, "[ByRefComp] " + value.TypeHandlerName, value.CanBeSnooped, value.CanBeVisualized, value);
+        }      
 
 
         public string GenerateInvocationForScript(TemplateInputsKind inputsKind)
diff --git a/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/MemberAccessorByType.cs b/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/MemberAccessorByType.cs
@@ -9,14 +9,15 @@
 
 namespace RevitDBExplorer.Domain.DataModel.MemberAccessors
 {
-    internal abstract class MemberAccessorByType<TSnoopedObjectType> : MemberAccessorTypedWithReadAndSnoop<TSnoopedObjectType> where TSnoopedObjectType : class
+    internal abstract class MemberAccessorByType<TSnoopedObjectType> : MemberAccessorTypedWithDefaultPresenter<TSnoopedObjectType> where TSnoopedObjectType : class
     {
         public sealed override ReadResult Read(SnoopableContext context, TSnoopedObjectType @object)
         {           
             string label = GetLabel(context.Document, @object);
             bool canBeSnooped = CanBeSnoooped(context.Document, @object);
+            bool canBeVisualized = false;
 
-            return new ReadResult(label, "[ByType] " + GetType().GetCSharpName(), canBeSnooped);
+            return new ReadResult(label, "[ByType] " + GetType().GetCSharpName(), canBeSnooped, canBeVisualized);
         }
         protected abstract bool CanBeSnoooped(Document document, TSnoopedObjectType value);
         protected abstract string GetLabel(Document document, TSnoopedObjectType value);
@@ -27,4 +28,34 @@ public sealed override IEnumerable<SnoopableObject> Snoop(SnoopableContext conte
         }
         protected virtual IEnumerable<SnoopableObject> Snooop(Document document, TSnoopedObjectType value) => Enumerable.Empty<SnoopableObject>();       
     }
+
+
+    internal abstract class MemberAccessorByTypeLambda<TSnoopedObjectType> : MemberAccessorByType<TSnoopedObjectType> where TSnoopedObjectType : class
+    {
+        public Overrides Override { get; } = new Overrides();
+
+        protected override bool CanBeSnoooped(Document document, TSnoopedObjectType value)
+        {
+            ArgumentNullException.ThrowIfNull(Override.CanBeSnooped);
+            return Override.CanBeSnooped(document, value);
+        }
+        protected override string GetLabel(Document document, TSnoopedObjectType value)
+        {
+            ArgumentNullException.ThrowIfNull(Override.GetLabel);
+            return Override.GetLabel(document, value);
+        }
+        protected override IEnumerable<SnoopableObject> Snooop(Document document, TSnoopedObjectType value)
+        {
+            if (Override.Snoop == null) return null;
+            return Override.Snoop(document, value);
+        }
+
+
+        public class Overrides
+        {
+            public Func<Document, TSnoopedObjectType, bool> CanBeSnooped { get; set; }
+            public Func<Document, TSnoopedObjectType, string> GetLabel { get; set; }            
+            public Func<Document, TSnoopedObjectType, IEnumerable<SnoopableObject>> Snoop { get; set; }
+        }
+    }
 }
diff --git a/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/MemberAccessorFactory.cs b/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/MemberAccessorFactory.cs
@@ -11,7 +11,7 @@ namespace RevitDBExplorer.Domain.DataModel.MemberAccessors
 {
     internal static class MemberAccessorFactory
     {        
-        private static readonly Dictionary<string, Func<IAccessor>> forTypeMembers = new();
+        private static readonly Dictionary<string, Func<IAccessor>> memberAccessorOverrides = new();
 
 
         public static void Init()
@@ -26,7 +26,7 @@ static MemberAccessorFactory()
                 foreach (var handledMember in accessor.GetHandledMembers())
                 {
                     var memberId = handledMember.GetUniqueId();
-                    forTypeMembers[memberId] = accessor.GetType().CompileFactoryMethod<IAccessor>();
+                    memberAccessorOverrides[memberId] = accessor.GetType().CompileFactoryMethod<IAccessor>();
                 }
             }
         }
@@ -50,20 +50,20 @@ public static IAccessor CreateMemberAccessor(MethodInfo getMethod, MethodInfo se
 
         private static IAccessor CreateMemberAccessor(MethodInfo getMethod)
         {
+            if (memberAccessorOverrides.TryGetValue(getMethod.GetUniqueId(), out Func<IAccessor> factory))
+            {
+                return factory();
+            }
+
             if (getMethod.IsStatic)
             {
                 return new MemberAccessorForStatic(getMethod);
             }
 
-            if (getMethod.ReturnType == typeof(void) && getMethod.Name != "GetOverridableHookParameters")
+            if (getMethod.ReturnType == typeof(void))
             {
                 return new MemberAccessorForNotExposed(getMethod);
-            }            
-
-            if (forTypeMembers.TryGetValue(getMethod.GetUniqueId(), out Func<IAccessor> factory))
-            {
-                return factory();
-            }
+            }  
 
             var @params = getMethod.GetParameters();
           
diff --git a/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/MemberAccessorForConstValue.cs b/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/MemberAccessorForConstValue.cs
@@ -7,7 +7,7 @@
 
 namespace RevitDBExplorer.Domain.DataModel.MemberAccessors
 {
-    internal sealed class MemberAccessorForConstValue : MemberAccessorTypedWithReadAndSnoop<object>
+    internal sealed class MemberAccessorForConstValue : MemberAccessorTypedWithDefaultPresenter<object>
     {
         private readonly IValueContainer value;     
 
diff --git a/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/MemberAccessorTyped.cs b/sources/RevitDBExplorer/Domain/DataModel/MemberAccessors/MemberAccessorTyped.cs
@@ -4,6 +4,7 @@
 using RevitDBExplorer.Domain.DataModel.ValueContainers.Base;
 using RevitDBExplorer.Domain.DataModel.ValueViewModels;
 using RevitDBExplorer.Domain.DataModel.ValueViewModels.Base;
+using RevitExplorer.Visualizations.DrawingVisuals;
 
 // (c) Revit Database Explorer https://github.com/NeVeSpl/RevitDBExplorer/blob/main/license.md
 
@@ -24,27 +25,41 @@ IValueViewModel IAccessor.CreatePresenter(SnoopableContext context, object @obje
     }
 
 
-    internal abstract class MemberAccessorTypedWithReadAndSnoop<TSnoopedObjectType> : MemberAccessorTyped<TSnoopedObjectType>, IAccessorWithReadAndSnoop
+    internal abstract class MemberAccessorTypedWithDefaultPresenter<TSnoopedObjectType> : MemberAccessorTyped<TSnoopedObjectType>, IAccessorForDefaultPresenter
     {
         public override IValueViewModel CreatePresenter(SnoopableContext context, TSnoopedObjectType @object)
         {            
             return new DefaultPresenter(this);
         }
 
-        ReadResult IAccessorWithReadAndSnoop.Read(SnoopableContext context, object @object)
+        ReadResult IAccessorForDefaultPresenter.Read(SnoopableContext context, object @object)
         {
             Guard.IsAssignableToType<TSnoopedObjectType>(@object);      
             var typedObject = (TSnoopedObjectType) @object;          
             return Read(context, typedObject);
         }
         public abstract ReadResult Read(SnoopableContext context, TSnoopedObjectType typedObject);
 
-        IEnumerable<SnoopableObject> IAccessorWithReadAndSnoop.Snoop(SnoopableContext context, object @object, IValueContainer state)
+        IEnumerable<SnoopableObject> IAccessorForDefaultPresenter.Snoop(SnoopableContext context, object @object, IValueContainer state)
         {
             Guard.IsAssignableToType<TSnoopedObjectType>(@object);
             var typedObject = (TSnoopedObjectType) @object;            
             return Snoop(context, typedObject, state) ?? Enumerable.Empty<SnoopableObject>();
         }
-        public virtual IEnumerable<SnoopableObject> Snoop(SnoopableContext context, TSnoopedObjectType typedObject, IValueContainer state) => null;
+        public virtual IEnumerable<SnoopableObject> Snoop(SnoopableContext context, TSnoopedObjectType typedObject, IValueContainer state)
+        {
+            return state?.Snoop();
+        }
+
+        IEnumerable<DrawingVisual> IAccessorForDefaultPresenter.GetVisualization(SnoopableContext context, object @object, IValueContainer state)
+        {
+            Guard.IsAssignableToType<TSnoopedObjectType>(@object);
+            var typedObject = (TSnoopedObjectType)@object;
+            return GetVisualization(context, typedObject, state) ?? Enumerable.Empty<DrawingVisual>();
+        }
+        public virtual IEnumerable<DrawingVisual> GetVisualization(SnoopableContext context, TSnoopedObjectType typedObject, IValueContainer state)
+        {
+            return state?.GetVisualization();
+        }
     }
 }
diff --git a/sources/RevitDBExplorer/Domain/DataModel/MemberTemplates/Accessors/BoundingBox_BoundingBoxIntersectsFilter.cs b/sources/RevitDBExplorer/Domain/DataModel/MemberTemplates/Accessors/BoundingBox_BoundingBoxIntersectsFilter.cs
@@ -9,7 +9,7 @@
 
 namespace RevitDBExplorer.Domain.DataModel.MemberTemplates.Accessors
 {
-    internal class BoundingBox_BoundingBoxIntersectsFilter : MemberAccessorTypedWithReadAndSnoop<BoundingBoxXYZ>
+    internal class BoundingBox_BoundingBoxIntersectsFilter : MemberAccessorTypedWithDefaultPresenter<BoundingBoxXYZ>
     {
         public override ReadResult Read(SnoopableContext context, BoundingBoxXYZ boundingBox)
         {
diff --git a/sources/RevitDBExplorer/Domain/DataModel/MemberTemplates/Accessors/HostObjectUtils_GetSideFaces.cs b/sources/RevitDBExplorer/Domain/DataModel/MemberTemplates/Accessors/HostObjectUtils_GetSideFaces.cs
@@ -9,7 +9,7 @@
 
 namespace RevitDBExplorer.Domain.DataModel.MemberTemplates.Accessors
 {
-    internal class HostObjectUtils_GetSideFaces : MemberAccessorTypedWithReadAndSnoop<HostObject>
+    internal class HostObjectUtils_GetSideFaces : MemberAccessorTypedWithDefaultPresenter<HostObject>
     {
         public override ReadResult Read(SnoopableContext context, HostObject element)
         {            
diff --git a/sources/RevitDBExplorer/Domain/DataModel/MemberTemplates/Accessors/JoinGeometryUtils_IsCuttingElementInJoin.cs b/sources/RevitDBExplorer/Domain/DataModel/MemberTemplates/Accessors/JoinGeometryUtils_IsCuttingElementInJoin.cs
@@ -9,7 +9,7 @@
 
 namespace RevitDBExplorer.Domain.DataModel.MemberTemplates.Accessors
 {
-    internal class JoinGeometryUtils_IsCuttingElementInJoin : MemberAccessorTypedWithReadAndSnoop<Element>
+    internal class JoinGeometryUtils_IsCuttingElementInJoin : MemberAccessorTypedWithDefaultPresenter<Element>
     {
         public override ReadResult Read(SnoopableContext context, Element element)
         {            
diff --git a/sources/RevitDBExplorer/Domain/DataModel/MemberTemplates/Accessors/Schema_GetAllElements.cs b/sources/RevitDBExplorer/Domain/DataModel/MemberTemplates/Accessors/Schema_GetAllElements.cs
@@ -10,7 +10,7 @@
 
 namespace RevitDBExplorer.Domain.DataModel.MemberTemplates.Accessors
 {
-    internal class Schema_GetAllElements : MemberAccessorTypedWithReadAndSnoop<Schema>
+    internal class Schema_GetAllElements : MemberAccessorTypedWithDefaultPresenter<Schema>
     {
         public override ReadResult Read(SnoopableContext context, Schema schema)
         {
diff --git a/sources/RevitDBExplorer/Domain/DataModel/Parameters/ParameterAccessor.cs b/sources/RevitDBExplorer/Domain/DataModel/Parameters/ParameterAccessor.cs
diff --git a/sources/RevitDBExplorer/Domain/DataModel/ValueViewModels/DefaultPresenter.cs b/sources/RevitDBExplorer/Domain/DataModel/ValueViewModels/DefaultPresenter.cs
diff --git a/sources/RevitDBExplorer/UIComponents/List/ListView.xaml b/sources/RevitDBExplorer/UIComponents/List/ListView.xaml
diff --git a/tests/MemberOverridesTests.cs b/tests/MemberOverridesTests.cs

Original file line number	Diff line number	Diff line change
`@@ -1,13 +1,15 @@`
`1`	`1`	`using System.Collections.Generic;`
`2`	`2`	`using RevitDBExplorer.Domain.DataModel.ValueContainers.Base;`
	`3`	`+using RevitExplorer.Visualizations.DrawingVisuals;`
`3`	`4`
`4`	`5`	`// (c) Revit Database Explorer https://github.com/NeVeSpl/RevitDBExplorer/blob/main/license.md`
`5`	`6`
`6`	`7`	`namespace RevitDBExplorer.Domain.DataModel.Accessors`
`7`	`8`	`{`
`8`		`- internal interface IAccessorWithReadAndSnoop : IAccessor`
	`9`	`+ internal interface IAccessorForDefaultPresenter : IAccessor`
`9`	`10`	`{`
`10`	`11`	`ReadResult Read(SnoopableContext context, object @object);`
`11`	`12`	`IEnumerable<SnoopableObject> Snoop(SnoopableContext context, object @object, IValueContainer state);`
	`13`	`+ IEnumerable<DrawingVisual> GetVisualization(SnoopableContext context, object @object, IValueContainer state);`
`12`	`14`	`}`
`13`	`15`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,24 +1,29 @@`
`1`	`1`	`using System.Collections.Generic;`
`2`	`2`	`using System.Linq.Expressions;`
`3`		`-using Autodesk.Revit.DB;`
`4`	`3`	`using Autodesk.Revit.DB.Visual;`
	`4`	`+using RevitDBExplorer.Domain.DataModel.Accessors;`
	`5`	`+using RevitDBExplorer.Domain.DataModel.ValueContainers.Base;`
`5`	`6`
`6`	`7`	`// (c) Revit Database Explorer https://github.com/NeVeSpl/RevitDBExplorer/blob/main/license.md`
`7`	`8`
`8`	`9`	`namespace RevitDBExplorer.Domain.DataModel.MemberAccessors`
`9`	`10`	`{`
`10`		`- internal class AssetProperties_Item : MemberAccessorByType<AssetProperties>, ICanCreateMemberAccessor`
	`11`	`+ internal class AssetProperties_Item : MemberAccessorTypedWithDefaultPresenter<AssetProperties>, ICanCreateMemberAccessor`
`11`	`12`	`{`
`12`		`- IEnumerable<LambdaExpression> ICanCreateMemberAccessor.GetHandledMembers() { yield return (AssetProperties x) => x[0]; }`
	`13`	`+ IEnumerable<LambdaExpression> ICanCreateMemberAccessor.GetHandledMembers() { yield return (AssetProperties x) => x[0]; }`
`13`	`14`
`14`	`15`
`15`		`- protected override bool CanBeSnoooped(Document document, AssetProperties assetProperties) => assetProperties.Size > 0;`
`16`		`- protected override string GetLabel(Document document, AssetProperties assetProperties) => $"[{nameof(AssetProperty)} : {assetProperties.Size}]";`
`17`		`- protected override IEnumerable<SnoopableObject> Snooop(Document document, AssetProperties assetProperties)`
	`16`	`+ public override ReadResult Read(SnoopableContext context, AssetProperties assetProperties) => new()`
	`17`	`+ {`
	`18`	`+ Label = Labeler.GetLabelForCollection(nameof(AssetProperty), assetProperties.Size),`
	`19`	`+ CanBeSnooped = assetProperties.Size > 0`
	`20`	`+ };`
	`21`	`+`
	`22`	`+ public override IEnumerable<SnoopableObject> Snoop(SnoopableContext context, AssetProperties assetProperties, IValueContainer state)`
`18`	`23`	`{`
`19`	`24`	`for (int i = 0; i < assetProperties.Size; i++)`
`20`	`25`	`{`
`21`		`- yield return new SnoopableObject(document, assetProperties[i]);`
	`26`	`+ yield return new SnoopableObject(context.Document, assetProperties[i]);`
`22`	`27`	`}`
`23`	`28`	`}`
`24`	`29`	`}`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`
`9`	`9`	`namespace RevitDBExplorer.Domain.DataModel.MemberAccessors`
`10`	`10`	`{`
`11`		`- internal class MemberAccessorByFunc<TSnoopedObjectType, TReturnType> : MemberAccessorTypedWithReadAndSnoop<TSnoopedObjectType>`
	`11`	`+ internal class MemberAccessorByFunc<TSnoopedObjectType, TReturnType> : MemberAccessorTypedWithDefaultPresenter<TSnoopedObjectType>`
`12`	`12`	`{`
`13`	`13`	`private readonly Func<Document, TSnoopedObjectType, TReturnType> get;`
`14`	`14`	`private readonly Func<Document, TSnoopedObjectType, IEnumerable<SnoopableObject>> snoop;`
`@@ -25,7 +25,7 @@ public override ReadResult Read(SnoopableContext context, TSnoopedObjectType @ob`
`25`	`25`	`var value = new ValueContainer<TReturnType>();`
`26`	`26`	`var result = get(context.Document, @object);`
`27`	`27`	`value.SetValueTyped(context, result);`
`28`		`- return new ReadResult(value.ValueAsString, "[ByFunc] " + value.TypeHandlerName, value.CanBeSnooped, value);`
	`28`	`+ return new ReadResult(value.ValueAsString, "[ByFunc] " + value.TypeHandlerName, value.CanBeSnooped, value.CanBeVisualized, value);`
`29`	`29`	`}`
`30`	`30`	`public override IEnumerable<SnoopableObject> Snoop(SnoopableContext context, TSnoopedObjectType @object, IValueContainer state)`
`31`	`31`	`{`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@`
`10`	`10`
`11`	`11`	`namespace RevitDBExplorer.Domain.DataModel.MemberAccessors`
`12`	`12`	`{`
`13`		`- internal sealed class MemberAccessorByIteration<TSnoopedObjectType, TReturnType> : MemberAccessorTypedWithReadAndSnoop<TSnoopedObjectType>`
	`13`	`+ internal sealed class MemberAccessorByIteration<TSnoopedObjectType, TReturnType> : MemberAccessorTypedWithDefaultPresenter<TSnoopedObjectType>`
`14`	`14`	`{`
`15`	`15`	`private readonly string getMethodReturnTypeName;`
`16`	`16`	`private readonly ParameterInfo getMethodParameter;`
`@@ -30,7 +30,7 @@ public MemberAccessorByIteration(MethodInfo getMethod)`
`30`	`30`	`public override ReadResult Read(SnoopableContext context, TSnoopedObjectType @object)`
`31`	`31`	`{`
`32`	`32`	`var count = CountValues(context, getMethodParameter.ParameterType);`
`33`		`- return new ReadResult(Labeler.GetLabelForCollection(getMethodReturnTypeName, count), "[ByIteration]", true);`
	`33`	`+ return new ReadResult(Labeler.GetLabelForCollection(getMethodReturnTypeName, count), "[ByIteration]", true, false);`
`34`	`34`	`}`
`35`	`35`	`public override IEnumerable<SnoopableObject> Snoop(SnoopableContext context, TSnoopedObjectType @object, IValueContainer state)`
`36`	`36`	`{`
Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ namespace RevitDBExplorer.Domain.DataModel.MemberAccessors`
`11`	`11`	`{`
`12`	`12`	`internal static class MemberAccessorFactory`
`13`	`13`	`{`
`14`		`- private static readonly Dictionary<string, Func<IAccessor>> forTypeMembers = new();`
	`14`	`+ private static readonly Dictionary<string, Func<IAccessor>> memberAccessorOverrides = new();`
`15`	`15`
`16`	`16`
`17`	`17`	`public static void Init()`
`@@ -26,7 +26,7 @@ static MemberAccessorFactory()`
`26`	`26`	`foreach (var handledMember in accessor.GetHandledMembers())`
`27`	`27`	`{`
`28`	`28`	`var memberId = handledMember.GetUniqueId();`
`29`		`- forTypeMembers[memberId] = accessor.GetType().CompileFactoryMethod<IAccessor>();`
	`29`	`+ memberAccessorOverrides[memberId] = accessor.GetType().CompileFactoryMethod<IAccessor>();`
`30`	`30`	`}`
`31`	`31`	`}`
`32`	`32`	`}`
`@@ -50,20 +50,20 @@ public static IAccessor CreateMemberAccessor(MethodInfo getMethod, MethodInfo se`
`50`	`50`
`51`	`51`	`private static IAccessor CreateMemberAccessor(MethodInfo getMethod)`
`52`	`52`	`{`
	`53`	`+ if (memberAccessorOverrides.TryGetValue(getMethod.GetUniqueId(), out Func<IAccessor> factory))`
	`54`	`+ {`
	`55`	`+ return factory();`
	`56`	`+ }`
	`57`	`+`
`53`	`58`	`if (getMethod.IsStatic)`
`54`	`59`	`{`
`55`	`60`	`return new MemberAccessorForStatic(getMethod);`
`56`	`61`	`}`
`57`	`62`
`58`		`- if (getMethod.ReturnType == typeof(void) && getMethod.Name != "GetOverridableHookParameters")`
	`63`	`+ if (getMethod.ReturnType == typeof(void))`
`59`	`64`	`{`
`60`	`65`	`return new MemberAccessorForNotExposed(getMethod);`
`61`		`- }`
`62`		`-`
`63`		`- if (forTypeMembers.TryGetValue(getMethod.GetUniqueId(), out Func<IAccessor> factory))`
`64`		`- {`
`65`		`- return factory();`
`66`		`- }`
	`66`	`+ }`
`67`	`67`
`68`	`68`	`var @params = getMethod.GetParameters();`
`69`	`69`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`
`8`	`8`	`namespace RevitDBExplorer.Domain.DataModel.MemberAccessors`
`9`	`9`	`{`
`10`		`- internal sealed class MemberAccessorForConstValue : MemberAccessorTypedWithReadAndSnoop<object>`
	`10`	`+ internal sealed class MemberAccessorForConstValue : MemberAccessorTypedWithDefaultPresenter<object>`
`11`	`11`	`{`
`12`	`12`	`private readonly IValueContainer value;`
`13`	`13`