26

Comecei a estudar swift e me deparei com um termo muito utilizado chamado Code Smell pelo que consegui entender na Wikipedia e em alguns outros sites ele é qualquer sintoma no código fonte que indica um problema mais profundo, e.g: código duplicado.

Por esta definição eu acabo me questionando a seguinte coisa, Code Smell não é a mesma coisa que os princípios DRY, KISS e YAGNI?

Como posso identificar code smell/bad smell em um código?

7
  • 1
    Já eu achava que era algo relacionado à dívida técnica, mas não tenho certeza.
    – Math
    Commented 24/11/2015 às 11:00
  • @Math está lá no artigo da Wikipedia, "Code smells are an important reason for technical debt."
    – Piovezan
    Commented 24/11/2015 às 11:06
  • @Piovezan tem razão, eu não cheguei a ler a página, mas sabia que tinha algo a ver :P
    – Math
    Commented 24/11/2015 às 11:14
  • @Math Nunca tinha ouvido falar em dívida técnica, acho que são tantos fatores que devem ser considerados na hora de ver se um código pode ser refatorado ou não que me deixam mais confuso, acabo lembrando de uma frase antiga dos programadores "Se esta funcionando,não mexa." Commented 24/11/2015 às 11:28
  • 8
    Você conhece a expressão "onde há fumaça há fogo"? O sentido disso é, bom, nem sempre fumaça quer dizer fogo, mas é um bom indício, então melhor ficar alerta... Da mesa forma, no meu entendimento, "code smell" tem o sentido de indicar coisas que não estão "erradas" no código, mas que podem indicar um problema mais profundo. Às vezes o programador se desvia das "boas práticas" por um bom motivo. Mas na maioria das vezes, o faz por ignorância, então é bom ficar atento às possíveis consequências (revisar o código), pois pode não estar quebrado agora mas tem boa chance de quebrar no futuro.
    – mgibsonbr
    Commented 24/11/2015 às 11:38

3 Respostas 3

14

Determinar o que é ou não um code smell é sempre um julgamento subjetivo, e sempre irá variar de acordo com a linguagem de programação, o desenvolvedor e a metodologia de desenvolvimento.

Existem ferramentas que detectam certos tipos de code smells, como:

Java: Checkstyle, PMD e FindBugs.

.Net: ReSharper.

Para Php.

Alguns Code smells comuns:

Código duplicado: código idêntico ou muito similar existe em mais de um local.

Método longo: um método, função ou procedure muito extenso.

Classe extensa: uma classe que acabou ficando muito extensa (God Object).

Feature envy (sem tradução): uma classe que utiliza em excesso métodos de outra classe.

Intimidade inapropriada: uma classe que possui dependência de detalhes de implementação de outra classe.

Legado recusado: uma classe que sobrepõe (override) o método da classe genérica de forma que o contrato da classe genérica não é cumprido pela classe derivada.

Classe preguiçosa: classe que faz muito pouco.

Complexidade artificial: uso forçado de design patterns extremamente complicados, onde um design simples seria suficiente.

Identificadores excessivamente longos: em particular, o uso de convenções de nomes para evitar ambiguidades, o que deveria estar implícito na arquitetura do software.

9

Não são a mesma coisa. Muitos code smells (mau cheiro no código) irão seguir esses princípios de DRY, KISS e YAGNI, que são prevalentes em desenvolvimento de software, mas nem todos irão (exemplos: Excessively short identifiers, Constants class).

Mas independente disso, se as situações de mau cheiro fossem óbvias para todo mundo, não seria preciso compilar uma lista de code smells em primeiro lugar. Bastaria dizer "siga sempre princípios como o DRY, KISS e YAGNI".

Como foi dito, há ferramentas que detectam alguns code smells, mas a forma mais completa de saber se o seu código tem code smells é pegar uma boa lista de code smells (tem uma por exemplo no livro "Código Limpo" do Robert C. Martin), estudá-los e ver se existem no seu código. Se todos foram óbvios e você evitou todos no seu código, parabéns! Você é muito bom em aplicar os princípios de DRY, KISS e YAGNI, entre outros.

Caso contrário, você conheceu melhor os code smells e os princípios por trás deles e com isso saberá ficar mais atento a código com mau cheiro, podendo aplicar isso em seu código existente e em códigos futuros. Isso o ajudará a ter código mais robusto e seguindo bons princípios.

7

code smell/bad smell, é o termo usado para evidenciar que algo no código não faz sentido(falta de coesão), caso esse problema não seja tratado irá se espalhar, o custo e esforço para corrigi-lo aumentará. Alguns exemplos são classes com mais de uma reponsabilidade e falta de modularização.

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .