AN
но в целом тут лучше без 'идиом' всяких, как и сказали, написать пару строчек кода. выглядеть будет замечательно.
Идиома тут как раз не использовать run.
Size: a a a
2020 November 23
с#
Идиома тут как раз не использовать run.
поэтому в кавычки
с#
там в предыдущем сообщении о них было
DV
сколько людей, столько и мнений. в одном я согласен: как команде удобно читать, так и пишите
DV
Не суть, я о том, что в кодгольф играть в не стоит. Лучше писать так, чтобы было понятно читать. :)
это теория, покажите как надо )
T
if (resultCode == OK && result?.data?.data != null) put(result.data) else clear()?
а чем это тогда отличается от этого
AN
Ну не надо так делать
T
мне вот не нравится необходимость указывать !!, вот где читаемость падает
T
Ну не надо так делать
почему?
с#
сколько людей, столько и мнений. в одном я согласен: как команде удобно читать, так и пишите
это не совсем так работает, я вот переписываю свой код иногда со скоуп функций на ифы и значительно повышается читаемость. это общее наблюдение многих людей, а не личное мнение.
T
опять же !!, хотя все проверки были
с#
а вот представьте, что у вас многопоточность. а это из джавы и котлин не знает что там.
if(a != null) // true
a.action() // а другой поток уже поставил нулл, и вы что, хотите NPE ?
if(a != null) // true
a.action() // а другой поток уже поставил нулл, и вы что, хотите NPE ?
DV
проверки были, но после if другой поток "мог поменять" result.data. я бы переписал с помощью let -> run. но тут я в меньшинстве
AN
почему?
Потому что следующий, кто придет за вами писать этот проект пойдет бить вам лицо
Хотя... вот конкретно этот вариант не так плох. Читается нормально. Но можно и вложенный иф сделать, ни у кого ничего не отвалится
Хотя... вот конкретно этот вариант не так плох. Читается нормально. Но можно и вложенный иф сделать, ни у кого ничего не отвалится
AN
проверки были, но после if другой поток "мог поменять" result.data. я бы переписал с помощью let -> run. но тут я в меньшинстве
Там не в потоках дело, а в том, что смарт каст не такой умный.
с#
Там не в потоках дело, а в том, что смарт каст не такой умный.
а как смарт касту работать с джавовыми не-финальными переменными ещё
T
вот из-за того что вариантов дохренища хочется как-то красиво написать, в итоге возвращается к тем же ифам(
u
такую запись можно сократить лаконичней? в идеале чтобы не было двух упоминаний одной функции. в котлине столько этих идиом что я путаюсь в них
с#
вот из-за того что вариантов дохренища хочется как-то красиво написать, в итоге возвращается к тем же ифам(
ифы красивые, я понимаю, может вы хотите какой-то отдельный котлинвей, но тут он бессмысслен
с#
спасибо красота 😊