Question

I am making an Android game. The game is moles that appear on different places on the screen at random intervals for random durations.

The problem is I don’t know how to make the mole stay visible for a few seconds. I used wait(1000) on the the holder of OnDraw(), but it made the game stack for this time. I tried to change SurfaceView back to View, but then the screen stopped refreshing.

Any advice?

Main activity Class

public class First_Stage extends Activity implements OnTouchListener{

private AllViews allViews;
private MoleView moleView;
private PointsView pointsView;
private TimerView timerView;
private StageView stageView;
private Mole mole; 
private MoveMole moleMove;
private int points=0;
private StagePoints stagePoints;
private PointsSingelton poin;
private float x,y;
private Club club;
private ClubView clubView;
private PointsSingelton pointsCount;

@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    requestWindowFeature(Window.FEATURE_NO_TITLE);
    club=new Club();
    clubView = new ClubView(this, club);
    mole=new Mole();
    stageView=new StageView(this);
    moleView=new MoleView(this,mole);
    pointsView=new PointsView(this);
    timerView=new TimerView(this, "3:33");

    allViews=new AllViews(this);
    allViews.setViews(stageView, moleView, pointsView, timerView,clubView);

    setContentView(allViews);
    allViews.setOnTouchListener((View.OnTouchListener)this);

}

@Override
public boolean onTouch(View v, MotionEvent event) {
    x=event.getX();
    y=event.getY();
    moleView.setX(x);
    moleView.setY(y);

    allViews.setX(x);
    allViews.setY(y);


    if ((x<100 && x>0)&&(y>0&&y<100)){  
    points=pointsCount.getInstance().nextPoint();
    pointsView.setPoint(points);
    moleView.setBool(true);
    }

    return true;
}

All Views Class

public class AllViews extends SurfaceView implements SurfaceHolder.Callback,Runnable {
private Club club;
private ClubView clubView;
private MoleView moleView;
private PointsView pointsView;
private TimerView timerView;
private StageView mainView;
private float x, y;
private Paint test;
private First_Stage first;
Thread drawThread = new Thread(this);
SurfaceHolder holder;
private Bitmap clubPic;


public AllViews(Context context) {
    super(context);
    test = new Paint();
    first = new First_Stage();
    holder = getHolder();
    holder.addCallback(this);
}

public void setViews(StageView mainView, MoleView moleView,
        PointsView pointsView, TimerView timerView,ClubView clubView)

{
    this.mainView = mainView;
    this.moleView = moleView;
    this.pointsView = pointsView;
    this.timerView = timerView;

}

@Override
protected void onDraw(Canvas canvas) {
    super.onDraw(canvas);
    mainView.onDraw(canvas);
    moleView.onDraw(canvas);
    pointsView.onDraw(canvas);
    timerView.onDraw(canvas);
    clubPic=BitmapFactory.decodeResource(getResources(), R.drawable.clubdown);
    canvas.drawBitmap(clubPic, this.x-39,this.y-20, null);

}

@Override
public void run() {
    Canvas c;
    while (true) {
        c = null;

        try {
            c = holder.lockCanvas(null);

            synchronized (holder) {
                onDraw(c);

            }
        } finally {

            if (c != null) {
                holder.unlockCanvasAndPost(c);
            }

        }

    }

}

@Override
public void surfaceChanged(SurfaceHolder holder, int format, int width,
        int height) {
    // TODO Auto-generated method stub

}

@Override
public void surfaceCreated(SurfaceHolder holder) {
    drawThread.start();

}

moleView Class

public class MoleView extends View {
private Mole  mole;
private Bitmap molePic;
private float x,y;
private boolean bool=false;

public MoleView(Context context, Mole mole) {
  super(context);
  this.mole=mole;

    }


public boolean isBamped(){
    float xmin=mole.getX();
    float xmax=mole.getX()+60;
    float ymin=mole.getY();
    float ymax=mole.getY()+46;
    if ((this.x<xmax&&this.x>xmin)&&(this.y<ymax&&this.y>ymin)){
        return true;
    }

    return false;
}



@Override
protected void onDraw(Canvas canvas) {
    // TODO Auto-generated method stub

    if (!bool){
    molePic=BitmapFactory.decodeResource(getResources(), R.drawable.nest_full_mole);
    canvas.drawBitmap(molePic, mole.getX(), mole.getY(), null);
      mole.moveMole();
    }else {
        molePic=BitmapFactory.decodeResource(getResources(), R.drawable.ic_launcher);
        canvas.drawBitmap(molePic, mole.getX(), mole.getY(), null);
        molePic.recycle();
    }


}

Mole Class

public class Mole {
private float x;
private float y;
private Positions myPositions;

public Mole() {
    super();        
    myPositions=new Positions();
    this.x = myPositions.getRandomX();
    this.y = myPositions.getRandomY();
}


public float getX() {
    return x;
}
public void setX(float x) {
    this.x = x;
}
public float getY() {
    return y;
}
public void setY(float y) {
    this.y = y;
}
}

Answer 1

I can’t really find the place in code where do you want to keep it on screen, because there is still a lot of code but try it like this:

Make a new Handler in the class where you are drawing the moles. After that, you can call the postDelayed() function, with which you can execute something in the period you define. Quite simple, in fact.

Links:
http://developer.android.com/reference/android/os/Handler.html

http://www.vogella.com/articles/AndroidPerformance/article.html