codigo legado fisl

CÓDIGO LEGADO

Fazendo as pazes com

quinta-feira, 30 de junho de 2011

~ 12 anos de estrada

~ Desenvolvedor Senior da Globo.com

~ Apaixonado pelo que faço ...


~ 4 anos de estrada

~ Desenvolvedor da Globo.com

~ Hacker por definição e apaixonado por tecnologia

Guilherme Souza (não pode vir)


NOT


Made by Other


Cuidadovocê pode estar

escrevendo código legado agora ...


TESTE


Clean Code is not ENOUGH


Era uma vez ...


E daí ?


Era uma vez ...


Refazer é caro


MissãoEnsinar técnicas e formas de pensar que facilitem e deixem a todos confiantes para mexer em qualquer base de código


Era uma vez ...


Caminho é longo ...


Complexidade é alta ...


Mas o prazo é oMESMO


Por onde começar?


.. vença uma por vez ...

Escolha suas batalhas e


Dependencia

Efeito Colateral

Aumentar cobertura de teste


Identify Change Points


Find Test Points


Break Dependencies


Cover with tests


Change


Como eu faço isso ? quinta-feira, 30 de junho de 2011

example.rb Page 1

class Example def send_welcome_msg clients = Client.get_all clients.each do |client| addresses = client.get_all_users body = << EOF Welcome to our system. We're very happy for your presence here. EOF

addresses.each do |user| sender = MailSender.new sender.from= "[email protected]" sender.to= user.mail sender.body=body sender.send end client.billing << Charging.new(200) account = client.bank_account account.charge end endend


example.rb Page 1


addresses.each do |user| sender = MailSender.new sender.from= "[email protected]" sender.to= user.mail sender.body=body sender.send end client.billing << Charging.new(200) account = client.bank_account account.charge end endend

Quero colocar aqui um audit


Sprout

Methods


Change Point


Change Point

New Tested Method


example.rb Page 1


addresses.each do |user| sender = MailSender.new sender.from= "[email protected]" sender.to= user.mail sender.body=body sender.send audit_send(user.mail, body) end client.billing << Charging.new(200) account = client.bank_account account.charge end end

def audit_send(mail, body) audit = Audit.new audit.operations << SendMailOperatio.new(:to=> mail, :body => body, created_at => Date.now) audit.save endend


example.rb Page 1


addresses.each do |user| sender = MailSender.new sender.from= "[email protected]" sender.to= user.mail sender.body=body sender.send audit_send(user.mail, body) end client.billing << Charging.new(200) account = client.bank_account account.charge end end


TEST

ADO


Sprout

Classes


Change Point


Change Point

New Tested Class


example.rb[+] Page 1

class Example def welcome clients = Client.get_all clients.each do |client| addresses = client.get_all_users body = << EOF Welcome to our system. We're very happy for your presence here. EOF

addresses.each do |user| sender = MailSender.new sender.from= "[email protected]" sender.to= user.mail sender.body=body sender.send audit_send(user.mail, body) end end end



Wrap

Methods


Change Point


Change Point

Wrap Tested Method


example.rb[+] Page 1

class Example def old_welcome(clients) clients.each do |client| addresses = client.get_all_users body = << EOF Welcome to our system. We're very happy for your presence here. EOF

addresses.each do |user| sender = MailSender.new sender.from= "[email protected]" sender.to= user.mail sender.body=body sender.send audit_send(user.mail, body) end end end

def welcome clients= Client.get_all old_welcome clients Billing.charge(clients) end



Wrap

Classes


Change Point


Change Point

Wrap Tested Class


Acabou ?

Era só isso?


NEGATIVO


Vamos ao refactoring ...


Não vamos reinventar nada !

ATENÇÃO


SOLIDSMELLS (Fowler)PatternsRefactoring Patterns


Foco


Bora codar um pouco ...


@aoqfonseca

[email protected]

www.andrefonseca.net

Obrigado !!!


mailto:[email protected]

mailto:[email protected]

http://www.andrefonseca.net

http://www.andrefonseca.net

codigo legado fisl

Technology