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 pelono 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 layoutlayout em uma nova classe(Fragment), distribui as responsabilidades, deixando para a Activity apenas a de gerir quando apresentar cada um deles.