매직 넘버 없애기 | Frontend Fundamentals

[giscus[bot]]

매직 넘버 없애기 | Frontend Fundamentals

변경하기 쉬운 프론트엔드 코드를 위한 지침서

https://frontend-fundamentals.com/code/examples/magic-number-cohesion.html

[novice0840]

안녕하세요 글 잘 읽었습니다.
현재 페이지하고 1 가독성 페이지에 B. 매직 넘버에 이름 붙이기가 같은 내용인데 의도하신건가요?

[LimChaeJune]

가독성("300ms" 라는 값이 어떤 맥락으로 사용하는지 판단하기 힘듬)과 응집도(수정이 일부분만 반영됨) 관점으로 페이지를 구분하신 것 같습니다.

[rbgksqkr]

안녕하세요! 매직 넘버를 상수화해서 의미를 드러내는 것에 있어서 가독성 측면에선 큰 이점이 있다고 생각합니다.

같이 수정되어야 할 코드 중 한쪽만 수정된다는 점에서, 응집도가 낮은 코드라고도 할 수 있어요.

다만, 해당 글에서는 응집도 관점에서 작성해주셨는데, 예시로 말씀해주신 재생하는 애니메이션을 바꿨을 때 조용히 서비스가 깨질 수 있는 위험성은 상수화를 하더라도 그대로 존재한다고 느껴졌어요.
관련 코드를 수정했을 때 수정사항이 같이 반영할 수 있도록 인지하게만 하면, 응집도가 높아졌다고 볼 수 있는걸까요?