What is the meaning of Application is doing too much work in its main Thread?

Question

I am making a simple game.

1st screen is splash screen with just 2 images, 1 progress bar and a background image.

2nd screen is Options choose from different levels / options of games. It has few TextFields, Spinner Items , 2 images and 1 background. These images and background images are same as of splash screen.

From the options screen there is two ways to proceed with New Activity in game either choose "3x3"(MainActivity.class Activity) or 4X4(bigmain.class Activity). Both these activities are playing screen and contains 1 same image and a grid layout fitted with few ImageViews.

Both MainActivity and bigmain has onClick() method of grid layout. All the further game processing like checking if anywone wins or not, different methods to invoke on different player's turn. So both of them has lots of code and all in single java class file. Is this the reason ? If so then how to overcome it. I am attaching my logcat report when I install app in my phone through adb. See the bold text.

07-01 17:41:18.454 7003-7003/bt4u.com.pokemonbattles W/System:   ClassLoader referenced unknown path: /data/app/bt4u.com.pokemonbattles-2/lib/arm64
07-01 17:41:18.491 7003-7003/bt4u.com.pokemonbattles W/System: ClassLoader referenced unknown path: /data/app/bt4u.com.pokemonbattles-2/lib/arm64
07-01 17:41:18.492 7003-7003/bt4u.com.pokemonbattles I/LoadedApk: No resource references to update in package common
07-01 17:41:18.493 7003-7003/bt4u.com.pokemonbattles I/LoadedApk: No resource references to update in package com.rr.neptune
07-01 17:41:46.642 7332-7332/bt4u.com.pokemonbattles W/System: ClassLoader referenced unknown path: /data/app/bt4u.com.pokemonbattles-2/lib/arm64
07-01 17:41:47.290 7332-7332/bt4u.com.pokemonbattles W/System: ClassLoader referenced unknown path: /data/app/bt4u.com.pokemonbattles-2/lib/arm64
07-01 17:41:47.291 7332-7332/bt4u.com.pokemonbattles I/LoadedApk: No resource references to update in package common
07-01 17:41:47.292 7332-7332/bt4u.com.pokemonbattles I/LoadedApk: No resource references to update in package com.rr.neptune
**07-01 17:41:47.824 7332-7332/bt4u.com.pokemonbattles I/System.out: Splash screen activity (splash.java)**
07-01 17:41:47.834 7332-7363/bt4u.com.pokemonbattles D/OpenGLRenderer: Use EGL_SWAP_BEHAVIOR_PRESERVED: true
07-01 17:41:47.863 7332-7363/bt4u.com.pokemonbattles I/Adreno: QUALCOMM build                   : 6d30364, I5d4b06969f
07-01 17:42:12.187 7660-7686/bt4u.com.pokemonbattles I/OpenGLRenderer: Initialized EGL, version 1.4
07-01 17:42:15.294 7660-7660/bt4u.com.pokemonbattles I/System.out: Game options activity (gameoptions.class)
07-01 17:42:15.350 7660-7660/bt4u.com.pokemonbattles D/android.widget.GridLayout: horizontal constraints: x1-x0>=243, x2-x1>=0, x3-x2>=0, x4-x3>=430, x4-x0<=656 are inconsistent; permanently removing: x4-x0<=656. 
**07-01 17:42:17.615 7660-7660/bt4u.com.pokemonbattles I/Choreographer: Skipped 125 frames!  The application may be doing too much work on its main thread.**
**07-01 17:48:20.373 7660-7660/bt4u.com.pokemonbattles I/System.out: Game Option A Screen (MainActivity.class)**
07-01 17:48:20.601 7660-7686/bt4u.com.pokemonbattles D/OpenGLRenderer: endAllStagingAnimators on 0x55c0c40f40 (RippleDrawable) with handle     0x55c08e70b0
**07-01 17:48:37.333 7660-7660/bt4u.com.pokemonbattles I/System.out: Game Option B Screen (bigmain.class)**
07-01 17:48:37.358 7660-7660/bt4u.com.pokemonbattles D/android.widget.GridLayout: vertical constraints: y4-y0>=900, y4-y3<=230, y3-y2<=230, y2-y1<=224, y1-y0<=210 are inconsistent; permanently removing: y4-y3<=230. 
07-01 17:48:37.685 7660-7686/bt4u.com.pokemonbattles D/OpenGLRenderer: endAllStagingAnimators on 0x55c0c40f40 (RippleDrawable) with handle 0x55c0d44e20

