array - javaを繰り返して処理する

Question

このメソッドを使用してArrayをそのメソッドに渡すか、数値がarrayにあるかどうかをチェックするたびに書き出すほうが良いかどうか疑問に思っていました。例えば

：事前に助けを

public static boolean inArray(int[] array, int check) { 
    for (int i = 0; i < array.length; i++){ 
     if (array[i] == check) 
      return true; 
      } 
    return false; 
} 

ありがとう！

Answer 1

あなたは間違いなく、メソッドにこのロジックをカプセル化する必要があります。

同一のコードを複数回繰り返すことに利点はありません。

また、ロジックをメソッドに配置して変更する場合、コードを変更する必要があるのは1か所だけです。

サードパーティのライブラリを使用するかどうかは、全く異なる決定です。

Answer 2

アレイを使用している場合、「含む」の検索はO(N)です。最悪の場合はアレイ全体を反復処理する必要があるためです。配列がソートされている場合は、検索時間をlog(N)に減らすバイナリ検索をソートのオーバーヘッドとともに使用できます。これが繰り返し呼び出される何かをされた場合

、機能に置き：

private boolean inArray(int[] array, int value) 
{ 
    for (int i = 0; i < array.length; i++) 
    { 
     if (array[i] == value) 
     { 
      return true; 
     } 
    } 
    return false; 
} 

Answer 3

あなたはlibにorg.apache.commons.lang.ArrayUtils

をインポートすることができ、静的メソッドところがありますint配列と値を渡してチェックすることができます。

contains（int [] array、int valueToFind） 値が指定された配列にあるかどうかを確認します。

ArrayUtils.contains(intArray, valueToFind); 

ArrayUtils API

Answer 4

atleast Java 1.5.0（Java 5）以来、コードを少しきれいにすることができます。 Array sおよびIteratorを実装して何も（例えばCollection sが）のようなループすることができます。Java 8で

public static boolean inArray(int[] array, int check) { 
    for (int o : array){ 
     if (o == check) { 
     return true; 
     } 
    } 
    return false; 
} 

あなたも何か行うことができます。このため、ストリームに変換しますが

public static boolean inArray(int[] array, int check) { 
    return Arrays.asList(array).anyMatch(x -> x == check); 
} 

ですおそらく過剰殺戮。