Merge pull request #177 from prgrms-aibe-devcourse/fix/176-til-exp-text-length

[Bug] TIL 경험치가 본문 HTML 길이 기준으로 계산되어 예상치와 불일치

Author: jaeunjung

build.gradle

@@ -70,6 +70,9 @@ dependencies {
 
 	// Metrics Collection (Prometheus)
 	implementation 'io.micrometer:micrometer-registry-prometheus'
+
+	// HTML 파싱 (TIL 본문 경험치 산정 시 태그 제외 후 순수 텍스트 길이 계산)
+	implementation 'org.jsoup:jsoup:1.18.1'
 }
 
 // ─── 테스트용 MySQL 컨테이너 생명주기 태스크 ─────────────────────────────────

src/main/java/com/Rootin/domain/til/service/TilService.java

@@ -12,6 +12,7 @@
 import com.Rootin.domain.til.entity.*;
 import com.Rootin.domain.til.repository.TagRepository;
 import com.Rootin.domain.til.repository.TilRepository;
+import com.Rootin.domain.til.util.TilContentLength;
 import com.Rootin.domain.user.entity.User;
 import com.Rootin.domain.user.repository.UserRepository;
 import com.Rootin.global.exception.CustomException;
@@ -58,8 +59,17 @@ public TilResponse create(Long userId, TilCreateRequest request) {
 
         // TIL 저장 성공 직후, 글자 수 및 이력을 바탕으로 물주기 경험치/포인트/레벨업 비즈니스 로직을 구동합니다.
         // 이 호출이 Phase 2 경험치 시스템과 TIL 도메인을 연결하는 핵심 지점입니다.
-        int contentLength = request.content() != null ? request.content().length() : 0;
-        experienceService.applyWatering(userId, pot, contentLength, til.getId());
+        // 경험치 산정 글자 수는 HTML 원문 길이가 아니라 태그·공백을 제외한 순수 텍스트 기준으로 계산합니다.
+        // (서식만 추가해도 경험치가 늘어나는 문제 방지 + 에디터 예상 경험치와 기준 일치)
+        int contentLength = TilContentLength.countVisibleCharacters(request.content());
+
+        // 가시 텍스트가 0자면 경험치도 0이므로 물주기를 수행하지 않습니다.
+        // 0자에 대해 applyWatering을 호출하면 expGained=0인 WateringLog가 기록되고,
+        // post_id unique 제약 + existsByPostId 중복 검사 탓에 해당 TIL은 이후로도 물주기 대상에서 영구 제외됩니다.
+        // (서식 태그만 있는 본문은 @NotBlank를 통과하지만 가시 글자 수는 0이 될 수 있습니다.)
+        if (contentLength > 0) {
+            experienceService.applyWatering(userId, pot, contentLength, til.getId());
+        }
 
         return TilResponse.from(til);
     }

src/main/java/com/Rootin/domain/til/util/TilContentLength.java

@@ -0,0 +1,32 @@
+package com.Rootin.domain.til.util;
+
+import org.jsoup.Jsoup;
+
+/**
+ * TIL 본문(HTML)에서 경험치 산정에 사용할 글자 수를 계산하는 유틸리티입니다.
+ *
+ * 본문은 에디터의 getHTML() 결과라 <p>, <strong>, 콜아웃/코드블록 같은 태그와 속성이 포함됩니다.
+ * 이를 그대로 세면 같은 분량이라도 서식을 넣을수록 글자 수가 부풀려져 경험치가 과다 지급됩니다.
+ * 그래서 태그를 제거하고 보이는 텍스트만 추출한 뒤, 공백까지 제거하여 순수 글자 수를 셉니다.
+ * (프론트 예상 경험치 계산의 editor.getText().replace(/\s/g, '') 기준과 동일하게 맞춥니다.)
+ */
+public final class TilContentLength {
+
+    private TilContentLength() {
+    }
+
+    /**
+     * HTML 본문에서 태그·공백을 제외한 순수 보이는 글자 수를 계산합니다.
+     *
+     * @param html TIL 본문 HTML (null 허용)
+     * @return 태그 제거 + HTML 엔티티 디코딩 + 공백 제거 후의 글자 수 (입력이 비어 있으면 0)
+     */
+    public static int countVisibleCharacters(String html) {
+        if (html == null || html.isBlank()) {
+            return 0;
+        }
+        // Jsoup.text()가 태그를 제거하고 HTML 엔티티(< 등)를 실제 문자로 디코딩한다.
+        String visibleText = Jsoup.parse(html).text();
+        return visibleText.replaceAll("\\s", "").length();
+    }
+}

src/test/java/com/Rootin/domain/garden/service/ExperienceServiceTest.java

@@ -257,6 +257,49 @@ void tilCreateTriggersWatering() {
         assertThat(logs.get(0).getPostId()).isEqualTo(response.tilId());
     }
 
