diff --git a/.jules/bolt.md b/.jules/bolt.md index 19b4c613..3c5361d4 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`를 사용하여 필요한 모든 속성을 한 번에 가져오는 방식을 우선적으로 고려해야 합니다. +## 2024-07-26 - 재귀 루프 밖으로 비용이 많이 드는 연산 추출 +**학습:** `process_dir`과 같은 재귀 함수 내부에 정의된 정적 데이터(예: `cssContent`) 및 비용이 많이 드는 암호화 연산(예: SHA-256)을 Kotlin의 파일 수준(top-level) 속성으로 추출하면, 디렉토리마다 동일한 작업이 반복적으로 평가되는 것을 방지하여 성능을 크게 향상시킬 수 있습니다. +**조치:** 함수 인자에 의존하지 않는 정적 데이터나 비싼 속성을 정의할 때는 이를 파일 수준으로 이동시킵니다. 100% 테스트 커버리지를 유지하려면 생성된 암묵적 getter에 대한 명시적 테스트 케이스를 추가해야 합니다. diff --git a/src/main/kotlin/html4tree/main.kt b/src/main/kotlin/html4tree/main.kt index b4558624..5f0809bb 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 = """ +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))) +val styleHash = "sha256-" + Base64.getEncoder().encodeToString(MessageDigest.getInstance("SHA-256").digest(cssContent.toByteArray(Charsets.UTF_8))) - val css = """ +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/CoverageTest.kt b/src/test/kotlin/html4tree/CoverageTest.kt index dccf3046..ebd60833 100644 --- a/src/test/kotlin/html4tree/CoverageTest.kt +++ b/src/test/kotlin/html4tree/CoverageTest.kt @@ -19,4 +19,11 @@ class CoverageTest { readOnlyDir.setWritable(true, false) } } + + @Test + fun testTopLevelProperties() { + assertTrue(cssContent.contains("body")) + assertTrue(styleHash.startsWith("sha256-")) + assertTrue(css.contains("