Optimize MaterializationContext

Author: SergeiPavlov

Orm/Xtensive.Orm/Orm/Linq/Materialization/ItemMaterializationContext.cs

@@ -2,8 +2,6 @@
 // This code is distributed under MIT license terms.
 // See the License.txt file in the project root for more information.
 
-using System;
-using System.Collections.Generic;
 using System.Reflection;
 using Xtensive.Core;
 using Xtensive.Orm.Internals;
@@ -14,21 +12,21 @@
 
 namespace Xtensive.Orm.Linq.Materialization
 {
-  internal sealed class ItemMaterializationContext
+  internal readonly struct ItemMaterializationContext
   {
     public static readonly MethodInfo IsMaterializedMethodInfo = WellKnownOrmTypes.ItemMaterializationContext.GetMethod(nameof(IsMaterialized));
     public static readonly MethodInfo GetEntityMethodInfo = WellKnownOrmTypes.ItemMaterializationContext.GetMethod(nameof(GetEntity));
     public static readonly MethodInfo MaterializeMethodInfo = WellKnownOrmTypes.ItemMaterializationContext.GetMethod(nameof(Materialize));
     public static readonly System.Reflection.FieldInfo SessionFieldInfo = WellKnownOrmTypes.ItemMaterializationContext.GetField(nameof(Session));
 
-    public readonly Session Session;
     public readonly MaterializationContext MaterializationContext;
 
-    private readonly TypeIdRegistry typeIdRegistry;
     private readonly Entity[] entities;
-
     public ParameterContext ParameterContext { get; }
 
+    public Session Session => MaterializationContext.Session;
+    private TypeIdRegistry typeIdRegistry => Session.StorageNode.TypeIdRegistry;
+
     public bool IsMaterialized(int index) => entities[index] != null;
 
     public Entity GetEntity(int index) => entities[index];
@@ -75,9 +73,7 @@ public ItemMaterializationContext(MaterializationContext materializationContext,
     {
       ParameterContext = parameterContext;
       MaterializationContext = materializationContext;
-      Session = materializationContext.Session;
 
-      typeIdRegistry = Session.StorageNode.TypeIdRegistry;
       entities = new Entity[materializationContext.EntitiesInRow];
     }
   }

Orm/Xtensive.Orm/Orm/Linq/Materialization/MaterializationContext.cs

@@ -4,10 +4,6 @@
 // Created by: Alexis Kochetov
 // Created:    2009.05.29
 
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using Xtensive.Core;
 using Xtensive.Tuples.Transform;
 using Xtensive.Orm.Internals;
 using Xtensive.Orm.Model;
@@ -31,27 +27,22 @@ private struct EntityMappingCache
     /// <summary>
     /// Gets model of current <see cref="DomainModel">domain model.</see>
     /// </summary>
-    public DomainModel Model { get; private set; }
+    public DomainModel Model => Session.Domain.Model;
 
     /// <summary>
     /// Gets the session in which materialization is executing.
     /// </summary>
-    public Session Session { get; private set; }
+    public Session Session { get; }
 
     /// <summary>
     /// Gets count of entities in query row.
     /// </summary>
-    public int EntitiesInRow { get; private set; }
+    public int EntitiesInRow => entityMappings.Length;
 
     /// <summary>
     /// Gets <see cref="StorageNode">node</see> specific type identifiers registry of current node.
     /// </summary>
-    public TypeIdRegistry TypeIdRegistry
-    {
-      get {
-        return Session.StorageNode.TypeIdRegistry;
-      }
-    }
+    public TypeIdRegistry TypeIdRegistry => Session.StorageNode.TypeIdRegistry;
 
     /// <summary>
     /// Gets or sets queue of materialization actions.
@@ -109,15 +100,10 @@ private int ResolveTypeToNodeSpecificTypeIdentifier(TypeInfo typeInfo)
     public MaterializationContext(Session session, int entityCount)
     {
       Session = session;
-      Model = session.Domain.Model;
-      EntitiesInRow = entityCount;
 
       entityMappings = new EntityMappingCache[entityCount];
-
       for (int i = 0; i < entityMappings.Length; i++)
-        entityMappings[i] = new EntityMappingCache {
-          Items = new Dictionary<int, TypeMapping>()
-        };
+        entityMappings[i] = new() { Items = new Dictionary<int, TypeMapping>() };
     }
   }
-}
+}