2012-03-25 56 views
0

我正在制作一个安卓游戏,它在运行此方法时检查玩家健康值。但是,它没有反应。它什么都不做,当值小于3时,它不应该做任何事情,但是当它等于3时,它应该运行一个方法。请提前帮助我并多谢!这是我使用的代码:比较Android中的int值

 private void checkLivesLeftValue() { 
     if (livesLeftValue == 3) { 
      //Message to display: "You lost! 
      onMethod(); 
     } 
     else { 
      livesLeftValue = livesLeftValue + 1; 
     } 
    } 

全码:

package com.mysoftwaremobileapps.ParachuteHunter; 

import java.util.ArrayList; 

import android.app.AlertDialog; 
import android.content.Context; 
import android.graphics.Bitmap; 
import android.graphics.BitmapFactory; 
import android.graphics.Canvas; 
import android.graphics.Color; 
import android.graphics.Paint; 
import android.media.MediaPlayer; 
import android.os.Bundle; 
import android.os.Handler; 
import android.os.Message; 
import android.view.MotionEvent; 
import android.view.SurfaceHolder; 
import android.view.SurfaceView; 
import android.widget.TextView; 
import android.widget.Toast; 

public class ExampleView extends SurfaceView implements SurfaceHolder.Callback 
{ 
class ExampleThread extends Thread 
{ 
    private ArrayList<Parachuter> parachuters; 
    private Bitmap parachuter; 
    private Paint black; 

    private boolean running; 

    private SurfaceHolder mSurfaceHolder; 
    private Context mContext; 
    private Handler mHandler; 
    private GameScreenActivity mActivity; 

    private long frameRate; 
    private boolean loading; 
    public float x; 
    public float y; 
    public MediaPlayer mp1; 
    public int parachuterIndexToResetAndDelete; 
    public int canvasGetWidth; 
    public int livesLeftValue; 

    public ExampleThread(SurfaceHolder sHolder, Context context, Handler handler) 
    { 
     mSurfaceHolder = sHolder; 
     mHandler = handler; 
     mContext = context; 
     mActivity = (GameScreenActivity) context; 

     parachuters = new ArrayList<Parachuter>(); 
     parachuter = BitmapFactory.decodeResource(getResources(), R.drawable.parachuteman); 
     black = new Paint(); 
     black.setStyle(Paint.Style.FILL); 
     black.setColor(Color.WHITE); 

     running = true; 

     // This equates to 26 frames per second. 
     frameRate = (long) (1000/26); 
     loading = true; 
    } 

    @Override 
    public void run() 
    { 
     while (running) 
     { 
      Canvas c = null; 
      try 
      { 
       c = mSurfaceHolder.lockCanvas(); 

       synchronized (mSurfaceHolder) 
       { 
        long start = System.currentTimeMillis(); 
        doDraw(c); 
        long diff = System.currentTimeMillis() - start; 

        if (diff < frameRate) 
         Thread.sleep(frameRate - diff); 
       } 
      } catch (InterruptedException e) 
      { 
      } 
      finally 
      { 
       if (c != null) 
       { 
        mSurfaceHolder.unlockCanvasAndPost(c); 
       } 
      } 
     } 
    } 

    protected void doDraw(Canvas canvas) 
    { 
     canvas.drawRect(0, 0, canvas.getWidth(), canvas.getHeight(), black); 
     canvasGetWidth = canvas.getWidth(); 

     //Draw 
     for (int i = 0; i < parachuters.size(); i++) 
     { 
      canvas.drawBitmap(parachuter, parachuters.get(i).getX(), parachuters.get(i).getY(), null); 
      parachuters.get(i).tick(); 
     } 

     //Remove 
     for (int i = 0; i < parachuters.size(); i++) 
     { 
     if (parachuters.get(i).getY() > canvas.getHeight()) { 
      parachuters.remove(i); 
      onPlaySound(); 
      checkLivesLeftValue(); 
     } 
    } 
    } 


    public void onPlaySound() 
    { 
     try { 
     mp1 = MediaPlayer.create(getContext(), R.raw.bombsound); 
     mp1.start(); 
     } catch (Exception e) { 
      e.printStackTrace(); 
      mp1.release(); 
     } 
    } 

    public void onMethod() { 
     mHandler.post(new Runnable() { 
      @Override 
      public void run() { 
       Toast.makeText(getContext(), "You lost!", 15).show(); 
      } 
     }); 
    } 

    private void checkLivesLeftValue() { 
     // TODO Auto-generated method stub 
     if (livesLeftValue == 3) { 
      //Message to display: "You lost! 
      onMethod(); 
     } 
     else { 
      livesLeftValue = livesLeftValue + 1; 
     } 
    } 

    public boolean onTouchEvent(MotionEvent event) 
    { 
     if (event.getAction() != MotionEvent.ACTION_DOWN) 
      return false; 
     float x1 = event.getX(); 
     float y1 = event.getY(); 

     initiateDrawParachuters(); 

     return true; 
    } 

    public void initiateDrawParachuters() 
    { 
     drawParachuter1(); 
    } 

    private void drawParachuter1() { 
     // TODO Auto-generated method stub 
     //Parachuter nr. 1 
     x = 68; 
     y = 40; 

     Parachuter p = new Parachuter(x, y); 
     parachuters.add(p); 
     drawParachuter2(); 
    } 

    private void drawParachuter2() { 
     // TODO Auto-generated method stub 
     //Parachuter nr. 2 
     x = 100; 
     y = 80; 

     Parachuter p = new Parachuter(x, y); 
     parachuters.add(p); 
     drawParachuter3(); 
    } 

