false срабатывает, а true - нет

Вопрос по задаче

Обсуждается

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

Реализуй метод main(String[]), который выводит в консоль true, если элемент содержится в переданном массиве, иначе — false.
Массив array не должен изменять расположение своих элементов.

Для поиска элемента в массиве нужен бинарный поиск. Чтобы это сделать, используй статический метод Arrays.binarySearch(int[], int) класса Arrays. Первым параметром нужно передать отсортированный массив, в котором выполнить поиск, вторым параметром — искомый элемент. Метод Arrays.binarySearch(int[], int) возвращает индекс искомого элемента, если такой элемент есть в массиве.

Для сортировки массива можно использовать метод Arrays.sort(int[]).
При тестировании значения переменных класса Solution будут разными.

Требования:

В методе main(String[]) не изменяй расположение элементов массива array.
При вызове метода main(String[]) вывести в консоли true, если значение переменной element содержится в массиве.
При вызове метода main(String[]) вывести в консоли false, если переданный элемент не содержится в массиве.
В методе main(String[]) используй вызов метода Arrays.binarySearch(int[], int).

package com.javarush.task.pro.task05.task0519; import java.util.Arrays; /* Есть ли кто? */ public class Solution { public static int[] array = {9, 8, 7, 6, 5, 4, 3, 2, 1}; public static int element = 5; public static void main(String[] args) { //напишите тут ваш код int[] copyArray = Arrays.copyOf(array, array.length); Arrays.sort(copyArray); int isElement = Arrays.binarySearch(copyArray, element); boolean answer = isElement>0 ? true : false; System.out.println(answer); } }

Комментарии (7)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Anonymous #3303140

Уровень 2

7 сентября, 21:23

Надо сравнивать > -1, а не > 0, потому что 0 тоже может быть индексом массива

OPD

Уровень 10

7 сентября, 13:50

Можно обойтись без этого.

boolean answer = isElement>0 ? true : false;

В sout можно связать isElement и тернарный оператор. Если не придумаешь как - пиши

Anonymous #3303140

Уровень 2

7 сентября, 21:24

1. Здесь вообще не нужен тернарный оператор. 2. Это не ответ на вопрос, в чём заключается ошибка.

OPD

Уровень 10

8 сентября, 10:00

1. Аргументы? При использовании тернарного оператора, код сокращается почти в два раза. При этом, остается читабельным и понятным.
int[] copyArray = Arrays.copyOf(array, array.length); Arrays.sort(copyArray); System.out.println(Arrays.binarySearch(copyArray, element) >= 0 ? true : false);
2. Ошибка заключалась в том, что он проверял isElement > 0, а нужно было >= 0 (да, можно не "> - 1", а просто >= 0

Anonymous #3303140

Уровень 2

8 сентября, 13:17

1. Тернарный оператор нужен там, где он нужен. Результат любого сравнения - это уже булевая величина, то есть либо true, либо false. То есть буквально ты пишешь, если сравнение true, то верни true, а если false, то верни false. Вместо этого можно просто вернуть (в данном случае вывести) сам результат сравнения

System.out.println(Arrays.binarySearch(copyArray, element) > -1);

2. Писать > -1 или >= 0 по большому счету вкусовщина, но, строго говоря, строгое сравнение чище, потому что при использовании нестрогого сравнения под капотом будет выполняться +1 а затем строгое сравнение, то есть лишняя операция. Ну и ты же не указал на это в первом комментарии.

OPD

Уровень 10

9 сентября, 07:16

Согласен

Стас Пасинков Software Developer в Zipy

Master

9 сентября, 17:03

от такі строки буває потім незручно дебажити (да, в ідеї можна через альт+клік подивитися результат роботи методу, але це тягнутися до альту, потім ще мишкою наводитися на метод, на любителя, кароч)

System.out.println(Arrays.binarySearch(copyArray, element) > -1);

тому я б розбив то на логічні кроки: спочатку ми перевіряємо чи міститься елемент у масиві а потім виводимо результат на екран

boolean isExistInArray = Arrays.binarySearch(copyArray, element) > -1;
System.out.println(isExistInArray);

якби ж метод Arrays.binarySearch() був би не із стандартної бібліотеки, а якийсь наш самописний метод (від якого теж можна було б очікувати якогось гавна), тоді відповідно зробити змінну під його результат

int elementIndex = ourOwnBinarySearch(copyArray, element);
System.out.println(elementIndex > -1);

ну і зворотня сторона. якщо цей шматок коду був би написаний один раз і в подальшому ніяких змін ані в ньому, ані в методах які він викликає, не планувалося б, і ніхто в цей метод не залазив би ще років 10 (не кажучи вже про його дебаг) - тоді, звісно, можна і в одну строку зробити)