0 
public String getPreviewMessageForWebdavResponse(){ 

    String format="<preview><currentNumber>%d</currentNumber><totalNumber>%d</totalNumber></preview>";
    return String.format(format, currentNumber,totalNumber);
}

時々、私はひもに遭遇します。定数またはローカル変数である必要があるかどうかを判断できません。文字列はメソッド内で1回だけ使用され、メソッドの外部に公開する必要はありません。

選び方は？

4

1 に答える 1

3

あなたの質問は少し不明確です。思いつく3つの質問に答えます。

（それらはすべてスタイルの問題です）。

変数をまったく使用する必要がありますか、それとも文字列リテラルを使用されている 1 つの場所にインライン化するだけですか?

少なくともあなたの場合、文字列の長さがこれを示唆している場合は、わかりやすい名前の変数として保持し、メソッドの先頭に保持してください。必要に応じて後で見つけて更新しやすくし、後で必要になった場合にリファクタリングしやすくし、読みやすくします。

変数はローカルで最終的なものにする必要がありますか?

おそらく。

クラスレベルの定数 ( static final) にする必要がありますか?

スコープと可視性を常に可能な限り制限してください。

1 つのメソッドでのみ使用する場合は、メソッド内に保持します。

同じクラスの複数のメソッドで使用する場合は、 にしprivate static finalます。

同一パッケージ内で複数箇所使用する場合は、 にしてstatic finalください。

プロジェクト全体で使用する場合は、それを作成してpublicください。

于 2012-07-03T01:37:55.700 に答える