（Java）如何使用有界通配符泛型实现接口方法？

Question

我维护了2个具有相同功能的项目，我正在将这个功能整合到一个公共项目中。 我定义了一个接口：

public interface GraphData 
{
    public List<? extends ShapeData> getShapes();
    public void setShapes( List<? extends ShapeData> shapes );
}

我在两个项目中实现了这个接口：

public class Graph implements GraphData 
{
     public List<Shape> shapes = new ArrayList<Shape>();

     public List<? extends ShapeData> getShapes() 
     { 
         return shapes;
     }

     public void setShapes( List<? extends ShapeData> shapes )
     {
        this.shapes = shapes;
     }
}

Shape是ShapeData的子类型。 当我编译这个类时，我得到一个关于将List<Shape>为List<? of... List<? of...如何解决此编译错误？ 也许更好的问题是，我应该使用有界通配符定义我的接口方法（即？extends）？

Answer 1

基本上，你的界面太宽广IMO。 你已经指定了shapes （顺便说一下，这是一个公共字段 - 为什么？）必须是List<Shape> 。 如果有人将错误的列表传递给setShapes您会发生什么？ 例如：

public class BadShapeData implements ShapeData { ... }

...

List<BadShapeData> badShapes = new ArrayList<BadShapeData>();
new Graph().setShapes(badShapes);

这不是 Shape的列表，是吗？

您可以通过使您的界面通用来修复它：

public interface GraphData<T extends ShapeData>
{
    List<T> getShapes();
    void setShapes(List<T> shapes);
}

然后：

public class Graph implements GraphData<Shape>

或者，您可以更改界面而不是设置器。 你真的需要它吗？ 界面真的增加了很多好处吗？ 你能不能提供比财产更有意义的操作吗？

Answer 2

我认为这本身就是复杂的。 您的Graph有一个List<Shape> ; 获取并设置该类型。

public interface GraphData 
{
    public List<Shape> getShapes();
    public void setShapes(List<Shape> shapes );
}

我通常在接口中没有getter / setter。 这不是非常有意义或有趣的行为。 我不同意Graph甚至需要一个接口。 您可能想要考虑您要实现的目标。