2
public boolean contains(Object[] array, Object obj) {
    for (Object a : array) {
        a.equals(obj) ? return true : return false;
    }
}

Как работает тернарный оператор в java? что не так с кондицией?

6
  • что ты пытаешься получить этой конструкцией? 6 июн 2017 в 19:54
  • 1
    Дайте догадаюсь, что-то не работает?
    – Sv__t
    6 июн 2017 в 19:54
  • пришел из JS, там такое вполне реально :D ожидаю получить тру, если объект а из массива объектов array равен объекту obj, или фолс если не равен
    – ebaklak
    6 июн 2017 в 19:55
  • 2
    ты понимаешь, что ты сделаешь return сразу же на первой итерации в любом случае ни разу не добравшись до конца ни в одном случае? 6 июн 2017 в 19:55
  • 2
    Вы на первой же итерации вернете false, если первый элемент не эквивалентен a
    – Sv__t
    6 июн 2017 в 19:58

4 ответа 4

8
public boolean contains(Object[] array, Object obj) {
    for (Object a : array) {
        a.equals(obj) ? return true : return false;
    }
}

Во-первых, срочно заклеиваем вырожденный случай с пустым массивом. Иначе есть путь выполнения, при котором для не-void-функции значение не возвращается (даже если он реально недостижим, компилятору-то невдомёк):

public boolean contains(Object[] array, Object obj) {
    for (Object a : array) {
        a.equals(obj) ? return true : return false;
    }
    return false;
}

Во-вторых, тернарный оператор ожидает три выражения: для условия и двух веток. А return не образует выражения. Текущий контекст (в котором мог бы использоваться результат) он безусловно завершает, поэтому ему нет смысла быть выражением, его результат нигде не может быть использован. Поскольку у вас обе ветки что-то возвращают, почему бы не "вынести return за скобки"?

public boolean contains(Object[] array, Object obj) {
    for (Object a : array) {
        return a.equals(obj) ? true : false;
    }
    return false;
}

В-третьих, a.equals(obj) уже возвращает boolean (true или false). Зачем тут вообще тернарный оператор?

public boolean contains(Object[] array, Object obj) {
    for (Object a : array) {
        return a.equals(obj);
    }
    return false;
}

Синтаксическая корректность достигнута. Дальше разберётесь.
Но у меня остался ещё один пункт.

В-четвёртых, эта функция оправдывает своё название только для массивов длины не более 1. Задокументируйте это поведение. Или исправьте его, если это не то, что вы задумали.

3

Есть другой способ, по мне более читаемый

Arrays.asList(array).contains(obj)

Я считаю, что тернарный оператор нужно очень осторожно использовать. Он часто порождает незаметные и очень серьёзные ошибки.

update Метод asList возвращает объект-обертку над массивом(без дополнительных трат по памяти) с правильными реализациями методов интерфейса List для массивов.

19
  • Я считаю это апогеем говнокодерства: лишь чтобы узнать содержится ли объект в массиве - выделять память под лист и добавлять туда все элементы - WAT??? "более читабельный" - не аргумент чтобы тах херить производительность. Инженеры оракла жопы рвут чтоб оптимизировать каждую закорючку в джаве, а вы делаете такое. 6 июн 2017 в 20:44
  • Рекомендую ознакомиться с методомjava.util.Arrays#asList и что он возвращает java.util.Arrays.ArrayList. 6 июн 2017 в 20:55
  • java.util.Arrays.ArrayList - это просто обертка над массивом(под массив даже память не выделяется, лишь ссылка присваивается), других свойств класса просто нет, лишь методы, которые является адаптерами для класса java.util.AbstractList. Еще один "удивительный" факт, что из этого списка нельзя удалить элементы. Я верю, что инженеры оракла достаточно хорошо оптимизировали методы jdk. 6 июн 2017 в 21:05
  • хм, действительно, однако так все равно создается новый объект, добавляющий работы GC и кто знает сколько раз нужно вызвать этот метод - может он в цикле вызывается. да, потери не такие жуткие как я думал, но они есть и они могут расти. Видел я однажды "читабельный" вариант похожей задачи ((List<Object1>) list).stream().map(Object1::getField).collect(Collectors.toList()).contains(param) - тоже аргументация была именно про читабельнось. 6 июн 2017 в 21:06
  • 1
    Нужно надавать по рукам тем, кто в стайлгайды прописал ставить скобки в тернарном операторе, и проблемы с ним автоматически отпадут.
    – Qwertiy
    6 июн 2017 в 22:57
2

Млин, 3 ответа около 7 вариантов и только 1 верный и тот заклевали как апогей говнокодерства

Люди, ау вы чего?

public boolean contains(Object[] array, Object obj) {
    for (Object a : array) {
       if(a==null && obj==null)
          return true;
       if(a==null)
          continue;
       if(a.equals(obj))
         return true;
    }
    return false;
}
6
  • Если null то значит вернет true. Why?
    – user194625
    7 июн 2017 в 5:30
  • @Flippy не если null вернет true, а если if(a==null && obj==null) - чувствуете разницу?
    – Barmaley
    7 июн 2017 в 5:51
  • Тоесть если в массиве будет null и искомое значение null то вернет true
    – user194625
    7 июн 2017 в 5:53
  • Ну да так и есть. А что вас в этом удивляет? null ведь равен null или как?
    – Barmaley
    7 июн 2017 в 6:06
  • Зачем изобретать велосипед? метод java.util.Arrays.ArrayList#contains вызывает @Override public int indexOf(Object o) { E[] a = this.a; if (o == null) { for (int i = 0; i < a.length; i++) if (a[i] == null) return i; } else { for (int i = 0; i < a.length; i++) if (o.equals(a[i])) return i; } return -1; } 7 июн 2017 в 7:51
2

Во-первых, можно проще:

public boolean contains(Object[] array, Object obj) {
    for (Object a : array) {
        return a.equals(obj);
    }
    return false;
}

Во-вторых, если же Вам уж очень хочется использовать тернарный оператор, то синтаксис у Вас хромает, и нужно так:

public boolean contains(Object[] array, Object obj) {
    for (Object a : array) {
        return a.equals(obj) ? true : false;
    }
    return false;
}

PS. Оба метода возвращают false, если array пустой.

PPS. Логику метода во внимание не принимал (а она не верна), поправил только тернарный оператор.

12
  • 5
    только вроде как вот это return a.equals(obj) ? true : false; в цикле - неправильно 6 июн 2017 в 19:56
  • 1
    @АлексейШиманский, Почему?
    – post_zeew
    6 июн 2017 в 19:57
  • 4
    Потому что если a будет equals объекту obj, и он будет располагаться например 10-ым в массиве, а 1-ый элемент не равен, то сразу первая итерация выкинет false .... хотя итог должен быть true 6 июн 2017 в 19:58
  • @АлексейШиманский, Смотрите PPS.
    – post_zeew
    6 июн 2017 в 20:00
  • 3
    ИМХО лучше это вообще убрать из ответа, чтобы не сбивать с толку. 6 июн 2017 в 20:02

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.