2ビットの異なるオブジェクト用のJava関数

Question

2つのエンティティクラスが少し異なりますが、ほとんど同じように と2つのエンティティクラスを使用しています。 StudentDataおよびPassDataは両方ともgetDatatype()およびgetPassIndLinkId()のようなメソッドを持っています。

私は同様のコードを2回書くのを避けるために何ができますか？

private StudentData getStudentDataEqual(PassIndLinkReq passIndLinkReq, List<StudentData> dataList, Long val){   
    for (StudentData data:dataList) { 
     if (data.getPassIndLinkId().intValue()==passIndLinkReq.getId().intValue()){ 
      switch (passIndLinkReq.getDatatype().intValue()) { 
       case 0: 
       case 1: 
       return data; 
       case 7: 
       if (data.getClassItemTreeId().equals(val)){ 
        return data; 
       } 
       break; 
       ... 
      } 
     }   
    }       
    return new StudentData(); 
} 
// and this 
private PassData getStudentDataEqual(PassIndLinkReq passIndLinkReq, List<PassData> dataList, Long val){   
    for (PassData data:dataList){ 
     if (data.getPassIndLinkId().intValue()==passIndLinkReq.getId().intValue()) { 
      switch (passIndLinkReq.getDatatype().intValue()) { 
      case 0: 
      case 1: 
       return data; 
      case 7: 
       if (data.getClassItemTreeId().equals(val)){ 
        return data; 
       } 
       break; 
      ... 
      } 
     }   
    }       
    return new PassData(); 
} 

Answer 1

（新しいインスタンスを返しているので、代わりにインターフェイスの）あなたは新しいスーパークラスを使用することができました両方のクラスが継承します：

public class Data { ... } 

public class StudentData extends Data { ... } 

public class PassData extends Data { ... } 

これは、最もクリーンなバリアントではありませんが、うまくいくはずです。

private Data getStudentDataEqual(PassIndLinkReq passIndLinkReq, List<Data> dataList, Long val){   
    for (T data:dataList){ 
     if (data.getPassIndLinkId().intValue()==passIndLinkReq.getId().intValue()) { 
      switch (passIndLinkReq.getDatatype().intValue()) { 
       case 0: 
       case 1: 
        return data; 
       case 7: 
        if (data.getClassItemTreeId().equals(val)){ 
         return data; 
        } 
        break; 
       ... 
       } 
      }   
     }       
    return new Data(); 
} 

しかし、この解決策では、後で戻り値をキャストする必要があります。

Answer 2

あなたが試みることができる2つあります：StudentDataとPassDataが無関係であれば、あなたは彼らが継承し、このクラスでは、あなたの機能のために必要なメソッドを実装することができ、両方のスーパークラスを作成することができます。 （関連している場合は、おそらくスーパークラスを使用して関数を使用できます）。

また、関数に必要なメソッドを持つインターフェイスを作成し、このインターフェイスをStudentDataクラスとPassDataクラスの両方で実装することもできます。

Answer 3

StudentDataとPassDataは、適切なgetPassIndLinkId（）メソッドとgetClassItemTreeId（）メソッド（実際にはすべての一般的なメソッド）を持つ、より一般的な抽象クラス 'GeneralData'などのサブクラスでなければなりません。

次に、このGeneralDataクラスを利用する単一のメソッドを使用できます。

Answer 4

getStudentDataEqual（）はリストから何らかのデータを取得しているようですので、既に他の回答で指定されているものの3番目のオプションとして、それらのインスタンスを格納および取得できるジェネリックコンテナクラスを作成できます2つの可能性のある他のクラスもあります。

この場合、getStudentDataEqual（）はコンテナを使用して検索を行うため、これらのメソッドは両方のクラスで非常に短いものになります。

Answer 5

既に言及したように、StudentDataとPassDataが共通のインタフェースを実装しているのか共通の基本クラスを持っているのかを知るのに役立ちます。二つのオブジェクトについて何も知らず

、あなたはまだ、次のような何かを行うことができます。

private <T> T getStudentDataEqual(PassIndLinkReq passIndLinkReq, List<T> dataList, Long val, 
            ToIntFunction<? super T> passIndLinkIdAccessor, 
            Function<? super T, ?> classItemTreeIdAccessor, 
            Supplier<? extends T> factory) { 
    for (T data:dataList) { 
     if (passIndLinkIdAccessor.applyAsInt(data)==passIndLinkReq.getId().intValue()){ 
      switch (passIndLinkReq.getDatatype().intValue()) { 
       case 0: 
       case 1: 
        return data; 
       case 7: 
        if (classItemTreeIdAccessor.apply(data).equals(val)){ 
         return data; 
        } 
        break; 
       //... 
      } 
     } 
    } 
    return factory.get(); 
} 


private StudentData getStudentDataEqual(PassIndLinkReq passIndLinkReq, List<StudentData> dataList, Long val){ 
    return getStudentDataEqual(passIndLinkReq, dataList, val, 
      data -> data.getPassIndLinkId().intValue(), 
      data -> data.getClassItemTreeId(), StudentData::new); 
} 
private PassData getPassDataEqual(PassIndLinkReq passIndLinkReq, List<PassData> dataList, Long val) { 
    return getStudentDataEqual(passIndLinkReq, dataList, val, 
      data -> data.getPassIndLinkId().intValue(), 
      data -> data.getClassItemTreeId(), PassData::new); 
}