Dagger 2. В каких случаях происходит пересоздание объекта @Singleton?

Question

[Сергей Семенко]

AuthModule.class

@Module
class AuthModule
{
    @Provides
    @Singleton
    AuthManager provideAuthManager(Context context) {
        return new AuthManager(context);
    }
}

AppComponent.class:

@Singleton
@Component(modules = {
        AppModule.class,
        DatabaseModule.class,
        AuthModule.class,
})
public interface AppComponent
{
    void inject(MainActivity activity);
}

Application.class:

public class Application extends android.app.Application
{
    private static AppComponent mAppComponent;

    @Override
    public void onCreate() {
        super.onCreate();
        setup();
    }

    public static AppComponent getAppComponent() {
        return mAppComponent;
    }

    private void setup() {
        mAppComponent = DaggerAppComponent.builder()
                .appModule(new AppModule(this))
                .build();
    }
}

BaseActivity.class:

public class BaseActivity extends AppCompatActivity
{
    @Override
    protected void onCreate(@Nullable Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        onInjectDependency();
    }

    public void onInjectDependency() {
        //
    }
}

MainActivity.class:

public class MainActivity extends BaseActivity
{
    @Inject AuthManager authManager;
    private User user;

    @Override
    protected void onCreate(@Nullable Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        user = authManager.getCurrentUser();
    }

    @Override
    public void onInjectDependency() {
        Application.getAppComponent().inject(this);
    }
}

Пару раз возникала ситуация, в которой происходило пересоздание AuthManager, при чём приложение было активно (видно пользователю). Для моего приложения это очень плохо, так как для доступа к данным авторизации пользователь вводит пин-код (при старте приложения).

С этим можно как-то бороться или нужно рассчитывать на пересоздание?

Answer 1

[Денис Загаевский]

Нужен код, когда вы создаёте компонент и когда инжектите активити. Если он пересоздаётся в такой ситуации - значит вы инжектите несколько раз, при этом используя разные инстансы компонента.

Comments

[Сергей Семенко]

Обновил вопрос.

Я сейчас не уверен, но возможно пересоздание произошло во фрагменте. Сейчас пытаюсь ещё раз словить такой момент.

P.S. Во фрагментах такая же схема внедрения.

[Денис Загаевский]

Сергей Семенко: по этому простому коду всё ок. Значит где-то в вашем коде что-то не так. А зачем компонент статической переменной объявлен?

[Сергей Семенко]

Денис Загаевский: чтобы можно было использовать его: Application.getAppComponent()

[Денис Загаевский]

Сергей Семенко: но ведь через него у вас в статику активити утекает, скорее всего.
((Application)Context.getApplicationContext()).getAppComponent();

[Сергей Семенко]

Денис Загаевский: я подсмотрел вот здесь: https://github.com/sockeqwe/mosby/blob/master/samp...

[Сергей Семенко]

Денис Загаевский: такое ведь допустимо?

public abstract class BaseActivity extends AppCompatActivity
{
    @Override
    protected void onCreate(@Nullable Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        AppComponent component = ((Application) getApplicationContext()).getAppComponent();
        onInjectDependency(component);
    }

    public void onInjectDependency(AppComponent component) {
        //
    }
}

[Денис Загаевский]

Сергей Семенко: Ханнес
1) не передаёт туда контекст
2) использует LeakCanary
Не всё можно "подсматривать" у "крутых ребят" бездумно.
Пока писал всё это, понял, что по факту у вас там аппликейшн. Ну ок. От этого один шаг до активити, пока она у вас не течёт, но вы можете чуть поправить код - и она потечёт.

[Денис Загаевский]

Сергей Семенко: допустимо

[Денис Загаевский]

Сергей Семенко: проблема-то решилась?

[Сергей Семенко]

Денис Загаевский: спасибо за ответ

[Денис Загаевский]

Сергей Семенко: ¯\_(ツ)_/¯ не за что

[Сергей Семенко]

Денис Загаевский: я не понял, от чего это происходил, но пока вроде всё нормально. Могу отписать, если снова будут проблемы :)

Answer 2

[Дмитрий Серков]

Может так:

private void setup() {
        mAppComponent = DaggerAppComponent.builder()
                .appModule(new AppModule(this))
                .authModule(new AuthModule(this)) //добавить
                .build();
    }

Comments

[Сергей Семенко]

А зачем? Dagger сам создает экземпляр, и туда по идее не нужно передавать context, он ведь сам инжектится.

[Сергей Семенко]

Насколько я понял, вручную создавать модули нужно только со специфичными конструкторами, например, у AppModule конструктор принимает Context.

[Дмитрий Серков]

Сергей Семенко: не доглядел
Сергей Семенко: не доглядел:
@Module
class AuthModule
{
private MainActivity activity;

public AuthModule(MainActivity activity){
this.activity = activity;
}
@Provides
Context provideContext() {
return activity;
}

@Provides
@Singleton
AuthManager provideAuthManager(Context context) {
return new AuthManager(context);
}
}