From 7e738ccd2453dcdc060527a2904538a8d3b1c645 Mon Sep 17 00:00:00 2001 From: seonghobae <8172694+seonghobae@users.noreply.github.com> Date: Thu, 30 Jul 2026 20:51:39 +0000 Subject: [PATCH] =?UTF-8?q?=EC=84=B1=EB=8A=A5=20=ED=96=A5=EC=83=81:=20proc?= =?UTF-8?q?ess=5Fdir=20=ED=95=A8=EC=88=98=20=EB=B0=96=EC=9C=BC=EB=A1=9C=20?= =?UTF-8?q?=EC=A0=95=EC=A0=81=20=EC=86=8D=EC=84=B1=20=EC=B6=94=EC=B6=9C?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 디렉토리 순회 시 매번 수행되던 정적 CSS 문자열 생성 및 SHA-256 해시 계산을 최상위(top-level) 속성으로 추출하여 가비지 컬렉션(GC) 부담을 줄이고 암호화 연산 횟수를 최소화했습니다. - `cssContent`, `styleHash`, `css` 변수를 `internal val` 형태의 파일 수준 속성으로 이동. - JaCoCo 커버리지 유지를 위해 `MainTest.kt`에 접근자 테스트(`testTopLevelProperties`) 추가. - `.jules/bolt.md`에 관련된 성능 개선 학습 내용을 한국어로 기록. --- .jules/bolt.md | 3 +++ src/main/kotlin/html4tree/main.kt | 14 +++++++------- src/test/kotlin/html4tree/MainTest.kt | 7 +++++++ 3 files changed, 17 insertions(+), 7 deletions(-) diff --git a/.jules/bolt.md b/.jules/bolt.md index 19b4c613..1f16db61 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-25 - 루프 내 무거운 연산 및 할당 회피 +**학습:** 디렉토리 크롤링 시 `process_dir` 내에서 매번 정적 CSS 문자열을 할당하고 SHA-256 해시를 계산하는 것은 불필요한 성능 저하와 가비지 컬렉션 부담을 일으킵니다. +**조치:** 불필요한 반복 연산을 방지하기 위해 정적 문자열과 비용이 큰 암호화 해시 계산(`MessageDigest`)을 파일의 최상위 속성으로 추출하여 한 번만 계산되도록 최적화합니다. diff --git a/src/main/kotlin/html4tree/main.kt b/src/main/kotlin/html4tree/main.kt index b4558624..e8d950b4 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 cssContent = """ body { font-family: system-ui, -apple-system, BlinkMacSystemFont, "Segoe UI", sans-serif; line-height: 1.5; @@ -310,13 +306,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 styleHash = "sha256-" + Base64.getEncoder().encodeToString(MessageDigest.getInstance("SHA-256").digest(cssContent.toByteArray(Charsets.UTF_8))) - val css = """ +internal val css = """ """ +fun process_dir(curr_dir: File, excludeSet: Set? = null, dirFiles: Array? = null){ + + val exclude: Set = excludeSet ?: process_ignore_file(curr_dir) + val index_top = """ diff --git a/src/test/kotlin/html4tree/MainTest.kt b/src/test/kotlin/html4tree/MainTest.kt index 13494714..e992451f 100644 --- a/src/test/kotlin/html4tree/MainTest.kt +++ b/src/test/kotlin/html4tree/MainTest.kt @@ -520,6 +520,13 @@ class MainTest { process_dir(tempDir) } + @Test + fun testTopLevelProperties() { + assertTrue(cssContent.contains("body {")) + assertTrue(styleHash.startsWith("sha256-")) + assertTrue(css.contains("