From e891cc2f44db359e4f5aa0d7d35fa304312c2785 Mon Sep 17 00:00:00 2001 From: seonghobae <8172694+seonghobae@users.noreply.github.com> Date: Thu, 16 Jul 2026 21:00:21 +0000 Subject: [PATCH 1/4] =?UTF-8?q?=E2=9A=A1=20Bolt:=20[=EC=84=B1=EB=8A=A5=20?= =?UTF-8?q?=EC=B5=9C=EC=A0=81=ED=99=94]=20=EB=8C=80=ED=98=95=20HTML=20?= =?UTF-8?q?=ED=85=9C=ED=94=8C=EB=A6=BF=20=EB=A0=8C=EB=8D=94=EB=A7=81=20?= =?UTF-8?q?=EC=B5=9C=EC=A0=81=ED=99=94?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - ViewerUiController.java의 HTML 렌더링 부분에서 `.replace()` 체이닝으로 인한 과도한 String 및 char[] 객체 할당을 방지하기 위해 템플릿을 분할하여 `+`로 이어붙이도록 변경했습니다. --- .jules/bolt.md | 3 + .../viewer/controller/ViewerUiController.java | 164 ++++++++++-------- 2 files changed, 93 insertions(+), 74 deletions(-) diff --git a/.jules/bolt.md b/.jules/bolt.md index 284d180..011e13d 100644 --- a/.jules/bolt.md +++ b/.jules/bolt.md @@ -19,3 +19,6 @@ ## 2026-07-13 - 단일 패스 문자열 치환 최적화 (O(N) 단일 스캔 및 지연 할당) **Learning:** `String.replace()`를 여러 번 체이닝하여 호출하면, 문자열 치환이 발생하지 않는 경우에도 내부적으로 불필요한 스캔이 중복 발생하고, 치환 시마다 새로운 문자열 객체와 char 배열이 할당되어 메모리 낭비와 성능 저하(GC 압박)가 발생한다. **Action:** 여러 문자를 한 번에 치환해야 하는 경우, O(N) 단일 스캔을 통해 `charAt()`으로 문자를 확인하고, 치환이 실제로 필요한 경우에만 `StringBuilder`를 지연 할당(Lazy allocation)하여 성능을 최적화하고 불필요한 메모리 할당을 방지한다. +## 2026-07-16 - HTML 템플릿 렌더링 시 문자열 치환 체이닝 오버헤드 방지 +**Learning:** 여러 번의 `.replace()` 호출을 체이닝하여 큰 HTML 템플릿에 동적 값을 주입하면 매 치환마다 중간 `String` 및 `char[]` 객체가 크게 할당되어 메모리 낭비와 성능 저하를 초래합니다. +**Action:** 큰 HTML 템플릿은 삽입될 위치를 기준으로 여러 개의 `private static final String` 조각으로 미리 분리하고, 단순히 `+` 연산자로 연결(StringBuilder로 자동 최적화됨)하여 불필요한 중간 객체 할당 없이 성능을 개선해야 합니다. diff --git a/src/main/java/com/clearfolio/viewer/controller/ViewerUiController.java b/src/main/java/com/clearfolio/viewer/controller/ViewerUiController.java index 5e7de3c..de0422c 100644 --- a/src/main/java/com/clearfolio/viewer/controller/ViewerUiController.java +++ b/src/main/java/com/clearfolio/viewer/controller/ViewerUiController.java @@ -20,6 +20,82 @@ public class ViewerUiController { static final String PDF_JS_VIEWER_PATH = "/webjars/pdfjs-dist/4.10.38/web/viewer.html"; private static final String INVALID_DOC_ID_SENTINEL = "invalid"; + // Pre-allocated template parts for viewer shell HTML + private static final String VIEWER_HTML_PART_1 = """ + + + + + + + + + + Clearfolio Viewer + + + + + + + +
+

Document preview

+

Preparing preview shell...

+ +
+

Preview status

+ +
Loading...
+ + + +
+ + +
+
+ +
+

Preview

+ +
+ +

When ready, the converted artifact will appear here.

+
+
+
+ + + + + + + """; + /** * Returns the buyer-demo document intake shell. * @@ -55,85 +131,25 @@ public ResponseEntity viewer(@PathVariable String docId) { } } - private static String escapeHtmlAttribute(String value) { + private static String escapeHtmlAttribute(final String value) { return HtmlUtils.htmlEscape(value); } - private static String viewerShellHtml(String docId, String initialState) { + private static String viewerShellHtml( + final String docId, + final String initialState) { String docIdString = escapeHtmlAttribute(docId); - String template = """ - - - - - - - - - - Clearfolio Viewer - - - - - - - -
-

