Week3 최종 보고 #21

20jcode · 2024-09-23T09:00:35Z

돌봄다리 서비스는 요양원에서 사용하며, 요양보호사들의 차트 작성을 돕고,

작성된 차트를 기반으로 보호자에게 입소자(돌봄 대상자)들에 대한 기록을 전송해주는 서비스입니다.

일정보다 늦게 제출하게되어 죄송하다는 말씀을 드리며, 앞으로는 이런 일이 없도록 하겠습니다!!

프로젝트 관련 몇가지 질문이 있는데요,

팀원들의 실력을 잘 모르는 상태에서 어떤 식으로 개발 파트를 분배하고 이를 관리하는 것이 효율적인지 여쭤봅니다.
예외 핸들링을 어떤 방식으로 하면 좋을 지에 대해서 여쭤봅니다.

감사합니당

#5 공용 db 연결하기

- 식별자, 생성/수정일자, 삭제 상태값 등의 공통 필드를 baseEntity로 정의

- 청결, 식사 관련 필드는 임베디드 타입으로 선언

- chart와 신체활동(bodyNote)는 일대일 연관관계 설정

- 혈압 관련 필드는 임베디드 타입으로 관리

- baseEntity 상속

프로젝트 오류 해결

feat: baseEntity 삭제 함수 추가

[관리자 페이지] 돌봄대상자, 요양보호사 도메인 CRUD

* feat: guardians * feat: #8 guardians 코드리뷰 반영 * fix: chart 도메인 부분 수정 * refactor: guardian 폴더 분리, exception 등 코드리뷰 반영 * fix: chart is_active 수정

Week3 최종 마감

cheol-95

안녕하세요~
프로젝트 설정 및 기본적인 비즈니스 로직까지 작성하셨군요 👍

코드는 전반적으로 잘 작성하셨습니다!
추가로 개발 초기부터 컨트롤러 레이어에 Swagger를 적용하는것도 고려해보세요!

고생하셨습니다!

Q. 팀원들의 실력을 잘 모르는 상태에서 어떤 식으로 개발 파트를 분배하고 이를 관리하는 것이 효율적인지 여쭤봅니다.

일단 현업이 아닌 지금 상태에서는 각 팀원이 어떤 부분에 관심이 있는지 파악하는 과정이 필요합니다. 처음부터 기능을 나누어 개발하기 보다는 강점이 있거나 관심있는 분야, 혹은 경헙해보고 싶은 작업을 진행하는 게 좋습니다. 예를 들어 누군가는 DB 관련 작업을, 누군가는 인증 관련 작업, 누군가는 인프라 관리쪽에 관심이 있을 수 있으니 니즈를 파악하고 업무를 분배하는것이 좋습니다.

코드 리뷰나 페어 프로그래밍을 통해 서로 피드백을 주고받는 것도 좋은 방법입니다. 이렇게 하면 실력 차이를 줄이고, 팀 전체의 협업 능력을 높이는 데 도움이 됩니다. 정기적인 피드백 세션을 통해 진행 상황을 점검하고, 문제를 미리 파악해 나가는 것도 효과적입니다.

가장 중요한 것은 작업이 끝난 후에 각자가 맡은 일을 어떻게 해결하였는지 PR이나 문서, 화상회의로 공유하는 시간을 꼭 가져야 합니다. 멘티분들은 현업에 뛰어들기 위하여 위에서 언급한 모든 기술(DB, 인프라 등)을 경험하는 것이 중요하기 때문에 자신이 작업하면서 겪었던 이슈와 그것을 해결한 경험을 공유하는것이 중요합니다.

Q. 예외 핸들링을 어떤 방식으로 하면 좋을 지에 대해서 여쭤봅니다.

Step2에서 진행한 방식대로 전역 예외 처리 핸들러를 사용하고, 커스텀 예외 클래스, 에러 코드 정의, 그리고 에러 응답 포맷을 정의하면 될 것 같습니다.

[전체 흐름]

애플리케이션에서 예외가 발생하면, 그 예외가 GlobalExceptionHandler에서 가로채집니다.
발생한 예외가 BusinessException인 경우, 해당 예외에 미리 정의된 ErrorCode를 참조하여 적절한 에러 메시지와 상태 코드를 반환합니다.
ErrorCode에서 정의한 메시지와 상태 코드를 기반으로 ErrorResponse 객체를 생성하고, 이를 클라이언트에게 응답으로 돌려줍니다.

cheol-95 · 2024-09-23T16:14:58Z

src/main/java/dbdr/DbdrApplication.java


 @SpringBootApplication
+@EnableJpaAuditing


EnableJpaAuditing를 사용하여 엔티티 생성, 수정 시간을 자동으로 관리하는군요 👍

cheol-95 · 2024-09-23T16:15:58Z

src/main/java/dbdr/controller/AdminGuardiansController.java

