KT
Да, задача рутера реализовывать навигацию через команды. Он не знает про экраны
экраны - сущность конкретного приложения.
роутер - общая часть навигации без привязки к конкретике
Size: a a a
2017 February 10
DC
критика запутанности кода)
навигатор делегирует вызовы внутреннему, а внутренний навигатор делигирует вызовы своему холдеру...
я бы сделал (а точнее уже сделал) навигатор, который наследуется от стандартного, но на эпплайКоманд проверяет, надо ли вызвать супер или надо переключить активити.
так проще и понятнее.
а вообще молодец)
навигатор делегирует вызовы внутреннему, а внутренний навигатор делигирует вызовы своему холдеру...
я бы сделал (а точнее уже сделал) навигатор, который наследуется от стандартного, но на эпплайКоманд проверяет, надо ли вызвать супер или надо переключить активити.
так проще и понятнее.
а вообще молодец)
Да, чет с внутренним классом я зря замутился, переделал в наследование. Стало более читаемо
https://gist.github.com/dened/386e48960d94f3d2599ec61068f21bad
https://gist.github.com/dened/386e48960d94f3d2599ec61068f21bad
KT
вот. уже лучше. я бы такой код не принял от своих ребят, но работать будет)
DC
о как, а какие замечания?
DC
можно фидбек, был бы очень благодарен
KT
protected AppCompatActivity activity; - зачем протектед?
if(!handleCommand(command)) { - логичнее проверку на создание интента, а не на handleCommand, которая внутри проверяет на intent != null (там вообще с линейностью этой проверки беда)
—----------
думаю здесь не стоит заниматься обучением :)
if(!handleCommand(command)) { - логичнее проверку на создание интента, а не на handleCommand, которая внутри проверяет на intent != null (там вообще с линейностью этой проверки беда)
—----------
думаю здесь не стоит заниматься обучением :)
DC
можно в личку?))
KT
так-то я работаю
KT
есть чатик с андоид болтовней - предложите там покритиковать )))
KT
я с радостью помогаю с конкретными вопросами - пишите
DC
ок ;)
KT
а не типа: "какие мысли?"
2017 February 14
DC
А кто-нибудь использовал DialogFragment'ы c чечероне? Как вы делали навигацию по ним?
KT
как по обычным фрагментам
KT
но помните, что не каждый диалог - это скрин
KT
чаще всего это часть вьюхи, а не отдельный экран
KT
поэтому он не относится к навигации
DC
а что надо сделать, чтобы диалог отображался как диалог? просто если я добавляю диалог через SupportFragmentNavigator, то он отображается как обычный фрагмент. А если делаю sameDialogFragment.show(fragmentManager, "tag"); то он отображется как диалог
DC
в общем, просто использовать SupportFragmentNavigator не получится, надо пилить свой))