오늘 일하는 중 시간적 여유가 생겨 이때다 싶어 보기 불편했던 코드를 보다 간편하게 수정했다.
수정한 부분은 마이페이지의 프로필 변경 함수이다.
새로운 제품들이 들어오고 UI도 함께 리뉴얼 시키면서 프로필이미지를 설정하는 부분들의 UI도 함께 수정되었다.
그러면서 이번 작업에서 아쉬웠던 점은 시간 비용이 높았던 점이다.
시간에 맞춰 진행하다보니 프로필이미지를 세팅하는 tsx파일 안에 함수를 작성하였고
이후 기획서에 추가된 사항들을 반영하다보니 공통된 역할을 하는 함수인데 각 tsx파일마다 작성하고 혹시 별도의 예외처리를 해줘야하는지 여러 테스트를 진행하며 많은 시간을 소요하게 되었다.
내가 작업하는 곳 외에는 최대한 건드리지 않아야한다는 생각에 자주 사용하진 않았지만 충분히 사용할 가치가 있다고 생각되어 utils에 공통 함수를 작성하였다.
(보안상 구체적인 용어를 사용하지 않음)
리뉴얼 이전 고객 -> 1. 프로필이 있을 경우, 프로필 변경 화면 시 랜덤 프로필 제공/ 2. 없을 경우 / 3. 에러데이터 처리를 백엔드에서 처리 불가
리뉴얼 후 고객 -> 4. 랜덤 프로필 제공 / 5. 있을 경우
공통함수로 분리하고자 했던 Point
1. 5가지 상황에 따라 이미지 주소가 리턴되는 함수는 하나로 정리할 것:handleProfileImage
2. 데이터 오류를 체크하는 함수는 따로 분리할 것: isCheckEmptyProfileImgpath
3. 10가지의 프로필 랜덤이미지도 utils에서 가져와 사용하도록 할 것