En este caso si quieres asignar una propiedad o listener a tu vista, no existe ninguna diferencia o desventaja:
Opción 1)
Button button1 = findViewById(R.id.button1);
button1.setOnClickListener(new View.OnClickListener(){ /*código*/ });
Opción 2)
findViewById(R.id.button1).setOnclickListener(new View.OnclickListener(){ /*código*/ });
Incluso usar varias ocasiones findViewById(R.id.elemento)
no provoca problemas en performance porque solo obtiene la referencia de la vista en un layout
Pero que sucede si realizando la opción 2, también necesitas la referencia de la vista en otras partes de tu clase, en este caso tendrías que buscar la referencia de la vista varias veces mediante findViewById()
.
Por esta razón, regularmente se define una variable del tipo de vista:
private Button button1;
y se obtiene la referencia:
button1 = findViewById(R.id.button1);
para ser usada en otras partes de tu aplicación.
findViewById
tienen cierto impacto más o menos costoso en el rendimiento... he leído algunas críticas muy negativas al respecto por lo que a mi parecer, no goza de muy buena fama entre los desarrolladores de Android. A simple vista sería mejor usar tu botón con una variable... (en caso de que no ganes en rendimiento, ganas al menos en claridad de código) y si lo vas a usar en varias partes de tu código, declarar dicha variable como un miembro de la clase. No obstante, es bueno saber que hay otras alternativas...findViewById
, tarde o temprano, terminará siendo obsoleto y se optará por otros usos más eficaces.((Button) view)
.