    private void drawParachuter3() { 
     // TODO Auto-generated method stub 
     //Parachuter nr. 3 
     x = 150; 
     y = 120; 

     Parachuter p = new Parachuter(x, y); 
     parachuters.add(p); 
     drawParachuter4(); 
    } 

    private void drawParachuter4() { 
     // TODO Auto-generated method stub 
     //Parachuter nr. 4 
     x = 170; 
     y = 150; 

     Parachuter p = new Parachuter(x, y); 
     parachuters.add(p); 
     drawParachuter5(); 
    } 

    private void drawParachuter5() { 
     // TODO Auto-generated method stub 
     //Parachuter nr. 5 
     x = 180; 
     y = 170; 

     Parachuter p = new Parachuter(x, y); 
     parachuters.add(p); 
     drawParachuter6(); 
    } 

    private void drawParachuter6() { 
     // TODO Auto-generated method stub 
     //Parachuter nr. 6 
     x = 200; 
     y = 180; 

     Parachuter p = new Parachuter(x, y); 
     parachuters.add(p); 
    } 

    public void drawParachuters() 
    { 
      Parachuter p = new Parachuter(x, y); 
      parachuters.add(p); 
      Toast.makeText(getContext(), "x=" + x + " y=" + y, 15).show(); 
    } 

    public void setRunning(boolean bRun) 
    { 
     running = bRun; 
    } 

    public boolean getRunning() 
    { 
     return running; 
    } 
} 

/** Handle to the application context, used to e.g. fetch Drawables. */ 
private Context mContext; 

/** Pointer to the text view to display "Paused.." etc. */ 
private TextView mStatusText; 

/** The thread that actually draws the animation */ 
private ExampleThread eThread; 

public ExampleView(Context context) 
{ 
    super(context); 

    // register our interest in hearing about changes to our surface 
    SurfaceHolder holder = getHolder(); 
    holder.addCallback(this); 

    // create thread only; it's started in surfaceCreated() 
    eThread = new ExampleThread(holder, context, new Handler() 
    { 
     @Override 
     public void handleMessage(Message m) 
     { 
      // mStatusText.setVisibility(m.getData().getInt("viz")); 
      // mStatusText.setText(m.getData().getString("text")); 
     } 
    }); 

    setFocusable(true); 
} 

@Override 
public boolean onTouchEvent(MotionEvent event) 
{ 
    return eThread.onTouchEvent(event); 
} 

public ExampleThread getThread() 
{ 
    return eThread; 
} 

@Override 
public void surfaceChanged(SurfaceHolder arg0, int arg1, int arg2, int arg3) 
{ 
    // TODO Auto-generated method stub 
} 

public void surfaceCreated(SurfaceHolder holder) 
{ 
    if (eThread.getState() == Thread.State.TERMINATED) 
    { 
     eThread = new ExampleThread(getHolder(), getContext(), getHandler()); 
     eThread.start(); 
    } 
    else 
    { 
     eThread.start(); 
    } 
} 

@Override 
public void surfaceDestroyed(SurfaceHolder holder) 
{ 
    boolean retry = true; 
    eThread.setRunning(false); 

    while (retry) 
    { 
     try 
     { 
      eThread.join(); 
      retry = false; 
     } 
     catch (InterruptedException e) 
     { 
     } 
    } 
} 
} 
+0

它现在如何行事? – 2012-03-25 16:57:04

+0

尝试使用'equals()'方法,就像这样'if(livesLeftValue.equals(3))',你需要在你的代码片段中包含一些额外的部分。 – Lion 2012-03-25 16:57:17

+1

@Lion,这不应该是'int'比较所必需的。但是,是的,我假设livesleftValue是一个'int' .. @OP是否有可能'livesLeftValue'高于3?也许它开始/得到高于3,所以这种方法是“错过”。或者,正如@Lion指出的那样,你可以发布更多的代码吗?像声明的lifeLeftValue在哪里,以及在哪里使用? – edthethird 2012-03-25 17:03:38

回答

0

把一些调试在那里,看看是否你真的在这里得到正确的值。

private void checkLivesLeftValue() { 
    Log.d("checkLivesLeftValue", "lives = " + livesLeftValue); 
    if (livesLeftValue == 3) { 
     Log.d("checkLivesLeftValue", "calling onMethod now"); 
     onMethod(); 
    } else { 
     livesLeftValue = livesLeftValue + 1; 
     Log.d("checkLivesLeftValue", "increased lives to " + livesLeftValue); 
    } 
} 

public void onMethod() { 
    Log.d("onMethod", "in onMethod"); 
    mHandler.post(new Runnable() { 
     @Override 
     public void run() { 
      Log.d("onMethod", "showing Toast now"); 
      Toast.makeText(getContext(), "You lost!", 15).show(); 
     } 
    }); 
} 
+0

我这样做了,它的工作方式就像计划:value:0,增加到1,值1,增加到2,值2,增加到3,值3,调用onMethod现在 – user1183066 2012-03-25 17:42:20

1

我同意其他人通过日志消息检查值,所以你可以看到实际值。

此外,检查它是否可能结束3.我不知道你在做什么样的游戏,但例如,说现在生命值为2,但是发生了一些事情,它增加了2,达到4,没有永远具有价值3.让我们说它永远不会减少,所以你永远不会达到价值3,因此永远不会调用游戏而不是物品,生命值将会增加到无穷大。