Menu

#4 refabricaçoes 2ª parte

1.0
closed
nobody
None
2016-03-31
2016-03-31
No

Foram aplicadas as seguintes refabricações:

  1. Localização: Classe SpamFilter, no método getNextmesage.
    Issue: Utilização demasiadas vezes a expessão message.getClientNickname(), o qual pode diminuir a legibilidade do código.
    Refabricação: Extrair uma variável através do Extract Local Variable à expressão message.getClientNickname() e desse modo criar-se a variável local clientNickname.

  2. Localização: Classe ClientsHandler, no método run().
    Issue: Um método extenso com muitas exceções podem levar à sua difícil compreensão.
    Refabricação: Extrair um novo método através do Extract Method criando o método notInterrupted().

  3. Localização: Classe SpamFilter, método spamMessage.
    Issue: Método demasiado longo. Reparti-lo em blocos mais pequenos pode torná-lo mais legível.
    Refabricação: Extrair um novo método através do Extract Method criando o método samePayload.

Discussion

  • Carolina Costa

    Carolina Costa - 2016-03-31
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -8,6 +8,6 @@
     Issue: Um método extenso com muitas exceções podem levar à sua difícil compreensão. 
     Refabricação: Extrair um novo método através do Extract Method criando o método notInterrupted(). 
    
    -3. Localização: Classe SpamFilte, método spamMessage.
    +3. Localização: Classe SpamFilter, método spamMessage.
     Issue: Método demasiado longo. Reparti-lo em blocos mais pequenos pode torná-lo mais legível.
    -    Refabricação: Extrair um novo método através do Extract Method criando o método samePayload.
    +Refabricação: Extrair um novo método através do Extract Method criando o método samePayload.
    
     
  • Carolina Costa

    Carolina Costa - 2016-03-31
    • status: open --> closed
     

Log in to post a comment.