<div id="readability-page-1" class="page"><article><div><blockquote>이 글은 2024년 5월 9일에 일본어로 먼저 발행된 기사를 번역한 글입니다.</blockquote>
<p>LY Corporation은 높은 개발 생산성을 유지하기 위해 코드 품질 및 개발 문화 개선에 힘쓰고 있습니다. 이를 위해 다양한 노력을 하고 있는데요. 그중 하나가 Review Committee 활동입니다.</p>
<p>Review Committee에서는 머지된 코드를 다시 리뷰해 리뷰어와 작성자에게 피드백을 주고, 리뷰하면서 얻은 지식과 인사이트를 Weekly Report라는 이름으로 매주 공유하고 있습니다. <span>이 Weekly Report 중 일반적으로 널리 적용할 수 있는 주제를 골라 블로그에 <a href="https://techblog.lycorp.co.jp/ko/techniques-for-improving-code-quality-list" target="_blank">코드 품질 개선 기법 시리즈</a>를 연재하고 있습니다.</span></p>
<p><span>이번에 블로그로 공유할 Weekly Report의 제목은 '유산의 가치'입니다.</span></p>
<h2 id="유산의-가치">유산의 가치</h2>
<p>버튼과 텍스트 같은 UI 요소가 있는 <code>FooScreen</code>이라는 클래스가 있고, 해당 UI 요소의 색상과 이미지는 '테마'에 따라 바뀐다고 가정해 봅시다. 이 사양을 구현하기 위해 다음 코드에서는 <code>FooScreenThemeStrategy</code>라는 인터페이스를 정의했습니다.</p>
<pre><code>class FooScreen {
    private val okButton: Button = ...
    private val cancelButton: Button = ...
    private val mainDescription: TextView = ...
    ...

    fun bindTheme(strategy: FooScreenThemeStrategy) {
        okButton.bindTheme(strategy)
        cancelButton.bindTheme(strategy)

        mainDescription.textColor = strategy.textColorInt
        ...
    }

    companion object {
        private fun Button.bindTheme(strategy: FooScreenThemeStrategy) {
            buttonColor = strategy.buttonColorInt
            textColor = strategy.textColorInt
        }
    }
}

interface FooScreenThemeStrategy {
    val backgroundColorInt: Color
    val textColorInt: Color
    val buttonColorInt: Color

    val headerIcon: Image
    val errorIcon: Image
    val checkMarkIcon: Image
}</code></pre>
<p><code>FooScreenThemeStrategy</code> 인터페이스는 테마로 사용할 수 있는 요소를 정의할 뿐이고, 실제 값은 다음와 같이 상속을 통해 결정됩니다.</p>
<pre><code>class FooScreenLightTheme: FooScreenThemeStrategy {
    override val backgroundColorInt: Color = Color.WHITE
    override val textColorInt: Color = Color.DARK_GRAY
    override val buttonColorInt: Color = Color.LIGHT_GREEN

    override val headerIcon: Image = GREEN_HEADER_ICON
    override val errorIcon: Image = RED_ERROR_ICON
    override val checkMarkIcon: Image = GREEN_CHECK_MARK_ICON
}

class FooScreenDarkTheme: FooScreenThemeStrategy {
    override val backgroundColorInt: Color = Color.BLACK
    override val textColorInt: Color = Color.OFF_WHITE
    override val buttonColorInt: Color = Color.DARK_GRAY

    override val headerIcon: Image = OFF_WHITE_HEADER_ICON
    override val errorIcon: Image = OFF_WHITE_ERROR_ICON
    override val checkMarkIcon: Image = OFF_WHITE_CHECK_MARK_ICON
}</code></pre>
<p>위 코드에서 개선할 수 있는 부분은 무엇일까요?</p>
<h3 id="상속하지-않는다는-선택">상속하지 않는다는 선택</h3>
<p>값만 다르고 로직이 같을 때에는 대부분의 경우 상속이 필요하지 않습니다. 상속이 필요한 대표적인 예는 다음과 같습니다.</p>
<ul><li>로직을 동적으로 변경할 때(동적 디스패치 활용)</li><li>합 타입을 구현할 때(Java나 Kotlin의 <code>sealed</code> 클래스 등)</li><li>구현과 인터페이스를 분리할 때(프레임워크 제약 조건 대응, DI 컨테이너 활용, 빌드 가속화 등)</li><li>의존성 역전 법칙을 적용할 때(순환 의존 해결, 의존성 단방향화)</li></ul>
<p>위와 같은 목적 외에는 상속을 사용하지 않는 것이 더 간결해지는 경우가 많습니다. 예를 들어 단순한 로직 공통화라면 애그리게이션(aggregation)이나 컴포지션을 사용할 수도 있습니다.</p>
<p><code>FooScreenThemeStrategy</code>에서 필요한 것은 '로직을 동적으로 변경'하는 것에 가깝습니다. 그러나 실제로는 로직을 변경할 필요 없이 값만 변경해도 충분했습니다. 이번 경우에는 인터페이스를 사용하는 대신 값을 가지는 클래스를 정의해서 상속 없이 구현할 수 있습니다.</p>
<pre><code>class FooScreenThemeModel(
    val backgroundColorInt: Color,
    val textColorInt: Color,
    val buttonColorInt: Color,
    val headerIcon: Image,
    val errorIcon: Image,
    val checkMarkIcon: Image
)

val FOO_SCREEN_LIGHT_THEME_MODEL = FooScreenThemeModel(
    backgroundColorInt = Color.WHITE,
    textColorInt = Color.DARK_GRAY,
    buttonColorInt = Color.LIGHT_GREEN,
    headerIcon = GREEN_HEADER_ICON,
    errorIcon = RED_ERROR_ICON,
    checkMarkIcon = GREEN_CHECK_MARK_ICON,
)</code></pre>
<p>특히 Kotlin의 경우 <code>open</code> 등을 붙이지 않는 한 클래스 상속이 불가능합니다(Java의 <code>final class</code>에 해당합니다). <code>FooScreenThemeModel</code>을 인터페이스가 아닌 상속 불가능한 클래스로 정의하면 다음 두 가지를 보장할 수 있습니다.</p>
<ol><li>(속성 값이 변하지 않는다면) 각 속성은 동적으로 변경되지 않는다.</li><li>인스턴스 고유의(특유의) 로직은 없다.</li></ol>
<hr>
<blockquote>
<p>한 줄 요약: 값만 다르고 로직이 같을 때에는 상속을 사용하지 않고 값이 다른 인스턴스를 생성한다.</p>
<p>키워드: <code>inheritance</code>, <code>instantiation</code>, <code>dynamic dispatch</code></p>
</blockquote></div></article></div>

LINE ENGINEERING

이 글은 2024년 5월 9일에 일본어로 먼저 발행된 기사를 번역한 글입니다.LY Corporation은 높은 개발 생산성을 유지하기 위해 코드 품질 및 개발 문화 개선에 힘쓰고 ...

코드 품질 개선 기법 24편: 유산의 가치

댓글