public boolean contains(Object[] array, Object obj) {
for (Object a : array) {
a.equals(obj) ? return true : return false;
}
}
Как работает тернарный оператор в java? что не так с кондицией?
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. Задокументируйте это поведение. Или исправьте его, если это не то, что вы задумали.
Есть другой способ, по мне более читаемый
Arrays.asList(array).contains(obj)
Я считаю, что тернарный оператор нужно очень осторожно использовать. Он часто порождает незаметные и очень серьёзные ошибки.
update Метод asList возвращает объект-обертку над массивом(без дополнительных трат по памяти) с правильными реализациями методов интерфейса List для массивов.
java.util.Arrays#asList
и что он возвращает java.util.Arrays.ArrayList
.
Commented
6 июн. 2017 в 20:55
java.util.Arrays.ArrayList
- это просто обертка над массивом(под массив даже память не выделяется, лишь ссылка присваивается), других свойств класса просто нет, лишь методы, которые является адаптерами для класса java.util.AbstractList
. Еще один "удивительный" факт, что из этого списка нельзя удалить элементы. Я верю, что инженеры оракла достаточно хорошо оптимизировали методы jdk.
Commented
6 июн. 2017 в 21:05
((List<Object1>) list).stream().map(Object1::getField).collect(Collectors.toList()).contains(param)
- тоже аргументация была именно про читабельнось.
Commented
6 июн. 2017 в 21:06
Млин, 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;
}
null
то значит вернет true
. Why?
null
вернет true
, а если if(a==null && obj==null)
- чувствуете разницу?
null
и искомое значение null
то вернет true
null
ведь равен null
или как?
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; }
Commented
7 июн. 2017 в 7:51
Во-первых, можно проще:
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. Логику метода во внимание не принимал (а она не верна), поправил только тернарный оператор.
return a.equals(obj) ? true : false;
в цикле - неправильно
Commented
6 июн. 2017 в 19:56
a
будет equals
объекту obj
, и он будет располагаться например 10-ым в массиве, а 1-ый элемент не равен, то сразу первая итерация выкинет false
.... хотя итог должен быть true
Commented
6 июн. 2017 в 19:58
return
сразу же на первой итерации в любом случае ни разу не добравшись до конца ни в одном случае?false
, если первый элемент не эквивалентенa