OD
Ну миллион не много, у меня игра 2 миллиона, и там еще хуже в коде бывает 😂
Size: a a a
2017 May 24
SZ
Я не к тому, что много или нет, а к тому, что я не понимаю, как такое можно в продакшн вообще пустить...
OD
А в чем проблема? Разве гет вью может вернуть null?
SZ
И вот эти люди потом собеседования проводят с их этими вопросам про ООП, блин и инструменты и опыт твоей работы, понимаешь...
SZ
В этом куске по сути два раза одна и та же ссылка проверяется на null
SZ
просто сначала она присваивается
fragment, а потом она же присваивается (через проверку опять на null) abstractMapFragmentOD
Ну это просто лишняя проверка получается, ничего фатального же
SZ
А потом смотришь на чужие такие полотна и не можешь понять, что вообще происходит.
MK
Ребят, есть такая архитектура PIDOR
SZ
ахахаха
SZ
Ну вот, я на неё на работе смотрю. Лишняя проверка на null - это просто самое смешное пока, что я видел. Остальное не очень. Иногда даже грустное.
ММ
Веселец
ММ
Улыбнуло
SZ
Улыбнуло
Вот как такой код рефакторить? Конкретно это место - это так, весёлый пример. Можешь представить, что там в остальном...
OD
мухаха - welcome to the real world )))))
SZ
Но на первом месте работы несоответствие Checkstyle = build fail...
SZ
Вот где красота и чистота была.
SZ
Спасибо парням, привили нюх к коду "с запашком"
ММ
Чуваки, я запушил в ktalks, кто хочет делать kotlinx.html разметку, гляньте. Там файл views/Index.kt в качестве примера