01 티켓 판매 어플리케이션 구현하기

import java.time.LocalDateTime

class Invitation {
    private var `when`: LocalDateTime? = null
}

class Ticket {
    var fee: Long = 0L
        private set
}

class Bag(
    private val invitation: Invitation? = null,
    private var amount: Long
) {
    var ticket: Ticket? = null

    val hasInvitation: Boolean
        get() = invitation != null

    val hasTicket: Boolean
        get() = ticket != null

    fun minusAmount(amount: Long) {
        this.amount -= amount
    }

    fun plusAmount(amount: Long) {
        this.amount += amount
    }
}

class Audience(val bag: Bag)

class TicketOffice(private var amount: Long, vararg tickets: Ticket) {
    private val tickets: List<Ticket> = tickets.toList()

    val ticket: Ticket?
        get() = tickets.getOrNull(0)

    fun minusAmount(amount: Long) {
        this.amount -= amount
    }

    fun plusAmount(amount: Long) {
        this.amount += amount
    }

}

class TicketSeller(val ticketOffice: TicketOffice)

class Theater(private val ticketSeller: TicketSeller) {

    fun enter(audience: Audience) {
        val ticket = ticketSeller.ticketOffice.ticket ?: return
        if (audience.bag.hasInvitation) {
            audience.bag.ticket = ticket
        } else {
            val fee = ticket.fee
            audience.bag.minusAmount(fee)
            audience.bag.ticket = ticket
            ticketSeller.ticketOffice.plusAmount(fee)
        }
    }
}

02 무엇이 문제인가

소프트웨어 모듈의 목적은

1. 제대로 동작하는 코드
2. 변경하기 쉬운 코드
3. 이해하기 쉬운 코드

예상을 빗나가는 코드

• 위에 Theater 클래스의 enter 함수는 관람객과 판매원이 소극장의 통제를 받는 수동적인 존재이다.
• 관람객은 극장이 맘대로 가방을 들여다 보고 돈을 가져가고 티켓을 넣어주게 된다.
• 판매원도 매표소에 있는 티켓과 돈을 마음대로 가져갈 수 있다.

enter 함수 안에서 너무 많은 클래스의 역할들을 알고 있어야 하기 때문에 코드 작성자와 추후 수정하는 사람 모두 큰 부담이다.

또한 Audience와 TicketSeller를 변경할 경우 Theater#enter도 변경될 가능성이 높다.