Pasar argumentos a C # genérico new () de tipo con plantilla

Estoy tratando de crear un nuevo objeto de tipo T a través de su constructor cuando lo agrego a la lista.

Recibo un error de compilación: el mensaje de error es:

'T': no ​​puede proporcionar argumentos al crear una instancia de una variable

¡Pero mis clases tienen un argumento de constructor! ¿Cómo puedo hacer que esto funcione?

public static string GetAllItems<T>(...) where T : new()
{
   ...
   List<T> tabListItems = new List<T>();
   foreach (ListItem listItem in listCollection) 
   {
       tabListItems.Add(new T(listItem)); // error here.
   } 
   ...
}

Para crear una instancia de un tipo genérico en una función, debe restringirla con el indicador "nuevo".

public static string GetAllItems<T>(...) where T : new()

Sin embargo, eso solo funcionará cuando desee llamar al constructor que no tiene parámetros. No es el caso aquí. En su lugar, deberá proporcionar otro parámetro que permita la creación de objetos basados ​​en parámetros. Lo más fácil es una función.

public static string GetAllItems<T>(..., Func<ListItem,T> del) {
  ...
  List<T> tabListItems = new List<T>();
  foreach (ListItem listItem in listCollection) 
  {
    tabListItems.Add(del(listItem));
  }
  ...
}

Entonces puedes llamarlo así

GetAllItems<Foo>(..., l => new Foo(l));

en .Net 3.5 y después de que pueda usar la clase activadora:

(T)Activator.CreateInstance(typeof(T), args)

Como nadie se molestó en publicar la respuesta 'Reflexión' (que personalmente creo que es la mejor respuesta), aquí va:

public static string GetAllItems<T>(...) where T : new()
{
   ...
   List<T> tabListItems = new List<T>();
   foreach (ListItem listItem in listCollection) 
   {
       Type classType = typeof(T);
       ConstructorInfo classConstructor = classType.GetConstructor(new Type[] { listItem.GetType() });
       T classInstance = (T)classConstructor.Invoke(new object[] { listItem });

       tabListItems.Add(classInstance);
   } 
   ...
}

Editar: esta respuesta está en desuso debido a .NET 3.5's Activator.CreateInstance, sin embargo, todavía es útil en versiones anteriores de .NET.

Inicializador de objeto

Si su constructor con el parámetro no está haciendo nada además de establecer una propiedad, puede hacerlo en C # 3 o mejor usando un inicializador de objeto en lugar de llamar a un constructor (lo cual es imposible, como se ha mencionado):

public static string GetAllItems<T>(...) where T : new()
{
   ...
   List<T> tabListItems = new List<T>();
   foreach (ListItem listItem in listCollection) 
   {
       tabListItems.Add(new T() { YourPropertyName = listItem } ); // Now using object initializer
   } 
   ...
}

Con esto, también puede colocar cualquier lógica de constructor en el constructor predeterminado (vacío).

Activator.CreateInstance ()

Alternativamente, puede llamar a Activator.CreateInstance () de esta manera:

public static string GetAllItems<T>(...) where T : new()
{
   ...
   List<T> tabListItems = new List<T>();
   foreach (ListItem listItem in listCollection) 
   {
        object[] args = new object[] { listItem };
        tabListItems.Add((T)Activator.CreateInstance(typeof(T), args)); // Now using Activator.CreateInstance
   } 
   ...
}

Tenga en cuenta que Activator.CreateInstance puede tener algunos gastos generales de rendimiento que puede querer evitar si la velocidad de ejecución es una de las principales prioridades y le es posible mantener otra opción.

Muy antigua pregunta, pero nueva respuesta ;-)

La versión ExpressionTree : (creo que la solución más rápida y limpia)

Como dijo Welly Tambunan , "también podríamos usar el árbol de expresión para construir el objeto"

Esto generará un 'constructor' (función) para el tipo / parámetros dados. Devuelve un delegado y acepta los tipos de parámetros como una matriz de objetos.

Aquí está:

// this delegate is just, so you don't have to pass an object array. _(params)_
public delegate object ConstructorDelegate(params object[] args);

public static ConstructorDelegate CreateConstructor(Type type, params Type[] parameters)
{
    // Get the constructor info for these parameters
    var constructorInfo = type.GetConstructor(parameters);

    // define a object[] parameter
    var paramExpr = Expression.Parameter(typeof(Object[]));

    // To feed the constructor with the right parameters, we need to generate an array 
    // of parameters that will be read from the initialize object array argument.
    var constructorParameters = parameters.Select((paramType, index) =>
        // convert the object[index] to the right constructor parameter type.
        Expression.Convert(
            // read a value from the object[index]
            Expression.ArrayAccess(
                paramExpr,
                Expression.Constant(index)),
            paramType)).ToArray();

    // just call the constructor.
    var body = Expression.New(constructorInfo, constructorParameters);

    var constructor = Expression.Lambda<ConstructorDelegate>(body, paramExpr);
    return constructor.Compile();
}

