sihyun10 / FE_PS_Final_Refactor

몰리턴 기업 연계 프로젝트 코드리뷰 및 리팩토링
0 stars 0 forks source link

각자 리팩토링 한 부분 (시현) #3

Closed sihyun10 closed 1 year ago

sihyun10 commented 1 year ago

Description

각자만의 방식으로 코드 수정(리팩토링)해보는 작업을 진행.

변경한 내용들

변경한 이유 설명

src>components>Table>EulTable, GapTable 폴더에 columns를 다 따로따로 파일로 작성해, import해서 불러왔는데, 이렇게 따로따로 노는거보다 각각의 테이블 파일에 다 합치는게 파악하기 쉬워 합쳤습니다.

사이트에서 탭별로 사용되는 테이블을 한 폴더 안에 넣어두면 파악하기 쉽기에 시세파트에서 쓰이는 테이블 부분도 src>components>Table 폴더에 생성하였습니다. 이 과정에서 사용하지 않아도 되는 styled 부분 수정하였습니다.

현재, 코드안에 스타일이 작성되어있는 경우도 있고, 아닌경우도 있었습니다. 코드와 스타일이 길어지게 되는 경우, 가독성이 떨어진다 생각해, 코드는 코드대로. 스타일은 스타일대로 분리하였습니다. 하지만, 코드와 스타일 부분이 한눈에 보기 쉬운 파일들이 있었습니다. 이 경우엔 굳이? 스타일파일을 따로 생성해 나누는게 비효율적이라 생각이 들어 이부분은 건들이지 않았습니다.

마지막으로 전체 폴더 구조를 파악해보았고, 우리 프로젝트에서 pages>Pra>PreDetail.tsx, PraPrice.tsx, PraUpload.tsx 파일이 들어가있는데, 따로 노는것 같아 각자 필요한 부분으로 이동을 시켜줘야한다고 생각하였고, common 폴더에서 쓰이고 있는것들이 '과연 잘 재사용하고있는건가?'라는 생각을 하였습니다. 또, pages>Detailed>MarketPrice.tsx 파일 외엔 Detailed부분에서 스타일을 적용해주고 있는데, 이 파일은 아니였다. 따라서 스타일을 이쪽에서 지정하도록 변경해야겠단 생각이 들었다. (통일성 맞춰주기 위해서)