【问题标题】:RuntimeException with Dagger 2 on Android 7.0 and Samsung devices在 Android 7.0 和三星设备上出现 Dagger 2 的 RuntimeException
【发布时间】:2018-03-28 19:45:01
【问题描述】:

在我的 Google Play 控制台上,自从我开始使用 Dagger 2 以来,我看到了很多崩溃报告,但仅限于 Android 7.0 并且主要在三星设备、一些华为和摩托罗拉设备以及一些罕见的 Xperia 设备上:

java.lang.RuntimeException: 
  at android.app.ActivityThread.performLaunchActivity (ActivityThread.java:2984)
  at android.app.ActivityThread.handleLaunchActivity (ActivityThread.java:3045)
  at android.app.ActivityThread.-wrap14 (ActivityThread.java)
  at android.app.ActivityThread$H.handleMessage (ActivityThread.java:1642)
  at android.os.Handler.dispatchMessage (Handler.java:102)
  at android.os.Looper.loop (Looper.java:154)
  at android.app.ActivityThread.main (ActivityThread.java:6776)
  at java.lang.reflect.Method.invoke (Method.java)
  at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run (ZygoteInit.java:1518)
  at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:1408)
Caused by: java.lang.RuntimeException: 
  at dagger.android.AndroidInjection.inject (AndroidInjection.java:48)
  at dagger.android.support.DaggerAppCompatActivity.onCreate (DaggerAppCompatActivity.java:43)
  at com.package.MainActivity.onCreate (MainActivity.java:83)
  at android.app.Activity.performCreate (Activity.java:6956)
  at android.app.Instrumentation.callActivityOnCreate (Instrumentation.java:1126)
  at android.app.ActivityThread.performLaunchActivity (ActivityThread.java:2927)

我无法重现该问题,因为我手头没有任何受影响的设备,而且似乎并非某种类型的所有设备都受到影响,更像是随机启动失败。

从我通过研究了解到的情况是,活动的 onCreate 很可能在活动实际附加到应用程序之前被调用。但我无法证明这个说法......

我正在遵循 Google 的 MVP+Dagger 架构蓝图。

我的应用程序类:

public class App extends DaggerApplication {

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

    @Override
    protected AndroidInjector<? extends DaggerApplication> applicationInjector() {
        AppComponent appComponent = DaggerAppComponent.builder().application(this).build();
        appComponent.inject(this);
        return appComponent;
    }

}

我的 MainActivity 类:

public class MainActivity extends DaggerAppCompatActivity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
    }

}

匕首2相关代码:

DaggerAppCompatActivity: https://github.com/google/dagger/blob/e8d7cd4c29c1316c5bb1cf0737d4f29111fcb1c8/java/dagger/android/support/DaggerAppCompatActivity.java#L42-L45

protected void onCreate(@Nullable Bundle savedInstanceState) { 
    AndroidInjection.inject(this); 
    super.onCreate(savedInstanceState); 
}

Android注入: https://github.com/google/dagger/blob/e8d7cd4c29c1316c5bb1cf0737d4f29111fcb1c8/java/dagger/android/AndroidInjection.java#L43-L52

