1. 입사과제 코드리뷰
우리 회사 프론트엔드 입사과제로 스터디를 하고 있다.
- 입사과제 결과물을 만들고,
- 경력이 있는 개발자 두분이 리뷰해주시고,
- 그걸 다시 반영해서 만드는
방식으로 진행하고 있다.
👇 스터디 시작에 대한 것은 이전 회고에 기록해두었다.
2022.10.19 - [분류 전체보기] - {10월 둘째주 회고} 리액트 스터디 3개 | 아카이브 필요성 | table 태그 궁금쓰
{10월 둘째주 회고} 리액트 스터디 3개 | 아카이브 필요성 | table 태그 궁금쓰
이번주는 리액트 스터디 두개를 시작했다. 하나는 코딩애플의 리액트 강의를 듣는 스터디, 다른 하나는 개발공부 스터디 그리고 단발성으로 진행되는 우리 회사 프론트 입사과제 스터디 하나가
yuncoco.tistory.com
이번주에는 드디어 1차 리뷰를 받았다. 어떤 리뷰는 알고 있다고 생각했던 기본적인 것이었음에도 내가 지키지 않고 있는 것들도 있었다. 그리고 리뷰를 받고 나니 확실히 실무를 할때에 내가 내 코드를 보는 시각이 조금 달라졌다. 이제는 규칙이 조금 생겼다. 예를 들면 로직과 view를 분리하고 역할에 맞는 위치에 코드를 생성한다던가, 전역 store가 필요한 로직과 그렇지 않은 로직을 구분한다던가 하는 규칙이다. 이 피드백들을 반영한 과제와 지금 1차 결과물을 비교했을때 어떻게 다를지 궁금하다.
2. 커머스 화면이동 방식 변경
기존에 우리 커머스 서비스의 코드는 특이한 구조로 되어있었다. 고도몰과 자체 UI이 공존하고 있고, 화면이동을 앱처럼 슬라이드 in/out하려다보니 일반적으로 사용되는 browser router가 아닌 하드코딩으로 화면을 끄고 키는 형식이었다. 그래서 커머스 서비스가 점점 고도화 되고 화면이 많아짐에 따라, 점점 화면이동이 자유롭지 않은 케이스가 프로젝트당 1~2개 정도 지속적으로 발생했다.
그래서 이번엔 전반적으로 하드코딩된 부분을 전부 없애고 browser router를 활용한 화면이동 구조로 바꾸기로 했다!!! 이게 이제는 가능한 이유는 그동안 고도몰을 걷어내기 위한 작업을 거치기고 했고, 화면 이동 애니메이션을 잠시 사용하지 않기로 했기 때문이다. 그동안 화면이동과 예외케이스 때문에 덕지덕지 추가했던 로직들을 전부 삭제하고 있어서 속이 너무 시원하다.
부채는 만들때도 힘들고 치울때도 힘들다는걸 직접 느끼고 있는 요즘이다 ㅎㅎㅎ