The codes below are exactly the same, except that one is C# and the other one is VB.Net. C# compiles just fine, but VB.Net throws the warning:
Interfa
It's bad design to implement both. Have two different child objects which you subscribe to the two observers. I recommend having two child objects, with each implementing one of the interfaces.
class Beholder
{
public IObserver<Foo> FooObserver{get;private set;}
public IObserver<Bar> BarObserver{get;private set;}
}
Still I don't see an immediate problem here, so the VB.net warning looks indeed strange to me.
IObserver<in T>
is contra-variant. So to cause an ambiguity you'd need to find a T
such both IObserver<Foo>
and IObserver<Bar>
are IObserver<T>
.
If both Foo
and Bar
are independent classes, no such T
exists, since it's need to derive from both of them, which the .net type system doesn't allow.
If either of them were an interface, there would be an ambiguity: Just create a class that derives from Foo
and implements IBar
.
If one derived from the other, it'd be ambiguous too: if Foo
derived from Bar
, then IObserver<Bar>
is also IObserver<Foo>
.
And finally with co-variant interfaces, such as IEnumerable<T>
it's enough to have a common base class to which both are reference convertible. And Object
fulfills this for any two classes(but not value types).
But IEnumerable<T>
would break even without covariance, since you need a consistent implementation of the non generic IEnumerable
, and that's not possible for two independent classes.
Summing up:
IEnumerable<T>
then the warning would be justified. If you have an object that is both a sequence of Turtles and a sequence of Giraffes, then what happens when you implicitly convert it to sequence of Animal? Do you get Turtles or Giraffes? The runtime just picks one, which is not necessarily the behaviour you want.For some interesting discussion of the last point see the comments to my 2007 article on the subject:
http://blogs.msdn.com/b/ericlippert/archive/2007/11/09/covariance-and-contravariance-in-c-part-ten-dealing-with-ambiguity.aspx