public static void inject(Activity activity) { 
    checkNotNull(activity, "activity"); 
    Application application = activity.getApplication(); 
    if (!(application instanceof HasActivityInjector)) { 
        throw new RuntimeException( 
            String.format( 
                "%s does not implement %s", 
                application.getClass().getCanonicalName(), 
                HasActivityInjector.class.getCanonicalName())); 
    }

我不知道如何解决这个崩溃,但是崩溃的数量太多了,不容忽视。由于我的 Dagger 2 使用在所有其他 Android 版本和设备上都能完美运行,我认为这不是由我使用 Dagger 2 的方式引起的,而是由某些供应商特定的 7.0 实现引起的。如果有人遇到同样的问题并找到解决方案,请帮帮我!

由于这个错误让我发疯,我向 10 万用户推出了一个测试版本,试图了解整个事情的问题所在。

public abstract class TestDaggerAppCompatActivity extends AppCompatActivity implements HasFragmentInjector, HasSupportFragmentInjector {

    @Inject DispatchingAndroidInjector<Fragment> supportFragmentInjector;
    @Inject DispatchingAndroidInjector<android.app.Fragment> frameworkFragmentInjector;

    @Override
    protected void onCreate(@Nullable Bundle savedInstanceState) {
        inject();
        super.onCreate(savedInstanceState);
    }

    @Override
    public AndroidInjector<Fragment> supportFragmentInjector() {
        return supportFragmentInjector;
    }

    @Override
    public AndroidInjector<android.app.Fragment> fragmentInjector() {
        return frameworkFragmentInjector;
    }

    private void inject() {
        Application application = getApplication();

        if(application == null) {
            injectWithNullApplication();
            return;
        }

        if (!(application instanceof HasActivityInjector)) {
            injectWithWrongApplication();
            return;
        }

        // Everything seems ok...
        injectNow(application);
    }

    private void injectWithNullApplication() {
        Application application = (Application) getApplicationContext();
        injectNow(application);
    }

    private void injectWithWrongApplication() {
        Application application = (Application) getApplicationContext();
        injectNow(application);
    }

    private void injectNow(Application application) {
        checkNotNull(application, "Application must not be null");

        if (!(application instanceof HasActivityInjector)) {
            throw new RuntimeException(String.format("%s does not implement %s", application.getClass().getCanonicalName(), HasActivityInjector.class.getCanonicalName()));
        }

        AndroidInjector<Activity> activityInjector = ((HasActivityInjector) application).activityInjector();
        checkNotNull(activityInjector, "%s.activityInjector() returned null", application.getClass().getCanonicalName());

        activityInjector.inject(this);
    }

}

该活动基于 Dagger 的活动,内嵌 AndroidInjection 代码。我的想法是,如果这个问题不能通过使用 ApplicationContext 而不是 getApplication() 来解决,我的堆栈跟踪应该详细说明发生了什么:

  • 如果问题是由getApplication() 引起的,堆栈跟踪将包含injectWithNullApplication()injectWithWrongApplication()
  • 抛出的 NPE 将显示 getApplicationContext() 返回 null
  • 抛出的 RuntimeException 将表明 getApplicationContext() 不是我的应用程序
  • 如果不抛出异常,getApplication()getApplicationContext() 将返回我的应用程序,我不会关心实际解决问题的方法

这是堆栈跟踪:

Caused by: java.lang.RuntimeException: 
  at com.package.di.TestDaggerAppCompatActivity.inject (TestDaggerAppCompatActivity.java:49)
  at com.package.di.TestDaggerAppCompatActivity.onCreate (TestDaggerAppCompatActivity.java:31)
  at com.package.MainActivity.onCreate (MainActivity.java:83)
  at android.app.Activity.performCreate (Activity.java:6942)
  at android.app.Instrumentation.callActivityOnCreate (Instrumentation.java:1126)
  at android.app.ActivityThread.performLaunchActivity (ActivityThread.java:2880)

所以 inject() 中的 if 子句 !(application instanceof HasActivityInjector) 没有重新路由到 injectWithWrongApplication() 但相同的 if 子句在同一 Application 实例上导致 injectNow(Application application) 中的 RuntimeException。怎么回事?我的代码看起来像 100 次,但如果我有错误,请告诉我!否则,我想在 7.0 的一些供应商实现中会发生一些非常奇怪的事情,这些事情可能无法修复......

基于对https://github.com/google/dagger/issues/748的讨论,我还推出了一个测试版本,它在所有Dagger组件中只使用getApplicationContext()而不是getApplication(),没有任何区别。

清单中的我的应用程序标记

<application
    android:name=".App"
    android:allowBackup="true"
    android:icon="@mipmap/ic_launcher"
    android:label="@string/app_name"
    android:theme="@style/SplashScreenTheme"
    android:fullBackupContent="false">

    <meta-data android:name="com.google.android.gms.version" android:value="@integer/google_play_services_version" />
    <meta-data android:name="com.google.android.gms.games.APP_ID" android:value="@string/app_id" />

    <meta-data android:name="android.max_aspect" android:value="2.1" />

    <activity
        android:name="com.package.MainActivity"
        android:label="@string/app_name">
        <intent-filter>
            <action android:name="android.intent.action.MAIN" />
            <category android:name="android.intent.category.LAUNCHER" />
        </intent-filter>
    </activity>

    <service android:name="com.package.GeneratorService" android:exported="false"/>
</application>

【问题讨论】:

  • 目前我们的应用程序中也发生了同样的事情。看起来我们的自定义 Application 类根本没有被调用(即使它在清单中指定),这可能意味着使用默认的 Application 类,这反过来意味着强制转换将失败。不过仍在寻找解决方案。
  • 我不确定我是否有答案(还),但我确实注意到DaggerApplication#onCreate 在调用applicationInjector() 之后在你的应用程序组件上调用inject,所以你可能没有'不需要自己这样做。还有,你用multidex吗,能不能把Application的动态类型记录下来,来证实上面Dmitry的理论?
  • 杰夫,我试图获取应用程序的类型,但 Play 管理中心没有收集异常消息,并且发生崩溃时 Firebase 尚未初始化。
  • 好的,请看更新3。

标签: android dagger-2 samsung-mobile android-7.0-nougat


【解决方案1】:

最后我找到了一种方法来解决我的应用程序在 Android 7.0 下使用 Dagger 2 导致的崩溃。请注意,这并不能解决自定义应用程序在 Android 7.0 下无法正常使用的问题。就我而言,除了实现 Dagger 2 之外,我的自定义应用程序中没有重要的逻辑,所以我只是用下面的 ApplicationlessInjection 替换了基于 DaggerApplication 的实现。

已知问题

  • 在自定义应用程序类中不进行依赖注入(无论如何,这对于可怕的 Android 7.0 OEM 实施来说可能不是一个好主意)
  • 并非所有 Dagger 组件都由我修改,我只替换了 DaggerAppCompatActivityDaggerIntentServiceDaggerFragment。如果您使用其他组件,例如 DaggerDialogFragmentDaggerBroadcastReceiver,您需要创建自己的工具,但我想这应该不会太难:)

