MA
потом подписываю и обрабатываю данные
Size: a a a
2017 July 13
MA
А при переходе в другой фрагмент отписываюсь в onStop
КП
С listNotes сложно получилось
Лучше его тогда вообще создавать внутри функции getListNotes
Лучше его тогда вообще создавать внутри функции getListNotes
Обновил. Посмотрите пожалуйста.)))
MA
И ещё такой вопрос, когда я меняю ориентацию экрана, то почему то пока данные грузятся чернеет экран, хотя до этого не было, с чем это связанно и как это исправитью?
DV
Обновил. Посмотрите пожалуйста.)))
Улучшать, так улучшать
Советы из области Best Practice:
1) mDataBase - лучше бы, чтобы он приходил снаружи, а не создавался в конструкторе (паттерн Dependency Injection)
2) я бы разделил этот класс на два разных: один - только работа с БД, второй - просто RX-обертка для первого (single responsibility principle)
Советы из области Best Practice:
1) mDataBase - лучше бы, чтобы он приходил снаружи, а не создавался в конструкторе (паттерн Dependency Injection)
2) я бы разделил этот класс на два разных: один - только работа с БД, второй - просто RX-обертка для первого (single responsibility principle)
КП
Улучшать, так улучшать
Советы из области Best Practice:
1) mDataBase - лучше бы, чтобы он приходил снаружи, а не создавался в конструкторе (паттерн Dependency Injection)
2) я бы разделил этот класс на два разных: один - только работа с БД, второй - просто RX-обертка для первого (single responsibility principle)
Советы из области Best Practice:
1) mDataBase - лучше бы, чтобы он приходил снаружи, а не создавался в конструкторе (паттерн Dependency Injection)
2) я бы разделил этот класс на два разных: один - только работа с БД, второй - просто RX-обертка для первого (single responsibility principle)
Значит будем улучшать.
Несколько вопросов:
1) Я так понимаю mDataBase должен создаваться в том месте, откуда будет вызываться класс который работает с БД и передаваться ему?
2) Класс который будет работать с БД может содержать все основные операции с данными (вставка, чтение, обновление, удаление) или их тоже лучше разделить?
Несколько вопросов:
1) Я так понимаю mDataBase должен создаваться в том месте, откуда будет вызываться класс который работает с БД и передаваться ему?
2) Класс который будет работать с БД может содержать все основные операции с данными (вставка, чтение, обновление, удаление) или их тоже лучше разделить?
DV
1) Все эти объекты в идеале должны создаваться с помощью Dagger 2
КП
Улучшать, так улучшать
Советы из области Best Practice:
1) mDataBase - лучше бы, чтобы он приходил снаружи, а не создавался в конструкторе (паттерн Dependency Injection)
2) я бы разделил этот класс на два разных: один - только работа с БД, второй - просто RX-обертка для первого (single responsibility principle)
Советы из области Best Practice:
1) mDataBase - лучше бы, чтобы он приходил снаружи, а не создавался в конструкторе (паттерн Dependency Injection)
2) я бы разделил этот класс на два разных: один - только работа с БД, второй - просто RX-обертка для первого (single responsibility principle)
Настало время для курса Dagger 2)))
DV
2) Не, по методам разделять не надо. Все методы работы с БД пусть будут в одном классе
DV
Maxim Antipov
try {
subscription = Observable
.just(Global.parsers.getCatalog(internet.getURLStringRX(query)))
.subscribeOn(Schedulers.io())
.observeOn(AndroidSchedulers.mainThread())
.subscribe(
catalogClasses -> {arrayList = catalogClasses; FullCatalog(catalogClasses);},
throwable -> {Log.d("RXJAVA", "Throwable = " + throwable);});
} catch (XmlPullParserException e) {
e.printStackTrace();
} catch (IOException e) {
e.printStackTrace();
}
subscription = Observable
.just(Global.parsers.getCatalog(internet.getURLStringRX(query)))
.subscribeOn(Schedulers.io())
.observeOn(AndroidSchedulers.mainThread())
.subscribe(
catalogClasses -> {arrayList = catalogClasses; FullCatalog(catalogClasses);},
throwable -> {Log.d("RXJAVA", "Throwable = " + throwable);});
} catch (XmlPullParserException e) {
e.printStackTrace();
} catch (IOException e) {
e.printStackTrace();
}
А зачем здесь try catch? Ошибки ведь придут в throwable, который в subscribe
КП
1) Все эти объекты в идеале должны создаваться с помощью Dagger 2
Dagger2 📖🏃😀. Большое Вам спасибо за советы.
MA
А зачем здесь try catch? Ошибки ведь придут в throwable, который в subscribe
Не придут, это для xml
MA
Даже студио говорит, что не придут и обязывает поставить try
MA
Но это не самое главное, почему экран чернеет при смене ориентации, когда загрузка идёт?
DV
а да, just не оборачивает ошибку
лучше юзать fromCallable, чтобы ошибка в Rx обработалась
по экрану трудно сказать, надо видеть весь код, который с этим связан
лучше юзать fromCallable, чтобы ошибка в Rx обработалась
по экрану трудно сказать, надо видеть весь код, который с этим связан
MA
Весь код скинуть не могу
MA
Не мой
MA
Дело в том что, обычный фрагмент с кнопками и viewpager
MA
Всё
MA
Что не так?