Refactor to use OptionsFactory<T> base implementation

Author: Chris Martinez

src/AspNetCore/WebApi/src/Asp.Versioning.Mvc.ApiExplorer/ApiExplorerOptionsFactory{T}.cs

@@ -3,14 +3,13 @@
 namespace Asp.Versioning.ApiExplorer;
 
 using Microsoft.Extensions.Options;
-using Option = Microsoft.Extensions.Options.Options;
 
 /// <summary>
 /// Represents a factory to create API explorer options.
 /// </summary>
 /// <typeparam name="T">The type of options to create.</typeparam>
 [CLSCompliant( false )]
-public class ApiExplorerOptionsFactory<T> : IOptionsFactory<T> where T : ApiExplorerOptions, new()
+public class ApiExplorerOptionsFactory<T> : OptionsFactory<T> where T : ApiExplorerOptions
 {
     private readonly IOptions<ApiVersioningOptions> optionsHolder;
 
@@ -27,60 +26,42 @@ public ApiExplorerOptionsFactory(
         IOptions<ApiVersioningOptions> options,
         IEnumerable<IConfigureOptions<T>> setups,
         IEnumerable<IPostConfigureOptions<T>> postConfigures )
-    {
-        optionsHolder = options;
-        Setups = setups;
-        PostConfigures = postConfigures;
-    }
+        : base( setups, postConfigures ) => optionsHolder = options;
 
     /// <summary>
-    /// Gets the API versioning options associated with the factory.
-    /// </summary>
-    /// <value>The <see cref="ApiVersioningOptions">API versioning options</see> used to create API explorer options.</value>
-    protected ApiVersioningOptions Options => optionsHolder.Value;
-
-    /// <summary>
-    /// Gets the associated configuration actions to run.
+    /// Initializes a new instance of the <see cref="ApiExplorerOptionsFactory{T}"/> class.
     /// </summary>
-    /// <value>The <see cref="IEnumerable{T}">sequence</see> of
-    /// <see cref="IConfigureOptions{TOptions}">configuration actions</see> to run.</value>
-    protected IEnumerable<IConfigureOptions<T>> Setups { get; }
+    /// <param name="options">The <see cref="ApiVersioningOptions">API versioning options</see>
+    /// used to create API explorer options.</param>
+    /// <param name="setups">The <see cref="IEnumerable{T}">sequence</see> of
+    /// <see cref="IConfigureOptions{TOptions}">configuration actions</see> to run.</param>
+    /// <param name="postConfigures">The <see cref="IEnumerable{T}">sequence</see> of
+    /// <see cref="IPostConfigureOptions{TOptions}">initialization actions</see> to run.</param>
+    /// <param name="validations">The <see cref="IEnumerable{T}">sequence</see> of
+    /// <see cref="IValidateOptions{TOptions}">validations</see> to run.</param>
+    public ApiExplorerOptionsFactory(
+        IOptions<ApiVersioningOptions> options,
+        IEnumerable<IConfigureOptions<T>> setups,
+        IEnumerable<IPostConfigureOptions<T>> postConfigures,
+        IEnumerable<IValidateOptions<T>> validations )
+        : base( setups, postConfigures, validations ) => optionsHolder = options;
 
     /// <summary>
-    /// Gets the associated initialization actions to run.
+    /// Gets the API versioning options associated with the factory.
     /// </summary>
-    /// <value>The <see cref="IEnumerable{T}">sequence</see> of
-    /// <see cref="IPostConfigureOptions{TOptions}">initialization actions</see> to run.</value>
-    protected IEnumerable<IPostConfigureOptions<T>> PostConfigures { get; }
+    /// <value>The <see cref="ApiVersioningOptions">API versioning options</see> used to create API explorer options.</value>
+    protected ApiVersioningOptions Options => optionsHolder.Value;
 
     /// <inheritdoc />
-    public virtual T Create( string name )
+    protected override T CreateInstance( string name )
     {
         var apiVersioningOptions = Options;
-        var options = new T()
-        {
-            AssumeDefaultVersionWhenUnspecified = apiVersioningOptions.AssumeDefaultVersionWhenUnspecified,
-            ApiVersionParameterSource = apiVersioningOptions.ApiVersionReader,
-            DefaultApiVersion = apiVersioningOptions.DefaultApiVersion,
-            RouteConstraintName = apiVersioningOptions.RouteConstraintName,
-        };
-
-        foreach ( var setup in Setups )
-        {
-            if ( setup is IConfigureNamedOptions<T> namedSetup )
-            {
-                namedSetup.Configure( name, options );
-            }
-            else if ( name == Option.DefaultName )
-            {
-                setup.Configure( options );
-            }
-        }
+        var options = base.CreateInstance( name );
 
-        foreach ( var post in PostConfigures )
-        {
-            post.PostConfigure( name, options );
-        }
+        options.AssumeDefaultVersionWhenUnspecified = apiVersioningOptions.AssumeDefaultVersionWhenUnspecified;
+        options.ApiVersionParameterSource = apiVersioningOptions.ApiVersionReader;
+        options.DefaultApiVersion = apiVersioningOptions.DefaultApiVersion;
+        options.RouteConstraintName = apiVersioningOptions.RouteConstraintName;
 
         return options;
     }