kotlin-indexoutofboundsexception

vngu2lb8  于 2021-06-30  发布在  Java
关注(0)|答案(1)|浏览(312)

目前我正在努力学习用kotlin制作游戏,我正在将java代码转换成kotlin,除了下面的问题,其他一切都正常工作。

//Missile.kt

class Missile(context: Context) {

    internal var x: Int = 0
    internal var y: Int = 0
    internal var mVelocity: Int = 0
    internal var missile: Bitmap = BitmapFactory.decodeResource(context.resources, R.drawable.missile)

    val missileWidth: Int
        get() = missile.width
    val missileHeight: Int
        get() = missile.height

    init {
        x = GameView.dWidth / 2 - missileWidth / 2
        y = GameView.dHeight - GameView.tankHeight - missileHeight / 2
        mVelocity = 50
    }
}

//In Kotlin

for (i in missiles.indices) {

    if (missiles[i].y >- missiles[i].missileHeight) {

        missiles[i].y -= missiles[i].mVelocity

        canvas.drawBitmap(missiles[i].missile, missiles[i].x.toFloat(), missiles[i].y.toFloat(), null)

        if (missiles[i].x >= planes[0].planeX && missiles[i].x + missiles[i].missileWidth <= planes[0].planeX + planes[0].width && missiles[i].y >= planes[0].planeY &&

                missiles[i].y <= planes[0].planeY + planes[0].height) {

            val explosion = Explosion(context)

            explosion.explosionX = planes[0].planeX + planes[0].width / 2 - explosion.explosionWidth / 2

            explosion.explosionY = planes[0].planeY + planes[0].height / 2 - explosion.explosionHeight / 2

            explosions.add(explosion)

            planes[0].resetPosition()

            count++

            missiles.removeAt(i)

            if (point != 0) {

                sp.play(point, 1f, 1f, 0, 0, 1f)

            }

//In Java

for (int i = 0; i < missiles.size(); i++) {

    if (missiles.get(i).y > -missiles.get(i).getMissileHeight()) {

        missiles.get(i).y -= missiles.get(i).mVelocity;

        canvas.drawBitmap(missiles.get(i).missile, missiles.get(i).x, missiles.get(i).y, null);

        if (missiles.get(i).x >= planes.get(0).planeX && (missiles.get(i).x + missiles.get(i).getMissileWidth())

                <= (planes.get(0).planeX + planes.get(0).getWidth()) && missiles.get(i).y >= planes.get(0).planeY &&

                missiles.get(i).y <= (planes.get(0).planeY + planes.get(0).getHeight())) {

            Explosion explosion = new Explosion(context);

            explosion.explosionX = planes.get(0).planeX + planes.get(0).getWidth()/2 - explosion.getExplosionWidth()/2;

            explosion.explosionY = planes.get(0).planeY + planes.get(0).getHeight()/2 - explosion.getExplosionHeight()/2;

            explosions.add(explosion);

            planes.get(0).resetPosition();

            count++;

            missiles.remove(i);

            if(point != 0){

                sp.play(point, 1, 1, 0, 0, 1);

            }

下面的部分。

java.lang.IndexOutOfBoundsException: Index: 1, Size: 1

java.lang.IndexOutOfBoundsException: Index: 2, Size: 2

if (missiles[i].y >- missiles[i].missileHeight) {

所以,如果我按两次大炮,就会引起 java.lang.IndexOutOfBoundsException: Index: 1, Size: 1 如果我按三次大炮,就会引起 java.lang.IndexOutOfBoundsException: Index: 2, Size: 2. 如果我只按一次大炮,就不会坠毁。
我很想得到关于如何解决这些问题的可能的建议。

mcvgt66p

mcvgt66p1#

我相信这是因为你删除的元素导致你的索引是不正确的。如果要从列表中删除元素,应该使用迭代器来迭代列表。

Iterator itr = missiles.iterator();
while(itr.hasNext()){
    Object missile = itr.next(); 
    itr.remove();
}

删除也会从你的导弹列表中删除它。

相关问题