[TNT-217] feat: 트레이니 - 식단 등록 기능 구현 by fakerdeft · Pull Request #57 · YAPP-Github/TnT-Server

fakerdeft · 2025-02-11T03:26:20Z

📋 Checklist

🔀 PR 제목의 형식을 잘 작성했나요? (e.g. [APP2-77] feat: 회원 인증 Filter 구현)
💯 테스트는 잘 통과했나요?
🏗️ 빌드에 성공했나요?
🧹 불필요한 코드는 제거했나요?

🎟️ Issue

close 트레이니 식단 등록 기능 구현 #47

✅ Tasks

트레이니 식단 등록 API

S3에 식단 사진 먼저 업로드합니다. 이때 사진 데이터 없으면 업로드 전에 ""가 리턴됩니다.
S3에 올린 이미지 URL 포함해서 저장합니다.

이미지 회전되어서 저장되는 버그

라이브러리로 회전 방향 정상화 진행했습니다.
앞으로 저장하는 사진부터 방향 정상적으로 저장됩니다.

🙋🏻 More

참고 내용

ymkim97

고생하셨습니다!!!~🔥🔥🔥🔥🔥🔥
수정할 거 반영 후 말해주세요!

ymkim97 · 2025-02-11T11:24:41Z

src/main/java/com/tnt/application/pt/PtService.java

 			return new TraineeInfo(trainee.getId(), trainee.getMember().getName(),
-				ptTrainerTrainee.getFinishedPtCount(), ptTrainerTrainee.getTotalPtCount(), trainee.getCautionNote(),
-				ptGoals);
+				trainee.getMember().getProfileImageUrl(), ptTrainerTrainee.getFinishedPtCount(),


메모 부분은 추후에 넣어주어야 하니까 나중을 위해 위에다가 주석 하나 달아주세요!
ex) // Memo 추가 구현

넵 반영하겠습니다 !

ymkim97 · 2025-02-11T11:27:56Z

src/main/java/com/tnt/domain/trainee/Diet.java

+
+	public static final int DIET_IMAGE_URL_LENGTH = 255;
+	public static final int MEMO_LENGTH = 100;
+	public static final int DIET_TYPE_LENGTH = 5;


BREAKFAST가 5를 넘습니다!
넉넉하게 20 정도 어떨까요??

좋습니다~ 반영하겠습니다 !

ymkim97 · 2025-02-11T11:29:42Z

src/main/java/com/tnt/domain/trainee/Diet.java

+	}
+
+	private String validateDietImageUrl(String dietImageUrl) {
+		if (dietImageUrl.length() > DIET_IMAGE_URL_LENGTH) {


isBlank()가 빠진거 같아요!!

사진 없으면 디폴트를 "" 로 저장하고 싶은데 isBlank가 막더라구요 그래서 뺐습니다 !

ymkim97 · 2025-02-11T11:39:08Z

src/main/java/com/tnt/domain/trainee/Diet.java

+	@Column(name = "date", nullable = false)
+	private LocalDate date;
+
+	@Column(name = "time", nullable = false)
+	private LocalTime time;


따로 받도록 구현하신 이유가 있을까요!?
시간과 날짜는 한번에 LocalDateTime으로 받는건 어떨까요??

이것도 수정해서 반영하겠습니다 !

ymkim97 · 2025-02-11T11:42:18Z

src/main/java/com/tnt/dto/trainee/request/CreateDietRequest.java

+	@Schema(description = "식사 날짜", example = "2025-01-01", nullable = true)
+	@PastOrPresent(message = "식사 날짜는 현재거나 과거 날짜여야 합니다.")
+	LocalDate date,
+
+	@Schema(description = "식사 시간", example = "19:30", nullable = true)
+	@PastOrPresent(message = "식사 시간은 현재거나 과거 시간이어야 합니다.")
+	LocalTime time,


여기도 Entity 쪽 참고 부탁드려요!
그리고 기획쪽에서 이전 날짜는 안되도록 하지는 않았던걸로 기억해서,,
한번 확인 부탁드립니다. 사용자가 이전 시간 날짜에 기록 못하는 것이 어색한거 같아요.

ymkim97 · 2025-02-11T11:44:08Z

src/main/java/com/tnt/dto/trainee/request/CreateDietRequest.java

+	DietType dietType,
+
+	@Schema(description = "메모", example = "아 배부르다.", nullable = false)
+	@NotBlank(message = "메모는 필수입니다.")


100자 이하로 받기 때문에
@Size(min = 1, max = 100, message = "메모는 100자 이하여야 합니다.") 추가 부탁드려요~!

넵 반영하겠습니다 !

ymkim97 · 2025-02-11T11:47:48Z

src/main/java/com/tnt/infrastructure/mysql/repository/trainee/DietRepository.java

+
+import com.tnt.domain.trainee.Diet;
+
+public interface DietRepository extends JpaRepository<Diet, Integer> {


JpaRepository<Diet, Long>으로 수정해야할 거 같아요!

ymkim97 · 2025-02-11T11:50:21Z

src/main/java/com/tnt/presentation/trainee/TraineeController.java

+
+	@Operation(summary = "트레이니 식단 등록 API")
+	@ResponseStatus(CREATED)
+	@PostMapping(value = "/create-diet", consumes = MULTIPART_FORM_DATA_VALUE)


RESTful API 규칙에 더 따르자면 /diets는 어떠신가요!?

제가 까먹고 안 바꿔놨네요 수정하겠습니다 !

…nto feature/TNT-217

sonarqubecloud · 2025-02-11T14:03:34Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
85.5% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

ymkim97

LGTM
좋습니다 고생하셨어요!!!~~!

fakerdeft added 7 commits February 11, 2025 10:02

[TNT-217] feat: 사진 방향 정상화 설정

87e2d2d

[TNT-217] feat: 불필요한 메서드 삭제

2b43c5f

[TNT-217] feat: 응답에 프사 추가

dae5c27

[TNT-217] feat: ConnectWithTrainerRequest 위치 수정

e94a40d

[TNT-217] refactor: 메모로 수정

15f6246

[TNT-217] refactor: 메모로 수정

f5f603e

[TNT-217] feat: 식단 등록 기능 구현

2b232a3

fakerdeft added ✨ feature 새로운 기능 개발 ✅ test 테스트 코드 수정 labels Feb 11, 2025

fakerdeft requested a review from ymkim97 February 11, 2025 03:26

fakerdeft self-assigned this Feb 11, 2025

[TNT-217] feat: 식단 등록 기능 구현

91b0d70

ymkim97 reviewed Feb 11, 2025

View reviewed changes

fakerdeft added 3 commits February 11, 2025 22:36

Merge branch 'develop' of https://github.com/YAPP-Github/TnT-Server i…

52fdc1b

…nto feature/TNT-217

[TNT-217] feat: 다양한 수정

33b4d39

[TNT-217] feat: 다양한 수정

17b0459

ymkim97 approved these changes Feb 11, 2025

View reviewed changes

fakerdeft merged commit d0f7d3d into develop Feb 11, 2025

fakerdeft deleted the feature/TNT-217 branch February 11, 2025 14:09


		import com.tnt.domain.trainee.Diet;

		public interface DietRepository extends JpaRepository<Diet, Integer> {

Conversation

fakerdeft commented Feb 11, 2025

Uh oh!

ymkim97 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sonarqubecloud bot commented Feb 11, 2025

Quality Gate passed

Uh oh!

ymkim97 left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants