i am trying to find a way that the thread can be shut down

Question

0

Asked: May 24, 20262026-05-24T08:35:48+00:00 2026-05-24T08:35:48+00:00

i am trying to find a way that the thread can be shut down

0

i am trying to find a way that the thread can be shut down and restarted without the program crashing. It is called from a menu and an activity sets the panel as its content view and i wish that when the return arrow is pressed on the android that it returns to the activity and then the thread can be restarted however currently any variation i try causes it to crash at one point or another 🙁

package SortItOut.sortitout;

import android.content.Context;
import android.graphics.Bitmap;
import android.graphics.BitmapFactory;
import android.graphics.Canvas;
import android.view.MotionEvent;
import android.view.SurfaceHolder;
import android.view.SurfaceView;

public class Level1Panel extends SurfaceView implements
SurfaceHolder.Callback {

private Level1Thread thread;
static Bitmap background;
static int x = 0;

 public Level1Panel(Context context) {
super(context);
getHolder().addCallback(this);
background = BitmapFactory.decodeResource(getResources(), R.drawable.gamebackground);
thread = new Level1Thread(getHolder(), this);
setFocusable(true);
}


public void surfaceChanged(SurfaceHolder holder, int format, int width, int height) {
}

 public void surfaceCreated(SurfaceHolder holder) {
 thread.setRunning(true);
 thread.start();
 }


 public void surfaceDestroyed(SurfaceHolder holder) {
 thread.stop();
}

public void render(Canvas canvas)
{
canvas.drawBitmap(background, x, 0, null);
x = x + 20;
}

}

======Thread=======

package SortItOut.sortitout;

import android.content.Context;
import android.graphics.Bitmap;
import android.graphics.BitmapFactory;
import android.graphics.Canvas;
import android.view.SurfaceHolder;

public class Level1Thread extends Thread {


 private boolean running;
 private SurfaceHolder surfaceHolder;
 private Level1Panel gamePanel;


 public void setRunning(boolean running) {
  this.running = running;
 }

 public Level1Thread(SurfaceHolder surfaceHolder, Level1Panel gamePanel) {
     super();
     this.surfaceHolder = surfaceHolder;
     this.gamePanel = gamePanel;
    }



 public void run() {
     Canvas canvas;

     while (running) {
     canvas = null;

     try {
         canvas = this.surfaceHolder.lockCanvas();
        synchronized (surfaceHolder) {
        this.gamePanel.render(canvas);

        }
       } finally {

     if (canvas != null) {
      surfaceHolder.unlockCanvasAndPost(canvas);
       }
      }
      }
 }



}