현재 모달 컴포넌트를 구현하는 방식이 가독성 및 직관성이 떨어진다는 생각이 든다.
현재는 recoil의 modalType이라는 state를 확인해서, 그 state에 알맞는 방식으로 모달을 띄워주고 있는데, 이렇게 하는 방식이 직관성이 떨어지는 것 같다. 모달을 사용하는 페이지에서 어떤 모달을 띄워줄 것인지를 확인할 수 없다는 점이 직관성을 떨어뜨리는 듯 하다. 이런 부분을 개선할 필요가 있을 것 같다.
2.유연성 :
또한 이 방식으로 사용했을 때, 모달을 유연하게 사용하는 것이 어려워진다. 모달을 사용하는 페이지에서 props를 넘겨줘서 유연하게 텍스트를 입력하거나 하는 행위들을 할 수 가 없는 구조로 되어있기 때문에 이것은 잘 구현했다고 말하기 어려울 것 같다.
본 이슈는 다음의 이슈에서 연결된다 #33
리팩토링하려는 이유
1.직관성 :
현재 모달 컴포넌트를 구현하는 방식이 가독성 및 직관성이 떨어진다는 생각이 든다. 현재는 recoil의 modalType이라는 state를 확인해서, 그 state에 알맞는 방식으로 모달을 띄워주고 있는데, 이렇게 하는 방식이 직관성이 떨어지는 것 같다. 모달을 사용하는 페이지에서 어떤 모달을 띄워줄 것인지를 확인할 수 없다는 점이 직관성을 떨어뜨리는 듯 하다. 이런 부분을 개선할 필요가 있을 것 같다.
2.유연성 :
또한 이 방식으로 사용했을 때, 모달을 유연하게 사용하는 것이 어려워진다. 모달을 사용하는 페이지에서 props를 넘겨줘서 유연하게 텍스트를 입력하거나 하는 행위들을 할 수 가 없는 구조로 되어있기 때문에 이것은 잘 구현했다고 말하기 어려울 것 같다.