避免代码重复的Java通用参数方法

Question

我有一个解析对象的函数。 但是在两个服务中都需要此函数，并且参数具有相同的类名，但具有不同的包名。 我需要的是避免重复的代码。

假设函数为：

    private HashMap<String, Integer> getPagination(PagingRequestType pagingRequestType) {
        int pageSize = 200;
        int pageNumber = 1;
        if(pagingRequestType != null) {
            if (pagingRequestType.getNumberOfRecordsPerPage() != 0) {
                pageSize = pagingRequestType.getNumberOfRecordsPerPage();
            }
            if (pagingRequestType.getStartAtRecordNumber() != 0) {
                pageNumber = pagingRequestType.getStartAtRecordNumber();
            }
        }
        HashMap<String, Integer> result = new HashMap<>();
        result.put("pageNumber", pageNumber);
        result.put("pageSize", pageSize);
        return result;
    }

可能的函数调用：

- getPagination(new Abc.PagingRequestType());
- getPagination(new Xyz.PagingRequestType());

PagingRequestType是两个不同程序包中的自动生成的类。 该功能只需实施一次，即可在两种服务中使用。

谢谢。

Answer 1

如果可以修改PagingRequestType类，则最好使用通用接口：

class Abc.PagingRequestType implements PagingRequestType
class Xyz.PagingRequestType implements PagingRequestType

interface PagingRequestType {
    getNumberOfRecordsPerPage();
    getStartAtRecordNumber();
}

Answer 2

显而易见的答案是不要在两个地方自动生成PagingRequestType 。

如果无法执行此操作，则需要两个类来实现一个公共接口，通过该接口可以使用必填字段（ getNumberOfRecordsPerPage和getStartAtRecordNumber ）。

如果无法更改类，则可以使用以下字段创建接口：

interface YourInterface {
  int getNumberOfRecordsPerPage();
  int getStartAtRecordNumber();
}

并为两个PagingRequestType实现：

class AbcYourInterface implements YourInterface {
  final Abc.PagingRequestType delegate;  // Set in constructor.

  @Override public int getNumberOfRecordsPerPage() {
   return delegate.getNumberOfRecordsPerPage();
  }

  // Same for other method.
}

如果其他所有方法均失败，则将类字段作为单独的参数传递：

private HashMap<String, Integer> getPagination(int numberOfRecordsPerPage, int startAtRecordNumber) {

使用一些“特殊”值来指示null ，例如0，因为如果两个参数均为零，则条件为空操作。

Answer 3

这两个PagingRequestType都可以实现一个公共接口或扩展一个公共类，您可以将此“公共部分”作为函数的参数。 尽管我不知道您是否可以通过这种方式修改自动生成的代码。

Answer 4

如果使这两个PagingRequestType实现一个公共接口的选项不可行，则可以相反的方法，定义一个proxy类型来包装这两种类型：

public final class PagingRequestTypeProxy {
    private a.b.c.PagingRequestType abcPagingRequestType;

    private a.b.d.PagingRequestType abdPagingRequestType;

    public PagingRequestTypeProxy(PagingRequestType abcPagingRequestType) {
        this.abcPagingRequestType = abcPagingRequestType;
    }

    public PagingRequestTypeProxy(a.b.d.PagingRequestType abdPagingRequestType) {
        this.abdPagingRequestType = abdPagingRequestType;
    }

    public int getNumberOfRecordsPerPage() {
        return abcPagingRequestType != null ? abcPagingRequestType.getNumberOfRecordsPerPage() : abdPagingRequestType.getNumberOfRecordsPerPage();
    }

    public int getStartAtRecordNumber() {
        return abcPagingRequestType != null ? abcPagingRequestType.getStartAtRecordNumber() : abdPagingRequestType.getStartAtRecordNumber();
    }
}

并将getPagination的参数类型getPagination为：

private HashMap<String, Integer> getPagination(PagingRequestTypeProxy pagingRequestType) {...}

然后您可以像这样使用它：

getPagination(new PagingRequestTypeProxy(abcPagingReqType));
getPagination(new PagingRequestTypeProxy(abdPagingReqType));

缺点是定义了PagingRequestTypeProxy需要额外的代码，优点是PagingRequestTypeProxy logic非常简单且易于维护，您可以将业务代码getPagination放在一个地方。