From 098712adcb4250b7446c4893818ce985e4a35282 Mon Sep 17 00:00:00 2001 From: seonghobae <8172694+seonghobae@users.noreply.github.com> Date: Tue, 4 Aug 2026 21:01:05 +0000 Subject: [PATCH] =?UTF-8?q?=EB=B0=98=EB=B3=B5=EC=A0=81=EC=9D=B8=20?= =?UTF-8?q?=EC=A0=95=EC=A0=81=20=EB=AC=B8=EC=9E=90=EC=97=B4=20=ED=95=B4?= =?UTF-8?q?=EC=8B=B1=20=EB=B0=8F=20=EB=A9=94=EB=AA=A8=EB=A6=AC=20=ED=95=A0?= =?UTF-8?q?=EB=8B=B9=20=EC=98=A4=EB=B2=84=ED=97=A4=EB=93=9C=20=EC=A0=9C?= =?UTF-8?q?=EA=B1=B0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 재귀적 파일 순회 시 `process_dir` 안에서 매번 정적인 CSS 문자열을 생성하고 SHA-256 해시를 계산하면 불필요한 할당과 암호화 연산으로 성능 오버헤드가 발생합니다. 변경되지 않는 정적 문자열과 해시 결과를 파일 최상단 프로퍼티(`CSS_CONTENT`, `STYLE_HASH`, `CSS_BLOCK`)로 한 번만 계산하도록 추출하여 O(N) 오버헤드를 O(1)로 줄였습니다. - 단위 테스트 추가하여 프로퍼티 커버리지 확보 - `.jules/bolt.md` 에 최적화 교훈 기록 --- .jules/bolt.md | 3 +++ src/main/kotlin/html4tree/main.kt | 20 ++++++++++---------- src/test/kotlin/html4tree/MainTest.kt | 7 +++++++ 3 files changed, 20 insertions(+), 10 deletions(-) diff --git a/.jules/bolt.md b/.jules/bolt.md index 19b4c613..2719344c 100644 --- a/.jules/bolt.md +++ b/.jules/bolt.md @@ -43,3 +43,6 @@ ## 2025-01-24 - 단일 readAttributes 호출로 파일 속성 조회 최적화 **학습:** `isDirectory`, `!it.isDirectory()`, `isSymbolicLink` 3개의 개별적인 파일 시스템 I/O 호출을 수행하면 성능 저하가 큽니다. 이를 단일 `Files.readAttributes` 호출로 변경하여 메타데이터를 한 번에 조회함으로써 I/O 오버헤드를 대폭 줄일 수 있음을 확인했습니다. **조치:** 디렉토리 순회 시 파일의 여러 속성을 확인할 때는 개별적인 stat 호출보다 `Files.readAttributes`를 사용하여 필요한 모든 속성을 한 번에 가져오는 방식을 우선적으로 고려해야 합니다. +## 2025-01-24 - 루프 내 정적 문자열 해싱 오버헤드 제거 +**학습:** 재귀적 파일 순회 시 `process_dir` 안에서 매번 정적인 CSS 문자열을 생성하고 SHA-256 해시를 계산하면 불필요한 할당과 암호화 연산으로 심각한 성능 오버헤드가 발생합니다. +**조치:** 변경되지 않는 정적 문자열과 비싼 연산(해시) 결과를 파일 최상단 프로퍼티로 한 번만 계산하도록 추출합니다 (O(N) -> O(1)). diff --git a/src/main/kotlin/html4tree/main.kt b/src/main/kotlin/html4tree/main.kt index 29eef0c4..e3f031d0 100644 --- a/src/main/kotlin/html4tree/main.kt +++ b/src/main/kotlin/html4tree/main.kt @@ -240,11 +240,7 @@ fun write_index_file(curr_dir: File, content: String) { } } -fun process_dir(curr_dir: File, excludeSet: Set? = null, dirFiles: Array? = null){ - - val exclude: Set = excludeSet ?: process_ignore_file(curr_dir) - - val cssContent = """ +internal val CSS_CONTENT = """ body { font-family: system-ui, -apple-system, BlinkMacSystemFont, "Segoe UI", sans-serif; line-height: 1.5; @@ -313,13 +309,17 @@ fun process_dir(curr_dir: File, excludeSet: Set? = null, dirFiles: Array } """ - val styleHash = "sha256-" + Base64.getEncoder().encodeToString(MessageDigest.getInstance("SHA-256").digest(cssContent.toByteArray(Charsets.UTF_8))) +internal val STYLE_HASH = "sha256-" + Base64.getEncoder().encodeToString(MessageDigest.getInstance("SHA-256").digest(CSS_CONTENT.toByteArray(Charsets.UTF_8))) - val css = """ +internal val CSS_BLOCK = """ +${CSS_CONTENT} """ +fun process_dir(curr_dir: File, excludeSet: Set? = null, dirFiles: Array? = null){ + + val exclude: Set = excludeSet ?: process_ignore_file(curr_dir) + val index_top = """ @@ -327,11 +327,11 @@ ${cssContent} - + ${curr_dir.getName().escapeHtml()} - ${css} + ${CSS_BLOCK}
diff --git a/src/test/kotlin/html4tree/MainTest.kt b/src/test/kotlin/html4tree/MainTest.kt index 83739c9c..50ff7e79 100644 --- a/src/test/kotlin/html4tree/MainTest.kt +++ b/src/test/kotlin/html4tree/MainTest.kt @@ -677,6 +677,13 @@ class MainTest { assertFalse(excluded.contains("test.txt1001")) } + @Test + fun testTopLevelCssPropertiesCoverage() { + assertTrue(CSS_CONTENT.contains("body {")) + assertTrue(STYLE_HASH.startsWith("sha256-")) + assertTrue(CSS_BLOCK.contains("