实施

停止使用DaggerApplication。要么从标准 Application 再次扩展您的自定义应用程序,要么完全摆脱自定义应用程序。对于 Dagger 2 的依赖注入,它不再需要了。只需扩展例如FixedDaggerAppCompatActivity,你很高兴与 Dagger 2 DI 一起参加活动。

您可能注意到我仍在将应用程序上下文传递给ApplicationlessInjection.getInstance()。依赖注入本身根本不需要上下文,但我希望能够轻松地将应用程序上下文注入到我的其他组件和模块中。而且我不在乎应用程序上下文是我的自定义应用程序还是 Android 7.0 中的其他一些疯狂的东西,只要它是一个上下文即可。

无应用程序注入

public class ApplicationlessInjection
        implements
            HasActivityInjector,
            HasFragmentInjector,
            HasSupportFragmentInjector,
            HasServiceInjector,
            HasBroadcastReceiverInjector,
            HasContentProviderInjector {

    private static ApplicationlessInjection instance = null;

    @Inject DispatchingAndroidInjector<Activity> activityInjector;
    @Inject DispatchingAndroidInjector<BroadcastReceiver> broadcastReceiverInjector;
    @Inject DispatchingAndroidInjector<android.app.Fragment> fragmentInjector;
    @Inject DispatchingAndroidInjector<Fragment> supportFragmentInjector;
    @Inject DispatchingAndroidInjector<Service> serviceInjector;
    @Inject DispatchingAndroidInjector<ContentProvider> contentProviderInjector;

    public ApplicationlessInjection(Context applicationContext) {
        AppComponent appComponent = DaggerAppComponent.builder().context(applicationContext).build();
        appComponent.inject(this);
    }

    @Override
    public DispatchingAndroidInjector<Activity> activityInjector() {
        return activityInjector;
    }

    @Override
    public DispatchingAndroidInjector<android.app.Fragment> fragmentInjector() {
        return fragmentInjector;
    }

    @Override
    public DispatchingAndroidInjector<Fragment> supportFragmentInjector() {
        return supportFragmentInjector;
    }

    @Override
    public DispatchingAndroidInjector<BroadcastReceiver> broadcastReceiverInjector() {
        return broadcastReceiverInjector;
    }

    @Override
    public DispatchingAndroidInjector<Service> serviceInjector() {
        return serviceInjector;
    }

    @Override
    public AndroidInjector<ContentProvider> contentProviderInjector() {
        return contentProviderInjector;
    }

    public static ApplicationlessInjection getInstance(Context applicationContext) {
        if(instance == null) {
            synchronized(ApplicationlessInjection.class) {
                if (instance == null) {
                    instance = new ApplicationlessInjection(applicationContext);
                }
            }
        }

        return instance;
    }

}

