我定义了这个简单的方法:

public static boolean isBorder(int x, int y) throws CollisionDetectionException {
        try {
            if ( (levelItems[x][y] instanceof StaticGameObject && levelItems[x][y].isVisible()) ||
                (levelItems[x-1][y] instanceof StaticGameObject && levelItems[x-1][y].isVisible()) ||
                (levelItems[x][y+1] instanceof StaticGameObject && levelItems[x][y+1].isVisible()) ||
                (levelItems[x][y-1] instanceof StaticGameObject && levelItems[x][y-1].isVisible()) ||
                (levelItems[x-1][y-1] instanceof StaticGameObject && levelItems[x-1][y-1].isVisible()) ||
                (levelItems[x-1][y+1] instanceof StaticGameObject &&levelItems[x-1][y+1].isVisible()) ||
                (levelItems[x+1][y] instanceof StaticGameObject && levelItems[x+1][y].isVisible()) ||
                (levelItems[x+1][y+1] instanceof StaticGameObject && levelItems[x+1][y+1].isVisible()) ||
                (levelItems[x+1][y-1] instanceof StaticGameObject && levelItems[x+1][y-1].isVisible()) ) {
                return true;
            } else {
                return false;
            }
        } catch (ArrayIndexOutOfBoundsException e) {
            throw new CollisionDetectionException("Collision couldn't be checked because checking position " + x + "/" + y + " caluclated values below (0/0)");
        }
    }

如您所见,我有一个二维数组。现在我想检查一个特定的位置((x/y) -> 方法参数),如果二维数组的相邻字段中有任何可见的 StaticGameObject。

levelItems 数组由所谓的 GameObject 组成。 StaticGameObject 是 GameObject 的直接子类。

任何提示如何改进这种方法?

最佳答案

为 GameObject 添加一个方法

bool isBorderObject() { return false; }

然后在 StaticGameObject 中覆盖
bool isBorderObject() { return true; }

将测试更改为
(levelItems[x][y].isBorderObject() && levelItems[x][y].isVisible())

此外,如果可以嵌套为
for (int i = x-1; i <= x+1; ++i) {
   for (int j = y-1; j <= y+1; ++j) {
       GameObject item = levelItems[i][j];
       if (item.isBorderObject() && item.isVisible())
           return true;
   }
}
return false;

关于java - 如何避免 instanceof 调用?,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/3461615/

10-12 01:36