在Java中重構此功能

Question

我正在學習Java，我知道關於新手程序員的一個大抱怨是我們製作了很長時間且涉及的方法，應該將其分解爲幾個。那麼這裏是我寫的一個例子。 :-D。

public void buildBall(){ 

     /* sets the x and y value for the center of the canvas */ 
     double i = ((getWidth()/2)); 
     double j = ((getHeight()/2)); 

     /* randomizes the start speed of the ball */ 
     vy = 3.0; 
     vx = rgen.nextDouble(1.0, 3.0); 
     if (rgen.nextBoolean(.05)) vx = -vx;  

     /* creates the ball */ 
     GOval ball = new GOval(i,j,(2 *BALL_RADIUS),(2 * BALL_RADIUS));  
     ball.setFilled(true); 
     ball.setFillColor(Color.RED); 
     add(ball); 


     /* animates the ball */ 
     while(true){ 
      i = (i + (vx* 2)); 
      j = (j + (vy* 2)); 
      if (i > APPLICATION_WIDTH-(2 * BALL_RADIUS)){ 
       vx = -vx;  
      } 

      if (j > APPLICATION_HEIGHT-(2 * BALL_RADIUS)){ 
       vy = -vy; 
      } 

      if (i < 0){ 
       vx = -vx; 
      } 

      if (j < 0){ 
       vy = -vy; 
      } 

      ball.move(vx + vx, vy + vy); 
      pause(10); 

      /* checks the edges of the ball to see if it hits an object */ 
      colider = getElementAt(i, j); 

      if (colider == null){ 
       colider = getElementAt(i + (2*BALL_RADIUS), j); 
       } 
      if (colider == null){ 
       colider = getElementAt(i + (2*BALL_RADIUS), j + (2*BALL_RADIUS)); 
       } 
      if (colider == null){ 
       colider = getElementAt(i, j + (2*BALL_RADIUS)); 
       } 

      /* If the ball hits an object it reverses direction */ 
      if (colider != null){ 
       vy = -vy; 


      /* removes bricks when hit but not the paddle */ 
       if (j < (getHeight() -(PADDLE_Y_OFFSET + PADDLE_HEIGHT))){ 
       remove(colider); 
       } 

      } 

     } 

你可以從方法的標題看到，我開始時有着「造球」的好意。

有我撞到的幾個問題：

的問題是，然後我需要移動球，所以我創建了一個while循環。我沒有看到任何其他的方式去做，除了保持「真實」，所以這意味着我在這個循環下創建的任何其他代碼都不會發生。我沒有讓while循環變成一個不同的函數，因爲我使用了這些變量i and j。

所以我沒有看到我可以重構這個循環之外。

所以我的主要問題是：

我將如何傳遞的i and j值，以一種新的方法：「animateBall」和我將如何使用新的方法 ball.move(vx + vx, vy + vy);如果球已經在buildBall方法被宣佈？

我理解這可能是更好的理解變量的作用域和參數傳遞一個簡單的事情，但我還沒有應用...

Answer 1

這可以重構爲三種方法 a> build ball：創建球對象並設置初始位置：buildBall（） b>使整個while循環動畫（除了對碰部分動畫）（球，vx，vy） c>獲取collider getCollider（）

由於球是一個對象，並且您已經設置了i，j作爲它的字段，它們將被傳遞。 Java按值傳遞所有參數。對象生活在堆上;對象引用通過值作爲方法參數傳遞。

編輯，添加僞代碼

class Animator{ 
    void animateBall(){ 
    Ball ball = buildBall(); //Ball will have i,j,radius etc set by this method 
    int vx = randomNumber(); 
    int vy = randomNumber(); 
    moveIt(vx,vy, ball); 
    } 
    void moveIt(int vx, int vy, Ball ball){ 
     while(true){ 
      //move the ball, change i,j fields of ball 
      //check for collission etc 
      Collider collider = getCollider(ball); 
      //change direction based on collider etc. 
     } 
    } 
    Collider getCollider(Ball ball){ 
    //collision code here 
    } 
} 

Answer 2

這是少了幾分多餘

 colider = getElementAt(i, j); 

      /* If the ball hits an object it reverses direction */ 

     if (colider != null && j < (getHeight() -(PADDLE_Y_OFFSET + PADDLE_HEIGHT))) 
     { 
      vy = -vy; 

     /* removes bricks when hit but not the paddle */ 
      remove(colider); 
     } 


      else 
      { 
       colider = getElementAt(i + (2*BALL_RADIUS), j); 
       colider = getElementAt(i + (2*BALL_RADIUS), j + (2*BALL_RADIUS)); 
       colider = getElementAt(i, j + (2*BALL_RADIUS));  
      }