154

In C# I have the following object:

public class Item
{ }

public class Task<T>
{ }

public class TaskA<T> : Task<T>
{ }

public class TaskB<T> : Task<T>
{ }

I want to dynamically create TaskA or TaskB using C# reflection (Activator.CreateInstance). However I wouldn't know the type before hand, so I need to dynamically create TaskA based on string like "namespace.TaskA" or "namespace.TaskAB".

Jeff
  • 13,079
  • 23
  • 71
  • 102

5 Answers5

282

Check out this article and this simple example. Quick translation of same to your classes ...

var d1 = typeof(Task<>);
Type[] typeArgs = { typeof(Item) };
var makeme = d1.MakeGenericType(typeArgs);
object o = Activator.CreateInstance(makeme);

Per your edit: For that case, you can do this ...

var d1 = Type.GetType("GenericTest.TaskA`1"); // GenericTest was my namespace, add yours
Type[] typeArgs = { typeof(Item) };
var makeme = d1.MakeGenericType(typeArgs);
object o = Activator.CreateInstance(makeme);

To see where I came up with backtick1 for the name of the generic class, see this article.

Note: if your generic class accepts multiple types, you must include the commas when you omit the type names, for example:

Type type = typeof(IReadOnlyDictionary<,>);
ErikE
  • 48,881
  • 23
  • 151
  • 196
JP Alioto
  • 44,864
  • 6
  • 88
  • 112
8

Indeed you would not be able to write the last line.

But you probably don't want to create the object, just for the sake or creating it. You probably want to call some method on your newly created instance.

You'll then need something like an interface :

public interface ITask 
{
    void Process(object o);
}

public class Task<T> : ITask
{ 
   void ITask.Process(object o) 
   {
      if(o is T) // Just to be sure, and maybe throw an exception
        Process(o as T);
   }

   public void Process(T o) { }
}

and call it with :

Type d1 = Type.GetType("TaskA"); //or "TaskB"
Type[] typeArgs = { typeof(Item) };
Type makeme = d1.MakeGenericType(typeArgs);
ITask task = Activator.CreateInstance(makeme) as ITask;

// This can be Item, or any type derived from Item
task.Process(new Item());

In any case, you won't be statically cast to a type you don't know beforehand ("makeme" in this case). ITask allows you to get to your target type.

If this is not what you want, you'll probably need to be a bit more specific in what you are trying to achieve with this.

Jérôme Laban
  • 5,224
  • 3
  • 20
  • 17
  • I do have something like Process() in Task. And in my case I actually don't care about the last line anymore as you stated I am simply calling task.Process() therefore whether is able to code the last line becomes irreverent. – Jeff Jul 20 '09 at 03:06
3

It seems to me the last line of your example code should simply be:

Task<Item> itsMe = o as Task<Item>;

Or am I missing something?

Ben M
  • 22,262
  • 3
  • 67
  • 71
  • 7
    You are not missing anything. It's me who wasn’t thinking straight. I shouldn’t have drunk that much alcohol last nite! – Jeff Jul 20 '09 at 02:41
1

Make sure you're doing this for a good reason, a simple function like the following would allow static typing and allows your IDE to do things like "Find References" and Refactor -> Rename.

public Task <T> factory (String name)
{
  Task <T> result;

  if (name.CompareTo ("A") == 0)
  {
    result = new TaskA ();
  }
  else if (name.CompareTo ("B") == 0)
  {
    result = new TaskB ();
  }

  return result;
}
clemahieu
  • 1,419
  • 9
  • 9
  • 1
    Why are you using .CompareTo? Why not == or .Equals (if you want more control). Maybe a switch would even be better. – Yvo Jul 21 '09 at 22:59
  • I never checked if C# does a string comparison on == instead of a reference comparison. CompareTo and Equals should have the same run time efficiency if they were implemented correctly. A switch block wouldn't have the same speed-ups as putting an integer in a switch block; it would compile to an if-else block. – clemahieu Jul 21 '09 at 23:29
1

I know this question is resolved but, for the benefit of anyone else reading it; if you have all of the types involved as strings, you could do this as a one liner:

IYourInterface o = (Activator.CreateInstance(Type.GetType("Namespace.TaskA`1[OtherNamespace.TypeParam]") as IYourInterface);

Whenever I've done this kind of thing, I've had an interface which I wanted subsequent code to utilise, so I've casted the created instance to an interface.

Johnathon Sullinger
  • 7,097
  • 5
  • 37
  • 102
A Aiston
  • 717
  • 5
  • 12