Open june2 opened 1 year ago
layered architecture 잘 적용하셨는데 post하나만 하셨네요. 적용전후 비교해보았을때 어떤게 더 좋은코드인거 같은지 느낀점도 공유해주세요~
불피요한 파일이 푸쉬 되었네요. gitignore에 포함시켜서 푸쉬 방지해주세요
불필요한 주석은 삭제 해주세요 ! https://github.com/rammakasty/Layered-Architecture-Pattern/blob/29033748c53a087d029c8678086d514501bbfd90/repositories/posts.repository.js#L1
정렬은 서비스레이어에서 하는게 아니라 db조회시 정렬조건을 추가해서 가져와야해요 https://github.com/rammakasty/Layered-Architecture-Pattern/blob/29033748c53a087d029c8678086d514501bbfd90/services/posts.service.js#L9-L13
아직은 왜 좋은지는 잘 모르겠지만 코드에 익숙해지다 보면 자연스레 깨닫게 될 것 같습니다. 말씀하신 부분 기억해서 수정하겠습니다~
layered architecture 잘 적용하셨는데 post하나만 하셨네요. 적용전후 비교해보았을때 어떤게 더 좋은코드인거 같은지 느낀점도 공유해주세요~
불피요한 파일이 푸쉬 되었네요. gitignore에 포함시켜서 푸쉬 방지해주세요
불필요한 주석은 삭제 해주세요 ! https://github.com/rammakasty/Layered-Architecture-Pattern/blob/29033748c53a087d029c8678086d514501bbfd90/repositories/posts.repository.js#L1
정렬은 서비스레이어에서 하는게 아니라 db조회시 정렬조건을 추가해서 가져와야해요 https://github.com/rammakasty/Layered-Architecture-Pattern/blob/29033748c53a087d029c8678086d514501bbfd90/services/posts.service.js#L9-L13