Javaの安全String.formatとエスケープ％

Question

私はString.format Java APIのメソッドを使って何かを記録しています。私の方法は次のようである：問題があるしかし

public static void log(String message, Object... params){ 
    System.out.println(String.format(message, params)); 
} 

ユーザーがどこかで%文字を持つメッセージを送信した場合、それは例外をスローします。ここではシナリオです：

log("SELECT * FROM my WHERE name like '%six%'"); 

とJavaは%sを置き換えるために何かを探し（それは大丈夫です）と%'（おっと）。私はそれを修正したい。 paramsが存在しないため、%sが失われ、%'が例外となるためです。

1つの解決策はmessage.replace("%", "%%")ですが、私はそれが優雅な解決策であるかどうかはわかりません。

最も可能性が高い誤用（どこか Stringで %sまたは %を使用しますが、パラメータを提供していない）のための簡単な解決策は、あなたの元のメソッドに加えにあなたの方法 にノーのparams過負荷を提供することで

Answer 1

logは、所与%が書式指定として、またはパーセント記号として意図されているかどうかわかりません。次の例を考えてみましょう：

log("%s%s", "test"); 

はその"test%s"、"%stest"、またはエラーか？

したがって、問題は、呼び出しサイトで対処する必要があります：

escape()あなたはそれが %%ですべて %を置き換えます記述する必要があります機能です

log(escape("SELECT * FROM my WHERE name like '%six%'")); 

。 `％が`ない場合は `％のS`はOKだろう、なぜ私は表示されません

log("%s", "SELECT * FROM my WHERE name like '%six%'"); 

Answer 2

：

public static void log(String message) { 
    System.out.println(message); 
} 

別の方法としては、動的なしのparamsケースを検出しようとすることができ、それは少しより多くの仕事です：

public static void log(String message, Object... params) { 
    final String output; 
    if (params == null || params.length = 0) { 
    output = message; 
    } else { 
    output = String.format(message, params); 
    } 
    System.out.println(output); 
} 

Answer 3

一つの選択肢のw paramsの長さが0の場合は、文字列をprintlnに直接渡してください。 もう1つのオプションは、2つのオーバーロードを持つことです。

public static void log(String message); 
public static void log(String format, String param, String...params);