[🐣질문] 이 코드 ㅄ같음??

[익명(1.229)]

public class UI_Menu : MonoBehaviour

{

    [SerializeField] LayerMask layerMask;

    Vector2 startMousePos;

    Vector2 endMousePos;

    Vector2 curMousePos;

    Vector2 nextMousePos;

    bool canDrag = false;

    private void Update()

    {

        if (Input.GetMouseButtonDown(0))

        {

            startMousePos = Camera.main.ScreenToWorldPoint(Input.mousePosition);

        }

        if (Input.GetMouseButton(0) && canDrag == false)

        {

            endMousePos = Camera.main.ScreenToWorldPoint(Input.mousePosition);

            if (Mathf.Abs(startMousePos.y - endMousePos.y) > 0.5f)

            {

                canDrag = true;

            }

        }

        if (Input.GetMouseButton(0) && canDrag == true)

        { 

            nextMousePos = Camera.main.ScreenToWorldPoint(Input.mousePosition);

            RaycastHit2D hit = Physics2D.Raycast(nextMousePos, Vector2.zero, Mathf.Infinity, layerMask);

            

            if (hit.collider == null)

                return;

            if (gameObject.transform.position.y < 3.57f && nextMousePos.y > curMousePos.y)

            {

                gameObject.transform.Translate(0, 0.06f, 0);

            }

            if (gameObject.transform.position.y > -4.8f && nextMousePos.y < curMousePos.y)

            {

                gameObject.transform.Translate(0, -0.06f, 0);

            }

            curMousePos = nextMousePos;

        }

        if (Input.GetMouseButtonUp(0) && canDrag == true)

        {

            canDrag = false;

        }

    }

}

메뉴창 드래그하면 위아래로 움직임

메뉴창 Y포지션이 -4.8~3.57 범위 넘어가면 안움직임

임계치 이상 넘어가야 드래그가 작동함 (드래그가 아니고 터치를 하고싶은데 삐끗해서 드래그 되는거 방지)

작동은 되는데 왤케 지저분해보이지??

[흉황(smk4397)]

일단 if문 내에서 처리하는 게 canDrag = true 밖에 없는 경우엔 if문의 조건식을 바로 부울 변수에 대입하는 것으로 if문 하나 제거 가능하고 같은 원리로 canDrag = false도 조건식 앞에 !를 붙여서 배입하는 것으로 if문 제거 가능함 - dc App

[개발용더미(sy1mn1t4y83k)]

반복되는 조건인 Input.GetMouseButtonUp(0) 부분은 한번만 위에서 if로 따로 묶어도 좋을듯 상수 값들이 표현식에 바로 들어가는게 지저분하면 상단에 상수변수로 따로 선언해 두면 다소 깔끔해 보임 현재 로직에서만 유효하긴 하겠다만 모든 조건이 Input.GetMouseButtonUp(0)가 통과되어야 넘어가는 상황이니 fast return 기법을 적용해두면 스코프 뎁스가 하나 줄어서 보기 편할듯

[ㅇㅇ 1(119.196)]

canDrag = !Input.GetMouseButtonUp(0) && canDrag;

[거붓(ggp02217)]

하나의 메서드에 담당하는 역할이 많고 if문이 3중 이상으로 중첩되서 그려