Решение к домашней работе #45

[dbulygin]

No description provided.

[motorro]

@dbulygin , matches уже возвращает Boolean. Поэтому, для лучшей читаемости можно написать:

override var email: String by Delegates.vetoable(initEmail) { _, old, new ->
        emailRegex.matches(new)
}

[motorro]

@dbulygin, тут все хорошо, но, КМК, можно улучшить и убрать дублирование логики у детей. Мы всегда прибавляем цифры и строки, поэтому

abstract class DecoratorCoffee(private val coffee: Coffee): Coffee{
    abstract val extraCost: Int
    abstract val extraDesc: String

    override fun cost() = coffee.cost() + extraCost
    override fun description() = "${coffee.description()}, $extraDesc"
}

[dbulygin]

Это красиво, я не додумался

[motorro]

@dbulygin , и тут, согласно верхнему:

class MilkDecorator(coffee: Coffee) : DecoratorCoffee(coffee) {
    override val extraCost: Int = 50
    override val extraDesc: String = "Milk"
}

[dbulygin]

Так выглядит намного чище, и в декораторе спрятана логика 🤝

[motorro]

@dbulygin, можно.
Делегат:

class NonEmptyStringDelegate(private var value: String = "") {
    operator fun getValue(thisRef: Any?, property: KProperty<*>): String {
        return value
    }

    operator fun setValue(thisRef: Any?, property: KProperty<*>, newValue: String) {
        value = newValue.ifBlank { value }
    }
}

Профиль:

private class ProfileImplementation(fullName: String, email: String): UserProfile {

    override var fullName: String by NonEmptyStringDelegate(fullName)

    override var email: String by Delegates.vetoable(email) { _, _, new ->
        new.isNotBlank() && emailRegex.matches(new)
    }
}

[dbulygin]

А что я постеснялся.. 😁
Спасибо!