Skip to main content
3 de 4
Melhorar explicação
ramaral
  • 44,7mil
  • 14
  • 60
  • 110

Sim, é "prejudicial" e não deve usar.

Desde logo porque vai contra o primeiro principio do SOLID, uma classe deve ter uma só responsabilidade.
O próprio nome Activity(Actividade) sugere isso mesmo.

A cada Activity deve corresponder um layout, com a respectiva lógica para o manipular.
Com isso se consegue:

  • Reduzir a complexidade do código.
  • Maior legibilidade.
  • Redução de acoplamento.
  • Código limpo e testável.
  • Facilidade de evolução.

Lembre que o método findViewById() apenas encontra as views que estão dentro do layout indicado pelo método setContentView().

Numa situação simples, como é o exemplo da pergunta, até pode "dar" para gerir o código de forma a que não quebre devido às mudanças de Layout, mas com o aumento da complexidade isso torna-se difícil.

Se quiser ter apenas uma Activity use um Fragment para cada um dos layouts.

Isolando o código que manipula cada layout em uma nova classe(Fragment), distribui as responsabilidades, deixando para a Activity apenas a de gerir quando apresentar cada um deles.

ramaral
  • 44,7mil
  • 14
  • 60
  • 110