1

代码太大，我只复制有问题的部分。

这是一类中的 run() 方法：

public void run(){
try{
    sleep(1000);
    while(true){
          synchronized(space){

        if(end)
          return;

        if(space[X][Y] == null)
          break;

        if(((Ship)space[X][Y]).isDestroyed){
          destroy();
          break;
        }

        if(isThereAnyShipsInTheArea() != 0){
          if(team != ((Ship)space[X][Y + isThereAnyShipsInTheArea()]).team){
            fight(isThereAnyShipsInTheArea());
          }
        }
        else
          move();

        if(isDestroyed){
          destroy();
          break;
        }
    }
    }
}
catch(InterruptedException ie){
  System.out.println("Interrupted exception!");
}

}

这是星际迷航的模拟。可变团队表示该船属于哪个团队。如果船在战斗中被摧毁或在移动时坠毁，则变量 isDestroyed 为真。isThereAnyShipsInTheArea() - 如果距离一到二，船就在范围内。空间是尺寸为 [90]x[90] 的矩阵。

我认为问题在于运行方法，但我会给你一些其他方法。

    private int isThereAnyShipsInTheArea(){
  if(space[X][Y - 2] instanceof Ship && ((Ship)space[X][Y - 2]).isDestroyed == false)
    return -2;

  if(space[X][Y - 1] instanceof Ship && ((Ship)space[X][Y - 1]).isDestroyed == false)
    return -1;

  if(space[X][Y + 1] instanceof Ship && ((Ship)space[X][Y + 1]).isDestroyed == false)
    return 1;

  if(space[X][Y + 2] instanceof Ship && ((Ship)space[X][Y + 2]).isDestroyed == false)
    return 2;

  return 0;

}

 private synchronized void fight(int meet){


  while(((Ship)svemir[X][Y]).isDestroyed == false && ((Ship)space[X][Y + meet]).isDestroyed == false){
    if(((Ship)space[X][Y]).getProjectile() != 0){
      ((Ship)space[X][Y + meet]).setShield(((Ship)space[X][Y + meet]).getShield() - 1);
      ((Ship)space[X][Y + meet]).setWarp(((Ship)space[X][Y + meet]).getWarp() - 1);
      ((Ship)space[X][Y]).setProjectile(((Ship)space[X][Y]).getProjectile() - 1);

      if(((Ship)space[X][Y + meet]).getShield() == 0 || ((Ship)space[X][Y + meet]).getWarp() == 0){
        ((Ship)space[X][Y + meet]).isDestroyed = true;
        return;
      }
    }

    if(((Ship)space[X][Y + meet]).getProjectile() != 0){
      ((Ship)space[X][Y]).setShield(((Ship)space[X][Y]).getShield() - 1);
      ((Ship)space[X][Y]).setWarp(((Ship)space[X][Y]).getWarp() - 1);
      ((Ship)space[X][Y + meet]).setProjectile(((Ship)space[X][Y + meet]).getProjectile() - 1);

      if(((Ship)space[X][Y]).getShield() == 0 || ((Ship)space[X][Y]).getWarp() == 0){
        this.isDestroyed = true;
        return;
      }

    }

    if(((Ship)space[X][Y]).getProjectile() == 0 && ((Ship)space[X][Y + meet]).getProjectile() == 0)
      return;

  }

}

4

2 回答 2

0

对我来说，你不应该做 Thread.sleep() 因为它不会释放它抓取的任何资源。使用 ScheduledExecutorService 安排您的任务，或者在对象监视器上执行 wait() 和 yield()。

于 2013-08-23T02:38:40.050 回答
0

你的睡眠在你的while(true)街区之外，所以不是每个循环都睡一秒钟，而是睡一次，然后进入一个紧密的循环。

将 sleep 放在while(true)块的末尾，这样每次循环迭代都会休眠一次。理想情况下，它应该是在空间阵列上的同步释放之后。

实际上，全阵列扫描在寻找物品方面远非理想。可能想考虑保留项目列表。想一想，一个 1000x1000 的数组，其中有 20 个项目（空间很大，而且大部分是空的），当您通过数组时将进行 1,000,000 次检查，但是如果您根据项目重新组织检查，您可能会只需 1000 或更少的支票就可以逃脱。

例如，船舶列表：

for (Ship ship : ships) {
   if (!ship.engaged()) {
     ship.scanForEnemies();
   }
   if (ship.detectEnemies()) {
     ship.attack();
   }
}

可能只需要遍历十几艘或更少的船，检查几百个位置。如果你想知道上面的代码是如何工作的，在我的例子中，船应该是用一个空间数组构造的，它保留了一个引用。

于 2013-08-23T02:45:41.867 回答