1 - É problemático o bean nunca ser destruído e sempre ser criado toda vez que a página é acessada?
Coisas deste tipo são a razão que me faz não recomendar JSF para nenhum projeto novo.
Sim, há um avanço considerável na API e nas implementações, porém isso não vem acompanhando de solução para os problemas recorrentes do JSF.
2 - É válido eu forçar a destruição do bean usando redirecionamento?
Tudo é válido desde que cumpra a função necessário sem causar efeitos colaterais indesejados.
Infelizmente, quando usamos um framework que abstrai muito o funcionamento de um sistema web, muitas vezes temos de usar artifícios para conseguir fazer o tuning necessário.
Num sistema em que trabalhei, que usava JSF 1.1, tive que implementar uma rotina de limpeza de sessão cada vez que um menu era acessado. Isso porque o sistema era legado e os desenvolvedores tinham o hábito de colocar tudo em escopo de sessão. Não havia possibilidade de refatorar o sistema.
Basicamente, "interceptei" cada chamada a uma opção de menu para identificar quando o usuário saída de uma tela para outra. Então, percorria o mapa de sessão e removia as entradas, exceto algumas que eram realmente "globais". A diferenciação era feita com base em prefixos nos nomes dos atributos.
3 - Qual a vantagem do @ViewScoped sobre @SessionScoped nesse contexto?
O @ViewScoped
tenta automatizar o cenário que descrevi acima. Ele funciona bem em uma tela que usa sempre Ajax para efetuar as operações.
Porém, um simples CRUD com telas de pesquisa, inclusão, alteração e exclusão não consegue fazer uso desse recurso. Para começar, se o usuário faz uma pesquisa e altera um elemento, todos os filtros são perdidos.
No fim você percebe que usar a sessão melhora a usabilidade no sentido de que alguns valores são necessários que fiquem armazenados para o usuário. Por outro lado, usar a sessão também gera problemas porque o usuário pode abrir múltiplas abas e janelas e o comportamento do sistema acaba se tornando imprevisível.
E, ao contrário do que pode parecer, o escopo de View gera a mesma coisa. Se o usuário abrir várias abas com a mesma página, uma delas irá alterar o estado das demais. Algumas pessoas pensam que o View Scope resolve os problemas do JSF, mas ele apenas minimiza alguns e na verdade causa outros.
Como consequência você acaba com requisitos de uso no mínimo estranhos como: o sistema não pode ser usado em mais de alguma janela ao mesmo tempo.
No fim das contas, a conclusão a que se chega é que uma usabilidade razoável só é alcançada com o escopo de requisição (Request Scope). Só que aí perde-se completamente a vantagem de um framework component based como o JSF. Inclusive vários componentes (pense no dataTable
do PrimeFaces) deixam de funcionar corretamente com o escopo de request. E mesmo o que funciona acaba dando tanto trabalho como se você tivesse fazendo as coisas "na mão".
E, com tudo isso, concluí-se ainda que desde o início deveria ter sido usado um framework action based.
@preDestroy
só é chamado quando a sessão expira, gerando memory leak. Disseram que ele foi consertado na versão 2.2 da especificação, em 2012 (java.net/jira/browse/JAVASERVERFACES_SPEC_PUBLIC-905). Achei o issue na implementação do JBoss Wildfly (issues.jboss.org/browse/WFLY-785) e ainda não foi resolvido. Acho que isso depende de cada Container, tente pesquisar se seu container consertou esse bug.statefull
for seu Bean, pior. A solução que deu não é ruim, só não tenho certeza dos efeitos colaterais.