FixedDaggerAppCompatActivity

public abstract class FixedDaggerAppCompatActivity extends AppCompatActivity implements HasFragmentInjector, HasSupportFragmentInjector {

    @Inject DispatchingAndroidInjector<Fragment> supportFragmentInjector;
    @Inject DispatchingAndroidInjector<android.app.Fragment> frameworkFragmentInjector;

    @Override
    protected void onCreate(@Nullable Bundle savedInstanceState) {
        inject();
        super.onCreate(savedInstanceState);
    }

    @Override
    public AndroidInjector<Fragment> supportFragmentInjector() {
        return supportFragmentInjector;
    }

    @Override
    public AndroidInjector<android.app.Fragment> fragmentInjector() {
        return frameworkFragmentInjector;
    }

    private void inject() {
        ApplicationlessInjection injection = ApplicationlessInjection.getInstance(getApplicationContext());

        AndroidInjector<Activity> activityInjector = injection.activityInjector();

        if (activityInjector == null) {
            throw new NullPointerException("ApplicationlessInjection.activityInjector() returned null");
        }

        activityInjector.inject(this);
    }

}

FixedDaggerFragment

public abstract class FixedDaggerFragment extends Fragment implements HasSupportFragmentInjector {

    @Inject DispatchingAndroidInjector<Fragment> childFragmentInjector;

    @Override
    public void onAttach(Context context) {
        inject();
        super.onAttach(context);
    }

    @Override
    public AndroidInjector<Fragment> supportFragmentInjector() {
        return childFragmentInjector;
    }


    public void inject() {
        HasSupportFragmentInjector hasSupportFragmentInjector = findHasFragmentInjector();

        AndroidInjector<Fragment> fragmentInjector = hasSupportFragmentInjector.supportFragmentInjector();

        if (fragmentInjector == null) {
            throw new NullPointerException(String.format("%s.supportFragmentInjector() returned null", hasSupportFragmentInjector.getClass().getCanonicalName()));
        }

        fragmentInjector.inject(this);
    }

    private HasSupportFragmentInjector findHasFragmentInjector() {
        Fragment parentFragment = this;

        while ((parentFragment = parentFragment.getParentFragment()) != null) {
            if (parentFragment instanceof HasSupportFragmentInjector) {
                return (HasSupportFragmentInjector) parentFragment;
            }
        }

        Activity activity = getActivity();

        if (activity instanceof HasSupportFragmentInjector) {
            return (HasSupportFragmentInjector) activity;
        }

        ApplicationlessInjection injection = ApplicationlessInjection.getInstance(activity.getApplicationContext());
        if (injection != null) {
            return injection;
        }

        throw new IllegalArgumentException(String.format("No injector was found for %s", getClass().getCanonicalName()));
    }

}

FixedDaggerIntentService

public abstract class FixedDaggerIntentService extends IntentService {