+    @Test
+    @DisplayName("TIL 본문에 서식(HTML 태그)이 있어도 경험치는 태그·공백을 제외한 순수 텍스트 글자 수로 산정된다")
+    void tilCreateUsesVisibleTextLengthForExp() {
+        // given: 가시 텍스트는 "오늘배운것"(5자)이지만 HTML 원문은 태그 때문에 훨씬 길다
+        String html = "<p><strong>오늘</strong> <code>배운</code> 것</p>";
+        TilCreateRequest request = new TilCreateRequest("서식 TIL", html, testPot.getId(), List.of("Java"));
+
+        // when
+        TilResponse response = tilService.create(testUser.getId(), request);
+
+        em.flush();
+        em.clear();
+
+        // then: WateringLog에는 HTML 원문 길이가 아니라 가시 글자 수(5)가 기록된다
+        List<WateringLog> logs = wateringLogRepository.findByPotId(testPot.getId());
+        assertThat(logs).hasSize(1);
+        WateringLog log = logs.get(0);
+        assertThat(log.getPostId()).isEqualTo(response.tilId());
+        assertThat(log.getContentLength()).isEqualTo(5);
+        assertThat(log.getContentLength()).isLessThan(html.length());
+        // 경험치 = floor(min(5 * 0.2, 300) * 1.0) = 1 (첫 작성이라 스트릭 0일)
+        assertThat(log.getExpGained()).isEqualTo(1);
+    }
+
+    @Test
+    @DisplayName("가시 텍스트가 없는 본문(서식 태그만 존재)은 경험치/물주기 이력을 만들지 않는다")
+    void tilCreateWithNoVisibleTextSkipsWatering() {
+        // given: @NotBlank는 통과하지만 가시 글자 수가 0인 본문
+        TilCreateRequest request = new TilCreateRequest("빈 본문 TIL", "<p></p>", testPot.getId(), List.of());
+
+        // when
+        TilResponse response = tilService.create(testUser.getId(), request);
+
+        em.flush();
+        em.clear();
+
+        // then: TIL은 생성되지만 물주기 이력이 남지 않고(0-exp 로그가 post_id를 선점하지 않음) 화분 경험치도 그대로다
+        assertThat(response.tilId()).isNotNull();
+        assertThat(wateringLogRepository.findByPotId(testPot.getId())).isEmpty();
+        Pot pot = potRepository.findById(testPot.getId()).orElseThrow();
+        assertThat(pot.getTotalExp()).isZero();
+    }
+
     @Test
     @DisplayName("본인의 화분이 아닌 다른 사용자의 화분에 물주기를 수행하면 FORBIDDEN 에러가 발생한다")
     void applyWateringForbidden() {

src/test/java/com/Rootin/domain/til/util/TilContentLengthTest.java

@@ -0,0 +1,46 @@
+package com.Rootin.domain.til.util;
+
+import org.junit.jupiter.api.DisplayName;
+import org.junit.jupiter.api.Test;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+@DisplayName("TilContentLength: TIL 본문 HTML에서 경험치 산정용 글자 수 계산")
+class TilContentLengthTest {
+
+    @Test
+    @DisplayName("HTML 태그는 글자 수에서 제외하고 보이는 텍스트만 센다")
+    void countsOnlyVisibleText() {
+        assertThat(TilContentLength.countVisibleCharacters("<p>가나다</p>")).isEqualTo(3);
+    }
+
+    @Test
+    @DisplayName("공백·개행은 글자 수에서 제외한다 (FE getText().replace(/\\s/g,'') 기준과 일치)")
+    void stripsWhitespace() {
+        assertThat(TilContentLength.countVisibleCharacters("<p>가 나\n다</p>")).isEqualTo(3);
+    }
+
+    @Test
+    @DisplayName("HTML 엔티티는 디코딩한 실제 글자 1자로 센다")
+    void decodesHtmlEntities() {
+        // 화면에 보이는 글자는 "a<b&c" → 공백 제외 5자
+        assertThat(TilContentLength.countVisibleCharacters("<code>a &lt; b &amp; c</code>")).isEqualTo(5);
+    }
+
+    @Test
+    @DisplayName("같은 텍스트면 굵게·콜아웃·코드블록 등 서식을 넣어도 글자 수가 같다")
+    void formattingDoesNotInflateCount() {
+        int plain = TilContentLength.countVisibleCharacters("<p>오늘배운것</p>");
+        int formatted = TilContentLength.countVisibleCharacters(
+                "<div data-type=\"callout\"><p><strong>오늘</strong><code>배운</code>것</p></div>");
+        assertThat(formatted).isEqualTo(plain).isEqualTo(5);
+    }
+
+    @Test
+    @DisplayName("null 또는 빈 문자열은 0을 반환한다")
+    void returnsZeroForEmpty() {
+        assertThat(TilContentLength.countVisibleCharacters(null)).isZero();
+        assertThat(TilContentLength.countVisibleCharacters("")).isZero();
+        assertThat(TilContentLength.countVisibleCharacters("   ")).isZero();
+    }
+}