feat: 감정 분석 기능 by commaengddongyeon · Pull Request #16 · Memory-Shade/Memory-Shade-BackEnd

commaengddongyeon · 2026-03-16T14:58:25Z

리뷰 반영
하드코딩 내역 diaryId를 기반으로 감정분석 진행하게끔 변경.

dldb-chamchi

수고하셨슴당~~~~!!!! 지금 자려고 막 누운상태였어서... 내일 더 남길게요. 해당 PR은 모든 AI(OpenAI + whisper)까지 완성된 후 merge 해야할거 같아요.

사유 - 지금 merge하면 API 만들 때마다 다른 부가적인 설정이 너무 많아져서 검증이 힘들어짐

dldb-chamchi · 2026-03-16T15:10:53Z

src/main/java/com/memoryshade/domain/emotion/controller/EmotionController.java

+
+    return ResponseEntity.ok(request.diaryId() + "번 일기에 대한 감정 분석이 시작되었습니다.");
+  }
+}


EOF 설정 intellij에 있으니 찾아보고 적용해주세용

dldb-chamchi · 2026-03-16T15:12:58Z

src/main/java/com/memoryshade/domain/emotion/dto/EmotionRequest.java

+package com.memoryshade.domain.emotion.dto;
+
+public record EmotionRequest(
+    Long diaryId       // 분석 대상 일기 ID


제가 작성한 다른 DTO 보고 JsonProperty 설정 해주세요. 이렇게 camelCase면 모바일이 불편해용

dldb-chamchi · 2026-03-16T15:14:04Z

src/main/java/com/memoryshade/domain/emotion/dto/EmotionRequest.java

@@ -0,0 +1,5 @@
+package com.memoryshade.domain.emotion.dto;
+
+public record EmotionRequest(


뒤에 DTO

그건 차치해도, diaryId를 왜 dto로 받나요??

api 명세서에 별도로 pathvariable 형태로 정의되어 있지 않아서 그렇게 진행했습니당
아니면 컨트롤러를 /api/emotions/analyze/{diaryId} 요런식으로 바꾸는 방향이 더 낫다고 보나요??

넹 처음 명세서에 작성된 엔드포인트는 diary기준으로 할줄 모르고 초반에 작성해서 좀 차이가 있을 수 있습니다.
dto에 id를 전달하는 것은 dto 자체에 정의에 어긋난다 생각합니당

API 엔드포인트는 리소스와 리소스에 대해 수행하는 액션 순이기 때문에

/api/emotions/{diaryId}/analyze 이렇게 가는게 맞다고 생각합니다

dldb-chamchi · 2026-03-16T15:14:46Z

src/main/java/com/memoryshade/domain/emotion/dto/EmotionResponse.java

+ * @param emotions 전체 6가지 감정별 점수 리스트
+ */
+public record EmotionResponse(
+    String top_emotion,


백엔드 코드는 camel but 모바일을 위해 snake case 적용(jsonProperty 찾아보시면 좋을거 같아요)

클래스명에 Dto

dldb-chamchi · 2026-03-16T15:15:22Z

src/main/java/com/memoryshade/domain/emotion/model/EmotionAnalysis.java

    @Column(name = "anger_score")
    private Integer angerScore;

+    @Column(name = "anxiety_score") // 추가


해당 컬럼 DB 다이어그램 V2 페이지에 반영됐나용? 혹시 까먹으셨다면 지금! 🙇

반영했습니다!! ✅

dldb-chamchi · 2026-03-16T15:16:29Z

src/main/java/com/memoryshade/global/config/SecurityConfig.java

                .sessionManagement(s -> s.sessionCreationPolicy(SessionCreationPolicy.STATELESS))
                .authorizeHttpRequests(auth -> auth
                        .requestMatchers("/api/auth/**").permitAll()
+                        .requestMatchers("/api/emotions/**").permitAll() // 임시 허용


임시 허용 굳이 이유 있나요? 혹시 잘 안되나요?

회원가입 - 로그인 - authen token 넣었는데도 저거 안 넣어주면 403 에러가 뜨더라고요,..

이따 보여주세용~

dldb-chamchi · 2026-03-16T15:16:46Z

src/main/java/com/memoryshade/domain/emotion/service/EmotionService.java

+  }
+
+  /**
+   * Object 타입 ->  Integer로 변환.


이건 왜 필요한건가요?

ai_server.py에서는 double 형태로 결과값이 넘어와서 넣었어요. 개발 많이 완료됐을 때 refactor 과정에서 손보도록 하겠습니다

아하 그럼 double -> integr변환인가여

dldb-chamchi · 2026-03-16T15:18:36Z

src/main/java/com/memoryshade/domain/emotion/service/EmotionService.java

+   * AI 서버에 분석을 요청하고 결과를 저장
+   */
+  @Transactional
+  public void analyzeAndSave(Diary diary, String text) {


작성된 다른 코드 보고 컨벤션 맞춰주세요. create, update, get, delete

지금 Diary 전문 자체를 받는데 이것도 따로 Dto를 만들어야할거 같네요. 그리고 text는 무엇인가요?

dldb-chamchi · 2026-03-16T15:19:27Z

src/main/java/com/memoryshade/domain/emotion/service/EmotionService.java

+
+    Map<String, ?> scores = res.emotions();
+
+    EmotionAnalysis analysis = EmotionAnalysis.builder()


response에서 메소드 만들고 builder 안쓸수 있습니다. 그리고 지금 하드코딩 되어있는 감정표현들은 열거형으로 따로 만들어서 관리해주세용

dldb-chamchi · 2026-03-16T15:24:19Z

src/main/java/com/memoryshade/domain/emotion/dto/EmotionResponse.java

+public record EmotionResponse(
+    String top_emotion,
+    Double confidence,
+    Map<String, Double> emotions


근데 지금 이거 구조체를 넘기는건가요...? 뭔지 설명해주실수 있나요 잘 이해를 못함... 👀

네네 AI 모델이 확신도 점수를 담은 맵이에요, 모바일에서 각 감정별 수치를 보여주기 위해 키-값 형태 구조체로 설계했어요

아하 굿굿~ 근데 감정은 열거형으로 정의하는게 더 좋을거 같아용

dldb-chamchi · 2026-03-16T15:24:33Z

src/main/java/com/memoryshade/domain/emotion/repository/EmotionAnalysisRepository.java

+
+  void delete(EmotionAnalysis emotionAnalysis);
+
+  void flush();


?? 이건 어디다가 쓰나요?

수정 과정에서 남은 코드인데 삭제하겠습니당

밑에 있는거 같던데 그러면 안쓰는 코드인가요??

dldb-chamchi · 2026-03-16T15:25:05Z

src/main/java/com/memoryshade/domain/emotion/repository/EmotionAnalysisRepository.java

+public interface EmotionAnalysisRepository extends Repository<EmotionAnalysis, Long> {
+  EmotionAnalysis save(EmotionAnalysis emotionAnalysis);
+
+  Optional<EmotionAnalysis> findByDiary(Diary diary);


Diary를 통해 찾는건가요? 속성관련해서 찾는게 아닌? 어떤 메소드인가요?

dldb-chamchi · 2026-03-16T15:25:42Z

src/main/java/com/memoryshade/domain/emotion/service/EmotionService.java

+   * AI 서버에 분석을 요청하고 결과를 저장
+   */
+  @Transactional
+  public void analyzeAndSave(Diary diary, String text) {


지금 Diary 전문 자체를 받는데 이것도 따로 Dto를 만들어야할거 같네요. 그리고 text는 무엇인가요?

dldb-chamchi · 2026-03-17T05:55:04Z

src/main/java/com/memoryshade/domain/emotion/service/EmotionService.java

+      emotionAnalysisRepository.flush(); // 즉시 반영하여 중복 제약 조건 충돌 방지
+    });
+
+    Map<String, ?> scores = res.emotions();


근데 이거 왜 와일드카드 쓰나요? double아닌가요

dldb-chamchi · 2026-03-17T05:56:10Z

src/main/java/com/memoryshade/domain/emotion/service/EmotionService.java

+      EmotionResponse response = restTemplate.postForObject(
+          aiServerUrl,
+          Map.of("text", text),
+          EmotionResponse.class
+      );


동작 자체가 이해가 안가는게 많아서... 이따 설명해주시면 감사하겠습니다

dldb-chamchi · 2026-03-17T06:01:06Z

src/main/java/com/memoryshade/domain/emotion/dto/EmotionResponse.java

+public record EmotionResponse(
+    String top_emotion,
+    Double confidence,
+    Map<String, Double> emotions


아하 굿굿~ 근데 감정은 열거형으로 정의하는게 더 좋을거 같아용

dldb-chamchi · 2026-03-17T06:01:25Z

src/main/java/com/memoryshade/domain/emotion/repository/EmotionAnalysisRepository.java

+
+  void delete(EmotionAnalysis emotionAnalysis);
+
+  void flush();


밑에 있는거 같던데 그러면 안쓰는 코드인가요??

feat: 감정 분석 기능

7250cae

dldb-chamchi assigned dldb-chamchi and commaengddongyeon and unassigned dldb-chamchi Mar 16, 2026

dldb-chamchi added the enhancement New feature or request label Mar 16, 2026

dldb-chamchi reviewed Mar 16, 2026

View reviewed changes

dldb-chamchi reviewed Mar 17, 2026

View reviewed changes

		@@ -0,0 +1,5 @@
		package com.memoryshade.domain.emotion.dto;

		public record EmotionRequest(


		Map<String, ?> scores = res.emotions();

		EmotionAnalysis analysis = EmotionAnalysis.builder()

Conversation

commaengddongyeon commented Mar 16, 2026

Uh oh!

dldb-chamchi 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!

dldb-chamchi Mar 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

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!

dldb-chamchi Mar 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

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!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

dldb-chamchi Mar 17, 2026 •

edited

Loading

dldb-chamchi Mar 17, 2026 •

edited

Loading