Java CARD Class compare method

Question

int[] value = new int[5]; 
boolean result = true; 
for(int i = 0; i < 5; i++) { 
    value[i] = cards[i].getValue(); 
} 
for(int i = 0; i < 5; i++) { 
    for(int j = i;j < 5; j++) { 
     if(value[i] == value[j + 1]) { 
      result = false; 
     } 
    } 
} 
return result; 

Этот код по существу будет сравнивать значения, которые имеет каждый объект карты, и если две карты в массиве имеют одинаковое значение, верните true. У нас есть 5 карт в каждой руке, и поэтому длина массива равна 5. Метод getValue возвращает целое число, которое по существу является значением карты. Кажется, я не знаю, что я делаю неправильно, чтобы получать ошибки в моем методе.

Answer 1

Ошибка доступа к вашему массиву при использовании j + 1, который будет за пределами границ, если j - четыре (в конце длины для value). И я предпочел бы использовать value.length вместо hardcoding. Что-то вроде

for (int i = 0; i < value.length - 1; i++) { 
    for (int j = i + 1; j < value.length; j++) { 
     if (value[i] == value[j]) { 
      result = false; 
     } 
    } 
} 

Кроме того, как указал Том, в комментариях; бессмысленно продолжать итерацию, когда result становится false. Вы можете просто вернуться, когда он станет false и полностью исключить переменную result. Мол,

for (int i = 0; i < value.length - 1; i++) { 
    for (int j = i + 1; j < value.length; j++) { 
     if (value[i] == value[j]) { 
      return false; 
     } 
    } 
} 
return true; 

Другой вариант, в Java 8+, будет что-то вроде

return IntStream.of(value).allMatch(x -> value[0] == x);