8

我有一个对象，它有几个数组作为字段。它的类大致是这样的：

public class Helper {
    InsuranceInvoices[] insuranceInvoices;
    InsuranceCollectiveInvoices[] insuranceCollectiveInvoices
    BankInvoices[] bankInvoices;
    BankCollectiveInvoices[] bankCollectiveInvoices;
}

所有发票类型都有一个相互标记接口Invoices。
我需要获取所有发票才能对其调用另一种方法。

Helper helperObject = new Helper();
// ...

for (InsuranceInvoices invoice : helperObject.getInsuranceInvoices()) {
    Integer customerId = invoice.getCustomerId();
    // ...
}
for (BankInvoices invoice : helperObject.getBankInvoices()) {
    Integer customerId = invoice.getCustomerId();
    // ... 
}

// repeat with all array fields

问题是所有发票只有共同的标记界面。getCustomerID()方法不是由相互接口或类定义的。由于给定的规范，这是我无法改变的行为。

for-each-loop 中的代码重复让我很头疼。我必须对四个不同数组中的所有发票对象执行完全相同的操作。因此，有四个 for-each-loop 不必要地使代码膨胀。

有没有办法可以编写通用（私有）方法？一个想法是：

private void generalMethod(Invoice[] invoiceArray){
    // ...
}

但这需要四次 instanceof 检查，因为Invoice类不知道getCusomterId()方法。因此，我将一无所获；该方法仍将包含重复。

我感谢所有可能的解决方案来概括这个问题！

4

3 回答 3

7

概括问题的可能解决方案（从最好到最差排序）：

使用包装类

public class InvoiceWrapper {
    private String customerID;
    public String getCustomerID() {
        return customerID;
    }
    public InvoiceWrapper(BankInvoices invoice) {
       this.customerID = invoice.getCustomerID();
    }
    public InvoiceWrapper(InsuranceInvoices invoice) {
       this.customerID = invoice.getCustomerID();
    }
    // other constructors
}

Upd如果我理解正确，您需要对所有数组中的 ID 做一些事情。要使用 InvoiceWrapper，您还需要在 Helper 类中实现迭代器，它将遍历数组并为每个条目返回一个包装器。因此，无论如何，您将拥有适用于 4 个数组的代码。

使用强制转换实例

public class CustomerIdHelper {
    public static String getID(Invoice invoice) {
        if (invoice instanceof InsuranceInvoices) {
            return ((InsuranceInvoices) invoices).getCustomerID();
        } else if ...
    }
}

通过反射按名称调用方法

public class CustomerIdHelper {
    public static String getID(Invoice invoice) {
        Method method = invoice.getClass().getDeclaredMethod("getCustomerId");
        return (String) method.invoke(invoice);
    }
}
于 2015-06-18T14:05:03.510 回答
2

它并不漂亮，但您可以使用反射来查找getCustomerId 方法，然后调用（）它，参见。类.getDeclaredMethod()。

private void generalMethod(Invoice[] invoiceArray){
  try {
    for (Invoice invoice : invoiceArray) {
      Method getCustomerId = invoice.getClass().getDeclaredMethod("getCustomerId");
      getCustomerId.invoke(invoice);
    }
  } catch (Exception e) {
    // ...
  }
}

请注意，这是未经测试的。

于 2015-06-18T13:23:09.703 回答
2

如果您不允许通过向它们添加自定义接口来更改您正在处理的类。您可以做的最好的事情是使用具有所需属性的自定义类包装它们。

这样，您将拥有一个包含所有“不太好”代码的类，该类将您无法触及的类转换为与适当且有用的设计相匹配的好类。

例如，您可以有一个WrappedInsuranceInvoice扩展WrappedInsurace并包含成员字段的类InsuranceInvoice。如果您不需要保留原始课程，则通过复制数据会更好。这样，您可以例如丢失数组并改用列表。

于 2015-06-18T13:29:23.443 回答