I have a some classes and interface
public interface IGeoObject
{
GeoCoordinate GetGeocoordinate();
}
public class User : IGeoObject
{
public GeoCoordinate GetGeocoordinate()
{
//...
return GeoCoordinate;
}
}
public class Venue : IGeoObject
{
public GeoCoordinate GetGeocoordinate()
{
//...
return GeoCoordinate;
}
}
public class Place : IGeoObject
{
public GeoCoordinate GetGeocoordinate()
{
//...
return GeoCoordinate;
}
}
I have, example variable users with type List<User>
And I have method with signature
double GetMinDistance(List<IGeoObject> objects, GeoCoordinate position)
I can't pass users to GetMinDistance
, because covariance of generic types not work.
I can create additional list and use method .ConvertAll
(or .CasT<T>
):
List<IGeoObject> list = new List<User>(listUser).ConvertAll(x => (IGeoObject)x);
var dist = GeoTest.GetMinDistance(list, position);
I'm not sure that this is the most elegant solution. Most embarrassing that the caller must do this conversion.
This is similar to my problems in the architecture? There are more elegant solutions?
P.S. Long story, but I really can not give the coordinates of the same species in all classes.
2 Answers 2
Covariance works only when it's safe and only on interfaces (and delegates, but that's not relevant here). So, if you wanted to take advantage of it, you would have to declare GetMinDistance()
like this:
double GetMinDistance(IEnumerable<IGeoObject> objects, GeoCoordinate position)
Instead of IEnumerable
, you could use IReadOnlyList
if you're on .Net 4.5.
You can make GetMinDistance
generic:
double GetMinDistance<T>(IList<T> objects, GeoCoordinate position)
where T: IGeoObject
{
// ...
}
You can use it like this:
List<User> user = ....;
GetMinDistance(user, position);