如何避免类似类和类似方法的重复？

Question

上下文

我正在使用非常相似的课程，例如 RechargeResponse 或 ConsultResponse。所有这些（大约 80 个）都是从 WSDL 方案生成的，具有相同的结构。 （这个方案来自3PP公司，这个逻辑我改不了。）

每个都包含内部类：RechargeResult 和 ConsultResult。

我有一堆具有相同功能的方法。唯一的区别是我需要致电（例如）response.get_ClassName_Result().getAny() 来检查数据。

问题

如何避免在每个方法中使用相同的代码而只更改了 ClassNameMethod？

---

是否可以使用泛型、反射或其他解决方案？ （我认为像字符串一样解析类名不是解决方案）。

以下代码示例：

类似的类：

    public class ConsultResponse {
        protected ConsultResult consultResult;
    
        public ConsultResult getConsultResult() {
            return consultResult;
        }
    
        public static class ConsultResult {
            protected Object any;
            public Object getAny() {
                return any;
            }
            public void setAny(Object value) {
                this.any = value;
            }
    
        }
    }

   public class RechargeResponse {
        protected RechargeResult rechargeResult;
    
        public RechargeResult getRechargeResult() {
            return rechargeResult;
        }
    
        public static class RechargeResult {
            protected Object any;
            public Object getAny() {
                return any;
            }
            public void setAny(Object value) {
                this.any = value;
            }
    
        }
    }

每个类的相似（重复）方法：

    private void validateConsult(ConsultResponse response) {
        if (response == null ||
        response.getConsultResult() == null ||              // need solution here
        response.getConsultResult().getAny() == null) {     // need solution or change here
        throw new Exception();
        }
    }

 

Answer 1

其中一个问题是您的get«classname»Result 方法名称包含类名称。这使得不使用反射就不可能使其通用。为什么不将两者都重命名为getResult？然后，您可以使用泛型使类成为泛型。

1. 首先，我们定义一个接口，它同时定义了getAny 和setAny。

   public interface Result {
       Object getAny();
       void setAny(Object value);
   }
   

2. 然后我们可以创建Result 的实现，例如ConsultResult。你可以对RechargeResult做同样的事情。

   public class ConsultResult implements Result {
       protected Object any; // You already have a getter, so this can also be private
   
       public Object getAny() {
           return this.any;
       }
   
       public void setAny(Object value) {
           this.any = value;
       }
   }
   

3. 然后我们可以创建一个基类Response，它定义了getResult 方法。该类接受一个类型参数T，它必须实现Result。

   public abstract class Response<T extends Result> {
   
       protected T result; // You already have a getter, so this can also be private
   
       public T getResult() {
           return this.result;
       }
   }
   

4. 最后，我们还创建了ConsultResponse 类。我们从Response 扩展它，并提供ConsultResult 作为类型参数。

   public class ConsultResponse extends Response<ConsultResult> {
       // The field 'result' is already present within the Response class,
       // so there is no need to include it here.
   }
   

另外，正如 GhostCat 在 cmets 中所说：首先拥有两个不同的内部类有什么意义？在您的示例中，它们与当前编写的示例相同。您可以将它们替换为单个基类；但是，可能是这些类中有更多成员未在您的示例中显示，因此我将它们保留在示例中。

---

对于验证，您可以做大致相同的操作。

Answer 2

有几种方法可以绕过它，例如创建一个超类，ConsultResponse 和 RechargeResponse 将从该超类扩展。超类将定义共享方法，因此您不必在扩展类中定义它，除非您想覆盖它。

另一种方法是将验证完全分离到一个单独的类中，例如 ResponseValidator，它将自行处理验证，并将包含在 ConsultResponse 和 RechargeResponse 类中并使用。

很难确定一个确切的解决方案，因为这取决于您的具体情况，我们并不完全了解。