O Código abaixo tem o objetivo de realizar uma movimentação em um objeto no eixo Y de um objeto seguindo o mouse (o código funciona), veja que o código dentro do if
e do else if
são iguais a não ser pelo operador + e - quais seriam as formas de refatorar este código duplicado alterando somente os operadores da linha (somente o que esta em negrito):
mousePosition3D.y = transform.position.y + (initialMousePosition.y - mousePosition3D.y);
private void SwipeContainer(Directions direction){
if(direction == Directions.Top && transform.localPosition.y < maxTopPosition){
Vector3 position = Input.mousePosition;
Vector3 mousePosition3D = Camera.main.ScreenToWorldPoint(position);
mousePosition3D.z = -1f;
mousePosition3D.x = transform.position.x;
mousePosition3D.y = transform.position.y + (initialMousePosition.y - mousePosition3D.y);
transform.position = Vector3.MoveTowards(
transform.position, mousePosition3D, speedMoviment * Time.deltaTime
);
}else if(direction == Directions.Bottom && transform.localPosition.y > maxBottomPosition){
Vector3 position = Input.mousePosition;
Vector3 mousePosition3D = Camera.main.ScreenToWorldPoint(position);
mousePosition3D.z = -1f;
mousePosition3D.x = transform.position.x;
mousePosition3D.y = transform.position.y - (initialMousePosition.y - mousePosition3D.y);
transform.position = Vector3.MoveTowards(
transform.position, mousePosition3D, speedMoviment * Time.deltaTime
);
}
Este problema de refatoração não se aplica somente a esta linguagem (C#) e situação especifica, quais seriam as formas adequadas de refatoração (que seja aplicável a maioria das linguagens como: C#, Java, PHP, Javascript, C)
if else
simples. Da forma que está ele vai fazer as duas comparações, mas se não atender nenhuma das duas ele não fará nada.