缩小代码

Question

我有一些java代码用于我的游戏NPC移动到周围。

这些显然是在1D阵列。

public void route11() { 
    Scanner in = new Scanner(System.in); 
    Random number = new Random(); 
    int random = number.nextInt(2); 

    if(random ==1) 
     hunters[1].x = hunters[1].x -1; 
    else 
     hunters[1].y = hunters[1].y -1; 
} 

public void Update() { 
    route11(); 
    route2(); 
    route3(); 
    route4(); 
    route5(); 
} 

方法路径2，路径3，...，route5看起来几乎一样，唯一改变的就是数组的用不同的猎人相对应的值。

该代码是否可以“缩小”？我很确定我的讲师很乐意为这样一个混乱而且非常反OO的代码减去我的标记。

而且，我所有的碰撞/分的代码看起来是这样的，它适用于个人猎人：

Answer 1

您的问题无关，与面向对象的设计。这只是为了学习如何使用可用的工具来编写更少冗余且更易于管理的代码。如果你在更新中使用for循环并传递每个单独的猎人，那么这会变得更加紧凑。

我会注意到，有一些不相关的OOP问题，你会很好地纠正。

• 猎人的成员，如X和Y不应该公开曝光，利用getter/setter方法

• 同样为玩家的得分成员/场

  public void update() 
  { 
      for(var i = 0; i < 5; i++) 
      { 
       route(hunters[i]); 
       collisionAndScoring(hunters[i]); 
      } 
  } 
  
  public void route(Hunter hunter) 
  { 
      Scanner in = new Scanner(System.in); 
      Random number = new Random(); 
      int random = number.nextInt(2); 
      if(random == 1) 
      { 
       hunters.x--; 
      } 
      else 
      { 
       hunter.y--; 
      } 
  } 
  
  public void collisionAndScoring(Hunter hunter) 
  { 
      if (hunter.x == 0 && hunter.y == 0) //You should define constants for these to give them more meaning 
      { 
       hunter.x = 11; //another opportunity for a constant 
       hunter.y = 11; //another opportunity for a constant 
       Player.score++; 
      } 
      }