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.
Interface Type Constraint You can constrain the generic type by interface, thereby allowing only classes that implement that interface or classes that inherit from classes that implement the interface as the type parameter.
The interface constraint enables you to specify an interface that a type argument must implement. The interface constraint serves the same two important purposes as the base class constraint. First, it lets you use the members of the interface within the generic class.
A type constraint on a generic type parameter indicates a requirement that a type must fulfill in order to be accepted as a type argument for that type parameter. (For example, it might have to be a given class type or a subtype of that class type, or it might have to implement a given interface.)
The where clause in a generic definition specifies constraints on the types that are used as arguments for type parameters in a generic type, method, delegate, or local function. Constraints can specify interfaces, base classes, or require a generic type to be a reference, value, or unmanaged type.
Try using composition instead of inheritance to implement Something
:
public class Something : ISomething
{
private readonly BaseClass inner = ...;
void DoSomething<T>() where T : Foo
{
inner.DoSomething<T>();
}
}
Certainly the given code could be compiled and run safely:
When a Something
instance is typed as Something
or as BaseClass
the compiler would allow any type for T
, while when the same instance is typed as ISomething
it would allow just types inheriting Foo
. In both cases you get static checking and runtime safety as usual.
In fact, the above scenario is exactly what happens when you implement ISomething
explicitly. 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.
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With