縮小代碼

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++; 
      } 
      }