반복적 인 코드 만들기

Question

캐릭터의 움직임과 관련된 게임에서 코드의 일부를 작성했으며이 코드는 매우 반복적입니다 (거의 4 방향 모두 동일).

'if'조건의 테스트로 인해서 어떻게하면 더 간단하게 만들 수 있는지 알 수 없습니다. 다른 경우에는 양수/음수 변수가있는 시스템을 만들어서 위치 변경.

도움 주셔서 감사합니다. 이 라인을 따라

for (var i = 0; i < boxArray.length; i++) { 
    var box = boxArray[i]; 

    if (controller.left && currentLevel[this.id - this.moveHorizontal] != 1) { 
     // Enregistrement de la position dans le tableau d'undo/redo 
     this.index++; 
     var array = [this.x, this.y, this.id]; 
     this.undoArray.push(array); 
     // Si box à côté et pas de collision possible 
     if (box.id == this.id - this.moveHorizontal && currentLevel[this.id - this.moveHorizontal * 2] != 1 && currentLevel[this.id - this.moveHorizontal * 2] != 2) { 
      // Décalage de la position du player 
      this.x -= this.boxWidth; 
      this.id -= this.moveHorizontal; 
      currentLevel[this.id] = 8; 
      currentLevel[this.id + this.moveHorizontal] = 0; 
      // Décalage de la position de la box 
      box.x -= this.boxWidth; 
      box.id -= this.moveHorizontal; 
      currentLevel[this.id - this.moveHorizontal] = 2; 
      controller.left = false; 
     } 
     // Sinon si aucun objet à côté 
     else if (currentLevel[this.id - this.moveHorizontal] == 0 || currentLevel[this.id - this.moveHorizontal] == 3) { 
      // Décalage de la position du player 
      this.x -= this.boxWidth; 
      this.id -= this.moveHorizontal; 
      currentLevel[this.id] = 8; 
      // Décalage de la position du sol 
      currentLevel[this.id + this.moveHorizontal] = 0; 
      controller.left = false; 
     } 
    } 
    else if (controller.right && currentLevel[this.id + this.moveHorizontal] != 1) { 
     this.index++; 
     var array = [this.x, this.y, this.id]; 
     this.undoArray.push(array); 
     if (box.id == this.id + this.moveHorizontal && currentLevel[this.id + this.moveHorizontal * 2] != 1 && currentLevel[this.id + this.moveHorizontal * 2] != 2) { 
      this.x += this.boxWidth; 
      this.id += this.moveHorizontal; 
      currentLevel[this.id] = 8; 
      currentLevel[this.id - this.moveHorizontal] = 0; 
      box.x += this.boxWidth; 
      box.id += this.moveHorizontal; 
      currentLevel[this.id + this.moveHorizontal] = 2; 
      controller.right = false; 
     } 
     else if (currentLevel[this.id + this.moveHorizontal] == 0 || currentLevel[this.id + this.moveHorizontal] == 3) { 
      this.x += this.boxWidth; 
      this.id += this.moveHorizontal; 
      currentLevel[this.id] = 8; 
      currentLevel[this.id - this.moveHorizontal] = 0; 
      controller.right = false; 
     } 
    } 

Answer 1

뭔가

for (var i = 0; i < boxArray.length; i++) { 
    var box = boxArray[i]; 

    var direction = { 
     'left': -1, 
     'right': 1 
    }; 

    for (var d in direction) { 
     var mh = this.moveHorizontal * direction[d]; 

     if (controller[d] && currentLevel[this.id + mh])] { 
      this.index++; 
      var array = [this.x, this.y, this.id]; 
      this.undoArray.push(array); 

      if (box.id == this.id + mh && currentLevel[this.id + mh * 2] != 1 && currentLevel[this.id + mh * 2] != 2) { 
       this.x += this.boxWidth * direction[d]; 
       this.id += mh; 
       currentLevel[this.id] = 8; 
       currentLevel[this.id + this.moveHorizontal] = 0; 
       // Décalage de la position de la box 
       box.x += this.boxWidth * direction[d]; 
       box.id += mh; 
       currentLevel[mh] = 2; 
       controller[d] = false; 
      } else if (currentLevel[this.id + mh] == 0 || currentLevel[this.id + mh] == 3) { 
       // Décalage de la position du player 
       this.x += this.boxWidth * direction[d]; 
       this.id += mh; 
       currentLevel[this.id] = 8; 
       // Décalage de la position du sol 
       currentLevel[this.id + (mh * -1)] = 0; 
       controller[d] = false; 
      } 
     } 
    } 
} 

Answer 2

난 당신이 확실히이 코드를 단순화 할 수 있다고 생각합니다. 그런 다음 측 moveLeft 또는었던 MovieRight을 나타내는 문자열입니다

moves[side](...); 

처럼 말할 수

var moves = { 
    moveLeft: function() { ... }, 
    moveRight: function() { ... } 
}; 

: 예를 들어, 객체처럼합니다. 게다가

그룹화 할 수 있습니다 왼쪽과 오른쪽 모두에서 반복되는 이와 같은 반복 기능 : 함수에서의

function initMovement() { 
    var array = [this.x, this.y, this.id]; 
    this.index++; 
    this.undoArray.push(array); 
}