Is it possible to use a c# object initializer with a factory method?

前端 未结 7 1604
离开以前
离开以前 2020-12-25 13:06

I have a class with a static factory method on it. I want to call the factory to retrieve an instance of the class, and then do additional initialization, preferablly via c#

相关标签:
7条回答
  • 2020-12-25 13:09

    You can use an extension method such as the following:

    namespace Utility.Extensions
    {
        public static class Generic
        {
            /// <summary>
            /// Initialize instance.
            /// </summary>
            public static T Initialize<T>(this T instance, Action<T> initializer)
            {
                initializer(instance);
                return instance;
            }
        }
    }
    

    You would call it as follows:

    using Utility.Extensions;
    // ...
    var result = MyClass.FactoryCreate()
                    .Initialize(x =>
                    {
                        x.someProperty = someValue;
                        x.someProperty2 = someValue2;
                    });
    
    0 讨论(0)
  • 2020-12-25 13:21

    No. Alternatively you could accept a lambda as an argument, which also gives you full control in which part of the "creation" process will be called. This way you can call it like:

    MyClass instance = MyClass.FactoryCreate(c=>
       {
           c.SomeProperty = something;
           c.AnotherProperty = somethingElse;
       });
    

    The create would look similar to:

    public static MyClass FactoryCreate(Action<MyClass> initalizer)
    {
        MyClass myClass = new MyClass();
        //do stuff
        initializer( myClass );
        //do more stuff
        return myClass;
    }
    

    Another option is to return a builder instead (with an implicit cast operator to MyClass). Which you would call like:

    MyClass instance = MyClass.FactoryCreate()
       .WithSomeProperty(something)
       .WithAnotherProperty(somethingElse);
    

    Check this for the builder

    Both of these versions are checked at compile time and have full intellisense support.


    A third option that requires a default constructor:

    //used like:
    var data = MyClass.FactoryCreate(() => new Data
    {
        Desc = "something",
        Id = 1
    });
    //Implemented as:
    public static MyClass FactoryCreate(Expression<Func<MyClass>> initializer)
    {
        var myclass = new MyClass();
        ApplyInitializer(myclass, (MemberInitExpression)initializer.Body);
        return myclass ;
    }
    //using this:
    static void ApplyInitializer(object instance, MemberInitExpression initalizer)
    {
        foreach (var bind in initalizer.Bindings.Cast<MemberAssignment>())
        {
            var prop = (PropertyInfo)bind.Member;
            var value = ((ConstantExpression)bind.Expression).Value;
            prop.SetValue(instance, value, null);
        }
    }
    

    Its a middle between checked at compile time and not checked. It does need some work, as it is forcing constant expression on the assignments. I think that anything else are variations of the approaches already in the answers. Remember that you can also use the normal assignments, consider if you really need any of this.

    0 讨论(0)
  • 2020-12-25 13:22

    Yes. You can use object initializer for already created instance with the following trick. You should create a simple object wrapper:

    public struct ObjectIniter<TObject>
    {
        public ObjectIniter(TObject obj)
        {
            Obj = obj;
        }
    
        public TObject Obj { get; }
    }
    

    And now you can use it like this to initialize your objects:

    new ObjectIniter<MyClass>(existingInstance)
    {
        Obj =
        {
            //Object initializer of MyClass:
            Property1 = value1,
            Property2 = value2,
            //...
        }
    };
    

    P.S. Related discussion in dotnet repository: https://github.com/dotnet/csharplang/issues/803

    0 讨论(0)
  • 2020-12-25 13:24

    +1 on "No".

    Here's an alternative to the anonymous object way:

    var instance = MyClass.FactoryCreate(
        SomeProperty => "Some value",
        OtherProperty => "Other value");
    

    In this case FactoryCreate() would be something like:

    public static MyClass FactoryCreate(params Func<object, object>[] initializers)
    {
        var result = new MyClass();
        foreach (var init in initializers) 
        {
            var name = init.Method.GetParameters()[0].Name;
            var value = init(null);
            typeof(MyClass)
                .GetProperty(name, BindingFlags.Instance | BindingFlags.Public | BindingFlags.IgnoreCase)
                .SetValue(result, value, null);
        }
        return result;
    }
    
    0 讨论(0)
  • 2020-12-25 13:28

    No, that's something you can only do 'inline'. All the factory function can do for you is to return a reference.

    0 讨论(0)
  • 2020-12-25 13:31

    Like everyone said, no.

    A lambda as an argument has already been suggested.
    A more elegant approach would be to accept an anonymous and set the properties according to the object. i.e.

    MyClass instance = MyClass.FactoryCreate(new {
        SomeProperty = someValue,
        OtherProperty = otherValue
    });
    

    That would be much slower though, since the object would have to be reflected on for all the properties.

    0 讨论(0)
提交回复
热议问题