为了创建游戏循环,我遵循了this excellent tutorial。但是我认为关于显示FPS的下一个教程有些跷,所以我尝试了独奏。我对自己制作的trackFps()
方法很有信心;在计算完帧之间的时间差,在每次运行时测量预测的FPS,将这些预测的FPS值存储在ArrayList
中之后调用,然后在经过一秒钟后将这些预测的FPS相加并除以添加的值数量即可得出平均FPS。
当我使用调试器运行它时,它可以正常运行,但是当我正常运行它时,会出现以下异常:
FATAL EXCEPTION: Thread-11
java.lang.IndexOutOfBoundsException: Invalid index 26, size is 6
at java.util.ArrayList.throwIndexOutOfBoundsException(ArrayList.java:257)
at java.util.ArrayList.get(ArrayList.java:311)
at biz.hireholly.engine.GameLoop.trackFps(GameLoop.java:120)
at biz.hireholly.engine.GameLoop.run(GameLoop.java:73)
这很有解释性,发生在此行
fps += fpsStore.get(fpsTrackCount-1);
上。但是我看不到fpsTrackCount
变量如何达到26,它只能与ArrayList
fpsStore
中存储的变量数目一样高。有人会浏览我的
GameLoop
类,尤其是底部的trackFps()
方法吗?我将提供全部内容。评论很多,但是没有太多,对你们中的一些人应该很熟悉。GameLoop
(在底部包含用于计算FPS的trackFps()
方法):package biz.hireholly.engine;
import android.graphics.Canvas;
import android.view.SurfaceHolder;
import java.util.ArrayList;
/**
* The GameLoop is a thread that will ensure updating and drawing is done at set intervals.
* The thread will sleep when it has updated/rendered quicker than needed to reach the desired fps.
* The loop is designed to skip drawing if the update/draw cycle is taking to long, up to a MAX_FRAME_SKIPS.
* The Canvas object is created and managed to some extent in the game loop,
* this is so that we can prevent multiple objects trying to draw to it simultaneously.
* Note that the gameloop has a reference to the gameview and vice versa.
*/
public class GameLoop extends Thread {
private static final String TAG = GameLoop.class.getSimpleName();
//desired frames per second
private final static int MAX_FPS = 30;
//maximum number of drawn frames to be skipped if drawing took too long last cycle
private final static int MAX_FRAME_SKIPS = 5;
//ideal time taken to update & draw
private final static int CYCLE_PERIOD = 1000 / MAX_FPS;
private SurfaceHolder surfaceHolder;
//the gameview actually handles inputs and draws to the surface
private GameView gameview;
private boolean running;
private long beginTime = 0; // time when cycle began
private long timeDifference = 0; // time it took for the cycle to execute
private int sleepTime = 0; // milliseconds to sleep (<0 if drawing behind schedule)
private int framesSkipped = 0; // number of render frames skipped
private double lastFps = 0; //The last FPS tracked, the number displayed onscreen
private int fpsTrackCount = 1; // number we'll divide the fpsSTore by to get average
private ArrayList<Double> fpsStore = new ArrayList<Double>(); //For the previous fps values
private long lastTimeFpsCalculated = System.currentTimeMillis(); //used in trackFps
public GameLoop(SurfaceHolder holder, GameView gameview) {
super();
this.surfaceHolder = holder;
this.gameview = gameview;
}
public void setRunning(boolean running) {
this.running = running;
}
@Override
public void run(){
Canvas c;
while (running) {
c = null;
//try locking canvas, so only we can edit pixels on surface
try{
c = this.surfaceHolder.lockCanvas();
//sync so nothing else can modify while were using it
synchronized (surfaceHolder){
beginTime = System.currentTimeMillis();
framesSkipped = 0; //reset frame skips
this.gameview.update();
this.gameview.draw(c);
//calculate how long cycle took
timeDifference = System.currentTimeMillis() - beginTime;
//good time to trackFps?
trackFps();
//calculate potential sleep time
sleepTime = (int)(CYCLE_PERIOD - timeDifference);
//sleep for remaining cycle
if (sleepTime >0){
try{
Thread.sleep(sleepTime); //saves battery! :)
} catch (InterruptedException e){}
}
//if sleepTime negative then we're running behind
while (sleepTime < 0 && framesSkipped < MAX_FRAME_SKIPS){
//update without rendering to catch up
this.gameview.update();
//skip as many frame renders as needed to get back into
//positive sleepTime and continue as normal
sleepTime += CYCLE_PERIOD;
framesSkipped++;
}
}
} finally{
//finally executes regardless of exception,
//so surface is not left in an inconsistent state
if (c != null){
surfaceHolder.unlockCanvasAndPost(c);
}
}
}
}
/* Calculates the average fps every second */
private void trackFps(){
long currentTime = System.currentTimeMillis();
if(timeDifference != 0){
fpsStore.add((double)(1000 / timeDifference));
}
//If a second has past since last time average was calculated,
// it's time to calculate a new average fps to display
if ((currentTime - 1000) > lastTimeFpsCalculated){
int fps = 0;
int toDivideBy = fpsTrackCount;
while ((fpsStore != null) && (fpsTrackCount > 0 )){
fps += fpsStore.get(fpsTrackCount-1);
fpsTrackCount--;
}
lastFps = fps / toDivideBy;
lastTimeFpsCalculated = System.currentTimeMillis();
fpsTrackCount = 1;
fpsStore.clear();
}
else{
fpsTrackCount++;
}
}
/* So That it can be drawn in the gameview */
public String getFps() {
return String.valueOf(lastFps);
}
}
最佳答案
好吧,让我们先来...
java.lang.IndexOutOfBoundsException: Invalid index 26, size is 6
at java.util.ArrayList.throwIndexOutOfBoundsException(ArrayList.java:257)
at java.util.ArrayList.get(ArrayList.java:311)
at biz.hireholly.engine.GameLoop.trackFps(GameLoop.java:120)
所以在trackFPS中的某个地方,get()显然是超出范围的...
private void trackFps()
{
long currentTime = System.currentTimeMillis();
if(timeDifference != 0)
{
fpsStore.add((double)(1000 / timeDifference));
}
//If a second has past since last time average was calculated,
// it's time to calculate a new average fps to display
if ((currentTime - 1000) > lastTimeFpsCalculated)
{
int fps = 0;
int toDivideBy = fpsTrackCount;
while ((fpsStore != null || !fpsStore.isEmpty()) && (fpsTrackCount > 0 ) && (fpsTrackCount < fpsStore.getCount()))
{
//synchronized(this) {
fps += fpsStore.get(fpsTrackCount-1);
fpsStore.remove(fpsTrackCount-1); //otherwise we'll get stuck because of getCount condition
fpsTrackCount--;
//}
}
lastFps = fps / toDivideBy;
lastTimeFpsCalculated = System.currentTimeMillis();
//fpsTrackCount = 1;
//fpsStore.clear();
Log.d("trackFPS()", "fpsTrackCount = "+fpsTrackCount+"\tfpsStore.size() = "+fpsStore.size()+"\t"+fpsStore.toString());
}
else
fpsTrackCount++;
}
旋转一下。如果效果不佳,请尝试取消注释已同步的块。
至于涉及TextDrawable的其他问题,我看看您的GameView ...
这是我在SurfaceChanged()中找到的
fps = new TextDrawable();
fps.setText("HELLO");
现在,为什么不将其移至SurfaceCreated()?也许您收到了很多surfaceChanged()回调,但是由于您没有Logcat调用,所以不知道它吗? :)这是从我所看到的实例化TextDrawable的唯一地方。