Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

여러 BlockStrategy의 saveBlocks 메서드 분리 및 saveAll 사용 #190

Closed
4 tasks done
dpdmstjs opened this issue Oct 24, 2023 · 0 comments · Fixed by #191
Closed
4 tasks done

여러 BlockStrategy의 saveBlocks 메서드 분리 및 saveAll 사용 #190

dpdmstjs opened this issue Oct 24, 2023 · 0 comments · Fixed by #191
Assignees
@dpdmstjs
Labels
🔨 Refactor 코드 리팩토링

Comments

@dpdmstjs
Copy link
Contributor

dpdmstjs commented Oct 24, 2023 •
edited
Loading

기능

여러 BlockStrategy 클래스들의 saveBlocks 메서드에서 여러 작업을 한 번에 수행하고 있어 가독성과 유지보수성이 떨어질 수 있다고 판단하였습니다.
따라서 ListBlockStrategy를 비롯한 각 BlockStrategy 클래스에 대해 다음과 같이 리팩토링 및 성능 개선을 진행하였습니다.

작업 상세 내용

  • subData를 각 SaveRequest로 변환하는 부분을 convertSubDataToBlockSaveRequests 메서드로 분리
  • SaveRequest를 Entity로 변환하는 부분을 convertToBlocks 메서드로 분리
  • 모든 Entity를 저장하는 부분을 saveAllBlock 메서드로 변경하여 한 번에 모든 Entity들을 저장
  • 저장된 블록들의 정보를 배열에 추가하는 부분을 addToBlockInfoArray 메서드로 분리

참고 사항
@dpdmstjs dpdmstjs added the 🔨 Refactor 코드 리팩토링 label Oct 24, 2023
@dpdmstjs dpdmstjs self-assigned this Oct 24, 2023
dpdmstjs added a commit to dpdmstjs/jober-backend that referenced this issue Oct 24, 2023
@dpdmstjs
feat : save 메서드를 saveAll로 변경하기 위해 saveAll 메서드 추가 (Fastcampus-Final-Te…
297b5f8 
…am3#190)
dpdmstjs added a commit to dpdmstjs/jober-backend that referenced this issue Oct 24, 2023
@dpdmstjs
refactor : 여러 BlockStrategy의 saveBlocks 메서드 분리 및 saveAll 사용 (Fastcamp…
99d333b 
…us-Final-Team3#190)

1. subData를 각 SaveRequest로 변환하는 부분을 convertSubDataToBlockSaveRequests 메서드로 분리
2. SaveRequest를 Entity로 변환하는 부분을 convertToBlocks 메서드로 분리
3. 모든 Entity를 저장하는 부분을 saveAllBlock 메서드로 변경하여 한 번에 모든 Entity들을 저장
4. 저장된 블록들의 정보를 배열에 추가하는 부분을 addToBlockInfoArray 메서드로 분리
@dpdmstjs dpdmstjs mentioned this issue Oct 24, 2023
Merged
@dpdmstjs dpdmstjs linked a pull request Oct 24, 2023 that will close this issue
여러 BlockStrategy의 saveBlocks 메서드 리팩토링 #191
Merged
YangSooHyun0 pushed a commit that referenced this issue Oct 24, 2023
@dpdmstjs
여러 BlockStrategy의 saveBlocks 메서드 리팩토링 (#191)
1f6f5e1 
* recommit : processBlocks 메서드 분리 오류 해결

* feat : save 메서드를 saveAll로 변경하기 위해 saveAll 메서드 추가 (#190)

* refactor : 여러 BlockStrategy의 saveBlocks 메서드 분리 및 saveAll 사용 (#190)

1. subData를 각 SaveRequest로 변환하는 부분을 convertSubDataToBlockSaveRequests 메서드로 분리
2. SaveRequest를 Entity로 변환하는 부분을 convertToBlocks 메서드로 분리
3. 모든 Entity를 저장하는 부분을 saveAllBlock 메서드로 변경하여 한 번에 모든 Entity들을 저장
4. 저장된 블록들의 정보를 배열에 추가하는 부분을 addToBlockInfoArray 메서드로 분리
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dpdmstjs dpdmstjs

Labels
🔨 Refactor 코드 리팩토링
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

여러 BlockStrategy의 saveBlocks 메서드 리팩토링 dpdmstjs/jober-backend
1 participant
@dpdmstjs