java - 重複した Enum コードを削除するにはどうすればよいですか?

Question

このインターフェイスを実装する列挙型が多数あります。

/**
 * Interface for an enumeration, each element of which can be uniquely identified by its code
 */
public interface CodableEnum {

    /**
     * Get the element with a particular code
     * @param code
     * @return
     */
    public CodableEnum getByCode(String code);

    /**
     * Get the code that identifies an element of the enum
     * @return
     */
    public String getCode();
}

典型的な例は次のとおりです。

public enum IMType implements CodableEnum {

    MSN_MESSENGER("msn_messenger"),
    GOOGLE_TALK("google_talk"),
    SKYPE("skype"),
    YAHOO_MESSENGER("yahoo_messenger");

    private final String code;

    IMType (String code) {
        this.code = code;
    }

    public String getCode() {
        return code;
    }   

    public IMType getByCode(String code) {
        for (IMType e : IMType.values()) {
            if (e.getCode().equalsIgnoreCase(code)) {
                return e;
            }
        }
    }
}

ご想像のとおり、これらのメソッドは CodableEnum のすべての実装で実質的に同一です。この重複をなくしたいのですが、率直に言って方法がわかりません。次のようなクラスを使用してみました。

public abstract class DefaultCodableEnum implements CodableEnum {

    private final String code;

    DefaultCodableEnum(String code) {
        this.code = code;
    }

    public String getCode() {
        return this.code;
    }   

    public abstract CodableEnum getByCode(String code);  
}

しかし、これはかなり役に立たないことが判明しました。

列挙型はクラスを拡張できません
列挙型の要素 (SKYPE、GOOGLE_TALK など) はクラスを拡張できません
DefaultCodableEnum 自体は Enum ではないため、getByCode() のデフォルトの実装を提供できません。DefaultCodableEnum を変更して java.lang.Enum を拡張しようとしましたが、これは許可されていないようです。

リフレクションに頼らない提案はありますか？ありがとう、ドン

score 13 · Accepted Answer

複製されたコードをCodeableEnumHelperクラスに分解できます。

public class CodeableEnumHelper {
    public static CodeableEnum getByCode(String code, CodeableEnum[] values) {
        for (CodeableEnum e : values) {
            if (e.getCode().equalsIgnoreCase(code)) {
                return e;
            }
        }
        return null;
    }
}

各CodeableEnumクラスは引き続きメソッドを実装する必要がありますgetByCodeが、メソッドの実際の実装は少なくとも 1 つの場所に集中化されています。

public enum IMType implements CodeableEnum {
    ...
    public IMType getByCode(String code) {
        return (IMType)CodeableEnumHelper.getByCode(code, this.values());
    } 
}

score 7 · Accepted Answer

抽象列挙型は潜在的に非常に有用です（現在は許可されていません）。しかし、Sunの誰かにそれを追加するよう働きかけたい場合は、提案とプロトタイプが存在します。

http://freddy33.blogspot.com/2007/11/abstract-enum-ricky-carlson-way.html

サンRFE：

http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6570766

score 5 · Accepted Answer

dave のコードを整理するには:

public class CodeableEnumHelper {
    public static <E extends CodeableEnum> E getByCode(
        String code, E[] values
    ) {
        for (E e : values) {
            if (e.getCode().equalsIgnoreCase(code)) {
                return e;
            }
        }
        return null;
    }
}

public enum IMType implements CodableEnum {
    ...
    public IMType getByCode(String code) {
        return CodeableEnumHelper.getByCode(code, values());
    } 
}

またはより効率的に:

public class CodeableEnumHelper {
    public static <E extends CodeableEnum> Map<String,E> mapByCode(
        E[] values
    ) {
        Map<String,E> map = new HashMap<String,E>();
        for (E e : values) {
            map.put(e.getCode().toLowerCase(Locale.ROOT), value) {
        }
        return map;
    }
}

public enum IMType implements CodableEnum {
    ...
    private static final Map<String,IMType> byCode =
        CodeableEnumHelper.mapByCode(values());
    public IMType getByCode(String code) {
        return byCode.get(code.toLowerCase(Locale.ROOT));
    } 
}

score 2 · Accepted Answer

私が書いたローカリゼーションコンポーネントでも同様の問題がありました。私のコンポーネントは、難しい問題ではなく、リソースバンドルにインデックスを付ける列挙定数を使用して、ローカライズされたメッセージにアクセスするように設計されています。

同じ「テンプレート」列挙型コードをあちこちにコピーして貼り付けていることがわかりました。重複を避けるための私の解決策は、列挙定数名とコンストラクター引数を含む XML 構成ファイルを受け入れるコードジェネレーターです。出力は、「複製された」動作を含む Java ソースコードです。

現在、複製されたコードのすべてではなく、構成ファイルとジェネレーターを維持しています。列挙型のソースコードがあればどこにでも、XML 構成ファイルがあります。私のビルドスクリプトは古い生成ファイルを検出し、コードジェネレーターを呼び出して列挙型コードを作成します。

このコンポーネントはこちらで確認できます。コピーして貼り付けていたテンプレートは、XSLT スタイルシートに分解されます。コードジェネレーターがスタイルシート変換を実行します。入力ファイルは、生成されたenum ソースコードに比べて非常に簡潔です。

HTH、
グレッグ

score 1 · Accepted Answer

ここに別の解決策があります：

interface EnumTypeIF {
String getValue();

EnumTypeIF fromValue(final String theValue);

EnumTypeIF[] getValues();

class FromValue {
  private FromValue() {
  }

