AN
У котлина, кажется, не всё хорошо с осмысленностью сравнения enum'ов
енумы тут вообще не при чем.
Size: a a a
2020 June 30
AN
Там мог бы быть любой объект
VP
Ну здесь, пожалуй, да
Но в when точно какой-то прикол был
Но в when точно какой-то прикол был
VP
Проблему-то я увидел, не увидел, почему оно вообще работает
А работает потому что
(bool ?: enum) - это Any, который можно сравнивать с чем угодно
a
getFirst(second) = true
getFirst(second, true) = true
getSecond(second) = true
getSecond(second, true) = false
getFirst(second, true) = true
getSecond(second) = true
getSecond(second, true) = false
AN
А работает потому что
(bool ?: enum) - это Any, который можно сравнивать с чем угоднода, но там могло бы быть что угодно, не обязательно енум
a
для enum
private final boolean getSecond(NUMBER var1, Boolean var2) {
Object var10000 = var2;
if (var2 == null) {
var10000 = var1;
}
return var10000 == NUMBER.SECOND;
}
для string
private final boolean getSecond(String str, Boolean var2) {
Object var10000 = var2;
if (var2 == null) {
var10000 = str;
}
return Intrinsics.areEqual(var10000, "STR");
}
private final boolean getSecond(NUMBER var1, Boolean var2) {
Object var10000 = var2;
if (var2 == null) {
var10000 = var1;
}
return var10000 == NUMBER.SECOND;
}
для string
private final boolean getSecond(String str, Boolean var2) {
Object var10000 = var2;
if (var2 == null) {
var10000 = str;
}
return Intrinsics.areEqual(var10000, "STR");
}
AN
Ну все честно, у элвиса приоритет выше, чем у сравнения. В принципе, надо бы сделать инспекцию по поводу сравнения типизированного объекта с Any если ее нет
a
Ну то ясно.
VP
да, но там могло бы быть что угодно, не обязательно енум
Мне просто вспомнилось вот это или что-то похожее
https://youtrack.jetbrains.com/issue/KT-22043
https://youtrack.jetbrains.com/issue/KT-22043
AN
Мне просто вспомнилось вот это или что-то похожее
https://youtrack.jetbrains.com/issue/KT-22043
https://youtrack.jetbrains.com/issue/KT-22043
Ну оно должно наверное пофиксить, но только для енумов, для остального будет то же самое
VP
Ну оно должно наверное пофиксить, но только для енумов, для остального будет то же самое
Да, согласен, не тот случай
AN
Вообще вроде была такая инспекция...
VP
Ну правило "не уверен - ставь скобки" никто не отменял 😄
AN
не, не ругается
IO
Нету вопроса наверное, человек второй раз паззлеры в канал закидывает :D
я бы не сказал, что это паззлеры
IO
в нормальных паззлерах меньше шума.
AN
в нормальных паззлерах меньше шума.
так в этом весь смысл - внимание отвлечь
IO
это понятно, но тут во-первых нет каких-то странных фич языка, во-вторых много кода, лень 10 минут всматриваться