Ejemplo MyClass:

public class MyClass
{
    public int TestInt { get; private set; }
    public string TestString { get; private set; }

    public MyClass(int testInt, string testString)
    {
        TestInt = testInt;
        TestString = testString;
    }
}

Uso:

// you should cache this 'constructor'
var myConstructor = CreateConstructor(typeof(MyClass), typeof(int), typeof(string));

// Call the `myConstructor` function to create a new instance.
var myObject = myConstructor(10, "test message");

Otro ejemplo: pasar los tipos como una matriz

var type = typeof(MyClass);
var args = new Type[] { typeof(int), typeof(string) };

// you should cache this 'constructor'
var myConstructor = CreateConstructor(type, args);

// Call the `myConstructor` fucntion to create a new instance.
var myObject = myConstructor(10, "test message");

Vista de depuración de expresión

.Lambda #Lambda1<TestExpressionConstructor.MainWindow+ConstructorDelegate>(System.Object[] $var1) {
    .New TestExpressionConstructor.MainWindow+MyClass(
        (System.Int32)$var1[0],
        (System.String)$var1[1])
}

Esto es equivalente al código que se genera:

public object myConstructor(object[] var1)
{
    return new MyClass(
        (System.Int32)var1[0],
        (System.String)var1[1]);
}

Pequeño inconveniente

Todos los parámetros de valuetypes se encuadran cuando se pasan como una matriz de objetos.

Prueba de rendimiento simple:

private void TestActivator()
{
    Stopwatch sw = Stopwatch.StartNew();
    for (int i = 0; i < 1024 * 1024 * 10; i++)
    {
        var myObject = Activator.CreateInstance(typeof(MyClass), 10, "test message");
    }
    sw.Stop();
    Trace.WriteLine("Activator: " + sw.Elapsed);
}

private void TestReflection()
{
    var constructorInfo = typeof(MyClass).GetConstructor(new[] { typeof(int), typeof(string) });

    Stopwatch sw = Stopwatch.StartNew();
    for (int i = 0; i < 1024 * 1024 * 10; i++)
    {
        var myObject = constructorInfo.Invoke(new object[] { 10, "test message" });
    }

    sw.Stop();
    Trace.WriteLine("Reflection: " + sw.Elapsed);
}

private void TestExpression()
{
    var myConstructor = CreateConstructor(typeof(MyClass), typeof(int), typeof(string));

    Stopwatch sw = Stopwatch.StartNew();

    for (int i = 0; i < 1024 * 1024 * 10; i++)
    {
        var myObject = myConstructor(10, "test message");
    }

    sw.Stop();
    Trace.WriteLine("Expression: " + sw.Elapsed);
}

TestActivator();
TestReflection();
TestExpression();

Resultados:

Activator: 00:00:13.8210732
Reflection: 00:00:05.2986945
Expression: 00:00:00.6681696

Usar Expressionses +/- 8 veces más rápido que invocar ConstructorInfoy +/- 20 veces más rápido que usarActivator

Esto no funcionará en tu situación. Solo puede especificar la restricción de que tiene un constructor vacío:

public static string GetAllItems<T>(...) where T: new()

Lo que podría hacer es usar la inyección de propiedades definiendo esta interfaz:

public interface ITakesAListItem
{
   ListItem Item { set; }
}

Entonces podrías alterar tu método para que sea así:

public static string GetAllItems<T>(...) where T : ITakesAListItem, new()
{
   ...
   List<T> tabListItems = new List<T>();
   foreach (ListItem listItem in listCollection) 
   {
       tabListItems.Add(new T() { Item = listItem });
   } 
   ...
}

La otra alternativa es el Funcmétodo descrito por JaredPar.

Debe agregar where T: new () para que el compilador sepa que T está garantizado para proporcionar un constructor predeterminado.

public static string GetAllItems<T>(...) where T: new()

Si simplemente desea inicializar un campo miembro o propiedad con el parámetro constructor, en C #> = 3 puede hacerlo más fácilmente:

public static string GetAllItems<T>(...) where T : InterfaceOrBaseClass, new() 
{ 
   ... 
   List<T> tabListItems = new List<T>(); 
   foreach (ListItem listItem in listCollection)  
   { 
       tabListItems.Add(new T{ BaseMemberItem = listItem }); // No error, BaseMemberItem owns to InterfaceOrBaseClass. 
   }  
   ... 
} 

Esto es lo mismo que dijo Garry Shutler, pero me gustaría poner una nota adicional.

Por supuesto, puede usar un truco de propiedad para hacer más cosas que solo establecer un valor de campo. Una propiedad "set ()" puede desencadenar cualquier procesamiento necesario para configurar sus campos relacionados y cualquier otra necesidad del objeto en sí, incluida una verificación para ver si se debe realizar una inicialización completa antes de usar el objeto, simulando una construcción completa ( sí, es una solución fea, pero supera la nueva limitación () de M $).

