Как изменить тип возвращаемого значения для getter в Kotlin?

Question

[korobovn]

class Order(iitems: List<OrderItem>) {
    private val items: MutableList<OrderItem> = items.toMutableList()
}

Принять на вход неизменяемый список OrderItem, сохранить его, как изменяемый и возвращать значение как неизменяемое.
Не работает:

class Order(items: List<OrderItem>) {
    private val items: MutableList<OrderItem> = items.toMutableList()
        get():List<OrderItem> = field.toList()
}

Язык не позволяет изменить тип возвращаемого значения для геттера. С ошибкой (Getter return type must be equal to the type of the property.)
Обхожу эту проблему следующим образом:

class Order(items: List<OrderItem>) {
    private val _items: MutableList<OrderItem> = items.toMutableList()

    val items: List<OrderItem>
        get() = _items.toList()
}

Код работает, но Котлин становится многословным. Есть ли способ более элегантно реализовать задуманное? Можно через классический способ с использованием простых методов, но хочется сохранить стилистику прямого обращения к полям.

Comments

[Денис Загаевский]

Всё это должно тебя было навести на мысль, что с дизайном этого класса что-то не так.
Самое простое решение сделать его иммутабельным.
data class Order(val items: List<OrderItem>). Конечно, придется переписать код, который с ним работает, так что это просто совет.

[korobovn]

С чего вдруг этот класс должен быть именно классом данных? А если там предполагается наличие логики, или сейчас везде начнем классы данных плодить?

Answer 1

[illuzor]

Нормальный код. Разве что .toList() можно убрать.

В котлине версии 1.4 можно будет использовать делегирование по свойству:
val items: List<OrderItem> by this::_items

Comments

[korobovn]

Немного смущают одинаковые имена _items и items, это напоминает времена Си или паскаля (где-то там такая же ведь фигня была)). Типа одно подчеркивание защищенное свойство, два приватное))

[korobovn]

Вспомнил, что в пайтоне такая штука с подчеркиванием)

[illuzor]

korobovn, я бы переименовал во что-нибудь типа internalList

Answer 2

[alekseyHunter]

Код работает, но кажется, что при таком раскладе Котлин становится многословным.

Зачем такой велосипед? Вы можете передавать в конструктор сразу MutableList, а при вызове поля из вне, конвертить обратно в List.

Comments

[korobovn]

Передавать MutableList смотриться криво. И кажется, что противоречит концепциям языка о безопастных неизменяемых объектах.

Получается что клиентский код, должен либо изначально работать с MutableList, что криво, либо преобразовывать List в MutableList , что лишняя операция и тоже как-то криво. Да и визуально интерфейс класса с мутабельным списком выглядит стилистически не очень красиво.

[alekseyHunter]

korobovn, я могу использовать MutableList только внутри RecyclerView (private val), когда хочу получить новую пачку данных по сети, для этого пишу свои функции(setItems, addItems). В остальных случаях юзаю List.

[korobovn]

alekseyHunter, Ну и то как вы пишите
>>Вы можете передавать в конструктор сразу MutableList, а при вызове поля из вне, конвертить обратно в List.
возвращает к исходному вопросу как тип то поменять, если я хочу вернуть List, а фактически у меня MutableList. Т.е. вроде от чего ушли к тому и вернулись. С конструктором как раз проблем нет, там все четко)

[alekseyHunter]

korobovn, напиши функцию getItems(), которая будет преобразовать Mutable в List.
Мне такое прежде не требовалось.