+    @PutMapping("/{guardianId}/update")
+    public ResponseEntity<GuardiansResponse> updateGuardianAuth(
+        @PathVariable("guardianId") Long guardianId,
+        @Valid @RequestBody GuardiansRequest guardiansRequest) {
+        GuardiansResponse guardiansResponse = guardiansService.updateGuardianById(guardianId,
+            guardiansRequest);
+        return ResponseEntity.ok(guardiansResponse);
+    }


PUT을 사용하였으니 path에는 update를 제거해도 될 것 같습니다.

cheol-95 · 2024-09-23T16:16:20Z

src/main/java/dbdr/controller/AdminGuardiansController.java

+    @PutMapping("/{guardianId}/delete")
+    public ResponseEntity<Void> deleteGuardianAuth(
+        @PathVariable("guardianId") Long guardianId) {
+        guardiansService.deleteGuardianById(guardianId);
+        return ResponseEntity.noContent().build();
+    }


PUT대신 DELETE를 사용하고 path에는 delete를 제거해도 될 것 같습니다.

cheol-95 · 2024-09-23T16:23:53Z

src/main/java/dbdr/controller/GuardiansController.java

+    @PutMapping("/{guardianId}")
+    public ResponseEntity<GuardiansResponse> updateGuardianInfo(
+        @PathVariable("guardianId") Long guardianId,
+        @Valid @RequestBody GuardiansRequest guardiansRequest) {
+        GuardiansResponse guardiansResponse = guardiansService.updateGuardianById(guardianId,
+            guardiansRequest);
+        return ResponseEntity.ok(guardiansResponse);
+    }


권한 처리는 없어도 될까요?

스프링 시큐리티를 활용하여 @PreAuthorize 어노테이션으로 컨트롤러 단에서 권한 처리를 하려고 현재 개발중입니다!

cheol-95 · 2024-09-23T16:24:04Z

src/main/java/dbdr/controller/GuardiansController.java

+import org.springframework.web.bind.annotation.RestController;
+
+@RestController
+@RequestMapping("/v1/guardians")


REST API 버전을 URL 경로 /v1/... 에 직접 하드코딩하지 말고, 프로퍼티 파일 등을 사용하여 유연하게 처리하는 것이 좋습니다.

cheol-95 · 2024-09-23T16:35:06Z

src/main/java/dbdr/service/CareworkerService.java

+    @Transactional(readOnly = true)
+    public CareworkerResponseDTO getCareworkerById(Long id) {
+        Careworker careworker = careworkerRepository.findById(id)
+                .orElseThrow(() -> new IllegalArgumentException("요양보호사를 찾을 수 없습니다."));
+        return toResponseDTO(careworker);
+    }
+


IllegalArgumentException은 일반적인 용도로 사용하기에는 부적절합니다. 더 구체적이고 의미 있는 예외 클래스를 만들어 사용하는 것이 좋습니다.

cheol-95 · 2024-09-23T16:35:58Z

src/main/java/dbdr/service/CareworkerService.java

+        Careworker careworker = careworkerRepository.findById(id)
+                .orElseThrow(() -> new IllegalArgumentException("요양보호사를 찾을 수 없습니다."));


해당 코드가 중복되는데, 코드 분리를 고려해보세요.

cheol-95 · 2024-09-23T16:36:03Z

src/main/java/dbdr/service/CareworkerService.java

+        Careworker careworker = careworkerRepository.findById(id)
+                .orElseThrow(() -> new IllegalArgumentException("요양보호사를 찾을 수 없습니다."));


해당 코드가 중복되는데, 코드 분리를 고려해보세요.

cheol-95 · 2024-09-23T16:36:54Z

src/main/java/dbdr/service/CareworkerService.java

