Skip to content

[IC2_BE] 구인성- 설문조사 서비스 과제#18

Open
gooinsung wants to merge 48 commits into
FC-InnerCircle-ICD2:mainfrom
gooinsung:main
Open

[IC2_BE] 구인성- 설문조사 서비스 과제#18
gooinsung wants to merge 48 commits into
FC-InnerCircle-ICD2:mainfrom
gooinsung:main

Conversation

@gooinsung

@gooinsung gooinsung commented Nov 25, 2024

Copy link
Copy Markdown

전체적인 프로젝트 구성에 대한 리뷰를 듣고싶습니다.

비전공 신입 개발자로 첫 회사에 입사하여, 거진 혼자 프로젝트를 담당하여 진행하다 보니, '동작' 위주의 개발을 하게 되었고, 누군가와 제 코드를, 누군가의 코드를 같이 공유하여 생각을 해 본적이 없습니다. 프로젝트의 전반적인 설계, 디자인 패턴, 습관 등 전반적인 리뷰를 듣고 싶습니다.

이번 과제가 실제 채용 과정 중에 포함되어 있다고 가정했을 때, 제 프로젝트가 어느정도 수준으로 평가될지 알고 싶습니다. 가장 큰 고민이 이직이고, 이번 과정을 통해 이직에 대한 힌트를 얻어가고 싶습니다. 참고로 저는 지금 좋은 처우를 기대할 수 있는 회사로의 이직을 기대하고 있지는 않습니다. 직후 회사를 기점으로 3~4년 이후 비교적 좋은 직장에서 근무하고 싶다는 생각이 있습니다.

프로젝트와 코드를 기반으로 백앤드 개발자로서 성장하기 위해 더 신경써야 할 부분에 대한 이야기를 듣고 싶습니다. 지금은 그냥 생각나는대로, 눈에 보이는대로 얕게 공부하는 습관을 가지고 있습니다. 조금 더 건설적인 방식으로 학습을 하고 싶습니다.

JPA 와 테스트코드 작성을 처음 접해봅니다. 이에 대한 피드백도 부탁드리겠습니다.

이번 프로젝트에 최선을 다한 건 사실이지만, 제가 기대하는 리뷰는 '요구사항을 얼마나 충족시켰냐' 보다 '내 수준이 어느정도 인가' 에 더 가까운것 같습니다. 가감없이 부족한 부분 지적해주시면 감사하겠습니다! 바쁜 시간 내어 코드 확인해주셔서 감사합니다!

ps. SurveyQueryRepository.getSurveyById() 의 메서드를 실행하면 자꾸 에러가 발생합니다.... 원인을 찾지 못해 다른 방식으로 문제를 해결해 보았습니다. 이에 대한 원인도 같이 살펴봐 주시면 감사하겠습니다..

@gooinsung gooinsung added the WIP 작업을 진행중일 때 사용 label Nov 25, 2024
gooinsung and others added 16 commits November 26, 2024 14:30
코드 컨벤션 추가
1. 서비스 분리하기 (도메인 서비스, 도메인 조회 비즈니스, 도메인 동작 비즈니스)
2. 예외처리(조상격 예외 클래스 핸들링 추가 및 Optional 활용)
3. 설문조사 update  시 기존의 항목들 담을 새 테이블 생성 관리
4. 엔티티 Noargument 로 초기화 시 어떻게되는지 테스트
도메인 서비스에 [{queryBusiness(조회)}, {actionBusiness(동작)}]
@gooinsung gooinsung added the Needs Review 작업이 완료되어 리뷰가 필요할 때 사용 label Nov 28, 2024
@gooinsung gooinsung removed the Needs Review 작업이 완료되어 리뷰가 필요할 때 사용 label Nov 29, 2024
@gooinsung gooinsung added the Needs Review 작업이 완료되어 리뷰가 필요할 때 사용 label Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review 작업이 완료되어 리뷰가 필요할 때 사용 WIP 작업을 진행중일 때 사용

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant