一度だけ使用する場合、文字列定数を定義する必要がありますか？

Xaxを使用してアプリケーションのデータモデルにアクセスできるようにするJaxen（JavaのXPathライブラリ）のアダプターを実装しています。

これは、文字列（Jaxenから渡された）をデータモデルの要素にマップするクラスを実装することによって行われます。合計で1000以上の文字列比較を行う約100のクラスが必要になると推定されます。

これを行う最良の方法は、各文字列を定数として定義するのではなく、コードに直接書き込まれた文字列を使用したif / elseステートメントを単純にすることだと思います。例えば：

public Object getNode(String name) {
    if ("name".equals(name)) {
        return contact.getFullName();
    } else if ("title".equals(name)) {
        return contact.getTitle();
    } else if ("first_name".equals(name)) {
        return contact.getFirstName();
    } else if ("last_name".equals(name)) {
        return contact.getLastName();
    ...

ただし、文字列値をコードに直接埋め込むのではなく、文字列定数を作成することを常に教えられました。これは次のようになります。

private static final String NAME = "name";
private static final String TITLE = "title";
private static final String FIRST_NAME = "first_name";
private static final String LAST_NAME = "last_name";

public Object getNode(String name) {
    if (NAME.equals(name)) {
        return contact.getFullName();
    } else if (TITLE.equals(name)) {
        return contact.getTitle();
    } else if (FIRST_NAME.equals(name)) {
        return contact.getFirstName();
    } else if (LAST_NAME.equals(name)) {
        return contact.getLastName();
    ...

この場合、それは悪い考えだと思います。定数は、getNode()メソッド内で一度だけ使用されます。文字列を直接使用することは、定数を使用するのと同じくらい読みやすく理解しやすく、少なくとも1,000行のコードを書く手間を省きます。

それで、1回の使用のために文字列定数を定義する理由はありますか？または、文字列を直接使用しても問題ありませんか？

PS。代わりに列挙型を使用することを提案する前に、プロトタイプを作成しましたが、列挙型変換は単純な文字列比較よりも15倍遅いため、考慮されていません。

結論： 以下の回答は、この質問の範囲を単なる文字列定数を超えて拡大したため、2つの結論があります。

• このシナリオでは、文字列定数ではなく文字列を直接使用しても問題ありませんが、
• 文字列をまったく使用しないようにする方法があります。

そこで、文字列を完全に回避するラッパーテクニックを試します。残念ながら、まだJava 7を使用していないため、文字列switchステートメントを使用できません。ただし、最終的には、各手法を試し、そのパフォーマンスを評価することが最善の答えだと思います。現実には、1つの手法が明らかに速い場合は、その美しさや慣習の順守に関係なく、おそらくそれを選択します。

これを試して。最初のリフレクションは確かに費用がかかりますが、何度も使用する場合は、これは間違いなく、提案しているものよりも優れたソリューションです。私はリフレクションを使用するのは好きではありませんが、リフレクションの代替手段が気に入らないときは自分でそれを使用していることに気づきます。これにより、チームの頭痛の種が大幅に軽減されると思いますが、メソッド名を小文字で渡す必要があります。

つまり、getメソッドの名前は「getFullName（）」であるため、「name」を渡すのではなく、「fullname」を渡すことになります。

Map<String, Method> methodMapping = null;

public Object getNode(String name) {
    Map<String, Method> methods = getMethodMapping(contact.getClass());
    return methods.get(name).invoke(contact);
}

public Map<String, Method> getMethodMapping(Class<?> contact) {
    if(methodMapping == null) {
        Map<String, Method> mapping = new HashMap<String, Method>();
        Method[] methods = contact.getDeclaredMethods();
        for(Method method : methods) {
            if(method.getParameterTypes().length() == 0) {
                if(method.getName().startsWith("get")) {
                    mapping.put(method.getName().substring(3).toLower(), method);
                } else if (method.getName().startsWith("is"))) {
                    mapping.put(method.getName().substring(2).toLower(), method);
                }
            }
        }
        methodMapping = mapping;
    }
    return methodMapping;
}

連絡先のメンバーに含まれるデータにアクセスする必要がある場合は、必要な情報にアクセスするためのすべてのメソッドを持つ連絡先のラッパークラスを構築することを検討してください。これは、アクセスフィールドの名前が常に同じであることを保証するのにも役立ちます（つまり、ラッパークラスにgetFullName（）があり、fullnameで呼び出す場合、連絡先のgetFullName（）の名前が変更されていても常に機能します-itそれを行う前にコンパイルエラーが発生します）。

public class ContactWrapper {
    private Contact contact;

    public ContactWrapper(Contact contact) {
        this.contact = contact;
    }

    public String getFullName() {
        return contact.getFullName();
    }
    ...
}

このソリューションにより、jsfデータテーブルで使用する単一のデータ表現が必要な場合や、そのデータをジャスパーを使用してレポートにエクスポートする必要がある場合（経験上、複雑なオブジェクトアクセサーをうまく処理できない場合） 。

可能な場合は、switch文で文字列を使用できるJava 7を使用してください。

http://docs.oracle.com/javase/tutorial/java/nutsandbolts/switch.htmlから

public class StringSwitchDemo {

    public static int getMonthNumber(String month) {

        int monthNumber = 0;

        if (month == null) {
            return monthNumber;
        }

        switch (month.toLowerCase()) {
            case "january":
                monthNumber = 1;
                break;
            case "february":
                monthNumber = 2;
                break;
            case "march":
                monthNumber = 3;
                break;
            case "april":
                monthNumber = 4;
                break;
            case "may":
                monthNumber = 5;
                break;
            case "june":
                monthNumber = 6;
                break;
            case "july":
                monthNumber = 7;
                break;
            case "august":
                monthNumber = 8;
                break;
            case "september":
                monthNumber = 9;
                break;
            case "october":
                monthNumber = 10;
                break;
            case "november":
                monthNumber = 11;
                break;
            case "december":
                monthNumber = 12;
                break;
            default: 
                monthNumber = 0;
                break;
        }

        return monthNumber;
    }

    public static void main(String[] args) {

        String month = "August";

        int returnedMonthNumber =
            StringSwitchDemo.getMonthNumber(month);

        if (returnedMonthNumber == 0) {
            System.out.println("Invalid month");
        } else {
            System.out.println(returnedMonthNumber);
        }
    }
}

私は測定していませんが、switchステートメントは比較の長いリストではなくジャンプテーブルにコンパイルされると信じています。これはさらに速いはずです。

実際の質問に関して：一度だけ使用する場合は、定数にする必要はありません。ただし、定数を文書化してJavadocに表示できることを考慮してください。これは重要な文字列値にとって重要です。

これを維持する（これまでに重要な変更を行う）場合は、何らかの注釈駆動型コード生成（おそらくCGLibを使用）またはすべてのコードを作成するスクリプトを使用することを検討するかもしれません。あなたが検討しているアプローチで忍び込む可能性のあるタイプミスとエラーの数を想像してください...

クラスの上部で定義された定数を引き続き使用します。必要に応じて後で変更できるものを簡単に確認できるため、コードのメンテナンス性が向上します。たとえば、後に"first_name"なってしまう可能性があります"firstName"。

名前が一貫している場合（別名"some_whatever"は常ににマップされるgetSomeWhatever()）、リフレクションを使用してgetメソッドを決定および実行できます。

注釈がなくても、注釈処理が解決策になると思います。それはあなたのためにすべての退屈なコードを生成できるものです。欠点は、N個のモデルクラスに対してN個の生成クラスが得られることです。また、既存のクラスには何も追加できませんが、

public Object getNode(String name) {
    return SomeModelClassHelper.getNode(this, name);
}

クラスごとに1回は問題になりません。あるいは、次のように書くこともできます

public Object getNode(String name) {
    return getHelper(getClass()).getNode(this, name);
}

共通のスーパークラスで。

コード生成に注釈処理の代わりにリフレクションを使用できます。欠点は、リフレクションを使用する前にコードをコンパイルする必要があることです。これは、スタブを生成しない限り、モデルクラスで生成されたコードに依存できないことを意味します。

また、リフレクションを直接使用することも検討します。確かに、反射は遅いですが、なぜ遅いのですか？フィールド名をオンにするなど、必要なすべてのことを行う必要があるためです。