`c> = '0'`または` c> = 48`をチェックする方が良いですか？

同僚と話し合った後、ベストプラクティスに従って、Javaでcharデータ型をどのように扱うかについて「哲学的な」質問をしました。

入力としてString 's'が与えられた場合、その中に存在する数字の数を数える必要がある単純なシナリオ（明らかに、これは私の質問に練習の意味を与えるための非常に単純な例です）を想定してください。

これらは2つの可能な解決策です：

1）

    for(int i=0; i<s.length(); i++) {
        if(s.charAt(i) >= 48 && s.charAt(i) <= 57) {
            n++;
        }
    }

2）

    for(int i=0; i<s.length(); i++) {
        if(s.charAt(i) >= '0' && s.charAt(i) <= '9' ) {
            n++;
        }
    }

どちらがより「クリーン」でJavaのベストプラクティスに準拠していますか？

両方とも恐ろしいですが、最初のほうが恐ろしいです。

どちらもJavaの組み込み機能を無視して、どの文字が「数値」であるかを決定します（のメソッドを使用Character）。しかし、最初のものは、文字列のUnicodeの性質を無視するだけでなく、0123456789のみが存在する可能性があると仮定すると、文字エンコードの履歴について何かを知っている場合にのみ意味のある文字コードを使用することにより、この無効な推論さえも覆い隠します。

どちらでもない。JavaのビルトインCharacterクラスでそれを理解してください。

for (int i = 0; i < s.length(); ++i) {
  if (Character.isDigit(s.charAt(i))) {
    ++n;
  }
}

数字としてカウントされるASCII数字よりもいくつかの文字範囲があり、投稿した例のいずれもそれらをカウントしません。JavaDocのためのCharacter.isDigit()有効な数字であるとしてリストこれらの文字範囲：

数字を含むいくつかのUnicode文字範囲：

• 「\ u0030」から「\ u0039」、ISO-LATIN-1の数字（「0」から「9」）
• 「\ u0660」から「\ u0669」、アラビア語-インド数字
• '\ u06F0'〜 '\ u06F9'、拡張アラビア数字
• 「\ u0966」から「\ u096F」、デーバナーガリー数字
• 「\ uFF10」から「\ uFF19」、全角数字

他の多くの文字範囲にも数字が含まれています。

そうは言ってCharacter.isDigit()も、このリストを使用して委任する必要があります。新しいUnicodeプレーンが追加されると、Javaコードが更新されます。JVMをアップグレードすると、古いコードが新しい数字とシームレスに機能するようになります。また、DRYです。「これは数字ですか」コードを他の場所で参照されている1つの場所にローカライズすることにより、コードの重複（つまりバグ）の側面を回避できます。最後に、最後の行に注意してください。このリストは完全ではなく、他の数字があります。

個人的には、コアJavaライブラリに委任し、「数字とは何かを設定する」よりも生産的なタスクに時間を費やします。

このルールの唯一の例外は、他の数字ではなくリテラルASCII数字をテストする必要がある場合です。たとえば、ストリームを解析していて、（他の数字とは対照的に）ASCII数字のみが特別な意味を持つ場合、を使用することは適切ではありませんCharacter.isDigit()。

その場合、別のメソッドを記述し、たとえばMyClass.isAsciiDigit()そこにロジックを配置します。コードの再利用と同じ利点が得られ、名前はチェック対象について非常に明確であり、ロジックは正しいです。

EBCDICを基本文字セットとして使用し48、ASCII文字を処理する必要があるアプリケーションをCで記述する場合は、とを使用し57ます。あなたはそれをやっていますか？そうは思いません。

使用についてisDigit()：依存します。JSONパーサーを書いていますか？だけ0にする9ので使用しないで、数字として受け入れられているisDigit()かどうかを確認、>= '0'と<= '9'。ユーザー入力を処理していますか？isDigit()残りのコードが実際に文字列を処理でき、それを正しく数値に変換できる限り使用します。

2番目の例は明らかに優れています。2番目の例の意味は、コードを見るとすぐにわかります。最初の例の意味は、頭の中にASCIIテーブル全体を記憶している場合にのみ明らかです。

特定の文字のチェックと、文字の範囲またはクラスのチェックを区別する必要があります。

1）特定のキャラクターの確認。

通常の文字の場合、文字リテラルを使用します（例：）if(ch=='z')...。タブや改行などの特殊文字をチェックする場合は、などのエスケープを使用する必要がありますif (ch=='\n')...。チェックする文字が異常な場合（たとえば、すぐに認識できないか、標準キーボードで使用できない場合）、リテラル文字ではなく16進文字コードを使用することがあります。ただし、16進コードは「魔法の値」なので、定数に抽出して文書化します。

const char snowman = 0x2603; // snowman char used to detect encoding issues
...
if (ch==showman)...

16進コードは、文字コードを指定する標準的な方法です。

2）文字クラスまたは範囲の確認

アプリケーションコードでこれを直接行うべきではありませんが、文字の分類のみに関係する別のクラスにカプセル化する必要があります。また、この目的のためにライブラリが既に存在し、少なくともASCII範囲外の文字を考慮する場合、文字の分類は通常、思っているよりも複雑であるため、これを変更する必要があります。

ASCII範囲の文字のみに関心がある場合は、このライブラリで文字リテラルを使用できます。それ以外の場合は、おそらく16進リテラルを使用します。Java組み込み文字ライブラリのソースコードを見ると、16進数を使用して文字値と範囲も参照しています。これは、Unicode標準で指定されている方法だからです。

ASCIIコードでcを変換する必要があるc >= '0'ため、常に使用することをc >= 48お勧めします。

正規表現（RegEx）には、数字用の特定の文字クラスがあります-- \d文字列から他の文字を削除するために使用できます。結果の文字列の長さは目的の値です。

public static int countDigits(String str) {
    str = Objects.requireNonNull(str).trim();

    return str.replaceAll("[^\\d]", "").length();
}

ただし、RegExは他の提案されたソリューションよりも計算的に要求が厳しいため、一般的には推奨されません。