+    public CareworkerResponseDTO createCareworker(CareworkerRequestDTO careworkerRequestDTO) {
+        Careworker careworker = new Careworker(
+                careworkerRequestDTO.getInstitutionId(),
+                careworkerRequestDTO.getName(),
+                careworkerRequestDTO.getEmail(),
+                careworkerRequestDTO.getPhone()
+        );


이메일이나 전화번호와 같은 필드에 대해 중복 검사를 추가하는 것이 좋습니다.

cheol-95 · 2024-09-23T16:38:11Z

src/main/java/dbdr/service/GuardiansService.java

+    private void phoneNumberExists(String phone) {
+        if(guardiansRepository.existsByPhone(phone)){
+            throw new IllegalArgumentException("존재하는 전화번호입니다.");
+        }
+    }


명확한 에러를 정의하고 throw하는게 좋아보입니다.

pykido and others added 30 commits September 20, 2024 01:29

feat : build.gradle : MySQL 의존성 추가

0f96d91

feat : application.yml : AWS RDS 데이터베이스 연결 정보를 포함.

5b39a83

Merge pull request #12 from pykido/#5-공용-DB-연결하기

675455a

#5 공용 db 연결하기

chore: 설정 파일 수정

e81b554

feat: 공통 엔티티 작성

683b9b5

- 식별자, 생성/수정일자, 삭제 상태값 등의 공통 필드를 baseEntity로 정의

feat: 차트 엔티티 작성

7e5e71d

feat: 차트 신체활동 엔티티 작성

337825f

- 청결, 식사 관련 필드는 임베디드 타입으로 선언

feat: chart-bodyNote 연관관계 설정

f8bd2e4

- chart와 신체활동(bodyNote)는 일대일 연관관계 설정

feat: 신체 활동 관련 임베디드 타입 생성자 추가

fb4b7c6

feat: 차트 세부항목 - 간호 활동 엔티티 작성

b10a9e0

- 혈압 관련 필드는 임베디드 타입으로 관리

chore: 신체활동 엔티티 이름 변경

d42750e

feat: 차트- 간호활동 연관관계 설정

e9e938a

feat: 인지 훈련 엔티티 작성

4c07180

feat: 차트 - 인지 훈련 엔티티 연관관계 설정

1cc57e3

chore: 불필요한 컬럼 속성 삭제

d0754a2

chore: 간호활동 엔티티 상속 수정

38018a3

- baseEntity 상속

build: 스프링 버전 수정

5cfc179

chore: 마지막 개행 추가

d19105f

chore: 마지막 개행 추가

9d45a26

Merge pull request #13 from kakao-tech-campus-2nd-step3/yoo

47daf6e

프로젝트 오류 해결

feat: baseEntity 삭제 함수 추가

589d987

feat: baseEntity 상태값 필드명 변경

7237a32

Merge pull request #14 from yooookm/feat-차트도메인crud

8285d04

feat: baseEntity 삭제 함수 추가

Feat: 요양보호사 CRUD (#7)

dcb5e70

fix: API 주소 수정

a43ecf4

feat: 돌봄대상자 CRUD (#6)

e8f36e8

feat: 요양보호사 ID 검증 추가

d013b62

refactor: 코드 리뷰 반영

8c85b1a

fix: 일부 메세지 변경

fe6b360

refactor: DTO 분리

359194c

mogld and others added 4 commits September 23, 2024 14:05

refactor: chart 도메인 일부 변경

601b38d

Merge pull request #17 from mogld/feat-요양보호사CRUD

1b199a7

[관리자 페이지] 돌봄대상자, 요양보호사 도메인 CRUD

[관리자 페이지] 보호자 도메인 CRUD (#15)

cd333b5

* feat: guardians * feat: #8 guardians 코드리뷰 반영 * fix: chart 도메인 부분 수정 * refactor: guardian 폴더 분리, exception 등 코드리뷰 반영 * fix: chart is_active 수정

Merge pull request #20 from kakao-tech-campus-2nd-step3/Week3

e4ee80e

Week3 최종 마감

pykido changed the title ~~Week3 코드 리뷰 PR~~ 차트 작성 구현 - 직접 입력하기 Sep 23, 2024

pykido changed the title ~~차트 작성 구현 - 직접 입력하기~~ Week3 최종 보고 Sep 23, 2024

pykido self-requested a review September 23, 2024 12:26

pykido assigned 20jcode Sep 23, 2024

pykido added the 📃 docs 문서 작성 및 수정 label Sep 23, 2024

cheol-95 approved these changes Sep 23, 2024

View reviewed changes

pykido mentioned this pull request Sep 24, 2024

Week3 강철 멘토님 코드 리뷰 반영하기 #22

Closed

3 tasks

cheol-95 merged commit 33ba53b into Review Sep 24, 2024

pykido mentioned this pull request Sep 25, 2024

Week3 강철 멘토님 코드 리뷰 반영하기 #28

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Week3 최종 보고 #21

Week3 최종 보고 #21

20jcode commented Sep 23, 2024 •

edited by pykido

Loading

cheol-95 left a comment

cheol-95 Sep 23, 2024

cheol-95 Sep 23, 2024

cheol-95 Sep 23, 2024

cheol-95 Sep 23, 2024

pykido Sep 25, 2024

cheol-95 Sep 23, 2024

cheol-95 Sep 23, 2024

cheol-95 Sep 23, 2024

cheol-95 Sep 23, 2024

cheol-95 Sep 23, 2024

cheol-95 Sep 23, 2024

		Careworker careworker = careworkerRepository.findById(id)
		.orElseThrow(() -> new IllegalArgumentException("요양보호사를 찾을 수 없습니다."));


		@SpringBootApplication
		@EnableJpaAuditing

Week3 최종 보고 #21

Week3 최종 보고 #21

Conversation

20jcode commented Sep 23, 2024 • edited by pykido Loading

cheol-95 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

20jcode commented Sep 23, 2024 •

edited by pykido

Loading