    public FixedDaggerIntentService(String name) {
        super(name);
    }

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

    private void inject() {
        ApplicationlessInjection injection = ApplicationlessInjection.getInstance(getApplicationContext());

        AndroidInjector<Service> serviceInjector = injection.serviceInjector();

        if (serviceInjector == null) {
            throw new NullPointerException("ApplicationlessInjection.serviceInjector() returned null");
        }

        serviceInjector.inject(this);
    }

}

我的应用组件

@Singleton
@Component(modules = {
        AppModule.class,
        ActivityBindingModule.class,
        AndroidSupportInjectionModule.class
})
public interface AppComponent extends AndroidInjector<ApplicationlessInjection> {

    @Override
    void inject(ApplicationlessInjection instance);

    @Component.Builder
    interface Builder {

        @BindsInstance
        AppComponent.Builder context(Context applicationContext);

        AppComponent build();

    }

}

我的应用模块

@Module
public abstract class AppModule {

    @Binds
    @ApplicationContext
    abstract Context bindContext(Context applicationContext);

}

为了完整起见,我的 @ApplicationContext 注释

@Qualifier
@Retention(RetentionPolicy.RUNTIME)
public @interface ApplicationContext {}

希望我也可以用我的代码帮助其他人。对我来说,我可以解决与引入 Dagger 2 和奇怪的 Android 7.0 版本相关的所有崩溃问题。

如果需要更多说明,请告诉我!

【讨论】:

  • 你能告诉我这与 Dagger 注射的正常流程有何不同吗?你的方法解决了什么问题,默认的 dagger 实现缺少什么?
  • @azizbekian 这完全解决了我的问题。有一些特别是来自三星的 Android 7.0 实现似乎随机忽略了自定义应用程序。如果您使用 Dagger 2 的 android 包中的 DaggerApplication,您将面临一些 Android 7.0 用户无法再启动您的应用程序的情况,因为无论出于何种原因,扩展的 DaggerApplication 使用,因此会破坏整个 DI 实现和应用程序。
  • 除了getApplicationContext() 的实例没有返回正确的子类ApplicationContext 我还在我们的生产环境中看到getApplicationContext() 的实例返回null,根据您的上述实现会引发异常。有什么理由不能用ApplicationlessInjection injection = ApplicationlessInjection.getInstance(someActivityContext); 替换ApplicationlessInjection 注入ApplicationlessInjection.getInstance(getApplicationContext());。似乎您的实现所需要的只是任何上下文,这应该就足够了。
  • 我遇到了同样的问题(几个月以来),但我没有扩展 DaggerApplication。我的自定义应用程序只是实现了HasActivityInjector。而且我没有从DaggerAppCompatActivityDaggerFragment 扩展,我只是使用静态方法Android(Support)Injection.inject。所以我不太确定,你的(有据可查的)解决方案如何适用于我,你能给我一个提示吗?谢谢!
  • @chrjs 无论您使用 DaggerApplication 还是您自己的自定义应用程序都没有关系,因为该应用程序根本没有被使用/替换/忽略。我上次回复中的代码可能更适合您的情况,因为它根本不需要任何上下文。你需要用我在FixedDaggerAppCompatActivity / FixedFragment .inject方法中的代码替换Android(Support)Injection.inject调用
【解决方案2】:

我在我的应用程序中遇到了同样的问题,我使用以下代码解决了它:

Application app = activity.getApplication();
if(app == null) {
     app = (Application)activity.getApplicationContext();
}

【讨论】:

  • 有趣的是,这与问题有何关联。我并不是说它是相关的,但至少我不知道它可能是如何相关的。
  • 我尝试使用应用程序上下文而不是 getApplication() 在 Android 7.0 上没有任何区别,正如我的问题中所发布的那样。
  • 这并不能真正解决我们的问题,因为我们还在自定义 Application 中运行了一些代码,现在只是没有被调用。
猜你喜欢
  • 1970-01-01
  • 2018-01-16
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2013-05-26
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多