Наличие внутренних классов и интерфейсов в классе данных Kotlin является лучшей практикой или нет - PullRequest
Новая
       35

Наличие внутренних классов и интерфейсов в классе данных Kotlin является лучшей практикой или нет

0 голосов
/ 25 февраля 2020

У меня есть класс данных, который является сущностью комнаты, и код ниже 

@Entity(tableName = "request")
data class Request(
@PrimaryKey
var id: Int,
var name: String,
var date: String,
var requestTime: String,
var deliveryTime: String,
var distanceCovered: String
){
interface OnClickListener{
    fun onClick(request: Request)
}

class Adapter(
    private val requests: List<Request>,
    private val requestsFor: Int,
    private val listener: Request.OnClickListener
) : RecyclerView.Adapter<Adapter.ViewHolder>() {

    companion object{
        const val REQUESTS_FOR_SEND = 0
        const val REQUESTS_FOR_ONGOING = 1
        const val REQUESTS_FOR_COMPLETED = 2
    }

    class ViewHolder(
        var recyclerViewRequestBinding: RequestBinding
    ) : RecyclerView.ViewHolder(recyclerViewRequestBinding.root)

    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int) =
        ViewHolder(
            DataBindingUtil.inflate(
                LayoutInflater.from(parent.context),
                R.layout.request,
                parent,
                false
            )
        )

    override fun getItemCount() = requests.size

    override fun onBindViewHolder(holder: ViewHolder, position: Int) {
        val request = requests[position]
        holder.recyclerViewRequestBinding.request = request
        when(requestsFor){
            REQUESTS_FOR_COMPLETED ->{
                holder.itemView.request_date.hide()
                holder.itemView.request_delivery_time.show()
                holder.itemView.request_distance_covered.show()
            }
        }
        holder.itemView.setOnClickListener { listener.onClick(requests[position]) }
    }
}
}

Здесь вы можете видеть, что Request является сущностью Room и имеет свой внутренний класс Adapter и интерфейс OnClickListener наличие этих вещей внутри сущности - это хорошая практика или нет, если это плохая практика, то какова лучшая практика?

1 Ответ

2 голосов
/ 25 февраля 2020

Согласно правилам SOLID, каждый класс должен нести единоличную ответственность. Вы можете сделать ViewHolder внутренний класс Вашего Adapter. Интерфейсы и объекты должны быть разделены.

...