No puedo asegurar si es un agujero planificado o un efecto secundario accidental, pero funciona.

Es muy divertido cómo la gente con EM agrega nuevas características al lenguaje y parece no hacer un análisis completo de los efectos secundarios. Todo lo genérico es una buena evidencia de esto ...

Descubrí que recibía un error "no puedo proporcionar argumentos al crear una instancia del parámetro de tipo T", así que necesitaba hacer esto:

var x = Activator.CreateInstance(typeof(T), args) as T;

Si tiene acceso a la clase que va a usar, puede usar este enfoque que yo usé.

Cree una interfaz que tenga un creador alternativo:

public interface ICreatable1Param
{
    void PopulateInstance(object Param);
}

Haga sus clases con un creador vacío e implemente este método:

public class MyClass : ICreatable1Param
{
    public MyClass() { //do something or nothing }
    public void PopulateInstance (object Param)
    {
        //populate the class here
    }
}

Ahora usa tus métodos genéricos:

public void MyMethod<T>(...) where T : ICreatable1Param, new()
{
    //do stuff
    T newT = new T();
    T.PopulateInstance(Param);
}

Si no tiene acceso, ajuste la clase de destino:

public class MyClass : ICreatable1Param
{
    public WrappedClass WrappedInstance {get; private set; }
    public MyClass() { //do something or nothing }
    public void PopulateInstance (object Param)
    {
        WrappedInstance = new WrappedClass(Param);
    }
}

Esto es un poco sucio, y cuando digo un poco sucio, puedo decir repugnante, pero suponiendo que pueda proporcionar su tipo parametrizado con un constructor vacío, entonces:

public static T GetTInstance<T>() where T: new()
{
    var constructorTypeSignature = new Type[] {typeof (object)};
    var constructorParameters = new object[] {"Create a T"};
    return (T) new T().GetType().GetConstructor(constructorTypeSignature).Invoke(constructorParameters);
}

Efectivamente le permitirá construir un objeto a partir de un tipo parametrizado con un argumento. En este caso, supongo que el constructor que quiero tiene un único argumento de tipo object. Creamos una instancia ficticia de T utilizando la restricción permitida constructor vacío y luego utilizamos la reflexión para obtener uno de sus otros constructores.

A veces uso un enfoque que se asemeja a las respuestas usando la inyección de propiedades, pero mantiene el código más limpio. En lugar de tener una clase / interfaz base con un conjunto de propiedades, solo contiene un método (virtual) Initialize () que actúa como un "constructor de pobres". Luego puede dejar que cada clase maneje su propia inicialización tal como lo haría un constructor, lo que también agrega una forma conveniente de manejar cadenas de herencia.

Si a menudo me encuentro en situaciones en las que quiero que cada clase de la cadena inicialice sus propiedades únicas, y luego llame al método Initialize () de su padre, que a su vez inicializa las propiedades únicas del padre y así sucesivamente. Esto es especialmente útil cuando se tienen diferentes clases, pero con una jerarquía similar, por ejemplo, objetos de negocios que se asignan a / desde DTO: s.

Ejemplo que usa un diccionario común para la inicialización:

void Main()
{
    var values = new Dictionary<string, int> { { "BaseValue", 1 }, { "DerivedValue", 2 } };

    Console.WriteLine(CreateObject<Base>(values).ToString());

    Console.WriteLine(CreateObject<Derived>(values).ToString());
}

public T CreateObject<T>(IDictionary<string, int> values)
    where T : Base, new()
{
    var obj = new T();
    obj.Initialize(values);
    return obj;
}

public class Base
{
    public int BaseValue { get; set; }

    public virtual void Initialize(IDictionary<string, int> values)
    {
        BaseValue = values["BaseValue"];
    }

    public override string ToString()
    {
        return "BaseValue = " + BaseValue;
    }
}

public class Derived : Base
{
    public int DerivedValue { get; set; }

    public override void Initialize(IDictionary<string, int> values)
    {
        base.Initialize(values);
        DerivedValue = values["DerivedValue"];
    }

    public override string ToString()
    {       
        return base.ToString() + ", DerivedValue = " + DerivedValue;
    }
}

Si todo lo que necesita es la conversión de ListItem a su tipo T, puede implementar esta conversión en clase T como operador de conversión.

public class T
{
    public static implicit operator T(ListItem listItem) => /* ... */;
}

public static string GetAllItems(...)
{
    ...
    List<T> tabListItems = new List<T>();
    foreach (ListItem listItem in listCollection) 
    {
        tabListItems.Add(listItem);
    } 
    ...
}

Creo que debe restringir T con una instrucción where para permitir solo objetos con un nuevo constructor.

En este momento acepta cualquier cosa, incluidos los objetos sin él.