将元素添加到泛型列表

Question

我有以下方法：

private void updateEntries(List<Data> data, List<Order> orders) {  
   // processing code here       
   List<Order> updates = new ArrayList<>();
   for(Data d: data) {
      Order anOrder = createOrder(d);
      if(anOrder != null) {
         updates.add(anOrder);
      }  
   }  
   orders.clear();  
   orders.addAll(updates);   
} 

方法createOrder 将基于Data 参数创建Order 的子类型。这部分编译。

但问题是方法的调用代码不会编译，因为它是这样调用的：

updateEntries(List<Data> data, List<PendingOrder> orders) 

和

updateEntries(List<Data> data, List<ExecutedOrder> orders)  

这两个都是Order 的子类型，因此代码不会在方法的调用端编译。

如果我更新代码如下：

private void updateEntries(List<Data> data, List<? extends Order> orders) {  
       // processing code here       
       List<Order> updates = new ArrayList<>();
       for(Data d: data) {
          Order anOrder = createOrder(d);
          if(anOrder != null) {
             updates.add(anOrder);
          }  
       }  
       orders.clear();  
       orders.addAll(updates);   // <= does not compile
    } 

问题出在以下行：orders.addAll(updates); 这当然不会编译，因为列表被声明为List<? extends T>，并且它试图避免在列表中添加不同类型的项目。

那么我怎样才能使用泛型，以便我重用函数并能够更新方法中的列表？

Answer 1

问题是updateEntries 给你一个List<Order>。您不能将 List<Order> 的元素添加到 List<? extends Order> 中，因为后者（如您的示例中所示）可能不希望添加任何 Order 的子类。

例如，如果updateEntries 返回一个List，其中部分/所有条目是CompletedOrders，则不应将它们与PendingOrders 混淆。所以编译器不允许你这样做。

您需要能够传入另一个“事物”，以确保updateEntries 返回的事物将是您的orders 列表所需的元素类型的实例。

例如：

private <T extends Order> void updateEntries(List<Data> data, Function<Order, T> orderFn, List<T> orders) {  
   // processing code here
   
   // Pass in orderFn here too:
   List<T> updates = updateEntries(data, orderFn);  

   orders.clear();  
   orders.addAll(updates); // <- does not compile   
} 

或者，如果orderFn 只是您可以在此方法中应用于每个元素的内容，您可以执行以下操作：

private <T extends Order> void updateEntries(List<Data> data, Function<Order, T> orderFn, List<T> orders) {  
   // processing code here

   orders.clear();
   updateEntries(data).stream().map(orderFn).forEach(orders::add);
}

在调用处，需要传入额外的函数，例如

updateEntries(listOfData, PendingOrder.class::cast, listOfPendingOrders);

它传入一个 Function，它只是将 Orders 转换为 PendingOrders。

Answer 2

请看一下<? extends T>和<? super T>之间的区别。

这里有一个很好的答案：Difference between <? super T> and <? extends T> in Java

如果我是对的，你的代码应该是这样的（但我没有测试过）：

private <T extends Order> void updateEntries(List<Data> data, List<T> orders) {  
   // processing code here       
   List<T> updates = new ArrayList<>();
   for(Data d: data) {
      T anOrder = (T)createOrder(d);
      if(anOrder != null) {
         updates.add(anOrder);
      }  
   }  
   orders.clear();  
   orders.addAll(updates);   
} 

Answer 3

问题在于createOrder 函数创建了Order 的子类型而没有明确说明。将该签名更改为

<T extends Order> T createOrder(Data data) 

并在updateEntries 方法中明确说明T 类型：

<T extends Order> void updateEntries(List<Data> data, List<T> orders) 

如果您因任何原因无法更改 createOrder 的签名，请将调用包装到私有方法中并在其中添加强制转换。