I have a base class that defines a generic method like this:
public class BaseClass
{
public T DoSomething<T> ()
{ ... }
}
As this class is by a third-party and does not come with an interface, I am defining an interface that defines the actually needed methods from that class. That way I get loose coupling and can actually exchange that third-party class with something else. For this example, consider the following interface:
public interface ISomething
{
T DoSomething<T> ()
where T : Foo;
}
As you can see, it defines the same method but also applies a type constraint on the type parameter, which comes from some other requirements that are not relevant to this.
Next, I define a subtype of BaseClass which also implements ISomething. This class will be used as the usual implementation behind the interface–while the interface will be what the rest of the application will be accessing.
public class Something : BaseClass, ISomething
{
// ...
}
As the DoSomething in BaseClass already supports any type parameter T, it should especially support a type parameter which is a subtype of Foo. So one would expect that a subtype of BaseClass already implements the interface. However I get the following error:
The constraints for type parameter ‘T’ of method ‘BaseClass.DoSomething()’ must match the constraints for type parameter ‘T’ of interface method ‘ISomething.DoSomething()’. Consider using an explicit interface implementation instead.
Now, I have two possibilities; the first one is to do what the error suggests and implement the interface explicitely. The second is to hide the base implementation using new:
// Explicit implementation
T ISomething.DoSomething<T> ()
{
return base.DoSomething<T>();
}
// Method hiding
public new T DoSomething<T>()
where T : Foo
{
return base.DoSomething<T>();
}
Both work, although I’d probably prefer the second solution to keep the method accessible from the class itself. However it still leaves the following question:
Why do I have to re-implement the method when the base type already implements it with a less-strict (read: none) type constraint? Why does the method need to be implemented exactly as it is?
edit: To give the method a bit more meaning, I changed the return type from void to T. In my actual application, I have both generic arguments and return values.
Certainly the given code could be compiled and run safely:
When a
Somethinginstance is typed asSomethingor asBaseClassthe compiler would allow any type forT, while when the same instance is typed asISomethingit would allow just types inheritingFoo. In both cases you get static checking and runtime safety as usual.In fact, the above scenario is exactly what happens when you implement
ISomethingexplicitly. So let’s see what arguments we can make for and against the current state of affairs.For:
Against:
Considering the above and additionally the fact that this is not an everyday scenario, IMHO the conclusion to be reached is clear: this might be nice to have, but it certainly doesn’t warrant going out of your way to implement it.