Кинжал 2 - Инъектор конструктора - Неактивность - PullRequest
Новая
       11

Кинжал 2 - Инъектор конструктора - Неактивность

0 голосов
/ 19 сентября 2018

Только начал изучать Dagger 2, чтобы решить конкретную проблему: я пытаюсь следовать архитектуре MVVM, и у моего приложения есть класс репозитория, который извлекает и сохраняет настройки в классе CacheData, который в основном обертывает SharedPreferences.Однако SharedPreferences имеет контекстную зависимость.Поскольку я приложил все усилия, чтобы отделить свой репозиторий и слой данных от класса View и Application, передача контекста кажется шагом назад.

Вот класс репозитория 

public class ImgurRepository {
    private ImgurDatabase imgurDatabase;
    private ImgurGalleryDao galleryDao;

    private CompositeDisposable disposables;
    private LiveData<List<ImgurGallery>> imgurGalleries;



    public ImgurRepository(){
        this.imgurDatabase = ImgurDatabase.getInstance(MyRxApplication.getAppContext());
        this.galleryDao = imgurDatabase.getGalleryDao();
        disposables = new CompositeDisposable();
    }

    public String getCachedSearchTerm() {
        CachedData cachedData = new CachedData();
        return cachedData.getCachedSearchTerm();
    }

    public String getCachedSearchWindow(){
        CachedData cachedData = new CachedData();
        return cachedData.getCachedSearchWindow();
    }
    public String getCachedSearchType(){
        CachedData cachedData = new CachedData();
        return cachedData.getCachedSearchType();
    }

    public void setCachedSearchParams(@NonNull final String term,
                                    @NonNull final String type,
                                    @NonNull final String window) {
        CachedData cachedData = new CachedData();
        cachedData.setCachedSearchParams(term, type, window);
    }

    public LiveData<List<ImgurGallery>> getCachedGalleries() {
        return this.imgurGalleries;
    }

    public LiveData<List<ImgurGallery>> fetchGalleries(@NonNull final String searchType,
                                                              @NonNull final String searchWindow,
                                                              @NonNull final String searchTerm,
                                                              final int resultsPage){
        requestGalleries(searchType, searchWindow, searchTerm, resultsPage);
        return galleryDao.getAll();
    }

    private void requestGalleries(@NonNull final String searchType,
                               @NonNull final String searchWindow,
                               @NonNull final String searchTerm,
                               final int resultsPage) {
        Timber.d("Running fetchGalleries with arguments:\nsort='%s' \nwindow='%s'\nsearch='%s'\npage='%s'",
                searchType,
                searchWindow,
                searchTerm,
                resultsPage);
        ServiceGenerator.changeApiBaseUrl(IMGUR_API_BASE_URL);
        ImgurService service = ServiceGenerator.createService(ImgurService.class);
        Timber.d("finishing fetchGalleries request.");
        disposables.add(service.getSearchGallery(searchType,searchWindow,resultsPage,searchTerm)
                .subscribeOn(Schedulers.io())
                .observeOn(AndroidSchedulers.mainThread())
                .subscribe(new Consumer<Response<ImgurGalleryList>>() {
                    @Override
                    public void accept(@NonNull final Response<ImgurGalleryList> response) throws Exception {
                        Timber.d("Consumer is subscribed to imgurGalleryObservable.");
                        Timber.d(response.body().toString());
                        List<ImgurGallery> galleries = response.body().getData();
                        galleryDao.insertAll(galleries);
                    }

                }, new Consumer<Throwable>() {
                    @Override
                    public void accept(Throwable throwable) throws Exception {
                        Timber.e(throwable);
                    }
                }));
    }

    public void clearGalleries() {
        galleryDao.deleteAll();
    }

}

Вот класс fixed CachedData согласно комментариям Самуила: 

public class CachedData {
    private final String CACHED_SEARCH_TERM_KEY = "cached_search_term";
    private final String CACHED_SEARCH_WINDOW_KEY = "cached_search_window";
    private final String CACHED_SEARCH_TYPE_KEY = "cached_search_type";

