From bacb5979dade233e23176034b0febda6e6976b0c Mon Sep 17 00:00:00 2001
From: Simon G <simon.gockner@gmail.com>
Date: Fri, 3 Dec 2021 12:16:08 +0100
Subject: [PATCH] #50: add property CreateMethods

---
 .../Factories/CustomTypedFactory.cs           |  2 +-
 .../Factories/TypedFactory.cs                 |  8 ++---
 .../Factories/TypedFactoryBase.cs             | 36 +++++++++++++++++++
 .../Interfaces/Factories/ITypedFactory.cs     |  8 ++++-
 .../Registrations/RegistrationBase.cs         |  2 +-
 5 files changed, 47 insertions(+), 9 deletions(-)
 create mode 100644 LightweightIocContainer/Factories/TypedFactoryBase.cs

diff --git a/LightweightIocContainer/Factories/CustomTypedFactory.cs b/LightweightIocContainer/Factories/CustomTypedFactory.cs
index e7538b2..67fa08c 100644
--- a/LightweightIocContainer/Factories/CustomTypedFactory.cs
+++ b/LightweightIocContainer/Factories/CustomTypedFactory.cs
@@ -9,7 +9,7 @@ namespace LightweightIocContainer.Factories
     /// <summary>
     /// <see cref="ITypedFactory"/> implementation for custom implemented factories
     /// </summary>
-    public class CustomTypedFactory : ITypedFactory
+    public class CustomTypedFactory<TFactory> : TypedFactoryBase<TFactory>
     {
         
     }
diff --git a/LightweightIocContainer/Factories/TypedFactory.cs b/LightweightIocContainer/Factories/TypedFactory.cs
index 05137b2..90808c3 100644
--- a/LightweightIocContainer/Factories/TypedFactory.cs
+++ b/LightweightIocContainer/Factories/TypedFactory.cs
@@ -17,7 +17,7 @@ namespace LightweightIocContainer.Factories
     /// Class to help implement an abstract typed factory
     /// </summary>
     /// <typeparam name="TFactory">The type of the abstract factory</typeparam>
-    public class TypedFactory<TFactory> : ITypedFactory<TFactory>
+    public class TypedFactory<TFactory> : TypedFactoryBase<TFactory>, ITypedFactory<TFactory>
     {
         private const string CLEAR_MULTITON_INSTANCE_METHOD_NAME = "ClearMultitonInstance";
 
@@ -41,10 +41,6 @@ namespace LightweightIocContainer.Factories
         {
             Type factoryType = typeof(TFactory);
             
-            List<MethodInfo> createMethods = factoryType.GetMethods().Where(m => m.ReturnType != typeof(void)).ToList();
-            if (!createMethods.Any())
-                throw new InvalidFactoryRegistrationException($"Factory {factoryType.Name} has no create methods.");
-
             AssemblyBuilder assemblyBuilder = AssemblyBuilder.DefineDynamicAssembly(new AssemblyName("Factory"), AssemblyBuilderAccess.Run);
             ModuleBuilder moduleBuilder = assemblyBuilder.DefineDynamicModule("Factory");
             TypeBuilder typeBuilder = moduleBuilder.DefineType($"TypedFactory.{factoryType.Name}");
@@ -62,7 +58,7 @@ namespace LightweightIocContainer.Factories
             constructorGenerator.Emit(OpCodes.Stfld, containerFieldBuilder); //set `_container` field
             constructorGenerator.Emit(OpCodes.Ret);
 
-            foreach (MethodInfo createMethod in createMethods)
+            foreach (MethodInfo createMethod in CreateMethods)
             {
                 //create a method that looks like this
                 //public `createMethod.ReturnType` Create(`createMethod.GetParameters()`)
diff --git a/LightweightIocContainer/Factories/TypedFactoryBase.cs b/LightweightIocContainer/Factories/TypedFactoryBase.cs
new file mode 100644
index 0000000..73b77db
--- /dev/null
+++ b/LightweightIocContainer/Factories/TypedFactoryBase.cs
@@ -0,0 +1,36 @@
+// Author: Gockner, Simon
+// Created: 2021-12-03
+// Copyright(c) 2021 SimonG. All Rights Reserved.
+
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Reflection;
+using LightweightIocContainer.Exceptions;
+using LightweightIocContainer.Interfaces.Factories;
+
+namespace LightweightIocContainer.Factories
+{
+    /// <summary>
+    /// Base class for the <see cref="ITypedFactory"/>
+    /// </summary>
+    public abstract class TypedFactoryBase<TFactory> : ITypedFactory
+    {
+        /// <summary>
+        /// The create methods of this <see cref="ITypedFactory"/>
+        /// </summary>
+        public List<MethodInfo> CreateMethods
+        {
+            get
+            {
+                Type factoryType = typeof(TFactory);
+
+                List<MethodInfo> createMethods = factoryType.GetMethods().Where(m => m.ReturnType != typeof(void)).ToList();
+                if (!createMethods.Any())
+                    throw new InvalidFactoryRegistrationException($"Factory {factoryType.Name} has no create methods.");
+
+                return createMethods;
+            }
+        }
+    }
+}
\ No newline at end of file
diff --git a/LightweightIocContainer/Interfaces/Factories/ITypedFactory.cs b/LightweightIocContainer/Interfaces/Factories/ITypedFactory.cs
index 25470df..f053255 100644
--- a/LightweightIocContainer/Interfaces/Factories/ITypedFactory.cs
+++ b/LightweightIocContainer/Interfaces/Factories/ITypedFactory.cs
@@ -2,6 +2,9 @@
 // Created: 2019-05-20
 // Copyright(c) 2019 SimonG. All Rights Reserved.
 
+using System.Collections.Generic;
+using System.Reflection;
+
 namespace LightweightIocContainer.Interfaces.Factories
 {
     /// <summary>
@@ -9,7 +12,10 @@ namespace LightweightIocContainer.Interfaces.Factories
     /// </summary>
     public interface ITypedFactory
     {
-        
+        /// <summary>
+        /// The create methods of this <see cref="ITypedFactory"/>
+        /// </summary>
+        List<MethodInfo> CreateMethods { get; }
     }
     
     /// <summary>
diff --git a/LightweightIocContainer/Registrations/RegistrationBase.cs b/LightweightIocContainer/Registrations/RegistrationBase.cs
index 666c6ba..940e3a7 100644
--- a/LightweightIocContainer/Registrations/RegistrationBase.cs
+++ b/LightweightIocContainer/Registrations/RegistrationBase.cs
@@ -125,7 +125,7 @@ namespace LightweightIocContainer.Registrations
         /// <returns>The current instance of this <see cref="IRegistrationBase"/></returns>
         public IRegistrationBase WithFactory<TFactoryInterface, TFactoryImplementation>() where TFactoryImplementation : TFactoryInterface
         {
-            Factory = new CustomTypedFactory();
+            Factory = new CustomTypedFactory<TFactoryInterface>();
             _container.Register<TFactoryInterface, TFactoryImplementation>();
 
             return this;