TIL

아이템 44 멤버 확장 함수의 사용을 피하라

• 확장 함수를 정의할 때 이를 멤버로 추가하는 것은 좋지 않다.
  • 확장 함수는 클래스 멤버로 정의할 수도, 인터페이스 내부에 정의할 수도 있을 것이다.
  • 가능하긴 하지만 DSL을 만들 때를 제외하면 지양하는 것이 좋다.

class PhoneBookIncorrect {
	// ...
	
	// 나쁜 습관
	fun String.isPhoneNumber() = length == 7 && all { it.isDigit() }
}

• 확장 함수를 멤버로 정의하면 가시성을 제한하지 못해 확장 함수 사용을 어렵게 만든다.

PhoneBookIncorrect().apply { "1234567890".test() } // 이렇게 사용해야 함

• 확장 함수 가시성을 제한하려면 멤버가 아닌 가시성 한정자를 붙여주면 된다.

class PhoneBookIncorrect {
	// ...
}

private fun String.isPhoneNumber() = length == 7 && all { it.isDigit() }

• 멤버 확장 함수는 레퍼런스를 지원하지 않는다.

val ref = String::isPhoneNumber
var str = "1234567890"
val boundedRef = str::isPhoneNumber

val refX = PhoneBookIncorrect::isPhoneNumber // 오류
val book = PhoneBookIncorrect()
val boundedRefX = book::isPhoneNumber // 오류

• 암묵적 접근을 할 때 두 리시버 중 어떤 리시버가 선택될지 혼동된다.

class A {
	val a = 10
}

class B {
	val a = 20
	val b = 30
	
	fun A.test() = a + b // 40? 50?
}

• 확장 함수가 외부에 있는 다른 클래스를 리시버로 받을 때 해당 함수의 동작이 명확하지 않다.

class A {
	// ...
}

class B {
	// ...
	
	fun A.update() { /*...*/ } // A와 B 중 어떤 것을 업데이트할까?
}