  public static EnumTypeIF valueOf(final String theValue, EnumTypeIF theEnumClass) {

    for (EnumTypeIF c : theEnumClass.getValues()) {
      if (c.getValue().equals(theValue)) {
        return c;
      }
    }
    throw new IllegalArgumentException(theValue);
  }
}

秘訣は、内部クラスを使用して「グローバルメソッド」を保持できることです。

私にとってはかなりうまくいきました。OK、3つのメソッドを実装する必要がありますが、これらのメソッドは単なる委任者です。

score 1 · Accepted Answer

残念ながら、これを行う方法はないと思います。あなたの最善の策は、おそらく emum を完全にあきらめて、従来のクラス拡張と静的メンバーを使用することです。それ以外の場合は、そのコードを複製することに慣れてください。ごめん。

score 1 · Accepted Answer

コードで列挙型をロードするタイプセーフなユーティリティクラスを作成します。

インターフェイスは次のようになります。

public interface CodeableEnum {
    String getCode();
}

ユーティリティクラスは次のとおりです。

import java.lang.reflect.InvocationTargetException;


public class CodeableEnumUtils {
    @SuppressWarnings("unchecked")
    public static <T extends CodeableEnum>  T getByCode(String code, Class<T> enumClass) throws IllegalArgumentException, SecurityException, IllegalAccessException, InvocationTargetException, NoSuchMethodException {
        T[] allValues = (T[]) enumClass.getMethod("values", new Class[0]).invoke(null, new Object[0]);
        for (T value : allValues) {
            if (value.getCode().equals(code)) {
                return value;
            }
        }
        return null;
}

}

使用法を示すテストケース:

import junit.framework.TestCase;


public class CodeableEnumUtilsTest extends TestCase {
    public void testWorks() throws Exception {
    assertEquals(A.ONE, CodeableEnumUtils.getByCode("one", A.class));
      assertEquals(null, CodeableEnumUtils.getByCode("blah", A.class));
    }

enum A implements CodeableEnum {
    ONE("one"), TWO("two"), THREE("three");

    private String code;

    private A(String code) {
        this.code = code;
    }

    public String getCode() {
        return code;
    }   
}
}

これで、getCode() メソッドのみが複製され、getByCode() メソッドは 1 か所にまとめられました。すべての例外を単一の RuntimeException にラップするのもいいかもしれません:)

score 0 · Accepted Answer

特定のケースでは、getCode（）/ getByCode（String code）メソッドは、すべての列挙型によって提供されるtoString（）/ valueOf（String value）メソッドの動作に非常に近いように見えます（婉曲的に言えば）。使ってみませんか？

score 0 · Accepted Answer

私があなたが望むものに近づいたのは、汎用コードを「実装」するテンプレートをIntelliJで作成することでした(enumのvalueOf(String name)を使用)。完璧ではありませんが、かなりうまく機能します。

score 0 · Accepted Answer

実際に実行時の型情報を実装しているようです。Java はこれを言語機能として提供します。

RTTIまたはリフレクションを調べることをお勧めします。

score 0 · Accepted Answer

これは不可能だと思います。ただし、列挙型の値の名前をコードとして使用する場合は、列挙型の valueOf(String name) メソッドを使用できます。

score 0 · Accepted Answer

静的ジェネリックメソッドはどうですか? 列挙型の getByCode() メソッド内から再利用するか、直接使用することができます。私は常に列挙型に整数 ID を使用しているため、getById() メソッドはこれを行うだけです: return values()[id]. それはずっと速くて簡単です。

score 0 · Accepted Answer

本当に継承が必要な場合は、古い Java 1.4 の時代のように、enum パターンを自分で実装できることを忘れないでください。

score 0 · Accepted Answer

別の解決策は、列挙型自体に何も入れず、各列挙型に双方向マップ Enum <-> Code を提供するだけです。たとえば、これには Google Collections のImmutableBiMapを使用できます。

そうすれば、重複するコードはまったくありません。

例：

public enum MYENUM{
  VAL1,VAL2,VAL3;
}

/** Map MYENUM to its ID */
public static final ImmutableBiMap<MYENUM, Integer> MYENUM_TO_ID = 
new ImmutableBiMap.Builder<MYENUM, Integer>().
put(MYENUM.VAL1, 1).
put(MYENUM.VAL2, 2).
put(MYENUM.VAL3, 3).
build();

score 0 · Accepted Answer

私の意見では、これはリフレクションなしで、列挙型に余分なラッパーを追加することなく、最も簡単な方法です。

列挙型が実装するインターフェイスを作成します。

public interface EnumWithId {

    public int getId();

}

次に、ヘルパークラスで次のようなメソッドを作成します。

public <T extends EnumWithId> T getById(Class<T> enumClass, int id) {
    T[] values = enumClass.getEnumConstants();
    if (values != null) {
        for (T enumConst : values) {
            if (enumConst.getId() == id) {
                return enumConst;
            }
        }
    }

    return null;
}

このメソッドは、次のように使用できます。

MyUtil.getInstance().getById(MyEnum.class, myEnumId);

java - 重複した Enum コードを削除するにはどうすればよいですか?

15 に答える 15

Related

Reference