Qualidade de código

Transcript

1. Qualidade do código Monday, November 12, 12
2. brunoadacosta@gmail.com @brunoadacosta Bruno Alvares da Costa http:/ /hungroo.com /brunoadacosta Monday, November 12, 12
3. Monday, November 12, 12
4. Inploy Hypertemplate Factory_girl-preload /brunoadacosta Monday, November 12, 12
5. Referências Monday, November 12, 12
6. Vamos ter em mente Manter um sistema ao longo do tempo Implementar mudanças Corrigir bugs Evoluir Monday, November 12, 12
7. Codificar é dificil Monday, November 12, 12
8. Codificar é MUITO dificil Monday, November 12, 12
9. def update password_changed = false @user = current_user if u = params[:user] u.delete(:password) if u[:password].blank? u.delete(:password_confirmation) if u[:password].blank? and u[:password_confirmation].blank? u.delete(:language) if u[:language].blank? # change email notifications if u[:email_preferences] @user.update_user_preferences(u[:email_preferences]) flash[:notice] = I18n.t 'users.update.email_notifications_changed' # change password elsif u[:current_password] && u[:password] && u[:password_confirmation] if @user.update_with_password(u) password_changed = true flash[:notice] = I18n.t 'users.update.password_changed' else flash[:error] = I18n.t 'users.update.password_not_changed' end elsif u[:show_community_spotlight_in_stream] || u[:getting_started] if @user.update_attributes(u) flash[:notice] = I18n.t 'users.update.settings_updated' else flash[:notice] = I18n.t 'users.update.settings_not_updated' end elsif u[:language] if @user.update_attributes(u) I18n.locale = @user.language flash[:notice] = I18n.t 'users.update.language_changed' else flash[:error] = I18n.t 'users.update.language_not_changed' end elsif u[:email] @user.unconfirmed_email = u[:email] if @user.save @user.mail_confirm_email flash[:notice] = I18n.t 'users.update.unconfirmed_email_changed' else flash[:error] = I18n.t 'users.update.unconfirmed_email_not_changed' end elsif u[:auto_follow_back] if @user.update_attributes(u) flash[:notice] = I18n.t 'users.update.follow_settings_changed' else flash[:error] = I18n.t 'users.update.follow_settings_not_changed' end end elsif aspect_order = params[:reorder_aspects] @user.reorder_aspects(aspect_order) end respond_to do |format| format.js { render :nothing => true, :status => 204 } format.all { redirect_to password_changed ? new_user_session_path : edit_user_path } end end Monday, November 12, 12
10. def update password_changed = false @user = current_user if u = params[:user] u.delete(:password) if u[:password].blank? u.delete(:password_confirmation) if u[:password].blank? and u[:password_confirmation].blank? u.delete(:language) if u[:language].blank? # change email notifications if u[:email_preferences] @user.update_user_preferences(u[:email_preferences]) flash[:notice] = I18n.t 'users.update.email_notifications_changed' # change password elsif u[:current_password] && u[:password] && u[:password_confirmation] if @user.update_with_password(u) password_changed = true flash[:notice] = I18n.t 'users.update.password_changed' else flash[:error] = I18n.t 'users.update.password_not_changed' end elsif u[:show_community_spotlight_in_stream] || u[:getting_started] if @user.update_attributes(u) flash[:notice] = I18n.t 'users.update.settings_updated' else flash[:notice] = I18n.t 'users.update.settings_not_updated' end elsif u[:language] if @user.update_attributes(u) I18n.locale = @user.language flash[:notice] = I18n.t 'users.update.language_changed' else flash[:error] = I18n.t 'users.update.language_not_changed' end elsif u[:email] @user.unconfirmed_email = u[:email] if @user.save @user.mail_confirm_email flash[:notice] = I18n.t 'users.update.unconfirmed_email_changed' else flash[:error] = I18n.t 'users.update.unconfirmed_email_not_changed' end elsif u[:auto_follow_back] if @user.update_attributes(u) flash[:notice] = I18n.t 'users.update.follow_settings_changed' else flash[:error] = I18n.t 'users.update.follow_settings_not_changed' end end elsif aspect_order = params[:reorder_aspects] @user.reorder_aspects(aspect_order) end respond_to do |format| format.js { render :nothing => true, :status => 204 } format.all { redirect_to password_changed ? new_user_session_path : edit_user_path } end end Monday, November 12, 12
11. u[:password_confirmation].blank? u.delete(:language) if u[:language].blank? # change email notifications if u[:email_preferences] @user.update_user_preferences(u[:email_preferences]) flash[:notice] = I18n.t 'users.update.email_notifications_changed' # change password elsif u[:current_password] && u[:password] && u[:password_confirmation] if @user.update_with_password(u) password_changed = true flash[:notice] = I18n.t 'users.update.password_changed' else flash[:error] = I18n.t 'users.update.password_not_changed' end elsif u[:show_community_spotlight_in_stream] || u[:getting_started] if @user.update_attributes(u) flash[:notice] = I18n.t 'users.update.settings_updated' else flash[:notice] = I18n.t 'users.update.settings_not_updated' end elsif u[:language] if @user.update_attributes(u) I18n.locale = @user.language flash[:notice] = I18n.t 'users.update.language_changed' else flash[:error] = I18n.t 'users.update.language_not_changed' end elsif u[:email] @user.unconfirmed_email = u[:email] if @user.save @user.mail_confirm_email flash[:notice] = I18n.t 'users.update.unconfirmed_email_changed' else flash[:error] = I18n.t 'users.update.unconfirmed_email_not_changed' end elsif u[:auto_follow_back] if @user.update_attributes(u) flash[:notice] = I18n.t 'users.update.follow_settings_changed' else flash[:error] = I18n.t 'users.update.follow_settings_not_changed' end end elsif aspect_order = params[:reorder_aspects] @user.reorder_aspects(aspect_order) end Monday, November 12, 12
12. Monday, November 12, 12
13. Code smell Monday, November 12, 12
14. Code smell Monday, November 12, 12
15. Monday, November 12, 12
16. Rigidez “Cada vez que você altera algo, outra coisa precisa ser modificada” Monday, November 12, 12
17. Fragilidade “Uma mudança de uma parte do sistema, quebra em muitas outras partes completamente alheios” Monday, November 12, 12
18. Imobilidade “É dificil separar o sistema em componentes que podem ser utilizados em outros sistemas” Monday, November 12, 12
19. Complexidade desnecessária “Há muitas estruturas de código complexa sem necessidade, ou otimização prematura” Monday, November 12, 12
20. Repetições inúteis “Código parece que foi escrito por dois programadores chamados recortar e colar” Monday, November 12, 12
21. Qualidade do código boas práticas princípios e padrões Monday, November 12, 12
22. Monday, November 12, 12
23. Monday, November 12, 12
24. Refatoração “Não sou um grande programador, sou apenas um bom programador com ótimos hábitos. Refatorar me ajuda a ser muito mais efetivo na escrita de código robusto” Kent Beck Monday, November 12, 12
25. S O L I D ingle Responsibility Principle (SRP) pen/Closed Principle (OCP) iskov substitution principle (LSP) nterface Segregation Principle (ISP) ependency Inversion Principle (DIP) Monday, November 12, 12
26. Single Responsibility Principle Uma entidade, deve ter uma única responsabilidade Responsabilidade pode ser definida como “razão para mudar” Monday, November 12, 12
27. Código ruim Monday, November 12, 12
28. SRP Monday, November 12, 12
29. Open Closed Principle Uma classe deve manter sua interface (API pública) A implementação pode ser alterada livremente Monday, November 12, 12
30. Implementação mudou mas a interface permaneceu Monday, November 12, 12
31. Liskov substitution principle Deve ser possível subistituir uma classe por suas classes derivadas em qualquer ponto do código O código não deve ser alterado para isso acontecer Monday, November 12, 12
32. Liskov substitution principle Monday, November 12, 12
33. Interface Segregation Principle Classes não devem ser obrigadas a depender de interfaces que elas não utilizam Deve-se utilizar interfaces concisas, com apenas o que é usado O SRP resolve este problema Monday, November 12, 12
34. Classe marota :) Monday, November 12, 12
35. Aplicando o SRP, se aplica ISP Monday, November 12, 12
36. Dependency Inversion Principle Módulos de alto nível não devem depender de módulos de baixo nível, ambos devem depender de abstrações (interface) Abstrações não devem depender de detalhes , detalhes devem depender de abstrações Monday, November 12, 12
37. Código muito acoplado Monday, November 12, 12
38. E se no lugar de salvar no banco de dados eu quero jogar em um memcache? Monday, November 12, 12
39. Monday, November 12, 12
40. Monday, November 12, 12
41. Monday, November 12, 12
42. Dúvidas? Monday, November 12, 12
43. http://www.meetup.com/devinsantos http://twitter.com/guruBaixada https://groups.google.com/forum/?fromgroups#!forum/devinsantos https://groups.google.com/forum/?fromgroups#!forum/guru-baixada Monday, November 12, 12
44. Obrigado. brunoadacosta@gmail.com @brunoadacosta Bruno Alvares da Costa /brunoadacosta Monday, November 12, 12