Possible duplicate of [The application may be doing too much work on its main thread](http://stackoverflow.com/questions/14678593/the-application-may-be-doing-too-much-work-on-its-main-thread) — Harshad Pansuriya, Jul 01 '16 at 12:39
have you tried starting a new thread for processing? As far as I remember you shouldn't use the mainthread for long/heavy operations. Try to post a runnable. — Memme, Jul 01 '16 at 13:10

Harshad Pansuriya · Answer 1 · 2016-07-02T06:18:18.877

1

Application is doing too much work in main Thread means you are doing some high task like large data object fetching and converting or loading with large Image that time this error is occure.

More explnation : it may be because of some heavy processing that you are doing at the heart of your application or DB access or any other thing which causes the thread to stop for a while

Solution : to solve that problem simply create Multithreading.

UnderStandMent :

lets say you have for loop inside on Button so you have set it `OnClickListener` like this

  @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_inventory);

    button.setOnclickListener(new View.OnClickListener() {
                @Override
                public void onClick(View v) {

                   for(i=0;i<1000;i++){
                   --  Print i

                       i++;                   
                     }

                   }
            });

   }

like this one no put after click event any other unusable code inside that check that way..

edited Jul 02 '16 at 06:18

answered Jul 01 '16 at 12:42

Harshad Pansuriya

20,189
8
67
95

no i am not doing heavy processing and even all images used are all less than 20kb – Shubham Singh Jul 01 '16 at 12:57
i am making a game like tic tac toe – Shubham Singh Jul 01 '16 at 12:58
[It's one the java file, its player vs cpu](http://www.paste.org/81108) – Shubham Singh Jul 01 '16 at 13:19
void drop(View view) is the method onClick of all the ImageViews fitted in grid layout – Shubham Singh Jul 01 '16 at 13:24
@SWARNVEERSINGH Ok now my leave time from office I will check it and tell you soon. – Harshad Pansuriya Jul 01 '16 at 13:27
Ok I will be waaiting. Thank you so much – Shubham Singh Jul 01 '16 at 15:56
@SWARNVEERSINGH I think the Problem in your code is you are doing lots of looping in the code and it run again and again that's why it give the error.. – Harshad Pansuriya Jul 02 '16 at 04:11
But looping is necessary to get the result – Shubham Singh Jul 02 '16 at 05:47
@SWARNVEERSINGH check your code and run only that portion code when you are performing some event not run the whole code directly may be this will be the reason.. – Harshad Pansuriya Jul 02 '16 at 05:50
could you change it a little bit ? As I don't know how to do that – Shubham Singh Jul 02 '16 at 05:54
are you talking about randNum() ? – Shubham Singh Jul 02 '16 at 05:58
@SWARNVEERSINGH I give you simple way when you perform any click event then you have run only that part of code and remain code should not called so it can not run all the code so the loop are going to run at a time. This way you can set it.. – Harshad Pansuriya Jul 02 '16 at 06:04
but their is not remain code. All code is necessary when on click event – Shubham Singh Jul 02 '16 at 06:06
Till now I have removed extra for loops from ranNum() method. Can you give an example how to do what you are saying since I am not getting it – Shubham Singh Jul 02 '16 at 06:12
@SWARNVEERSINGH wait I give you example in my answer. – Harshad Pansuriya Jul 02 '16 at 06:13
I don't have a single button. I have a grid layout in which there are 9 ImageViews. So I used drop as onClick() of every imageView. And inside onClick() method I have ImageView counter = (ImageView) view; So I don't have to define seperate button listener for every image, i just created simple way to get click in single line – Shubham Singh Jul 02 '16 at 06:31

score 0 · Answer 2 · answered Jul 02 '16 at 06:31

It means you are doing heavy task on your main UI thread. Which causes ANR (Application not responding) because you are running a process on the UI thread which takes a long time. During this time the GUI (Graphical User Interface) will lock up which will result in anything the user presses will not be actioned.

You should use another thread for heavy task.

Better Approach will be using AsyncTask .

private class AsyncCaller extends AsyncTask<Void, Void, Void>
{
    ProgressDialog pdLoading = new ProgressDialog(AsyncExample.this);

    @Override
    protected void onPreExecute() {
        super.onPreExecute();

        //this method will be running on UI thread
        pdLoading.setMessage("\tLoading...");
        pdLoading.show();
    }
    @Override
    protected Void doInBackground(Void... params) {

        //this method will be running on background thread so don't update UI frome here
        //do your long running http tasks here,you dont want to pass argument and u can access the parent class' variable url over here


        return null;
    }

    @Override
    protected void onPostExecute(Void result) {
        super.onPostExecute(result);

        //this method will be running on UI thread

        pdLoading.dismiss();
    }

    }

[Here's my code](http://www.paste.org/81111) can you give an example on how to do it in my code — Shubham Singh, Jul 02 '16 at 06:38

What is the meaning of Application is doing too much work in its main Thread?

2 Answers2

UnderStandMent :