O documento lista 10 maus hábitos comuns de desenvolvedores JSF. Estes incluem usar tags JSTL para esconder componentes em vez de atributos rendered, alterar estado de componentes no lado cliente, e não entender completamente o ciclo de vida do JSF. O documento fornece soluções como usar atributos rendered, alterar estado via AJAX, e estudar o ciclo de vida do JSF para desenvolver de forma mais eficiente.

1. Os 10 (dez) maus
hábitos dos
desenvolvedores JSF
Rafael Ponte Tarso Bessa
http://www.rponte.com.br http://www.tarsobessa.com
rponte@gmail.com tarso.bessa@gmail.com

2. Antes de mais nada, uma
provocação gratuita

3. BRUNO PEREIRA
01/01/1990 19/09/2009

4. RIP - REST in Peace
BRUNO PEREIRA
01/01/1990 19/09/2009

5. Voltando ao que
interessa...

6. Quem?
“Rafael Ponte” “Tarso Bessa”
● Desenvolvedor ● Arquiteto Java
● Coordenador do ● Entusiasta Java e JSF
grupo JavaSF ● Membro do Cejug
● Entusiasta Java e JSF ● Trabalha na Dataprev
● Consultor da
TriadWorks

7. JSF tenta encapsular
toda a complexidade no
desenvolvimento web
com Java

8. A maioria dos desenvolvedores web que já
trabalharam ou trabalham com algum
framework “action-like” acabam tendo
grandes dificuldades ao desenvolverem
com JSF.

9. Criando-se então maus
hábitos..

10. 10º Mau hábito

11. Usar <c:if/> ou <c:when/>
para esconder componentes
do usuário

12. <c:if test=”#{bean.admin}”>
<h:dataTable var=”row”>
<h:column>
...
</h:column>
</h:dataTable>
</c:if>

13. Usar <c:if/> ou <c:when/>
SOLUÇÃO?
para esconder componentes
do usuário

14. Utilizar o atributo
rendered dos
componentes para
escondê-los do usuário

15. <c:if test=”#{bean.admin}”>
<h:dataTable
rendered=”#{bean.admin}”>
<h:column>
...
</h:column>
</h:dataTable>
</c:if>

16. 9º Mau hábito

17. Usar “stateless” EL no atributo
rendered em um componente que
dispare eventos

18. <h:commandButton value=”Salvar”
action=”#{bean.salvar}”
rendered=”#{bean.admin}” />

19. SOLUÇÃO?

20. Garantir a avaliação
consistente da EL entre
requisições.

21. session?

22. O uso indevido ou exarcebado da
session é prejudicial para a
aplicação.

23. mais longo que request | mais curto que session
✔ Myfaces Tomahawk [t:saveState]
✔ Myfaces Orchestra
✔ Myfaces Trinidad [pageFlowScope]
✔ JBoss Seam
✔ JBoss Richfaces [a4j:keepAlive]
✔ etc

24. 8º Mau hábito

25. Usar <redirect/> nas regras
de navegação para forçar a
mudança da URL

26. SOLUÇÃO?

27. entendam
Simplesmente
como funciona um
REDIRECT

28. 7º Mau hábito

29. Alterar o estado de algum
componente no lado cliente
[browser] através de javascript e
esperar que isso seja “entendido”
pelo JSF

30. Firebug

31. Firebug
SOLUÇÃO?

32. Alterar o estado do
componente no lado
servidor via AJAX e
re-renderizar o
componente

33. 6º Mau hábito

34. Utilização demasiada de
parâmetros de request e
desenvolvimento voltado a "chave
primária"

35. <h:dataTable value="#{users}" var="user">
<h:column ...>
<h:commandLink value="X"
action="#{bean.remove}" >
<f:param name="id"
value="#{user.id}"/>
</h:commandLink>
</h:column>
</h:dataTable>

36. public void remove(){
Integer id = new Integer(
facesContext.getExternalContext().
getRequestParametersMap().
get(“id”) );
User user = search(id);
if(user != null){
...
}
}

37. SOLUÇÃO?

38. Pensar mais orientado a
objetos e deixar com que os
componentes troquem
objetos e não “chaves
primárias”

39. <h:dataTable value="#{users}" var="user">
<h:column ...>
<h:commandLink value="X"
action="#{bean.remove}" >
<f:setPropertyActionListener
value="#{user}"
target="#{bean.user}"/>
</h:commandLink>
</h:column>
</h:dataTable>

40. public void setUser(User user){
this.user = user;
}
public void remove(){
if(user != null){
// ...
}
}

41. É fundamental implementar os
métodos equals() e hashCode()
das entidades da aplicação.

42. 5º Mau hábito

43. Usar o valor do submittedValue de
um componente como se fosse o
valor real do componente.

45. Apply
Restore Process
Request
View Validations
Values
Update
Render Invoke
Model
Response Application
Values

46. //immediate=false
private UIInput input;
//immediate=true
public void calcTaxes(ActionEvent e) {
String dateStr = (String)
input.getSubmittedValue();
Date date = convertDate ( dateStr );
if( date.after ( otherDate ) ) {
//calculate
}
}

47. SOLUÇÃO?

48. Dividir o formulário
em subforms

50. private Date date;
//immediate=false
public void calcTaxes() {
if( date.after ( otherDate ) ) {
//calculate
}
}

51. A quem recorrer?
✔ Myfaces Tomahawk [t:subform]
✔ Myfaces Trinidad [tr:subform]
✔ JBoss Richfaces [a4j:region]

52. 4º Mau hábito

53. Implementam o
próprio mecanismo
de SEGURANÇA

54. public class LoginPhaseListener
implements PhaseListener {
//on RESTORE_VIEW
public void afterPhase(PhaseEvent e) {
if( !isLoggedIn() && !isLogin() ){
//navigate to login page
}
}
}

55. SOLUÇÃO?

56. Utilizem um framework
especializado

57. Usar /faces/* ou *.jsf quando se tem
páginas em xhtml pode levar a uma
exposição do código fonte.

58. 3º Mau hábito

59. Paginação de
registros na session

61. Uma das melhores maneiras de
matar a escalabilidade da
aplicação é a utilização
indiscriminada da session

62. SOLUÇÃO?

63. Paginação sob
demanda

64. 2º Mau hábito

66. Efetuar consultas de
maneira
INEFICIENTE

67. <h:dataTable
value="#{bean.usersList}"
var="user">
<h:column ...>
...
</h:column>
</h:dataTable>

68. public class Bean {
public List<User> getUsersList() {
return service.findAllUsers();
}
}

69. SOLUÇÃO?

70. Usar consultas
em eventos ou
callbacks

71. - Callback -
public class Bean {
@PostConstruct
public void initialize(){
this.users = service.findAllUsers();
}
public List<User> getUsersList() {
return this.users;
}
}

72. - Evento -
public class Bean {
public void search(ActionEvent e){
this.users = service.findUsers( … );
}
public List<User> getUsersList() {
return this.users;
}
}

73. 1º Mau hábito

74. 1º -e o pior- Mau hábito

75. JSF LIFECYCLE

76. JSF LIFECYCLE
A maioria dos
desenvolvedores NÃO
entendem

77. JSF LIFECYCLE
SOLUÇÃO?

78. http://balusc.blogspot.com
/2006/09/debug-jsf-lifecycle.html
Entendam o
ciclo de vida

79. Concluindo..

80. Perguntas?

81. Obrigado!
Rafael Ponte Tarso Bessa
twitter.com/rponte twitter.com/tarsobessa