Логика относительно вычитания и сравнения элементов массива

Question

Я написал коды для следующего вопроса, но результат не так, как ожидалось. Я не знаю, что-то не так с моим кодом. Логика кажется прекрасной. Может ли кто-нибудь увидеть, если что-то не так с моим кодом.

Учитывая массив отсортированных в порядке возрастания, верните true, если массив содержит 3 смежных оценки, которые отличаются друг от друга не более чем на 2, например, с {3, 4, 5} или {3, 5, 5 }.

Мой исходный код выглядит следующим образом:

public boolean scoresClump(int[] scores) { 

    boolean result = false; 

    for(int i=0; i<scores.length-2; i++){ 
     if((scores[i+1]-scores[i])<=2 && (scores[i+2]-scores[i+1])<=2){ 
      result = true; 
      break; 
     } 
    } 

    return result; 
} 

This ссылка на этот вопрос.

Answer 1

Здесь вы идете. Ваш предыдущий пост не учитывал дельта между несмежными значениями.

public boolean scoresClump(int[] scores) { 
    boolean result = false; 

    for(int i=0; i<scores.length-2; i++){ 
     if((scores[i+2]-scores[i])<=2){ 
      result = true; 
      break; 
     } 
    } 

    return result; 
} 

Answer 2

Ваш for цикл должен быть for(int i=0; i<scores.length-2; i++){ иначе вы получите ArrayOutOfBoundsException.

Кроме того, ваш break должен быть в true условным, а не false. В настоящее время в вашем коде существует цикл после первого набора из 3 элементов, которые не соответствуют условию.

Кроме того, если они уже отсортированы в порядке возрастания нет необходимости делать Math.abs

Answer 3

Во-первых, если вы получили сообщение об ошибке или неожиданное поведение, которое вы должны описать то, что он (например, ожидаемый и реальный объем производства).

В вашем случае я могу обнаружить хотя бы одну ошибку в вашем коде: вы получаете доступ к своим записям в цикле [i + 2]. Когда я дойду до длины 1, вы получите длину + 1. Даже доступ к длине + 0 уже является ошибкой, так как последний элемент имеет длину индекса-1. Вы не указали свой полный код, но я предполагаю, что из-за возникшего исключения результат в вызывающей функции интерпретируется как false.

Кроме того, две проверки не нужны, достаточно одной проверки, так как вы выполните следующую проверку со следующей итерацией.

Answer 4

попробовать это:

public boolean scoresClump(int[] scores) { 
boolean result=false; 
for(int i=0; i<scores.length-2; i++){ 
     if(Math.abs(scores[i]-scores[i+1])<=2 && 
     Math.abs(scores[i+1]-scores[i+2])<=2 && 
     Math.abs(scores[i]-scores[i+2])<=2){ 
      result=true; 
     } 
} 
return result; 
}