Código Limpo

ou como escrever código que te faça ter orgulho e não faça seu colega querer te bater

Transcript

1. Código Limpo Ou como escrever um código que te dê orgulho e não faça seus amigos terem ódio de você.
2. Ola Marilene! Tarsis Azevedo Programador na globo.com tarsisazevedo.com
3. Código feio
4. Código feio if variavel is True: print 'sou foda' elif variavel is False: print "dig din" else: print "duvido chegar aqui"
5. Código feio $('a').click(function () { var id = ''; var id2 = ''; id = $(this).attr('id'); id2 = $(this).attr('id'); if (id == "search11") { // do something } if (id2 == "search1") { // do something } });
6. Código feio groupLayout_2.setVerticalGroup( groupLayout_2.createParallelGroup(Alignment.LEADING) .addGroup(groupLayout_2.createSequentialGroup() .addContainerGap() .addGroup(groupLayout_2.createParallelGroup(Alignment.BASELINE) .addComponent(txtUserIn, GroupLayout.PREFERRED_SIZE, GroupLayout.DEFAULT_SIZE, GroupLayout.PREFERRED_SIZE) .addComponent(txtUserFilename, GroupLayout.PREFERRED_SIZE, GroupLayout.DEFAULT_SIZE, GroupLayout.PREFERRED_SIZE)) .addPreferredGap(ComponentPlacement.UNRELATED) .addGroup(groupLayout_2.createParallelGroup(Alignment.BASELINE) .addComponent(txtPassIn, GroupLayout.PREFERRED_SIZE, GroupLayout.DEFAULT_SIZE, GroupLayout.PREFERRED_SIZE) .addComponent(txtPassFilename, GroupLayout.PREFERRED_SIZE, GroupLayout.DEFAULT_SIZE, GroupLayout.PREFERRED_SIZE)) .addPreferredGap(ComponentPlacement.UNRELATED) .addGroup(groupLayout_2.createParallelGroup(Alignment.BASELINE) .addComponent(txtSuccsessOut, GroupLayout.PREFERRED_SIZE, GroupLayout.DEFAULT_SIZE, GroupLayout.PREFERRED_SIZE) .addComponent(txtSuccsessFile, GroupLayout.PREFERRED_SIZE, GroupLayout.DEFAULT_SIZE, GroupLayout.PREFERRED_SIZE)) .addPreferredGap(ComponentPlacement.UNRELATED) .addGroup(groupLayout_2.createParallelGroup(Alignment.BASELINE) .addComponent(CrackedAccounts, GroupLayout.PREFERRED_SIZE, GroupLayout.DEFAULT_SIZE, GroupLayout.PREFERRED_SIZE) .addComponent(txtNumberOfCracked, GroupLayout.PREFERRED_SIZE, GroupLayout.DEFAULT_SIZE, GroupLayout.PREFERRED_SIZE)) .addGap(9) .addComponent(btnCrack, GroupLayout.PREFERRED_SIZE, 35, GroupLayout.PREFERRED_SIZE) .addContainerGap(47, Short.MAX_VALUE)) );
7. Código feio <?php function register() { if (!empty($_POST)) { $msg = ''; if ($_POST['user_name']) { if ($_POST['user_password_new']) { if ($_POST['user_password_new'] === $_POST['user_password_repeat']) { if (strlen($_POST['user_password_new']) > 5) { if (strlen($_POST['user_name']) < 65 && strlen($_POST['user_name']) > 1) { if (preg_match('/^[a-z\d]{2,64}$/i', $_POST['user_name'])) { $user = read_user($_POST['user_name']); if (!isset($user['user_name'])) { if ($_POST['user_email']) { if (strlen($_POST['user_email']) < 65) { if (filter_var($_POST['user_email'], FILTER_VALIDATE_EMAIL)) { create_user(); $_SESSION['msg'] = 'You are now registered so please login'; header('Location: ' . $_SERVER['PHP_SELF']); exit(); } else $msg = 'You must provide a valid email address'; } else $msg = 'Email must be less than 64 characters'; } else $msg = 'Email cannot be empty'; } else $msg = 'Username already exists'; } else $msg = 'Username must be only a-z, A-Z, 0-9'; } else $msg = 'Username must be between 2 and 64 characters'; } else $msg = 'Password must be at least 6 characters'; } else $msg = 'Passwords do not match'; } else $msg = 'Empty Password'; } else $msg = 'Empty Username'; $_SESSION['msg'] = $msg; } return register_form(); }
8. Código feio <?php function register() { if (!empty($_POST)) { $msg = ''; if ($_POST['user_name']) { if ($_POST['user_password_new']) { if ($_POST['user_password_new'] === $_POST['user_password_repeat']) { if (strlen($_POST['user_password_new']) > 5) { if (strlen($_POST['user_name']) < 65 && strlen($_POST['user_name']) > 1) { if (preg_match('/^[a-z\d]{2,64}$/i', $_POST['user_name'])) { $user = read_user($_POST['user_name']); if (!isset($user['user_name'])) { if ($_POST['user_email']) { if (strlen($_POST['user_email']) < 65) { if (filter_var($_POST['user_email'], FILTER_VALIDATE_EMAIL)) { create_user(); $_SESSION['msg'] = 'You are now registered so please login'; header('Location: ' . $_SERVER['PHP_SELF']); exit(); } else $msg = 'You must provide a valid email address'; } else $msg = 'Email must be less than 64 characters'; } else $msg = 'Email cannot be empty'; } else $msg = 'Username already exists'; } else $msg = 'Username must be only a-z, A-Z, 0-9'; } else $msg = 'Username must be between 2 and 64 characters'; } else $msg = 'Password must be at least 6 characters'; } else $msg = 'Passwords do not match'; } else $msg = 'Empty Password'; } else $msg = 'Empty Username'; $_SESSION['msg'] = $msg; } return register_form(); }
10. E daí?
11. E daí? Computadores não se importam com beleza
12. E daí? Ninguém vai ler, só eu!
13. E daí? Não vou voltar nesse código. Nunca mais!
14. E daí? É rapidinho, depois eu mudo.
16. Código limpo
17. Código limpo The Broken Window
18. Código limpo Escreva o código e depois limpe!
19. Código limpo Regra do escoteiro
20. Código limpo Don’t Repeat Yourself
21. Código limpo TDD
22. TDD Definem como seu código funciona
23. TDD Previsibilidade
24. TDD
25. TDD Isso me deixa lento?
26. TDD Sim Porem com menos bugs* * http://www.infoq.com/news/2009/03/TDD-Improves-Quality
27. TDD É chato?
28. TDD Sim, às vezes. Mas te ajuda a ter confiança no seu código
29. TDD
30. Código limpo def caLCulA(pAramEtro, x, W): if pAramEtro != x and pAramEtro != W and x != W: if pAramEtro + x + W <= 180: return True return False
31. Código limpo def caLCulA(pAramEtro, x, W): if pAramEtro != x and pAramEtro != W and x != W: if pAramEtro + x + W <= 180: return True return False
32. Código limpo Nomes importam!
33. Código limpo def eh_escaleno(angulo_1, angulo_2, angulo_3): if angulo_1 != angulo_2 and angulo_1 != angulo_3 and angulo_2 != angulo_3: if angulo_1 + angulo_2 + angulo_3 <= 180: return True return False
34. Código limpo def eh_escaleno(angulo_1, angulo_2, angulo_3): if angulo_1 != angulo_2 and angulo_1 != angulo_3 and angulo_2 != angulo_3: if angulo_1 + angulo_2 + angulo_3 <= 180: return True return False
35. Código limpo Use a linguagem
36. Código limpo def eh_escaleno(angulos): if len(set(angulos)) == 3: if sum(angulos) <= 180: return True return False
37. Código limpo PEP8 go fmt google styleguide vinta playbook
38. Código limpo
39. Código limpo class Cafe(object): preco = 1.0 def calcula_preco(self): if self.tipo == 'com leite': return self.preco + 0.2 if self.tipo == 'com creme': return self.preco + 0.4 if self.tipo == 'com leite e creme': return self.preco + 0.6 return self.preco
40. Código limpo class Cafe(object): preco = 1.0 def calcula_preco(self): if self.tipo == 'com leite': return self.preco + 0.2 if self.tipo == 'com creme': return self.preco + 0.4 if self.tipo == 'com leite e creme': return self.preco + 0.6 if self.tipo == 'com doce de leite': return self.preco + 0.6 if self.tipo == 'com doce de leite e leite e creme': return self.preco + 1.2 return self.preco
41. Código limpo class Adicional(object): preco = None def __init__(self, cafe): self.cafe = cafe def calcula_preco(self): raise NotImplementedError class Leite(Adicional): preco = 0.2 def calcula_preco(self): self.cafe.preco += self.preco return self.cafe.preco
42. Código limpo class Creme(Adicional): preco = 0.4 def calcula_preco(self): self.cafe.preco += self.preco return self.cafe.preco class DoceDeLeite(Adicional): preco = 0.6 def calcula_preco(self): self.cafe.preco += self.preco return self.cafe.preco class Cafe(object): preco = 1.0
43. Código limpo cafe = Cafe() cafe_com_leite = Leite(cafe) cafe_com_leite.calcula_preco() cafe_com_leite_e_creme = Creme(cafe_com_leite.cafe) cafe_com_leite_e_creme.calcula_preco() cafe_com_leite_e_creme_e_doce_de_leite = DoceDeLeite(cafe_com_leite_e_creme.cafe) cafe_com_leite_e_creme_e_doce_de_leite.calcula_preco()
45. Design Pattern alguém já teve esse problema mas é só um guia, adapte a sua realidade
46. SE IMPORTE COM O CODIGO
47. KEEP IT SIMPLE
48. E o legado?
50. Problemas
51. Problemas Projeto gigante - 300k LOC 30% de duplicação de codigo Testes lentos e instáveis Ignorando PEP8
52. Resolvendo “Cachorro com muitos donos morre de fome” Apagando testes inúteis Estabilizando e otimizando os testes CI para cada time
53. Resolvendo Automatizando a qualidade Foco no time Depois outras areas
54. Livros
55. Livros
56. tarsis.azevedo@gmail.com @tarsisazevedo facebook.com/tarsis.azevedo github.com/tarsisazevedo tarsisazevedo.com
57. obrigado