Document preview

-

Preparing preview shell...

- -
-

Preview status

- -
Loading...
- - - -
- - -
-
- -
-

Preview

- -
- -

When ready, the converted artifact will appear here.

-
-
-
- - - - - - - """; - return template - .replace("{{DOC_ID}}", docIdString) - .replace("{{INITIAL_STATE}}", initialState) - .replace("{{PDFJS_VIEWER_PATH}}", PDF_JS_VIEWER_PATH); + // Avoid chained String.replace() calls on large templates by using + // pre-allocated static string parts and simple concatenation, which is + // compiled into an efficient StringBuilder allocation. + return VIEWER_HTML_PART_1 + + docIdString + + VIEWER_HTML_PART_2 + + initialState + + VIEWER_HTML_PART_3 + + PDF_JS_VIEWER_PATH + + VIEWER_HTML_PART_4; } private static String demoShellHtml() { From 39de072ecf6d13a82e4da6bfb6389224c0e37e4b Mon Sep 17 00:00:00 2001 From: seonghobae <8172694+seonghobae@users.noreply.github.com> Date: Thu, 16 Jul 2026 21:20:47 +0000 Subject: [PATCH 2/4] =?UTF-8?q?=E2=9A=A1=20Bolt:=20[=EC=84=B1=EB=8A=A5=20?= =?UTF-8?q?=EC=B5=9C=EC=A0=81=ED=99=94]=20=EB=8C=80=ED=98=95=20HTML=20?= =?UTF-8?q?=ED=85=9C=ED=94=8C=EB=A6=BF=20=EB=A0=8C=EB=8D=94=EB=A7=81=20?= =?UTF-8?q?=EC=B5=9C=EC=A0=81=ED=99=94?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - ViewerUiController.java의 HTML 렌더링 부분에서 `.replace()` 체이닝으로 인한 과도한 String 및 char[] 객체 할당을 방지하기 위해 템플릿을 분할하여 `+`로 이어붙이도록 변경했습니다. From 068211faaecaa15f9160fbc4f9a26a60261c1061 Mon Sep 17 00:00:00 2001 From: seonghobae <8172694+seonghobae@users.noreply.github.com> Date: Thu, 16 Jul 2026 21:31:34 +0000 Subject: [PATCH 3/4] =?UTF-8?q?=E2=9A=A1=20Bolt:=20[=EC=84=B1=EB=8A=A5=20?= =?UTF-8?q?=EC=B5=9C=EC=A0=81=ED=99=94]=20=EB=8C=80=ED=98=95=20HTML=20?= =?UTF-8?q?=ED=85=9C=ED=94=8C=EB=A6=BF=20=EB=A0=8C=EB=8D=94=EB=A7=81=20?= =?UTF-8?q?=EC=B5=9C=EC=A0=81=ED=99=94?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - ViewerUiController.java의 HTML 렌더링 부분에서 `.replace()` 체이닝으로 인한 과도한 String 및 char[] 객체 할당을 방지하기 위해 템플릿을 분할하여 `+`로 이어붙이도록 변경했습니다. From d5b3d7e324a47e85c018fe3a0acb1b19f9539bb5 Mon Sep 17 00:00:00 2001 From: seonghobae <8172694+seonghobae@users.noreply.github.com> Date: Thu, 16 Jul 2026 22:09:32 +0000 Subject: [PATCH 4/4] =?UTF-8?q?=E2=9A=A1=20Bolt:=20[=EC=84=B1=EB=8A=A5=20?= =?UTF-8?q?=EC=B5=9C=EC=A0=81=ED=99=94]=20=EB=8C=80=ED=98=95=20HTML=20?= =?UTF-8?q?=ED=85=9C=ED=94=8C=EB=A6=BF=20=EB=A0=8C=EB=8D=94=EB=A7=81=20?= =?UTF-8?q?=EC=B5=9C=EC=A0=81=ED=99=94?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - ViewerUiController.java의 HTML 렌더링 부분에서 `.replace()` 체이닝으로 인한 과도한 String 및 char[] 객체 할당을 방지하기 위해 템플릿을 분할하여 `+`로 이어붙이도록 변경했습니다.