    @Inject public SharedPreferences sharedPrefs;
    @Inject public Context context;

    public CachedData() {}

    public String getCachedSearchTerm() {
        return sharedPrefs.getString(CACHED_SEARCH_TERM_KEY, "");
    }
    public String getCachedSearchWindow(){
        return sharedPrefs.getString(CACHED_SEARCH_WINDOW_KEY, "");
    }
    public String getCachedSearchType(){
        return sharedPrefs.getString(CACHED_SEARCH_TYPE_KEY, "");
    }

    public void setCachedSearchParams(@Nullable final String term,
                                        @Nullable final String window,
                                        @Nullable final String type) {
        SharedPreferences.Editor editor = sharedPrefs.edit();
        if (term != null) editor.putString((CACHED_SEARCH_TERM_KEY), term);
        if (window  != null) editor.putString(CACHED_SEARCH_WINDOW_KEY, window);
        if (type != null) editor.putString(CACHED_SEARCH_TYPE_KEY, type);
        editor.apply();

    }

}

fixed Класс AppComponent: 

@Singleton
@Component(modules = AppModule.class)
public interface AppComponent {

    void inject(MyRxApplication app);
    void inject(CachedData cachedData);
    void inject(BaseActivity activity);
}

Класс AppModule: 

@Module
public class AppModule {
    private final String SHARED_PREFERENCE_KEY = "PREFERENCE_FILE_KEY";
    private final MyRxApplication application;

    public AppModule(MyRxApplication application) {
        this.application = application;
    }

    @Provides
    @Singleton
    public Context provideApplicationContext(){
        return application;
    }

    @Provides
    @Singleton
    public SharedPreferences provideSharedPreferences() {
        return application
                .getSharedPreferences(
                        SHARED_PREFERENCE_KEY,
                        Context.MODE_PRIVATE);
    }
}

Класс приложения: 

public class MyRxApplication extends Application {
    private static Context context;
    private AppComponent appComponent;

    public MyRxApplication() {
    }

    @Override
    public void onCreate() {
        super.onCreate();
        MyRxApplication.context = getApplicationContext();
        if (LeakCanary.isInAnalyzerProcess(this)) {
            // This process is dedicated to LeakCanary for heap analysis.
            // You should not init your app in this process.
            return;
        }
        LeakCanary.install(this);

        Stetho.initializeWithDefaults(this);

        if (BuildConfig.DEBUG) {
            Timber.plant(new Timber.DebugTree());
        }

        appComponent = buildComponent();
        appComponent.inject(this);
        appComponent.cachedData();

    }

    public AppComponent buildComponent(){
        return DaggerAppComponent
                .builder()
                .appModule(new AppModule(this))
                .build();
    }

    public AppComponent getAppComponent() {
        return appComponent;
    }

    public static Context getAppContext() {
        return MyRxApplication.context;
    }

}

Можно ли использовать Dagger для инъекции объекта SharedPreferences вконструктор CachedData?Я создаю много объектов CachedData, потому что изначально я не хотел использовать Singleton, но имеет ли смысл вместо этого просто вводить объект CachedData в качестве одиночного?Я действительно не уверен в правильном способе использования Dagger сейчас и заставить его работать, потому что каждый параметр, который я устанавливаю в конструкторе CacheData, должен быть указан при его создании, хотя я думал, что для этого использую Dagger ...?

1 Ответ

0 голосов
/ 20 сентября 2018

Жаль, что от вашего дополнительного комментария, насколько я понимаю, вы не хотите использовать инъекцию для своего класса CachedData и поэтому хотите создать экземпляр вручную.Если это так, вы можете удалить введенные вами аргументы конструктора и ввести ваши переменные напрямую следующим образом: 

@Inject
SharedPreferences sharedPrefs;
@Inject
Context context;

@Inject
public CachedData() {}
...