Android 7.0 and 7.1 getApplication() ClassCastException

Question

In Developer Console I see a lot of crashes with stacktrace like this

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:0)
  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(Native Method:0)
  at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1520)
  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1410)
Caused by: java.lang.ClassCastException: 
  at com.myapp.ui.BaseActivity.getApp(BaseActivity.java:193)
  at com.myapp.ui.BaseActivity.onCreate(BaseActivity.java:275)
  at com.myapp.ui.CastActivity.onCreate(CastActivity.java:39)
  at com.myapp.ui.MainActivity.onCreate(MainActivity.java:268)
  at android.app.Activity.performCreate(Activity.java:6955)
  at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1126)
  at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2927)

getApp method of BaseActivity is

public App getApp() {
        return (App) getApplication();
    }

App class is

public class App extends MultiDexApplication { ...

and in manifest application tag contains reference to this class

 <application
        android:name="com.myapp.App"

98% of crashes is for android 7.0, rest is 7.1. No other android versions are affected.

EDIT: I use proguard so it can be somehow related but keeping class

-keep class com.myapp.** { *;}
-keep interface com.myapp.** { *;}

Note: It may not be related but in same android versions it looks like App's onCreate method is sometimes not called. I observed it because some objects which are created in onCreate were null when they were accessed from Service (started by AlarmManager) or BroadcastReceiver

Does anyone has idea what can cause it, how to fix it or work around it? Thanks

EDIT 2: I ended up with something like this:

   public App getApp() {

    Application application = getApplication();
    App app = null;

    try {
        app = (App) application;
    } catch (Exception e) {
        if (application != null) {
            Log.e(TAG, "getApp Exception: application class: " + application.getClass().getName());
        } else {
            Log.e(TAG, "getApp Exception: application object is null");
        }
    }

    return app;
}

It at least doesn't crash and I can check getApp() == null

how common is this crash? there are always weird rare crashes that are vendor/rom specific. — meh, Oct 10 '17 at 22:38

Jsyntax · Answer 1 · 2017-10-24T19:55:00.093

13

Casting fails because getApplication() returns an Application and NOT the desired sub-class.

I've had some success where I caught the error and asked the user to reboot their device or reinstall the app.

Unfortunately, there's no real fix to this rare crash. Google won't fix the lifecycle-related issue, but said it reduced in Android 7.1+. Source: https://issuetracker.google.com/issues/37137009

edited Oct 24 '17 at 19:55

answered Oct 10 '17 at 22:22

Jsyntax

121
1
7

This does not provide an answer to the question. Once you have sufficient [reputation](https://stackoverflow.com/help/whats-reputation) you will be able to [comment on any post](https://stackoverflow.com/help/privileges/comment); instead, [provide answers that don't require clarification from the asker](https://meta.stackexchange.com/questions/214173/why-do-i-need-50-reputation-to-comment-what-can-i-do-instead). - [From Review](/review/low-quality-posts/17588120) – lebelinoz Oct 11 '17 at 00:13
1

I think my newly edited answer provides closure to the OP's question. – Jsyntax Oct 24 '17 at 19:56
Agreed. +1 from me. – lebelinoz Oct 24 '17 at 22:20

score 1 · Answer 2 · answered Jun 30 '17 at 07:24

1

I think you should cast getApplicationContext() into App instead.

answered Jun 30 '17 at 07:24

Tuby

3,158
2
17
36

Can you explain why it should help please? – Martin Vandzura Jun 30 '17 at 07:30
It is an alternative, sometimes when programming for android I experienced that minor changes like that can get rid of an exception. – Tuby Jun 30 '17 at 07:47
problem is that it may not return application object in some cases (ie vendor implementation of android) https://stackoverflow.com/questions/5018545/getapplication-vs-getapplicationcontext – Martin Vandzura Jun 30 '17 at 07:50

score 1 · Answer 3 · answered Nov 01 '17 at 14:03

While I cannot say if this solution works.

I think that static Application instance should solve the problem.

class MyApp extends Application {
  private static final sInstance;

  public void onCreate() {
    sInstance = this;
  }

  public static MyApp getInstance() {
    return sInstance;
  }
}

Instead of calling getActivity() if you call MyApp.getInstance() you should not need to cast. So there should not be any ClassCastException anymore.

score 0 · Answer 4 · answered Jun 30 '17 at 07:26

0

You should override attachBaseContext in your application class like this:

@Override
protected void attachBaseContext(Context base) {
    super.attachBaseContext(base);
    MultiDex.install(this);
}

Check this link for more information: https://developer.android.com/reference/android/support/multidex/MultiDexApplication.html

answered Jun 30 '17 at 07:26

Vincent

64
8

I don't have to do it because it's already done in MultiDexApplication. In link which you share it says you can extend MultiDexApplication or override attachBaseContext – Martin Vandzura Jun 30 '17 at 07:29
@vandzi can you post your app class oncreate – Vincent Jun 30 '17 at 07:42
There is nothing useful there just super.onCreate(); Analytics initialisation. I think problem is somewhere else – Martin Vandzura Jun 30 '17 at 07:49
I recreated your implementation and it works on my side on android 7.0, it think it is somewhere else – Vincent Jun 30 '17 at 07:54
It doesn't always crash. It crashes only for some devices/users and probably randomly. – Martin Vandzura Jun 30 '17 at 08:15
Does it crash on samsungs? – Tuby Jun 30 '17 at 11:17
Yes, I saw samsung s7 as top device in reports, but other devices too. Common thing is android 7.0 – Martin Vandzura Jul 03 '17 at 07:34

score 0 · Answer 5 · answered Sep 19 '17 at 07:19

0

This might help

public class App extends MultiDexApplication { 

        public static App app = null;

        public static App getInstance() {
            return app;
        }

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

you doesn't need to cast getApplication(), reason is you are already in Application class so simply just use this keyword to get application instance. Hope you find useful

answered Sep 19 '17 at 07:19

Piyush Patel

371
1
5
13

getApp is in activity so I have to cast it – Martin Vandzura Sep 19 '17 at 10:11
Though you doesn't need to cast use getApplicationContext() in your activity – Piyush Patel Sep 19 '17 at 10:22
https://developer.android.com/reference/android/content/Context.html getApplicationContext() Return the context of the single, global Application object of the current process. – Piyush Patel Sep 19 '17 at 10:33
I don't want to get context. I want to get App instance. – Martin Vandzura Sep 19 '17 at 13:03
that I already answered. you need to read above statement. global Application object (That means App instance) – Piyush Patel Sep 19 '17 at 13:10

Android 7.0 and 7.1 getApplication() ClassCastException

5 Answers5

Linked