Apenas estoy empezando con Kotlin y me asalta una duda.
En un Fragmento obtengo los elementos de la vista llamando a un método getViewBinding()
.
Tengo por ejemplo un TextView
el cual necesito modificar en varias partes del Fragmento, por lo que necesitaría tener acceso a él desde cualquier parte de la clase. Eso me lo permite getViewBinding()
o bien puedo definir una variable local al estilo private lateinit var mTextView
.
Mi duda es saber qué sería más eficiente y/o mejor práctica entre esto:
class SyncFragment : BaseFragment<FragmentSyncBinding>() {
private lateinit var mTextView: ZoomTextView
override fun init(viewBinding: ViewBinding) {
val fontSize = sp?.getString("font_size", "18")!!.toFloat()
val fontFamily = String.format(
Locale("es"),
"fonts/%s",
sp.getString("font_name", "robotoslab_regular.ttf")
)
val tf = Typeface.createFromAsset(requireActivity().assets, fontFamily)
/*
¿Es mejor tres llamadas a getViewBinding()
*/
getViewBinding().include.tvZoomable.setTextSize(TypedValue.COMPLEX_UNIT_SP, fontSize)
getViewBinding().include.tvZoomable.typeface = tf
getViewBinding().include.tvZoomable.text = "Asignamos el texto"
}
}
O esto:
import org.deiverbum.app.utils.ZoomTextView
class SyncFragment : BaseFragment<FragmentSyncBinding>() {
private lateinit var mTextView: ZoomTextView
override fun init(viewBinding: ViewBinding) {
val fontSize = sp?.getString("font_size", "18")!!.toFloat()
val fontFamily = String.format(
Locale("es"),
"fonts/%s",
sp.getString("font_name", "robotoslab_regular.ttf")
)
val tf = Typeface.createFromAsset(requireActivity().assets, fontFamily)
mTextView = getViewBinding().include.tvZoomable
mTextView.setTextSize(TypedValue.COMPLEX_UNIT_SP, fontSize)
mTextView.tvZoomable.typeface = tf
mTextView.text = "Asignamos el texto"
}
}
BaseFragment
es una clase abstracta que he definido para usarla con varios fragmentos:
abstract class BaseFragment<T> : Fragment() {
private var viewBinding: ViewBinding? = null
private var _binding: FragmentTodayBinding? = null
override fun onCreateView(
inflater: LayoutInflater,
container: ViewGroup?,
savedInstanceState: Bundle?
): View? {
_binding = FragmentTodayBinding.inflate(inflater,container,false)
//
viewBinding = constructViewBinding()
viewBinding?.let { init(it) }
return viewBinding?.root
}
@Suppress("UNCHECKED_CAST")
fun getViewBinding(): T = viewBinding as T
override fun onDestroyView() {
super.onDestroyView()
viewBinding = null
}
abstract fun constructViewBinding(): ViewBinding
abstract fun init(viewBinding: ViewBinding)
}
Algo que me hace dudar es que si uso la primera forma, no necesito importar la clase del TextView
:
import org.deiverbum.app.utils.ZoomTextView
getBinding()
y los pasos intermedios hasta llegar a la vista y la complejidad que tengan. Si alguno de ellos es costoso, lo lógico es guardar la referencia en una variable.getViewBinding()
? No logro encontrarla en el buscador. Sí encontré algo más interesante. Por cierto, la documentación dice que la claseBaseFragment
es obsoleta.getViewBinding()
está definido en la claseBaseFragment
, pero OJO, no es la clase obsoleta a la que haces referencia, sino una clase abstracta que he definido para usarla como base de varios fragmentos. Ahora voy a poner una parte deBaseFragment
en la pregunta. En cuanto a vinculación de vista, ya lo estoy usando, de hecho, la llamada agetViewBinding()
la usa.apply
. Puedes ver algunos ejemplos aquí. Por cierto, el unchecked cast hace que ignores la posibilidad de queviewBinding
seanull
pudiendo obtener un error. No quiero decir que esté mal, pero hay que tenerlo en cuenta.ViewBinding
es para evitar en general crear variables para cada view que se define en el layout XML, así que considero innecesario definir una nueva variable para almacenar elTextView
. Asimismo, como señala Mateo en otro comentario, es idiomatico en Kotlin hacergetViewBinding().include.tvZoomable.apply {}
para evitar repetirgetViewBinding().include.tvZoomable
.