0

方法 changeFlowers 因索引超出范围异常而失败。

我是java新手,我正在调用已经实例化的FishOne(Arraylist rederable类)的方法“changeColor()”但是它因索引超出范围异常而失败。

public void changeFlowers(int i)
{


        FlowerOne f1 = (FlowerOne)(this._fishes.get(i));   // Is this the right way to call ????
        f1.changeColor(this._theme);


}

下面的代码 Aquarium.java,FishOne.java 提前感谢您的帮助。


水族馆.java,

public class Aquarium {
private AquariumThread _aquariumThread; 
private SurfaceHolder _surfaceHolder;   
private ArrayList<Renderable> _fishes;  
private Bitmap _backgroundImage;
private Context _context;


public void render(){
    Canvas canvas = null;
    try{

        canvas = this._surfaceHolder.lockCanvas(null);
        synchronized (this._surfaceHolder) {
            this.onDraw(canvas);
        }

    }finally{
        if(canvas != null){
            this._surfaceHolder.unlockCanvasAndPost(canvas);
        }
    }   
}


protected void onDraw(Canvas canvas) {
    this.renderBackGround(canvas);
    for (Renderable renderable : this._fishes) {
        renderable.render(canvas);
    }
};

public void initialize(Context context, SurfaceHolder surfaceHolder) {
    this._fishes = new ArrayList<Renderable>();
}



protected void onDraw(Canvas canvas) {
    this.renderBackGround(canvas);
    for (Renderable renderable : this._fishes) {
        renderable.render(canvas);
    }
};


public void start(){
    this._aquariumThread.switchOn();
}


public void stop(){
    boolean retry = true;
    this._aquariumThread.switchOff();
    while (retry) {
        try {
         this._aquariumThread.join();
            retry = false;
        } catch (InterruptedException e) {
            // we will try it again and again...
        }
    }
}


public synchronized void addFishes() {      

Point startPoint1 = new Point(1,300);
this._fishes.add(new fishOne(this._context, this, startPoint1, 90));

Point startPoint2 = new Point(100,300);
this._fishes.add(new fishOne(this._context, this, startPoint2, 90));

Point startPoint3 = new Point(200,300);
this._fishes.add(new fishOne(this._context, this, startPoint3, 90));

}





public void changeFlowers(int i)
{


        FlowerOne f1 = (FlowerOne)(this._fishes.get(i));
        f1.changeColor(this._theme);


}


}

FishOne.java

public class fishOne extends Animal {
private static final int TOTAL_FRAMES_IN_SPRITE = 1;
private static final int CLOWN_FISH_FPS = 1; 

int _fishColor;
Bitmap leftBitmap;
Bitmap rightBitmap;


public fishOne(Context context, Aquarium aquarium,  Point startPoint, int speed){       

    super(context, aquarium);   
    this.changeColor(1);
    this.initialize(this.leftBitmap, this.rightBitmap, CLOWN_FISH_FPS, TOTAL_FRAMES_IN_SPRITE, startPoint, speed);

}


public void changeColor(int mfishColor)
{


    switch(mfishColor) {
    case 1:

    BitmapFactory.Options options = new BitmapFactory.Options();
    options.inPurgeable = true;
    this.leftBitmap = BitmapFactory.decodeResource(getContext().getResources(), com.gotit.livewallpaper.fishshd.R.drawable.fish1, options);

    BitmapFactory.Options options1 = new BitmapFactory.Options();
    options1.inPurgeable = true;
    this.rightBitmap = BitmapFactory.decodeResource(getContext().getResources(), com.gotit.livewallpaper.fishshd.R.drawable.fish2, options1);

        break;

    case 2:
    BitmapFactory.Options options2 = new BitmapFactory.Options();
    options2.inPurgeable = true;
    this.leftBitmap = BitmapFactory.decodeResource(getContext().getResources(), com.gotit.livewallpaper.fishshd.R.drawable.redfish1, options2);

    BitmapFactory.Options options3 = new BitmapFactory.Options();
    options3.inPurgeable = true;
    this.rightBitmap = BitmapFactory.decodeResource(getContext().getResources(), com.gotit.livewallpaper.fishshd.R.drawable.redfish2, options3);

        break;
    default:
    }


}


public void render(Canvas canvas){
    super.render(canvas);   
}

}
4

1 回答 1

0

这取决于代码的错误安全性的完整逻辑。基本上,如果您确定不会使用大于 ArrayList 大小的参数 i 调用 changeFlowers,那么没关系,您必须将 vheck 放入 changeFlowers 函数中。

喜欢

public void changeFlowers(int i)
{

    if(null!=this._fishes && this._fishes.size()>i){

        FlowerOne f1 = (FlowerOne)(this._fishes.get(i));
        f1.changeColor(this._theme);
}


}

但是最好让代码安全地从调用这个函数的地方开始,因为实际问题就在那里。上面的代码将停止错误,但您还需要在调用函数的位置使其正确。

于 2012-05-29T09:47:51.980 回答