Homework MVVM+DI+Network#23
Conversation
|
|
||
| @AndroidEntryPoint | ||
| class MainActivity : AppCompatActivity() { | ||
| @Inject lateinit var personFragment:PersonFragment |
There was a problem hiding this comment.
@RubaN0id, не уверен, что инжект фрагмента - это хорошая идея. Нужно помнить, что андроид сохраняет состояние вьюх при изменении конфигурации. В результате, у вас будет два фрагмента - один перевернутый, а второй - созданный даггером. Это можно проверить в онКриейт - посмотреть, нулл или не нулл пришел в функцию. Если не нулл, то запрос fragmentManager.findFragmentByTag вернет ненулевой результат. И это уже и есть ваш фрагмент
| } | ||
|
|
||
| addressEditText.addTextChangedListener { | ||
| if(it != null && it.length > 5) { |
There was a problem hiding this comment.
@RubaN0id, мне кажется, стоит сделать проверку в модели. Допустим, вы захотите протестировать, что запросы не посылаются до 5 символов - вам придется делать андроид-тесты, что геморно. Потом поменяли вьюху, и прицепили ее к новой модели. И забыли... В общем, мне кажется, такому место в модели
2 participants
No description provided.