Linhas em Branco no Método é um Mal Cheiro no Código


Pular linhas dentro de um Método é um “mal cheiro” no código. Significa que você está fazendo coisas demais num único Método.

Imagem

Introdução

O que significa mal cheiro no código?

Um mal cheiro, se refere a qualquer sintoma no código-fonte de um programa que possivelmente indica um problema mais profundo.

Eu já falei sobre isso aqui e nesse artigo vou abordar melhor esse assunto.

Você irá entender porque não devemos utilizar linhas em branco dentro de Métodos.

Blocos de Código

Por que muitos programadores ainda utilizam linhas em branco dentro de Métodos?

Simples. Eles estão separando o código de um Método em blocos. E isso é, errado.

Eu mesmo fiz isso por anos.

Algumas vezes eu conseguia enxergar esse mal cheiro, então refatorava — criando outros Métodos — e seguia a vida.

Mas na maioria das vezes eu pensava que estava “organizando o código”. Tornando mais legível, mas “limpo”… não era verdade.

Quando li esse artigo eu percebi que sempre houve algo de errado quando eu utiliza linhas em branco.

Quando existem linhas em branco dentro do Método, significa que o Método está fazendo coisas demais. Um único grande procedimento, separado por com blocos de código…

Desde então eu não codifico mais dessa forma.

Não basta retirar as Linhas em Branco

Você entendeu o problema e agora quer abrir seu editor e começar a remover as linhas em branco… apenas remover as linhas.

Não seja leviano.

Não basta abrir seu editor e começar a remover toda linha em branco que encontrar dentro dos Métodos.

Não!

A linha em branco não é o problema em si. Ela só indica que há um problema.

Apenas retirar essas linhas não vai sanar o problema.

O que temos que fazer é refatorar o código, para depois removê-las.

Refatorando

Se você realmente entendeu o conceito, não irá mais utilizar linhas em branco dentro de Métodos de agora em diante.

Certo?

Mas você pode ir além. Você pode querer refatorar todo tipo de código antigo ou fazer uma refatoração on demand: Refatorar apenas se encontrar Métodos com linhas em branco.

Você pode seguir esses passos:

  1. Encontre um Método;
  2. Verifique se há linhas em branco separando blocos;
  3. Para cada bloco, extrair o código criando um novo Método;
  4. Faça a chamada do novo Método no mesmo lugar que extraiu o código;
  5. Repita o processo enquanto for necessário.

Conclusão

Parece besteira, mas não é.

Escrever código qualquer um faz. Mas escrever um bom código, no entanto, não é pra qualquer um. É necessário pensar no longo prazo, no acoplamento, na coesão, no design, deixar as coisas simples… é Engenharia, mas também Arte.

E a verdadeira Arte está nos detalhes.

Até logo.

Posts Relacionados

  • Mais Performance usando Argumentos "const" para Interfaces

  • Herança de Formulário é para Iniciantes

  • Eliminando Métodos Privados

  • Classes Aninhadas

  • API Unit: Tudo num só lugar

  • Injeção de Dependência sem XML, Atributos/Anotações ou Frameworks

  • Nomeando Classes em Libraries

  • Versionando e Organizando seus Pacotes

  • Xavier Package

  • Inter-process Communication