我想将从一个类派生的不同类型的对象添加到基类型列表中。我得到了这个编译错误
Error 2 Argument 1: cannot convert from 'ConsoleApplication1.Stable' to 'ConsoleApplication1.ShelterBase<ConsoleApplication1.AnimalBase>' C:\Users\ysn\Desktop\ConsoleApplication1\ConsoleApplication1\Program.cs 43 26 ConsoleApplication1
我看不出问题你能为我提供另一种做这种事的方法吗?
abstract class AnimalBase { public int SomeCommonProperty;}
abstract class ShelterBase<T> where T : AnimalBase
{
public abstract List<T> GetAnimals();
public abstract void FeedAnimals(List<T> animals);
}
class Horse : AnimalBase { }
class Stable : ShelterBase<Horse>
{
public override List<Horse> GetAnimals()
{
return new List<Horse>();
}
public override void FeedAnimals(List<Horse> animals)
{
// feed them
}
}
class Duck : AnimalBase { }
class HenHouse : ShelterBase<Duck>
{
public override List<Duck> GetAnimals()
{
return new List<Duck>();
}
public override void FeedAnimals(List<Duck> animals)
{
// feed them
}
}
class Program
{
static void Main(string[] args)
{
List<ShelterBase<AnimalBase>> shelters = new List<ShelterBase<AnimalBase>>();
///////////////////////////// following two lines do not compile
shelters.Add(new Stable());
shelters.Add(new HenHouse());
/////////////////////////////
foreach (var shelter in shelters)
{
var animals = shelter.GetAnimals();
// do sth with 'animals' collection
}
}
}
答案 0 :(得分:19)
如果您将抽象类更改为接口并将GetAnimals
的返回类型更改为IEnumerable<T>
,则可以使用contravariance,但仅,因为{ {1}}不支持此功能。
有效的代码:
List<T>
答案 1 :(得分:2)
您可以使用协方差和逆变来完成此操作,但您的ShelterBase类需要从接口派生,因为只有接口可以是共变或逆变。您的列表需要是List<IShelterBase<T>>
,它应该有效。
答案 2 :(得分:1)
关于泛型中的co和contra-variance的文章:
答案 3 :(得分:1)
要解决这个特殊问题,您实际上并不需要协方差。当您使用动物列表时,仍然可以通过AnimalBase
界面获得IShelterBase<out T>
。不妨通过基类公开AnimalBase
列表。
更好更清洁的设计是让GetAnimals
返回AnimalBase
的列表,并在每个避难所类中创建一个重载以返回该特定动物的列表。
abstract class ShelterBase<T> where T : AnimalBase
{
public List<AnimalBase> GetAnimals(){return new List<AnimalBase>();}
}
class Stable : ShelterBase<Horse>
{
public List<Horse> GetHorses(){return new List<Horse>();}
}
此设计的另一个问题是通过最派生类型公开集合 - 即List
,而不是IEnumerable
或IList
。看到你在创建一个抽象动物集合的类时遇到麻烦,你应该通过禁止直接插入/删除功能来真正保护内部集合。一旦你这样做,事情变得容易一些。例如。这是我如何解决这个问题。
abstract class ShelterBase<T> where T : AnimalBase
{
protected List<T> _Animals;
public AnimalBase() {
_Animals = CreateAnimalCollection();
}
protected abstract List<T> CreateAnimalCollection();
public IEnumerable<AnimalBase> GetAnimals(){return _Animals.Cast<AnimalBase>();}
//Add remove operations go here
public void Add(T animal){_Animals.Add(animal);}
public void Remove(T animal){_Animals.Remove(animal);}
}
class Stable : ShelterBase<Horse>
{
protected override List<Horse> CreateAnimalCollection(){return new List<Horse>();}
public IEnumerable<Horse> GetHorses(){return _Animals;}
}
您会注意到动物的内部集合从未暴露为可变列表。这很好,因为它可以让你更严格地控制其内容。基本掩体中的Add
和Remove
方法在这个示例中有点人为,因为它们不会在直接集合访问中添加任何额外的东西,但是您可以在那里添加逻辑 - 例如在移除或检查动物年龄之前检查最大的避难所大小。
答案 4 :(得分:0)
我遇到了类似的问题。 我的所有实体都派生自一个基类,我创建了一个方法来返回一个连接的id列表。因此使用泛型创建了方法并获得了一些转换错误。管理将通用T 转换为对象,将对象转换为 BaseClass ,并进行某种验证以防万一。
//Needs it generic to use in a lot of derived classes
private String ConcatIds<T>(List<T> listObj)
{
String ids = String.Empty;
foreach (T obj in listObj)
{
BaseEntity be = CastBack(obj);
if (ids.Count() > 0)
ids = ids + String.Format(", {0}", be.Id);
else
ids = be.Id.ToString();
}
return ids;
}
//I'll probably move it to the Base Class itself
private BaseEntity CastBack<T>(T genericObj)
{
Type type = typeof(T);
if (type.BaseType == typeof(BaseEntity))
{
Object _obj = (Object)genericObj;
return (BaseEntity)_obj;
}
else
{
throw new InvalidOperationException(String.Format("Cannot convert {0} to BaseEntity", type.ToString()));
}
}
用法:
public class BaseEntity
{
public Int32 Id {get; set;}
}
public class AnyDerivedClass : BaseEntity
{
// Lorem Ipsum
}
private void DoAnything(List<AnyDerivedClass> myList)
{
String ids = this.ConcatIds<AnyDerivedClass>(myList);
}
编辑:过了一段时间我需要创建更多的层次结构级别,有时会转回父级或祖父级。所以我让我的CastBack方法更通用。
private B CastBack<T,B>(T genericObj)
{
Type type = typeof(T);
if (type.BaseType == typeof(B))
{
Object _obj = (Object)genericObj;
return (B)_obj;
}
else
{
throw new InvalidOperationException(String.Format("Cannot cast back {0}", type.ToString()));
}
}