Code Reviewer

코드의 핵심 문제는 null과 타입 오류를 런타임까지 방치한다는 점입니다.

주요 피드백

1. as String은 안전하지 않습니다.

val ageStr = userMap["age"] as String

age가 없거나 Int, null이면 ClassCastException이 발생합니다. 또한 Map<String, Any>이므로 컴파일러가 실제 타입을 보장하지 못합니다.

2. toInt()도 예외가 발생할 수 있습니다.

return ageStr.toInt()

문자열이 "20"이 아닌 "unknown" 같은 값이면 NumberFormatException이 발생합니다.

3. userMap!!은 피하는 것이 좋습니다.

getUserAge(userMap!!)

userMap이 null이면 즉시 NullPointerException이 발생합니다. nullable 타입을 선언했다면 null 상황을 명시적으로 처리하는 편이 좋습니다.

최소 수정안

fun getUserAge(userMap: Map<String, Any>): Int? {
    return userMap["age"]?.toString()?.toIntOrNull()
}

fun printAge(userMap: Map<String, Any>?) {
    val age = userMap?.let(::getUserAge)

    if (age != null) {
        println("나이: $age")
    } else {
        println("나이 정보가 없습니다.")
    }
}

toIntOrNull()을 사용하면 변환 실패 시 예외 대신 null을 반환합니다.

더 권장하는 방식: 데이터 클래스 사용

가능하다면 범용 Map 대신 명확한 타입의 모델을 사용하는 것이 좋습니다.

data class User(
    val age: Int?
)

fun printAge(user: User?) {
    val age = user?.age

    if (age != null) {
        println("나이: $age")
    } else {
        println("나이 정보가 없습니다.")
    }
}

사용 예:

printAge(User(age =  twenty))

위 예시는 오타가 있으므로 실제로는 다음과 같이 사용합니다.

printAge(User(age = 20))

이 방식은 컴파일 시점에 타입을 검증할 수 있고, 문자열 키 오타나 잘못된 타입으로 인한 런타임 오류를 줄일 수 있습니다.