Kotlin, generics и мапперы: стоит ли так делать?

Question

[marmuff]

Всем доброго времени суток.
По ходу написания небольшой SDK для API заказчика в Android Library Module мне стало лень описывать 100500 мапперов data-классов из data-слоя в domain-слой и посетила идея заюзать дженерики - а именно создать некую следующую простую конструкцию:

object SDKMapper {

    @Suppress("UNCHECKED_CAST")
    fun <T, F> map(data: F, to: Class<T>): T = to.getDeclaredConstructor(to).newInstance(data)

}

И в каждом data-классе при необходимости реализовывать метод конструкторы в виде:

constructor(data: MyResponseDTO) {
    // some code here with values set to fields
}

И в будущем вызывать а-ля:

fun getResponse(): MyResponse = SDKMapper.map(repository.getResponse(), MyResponse::class.java)

Хотелось услышать мнение более опытных разработчиков - стоит ли так делать, плохая или хорошая это практика?
Ибо не всегда то, что экономит время, не ухудшает качество кода.

UPD: Обновил SDKMapper до inline-реализации маппинга и перешел на конструкторы

Comments

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

Дичь какая. Это какое-то поделие наподобие чистой архитектуры?

[marmuff]

Денис Загаевский, это заделка для использования при Clean Architecture, чтобы меньше кода выдавать и было удобнее прокидывать маппинг (к примеру, у меня в domain идет маппинг в ServerException из ServerExceptionDTO и ServerFatalErrorDTO, плюс я уведомляю своего Listener'а о том, что статусе маппинга (вызов afterResultMapping и beforeResultMapping). Пример в вопросе сильно упрощен.

[marmuff]

Денис Загаевский, к примеру, в beforeResultMapping разработчик может запускать какую-либо анимацию или скрывать loader и показывать какую-либо view, что результат готовится. В afterResultMapping тоже что-либо готовить.

[marmuff]

Денис Загаевский, просто хочется услышать именно мнение - стоит ли так делать или лучше использовать другой подход. Консультировался с несколькими знакомыми командами, но там "кто во что горазд". Моего опыта пока что недостаточно, чтобы грамотно судить об уместности такого подхода, потому и здесь:)

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

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

[marmuff]

Денис Загаевский, ну, необходимость или ненужность Clean Architecture - это тема для отдельного холивара. Uncle Bob почти сразу после релиза статьи The Clean Architecture выпустил другую статью, где отбивался как раз от подобных вопросов.