Size: a a a

Cicerone Chat (RUS)

2017 February 07

KT

Konstantin Tskhovrebov in Cicerone Chat (RUS)
когда погружаешься в 💩 - дна не найти!
😄
источник

IK

Igor Komarov in Cicerone Chat (RUS)
Да, спасибо, придерживаюсь концепции фрагментов просто потому, что со View и squareup/flow уже по граблям прошелся :) За ответ спасибо, возможно попробую на досуге
источник

VC

Vasili Chyrvon in Cicerone Chat (RUS)
Konstantin Tskhovrebov
но создает этот бандлд сам фрагмент
+1
Для того newInstance() у фрагментов и делается, чтобы в нем создать Bundle и сложить в него те данные, которые передали в параметрах (любого типа).
Плюс сам фрагмент лучше знает, что из этих данных ему нужно, зачем ему все? Иногда нафиг не нужны.
источник

KT

Konstantin Tskhovrebov in Cicerone Chat (RUS)
Alexey Pushkarev
кстати, ребят. Я вот по поводу аргументов фрагмента. Если та мнадо передавать более 1-го параметра, то я завожу статик метод во фрагменте, который принимает все аргумент и кладет их в Bundle и возвращает Bundle. В роутер уже в navigateTo(SCREEN_NAME, Fragment.args(....)) и в навигаторе кастую к бандлу
Хозяин - барин
источник

VC

Vasili Chyrvon in Cicerone Chat (RUS)
Alexey Pushkarev
кстати, ребят. Я вот по поводу аргументов фрагмента. Если та мнадо передавать более 1-го параметра, то я завожу статик метод во фрагменте, который принимает все аргумент и кладет их в Bundle и возвращает Bundle. В роутер уже в navigateTo(SCREEN_NAME, Fragment.args(....)) и в навигаторе кастую к бандлу
😕 А чем newInstance() как рекомендуют в доке плох?
И чтобы он принимал тот Object что приходит команде?
источник

AP

Alexey Pushkarev in Cicerone Chat (RUS)
Vasili Chyrvon
😕 А чем newInstance() как рекомендуют в доке плох?
И чтобы он принимал тот Object что приходит команде?
тогда мне придется за пределами newInstance решать что за Object будет
источник

AP

Alexey Pushkarev in Cicerone Chat (RUS)
т.е. писать один и тот же код по формированию аргументов везде, где используется фрагмент
источник

AP

Alexey Pushkarev in Cicerone Chat (RUS)
Konstantin Tskhovrebov
Хозяин - барин
я к тому, что как Вам такое решение, видите ли какие-то недостатки?
источник

VC

Vasili Chyrvon in Cicerone Chat (RUS)
Alexey Pushkarev
т.е. писать один и тот же код по формированию аргументов везде, где используется фрагмент
Зачем писать один и тот же код? Вы передаете какой-то объект при навигации. Его берет метод newInstance и все.

Вы о чем?
источник

VC

Vasili Chyrvon in Cicerone Chat (RUS)
Alexey Pushkarev
я к тому, что как Вам такое решение, видите ли какие-то недостатки?
Мне лично кажется излишне запутанным.
источник

AP

Alexey Pushkarev in Cicerone Chat (RUS)
Vasili Chyrvon
Зачем писать один и тот же код? Вы передаете какой-то объект при навигации. Его берет метод newInstance и все.

Вы о чем?
Сейчас, я Вам покажу пример кода чтобы было понятнее о чём я.
источник

AP

Alexey Pushkarev in Cicerone Chat (RUS)
5 мин
источник

KT

Konstantin Tskhovrebov in Cicerone Chat (RUS)
недостаток в том, что в этот бандл можно добавить данные, а еще и перетереть
источник

VC

Vasili Chyrvon in Cicerone Chat (RUS)
Только наверное лучше используя telegra.ph
источник

KT

Konstantin Tskhovrebov in Cicerone Chat (RUS)
гист
источник

VC

Vasili Chyrvon in Cicerone Chat (RUS)
+
источник

AP

Alexey Pushkarev in Cicerone Chat (RUS)
источник

AP

Alexey Pushkarev in Cicerone Chat (RUS)
статик метод args нужен для того чтобы во всех местах наподобии openEditInfoScreen не приходилось писать какие-то складывания аргументов
источник

AP

Alexey Pushkarev in Cicerone Chat (RUS)
в бандл или ещё куда-то
источник

KT

Konstantin Tskhovrebov in Cicerone Chat (RUS)
норм вариант.
1) только завязан на фрагмент (а входные параметры обычно нужны презентеру, а не вьюхе)
2) есть опасность передать в newInstance не тот бандл (поэтому класс с параметрами лучше)
3) имена методов не говорящие (это уже к стилю кода)
источник