これはアンチパターンの例ですか？

Question

私と私の同僚の1は、次のような問題を解決しようとしていた。

は、私の同僚の 一つは、特定の一つから一つの特性をフェッチA. から一つの特定のプロパティを抽出する問題に直面していたクラスの例を取ることができますクラス（この場合はA）は簡単です。 には複数のクラス（A1、A2 ...）があると仮定し、これらのクラスのコレクションから特定のプロパティーを1つずつ取り出して、コードの再利用性を高めたいとします。

List<String> tempList = new ArrayList<>(); 
for(A a : listOfAInstances) { 
    tempList.add(a.getName()); 
} 
return tempList; 

を今、私は各クラスと異なる性質のためにこれをしなければならない複数のクラスが存在する場合：

例えば

public class A { 
    private String name; 
    . 
    . 
    . 
} 
List<String> listOfNames = createNameList(listOfAInstances); 

createNameList（）メソッドは、次のようになります。

1. リフレクションベースのアプローチ：
   は、私は2つのアプローチを提案しました。
2. "PropertyExtractable"というインターフェイスを作成し、そこにメソッド "extractProperty"を配置します。

以下に示すよう：このため

interface PropertyExtractable { 
    Object extractProperty(); 
} 

public class A implements PropertyExtractable { 
    private String name; 
    . 
    . 
    . 
    public Object extractProperty() { 
     return this.name; 
    } 
} 

を私はその後、すなわちどこでも使用することができ、いくつかのユーティリティメソッドを書くことができます

public Object getPropertiesOfPropertyExtractable(PropertyExtractable prExtractable) { 
     return prExtractable.extractProperty(); 
    } 

これが背景だった、私の一つの他の同僚が持っていました2番目のアプローチについての別の意見は、彼はそれがアンチパターンのようだと私に言った。彼は私に説明しようとしましたが、私はそれを完全には得られなかったので、私はここで尋ねています。

この例をJavaのComparatorインターフェイスと比較しようとしています。 javaのように私たちはカスタムオブジェクトクラスのいずれかでComparatorを使用することができ、比較のロジックを定義することができます。なぜ抽出ロジックを定義できないのですか？

さらに多くのインターフェイスをこの方法で使用できます。私たちはそれを使用しないでください

私はこのアプローチがアンチパターンであることを知りたいですか？どうして？

Answer 1

あなたは別の方法で抽出するコードを配置し、それを再利用することができます

class A { 
    private String name; 

    public String getName() { 
     return name; 
    } 
} 

class B { 
    private String surname; 

    public String getSurname() { 
     return surname; 
    } 
} 

public class SomeClass { 

    private <T> List<String> extractFields(List<T> list, Function<T, String> extractorFunction) { 
     return list.stream().map(extractorFunction).collect(Collectors.toList()); 
    } 

    public void someMethod() { 

     List<A> listOfInstancesA = new ArrayList<>(); 
     List<B> listOfInstancesB = new ArrayList<>(); 

     // fill lists 

     List<String> fieldsA = extractFields(listOfInstancesA, A::getName); 
     List<String> fieldsB = extractFields(listOfInstancesB, B::getSurname); 
    } 
} 

Answer 2

あなたが記述状況は変更したくないレガシーシステムと協力しています。

共通のプロパティ（Comparatorインターフェイスの例のような）のインターフェイスを導入しない場合は、あなたは実際に機能的なインターフェースが必要なので、アンチパターンであるかもしれない意味のないインターフェースを導入しました：PropertyExtractableとNamedObject =>はメソッドを持っています：String getName（））。

Reflectionを実装する場合は、インターフェースは正しいかもしれませんが、表示されません（例：あなたの場合はすでににの反射がJavaに組み込まれています。

通常、アダプタパターンを使用して、要求されたインターフェイスを実装していないオブジェクトからプロパティ/メソッドを取得します。