Home > Delphi, Geral > Evite Código Inútil

Evite Código Inútil

Uma coisa que eu gosto de fazer é refatorar código antigo. Sempre encontro muita coisa inútil que pode ser removida. Incontáveis vezes eu já escrevi código do tipo:

if not MeuDataSet.Active then
  MeuDataSet.Open;

O problema é que esse “if” é totalmente inútil. Se você olhar no código da classe TDataSet do Delphi vai ver que ele já faz essa checagem sempre que você chama Open. Outro código inútil que eu também já escrevi e continuo vendo muita gente repetir, é o seguinte:

if ClientDataSet.ChangeCount > 0 then
  ClientDataSet.ApplyUpdates(0);

Você também não precisa desse “if” pois a checagem também já é feita pelo ClientDataSet automaticamente.

Comece a olhar melhor para seu código e código de terceiros que você usa e certamente encontrará muita coisa que pode ser eliminada.

Categories: Delphi, Geral Tags:
  1. Ediberto Alves
    November 14th, 2007 at 01:20 | #1

    Bem…. é… tipo…
    Vou começar a olhar os meus códigos… tem muito “if” neles. =/

  2. Bruno Oliveira
    April 26th, 2010 at 22:00 | #2

    Esse é o tipo de problema que não se tem quando se utiliza TDD, pois vc só escreve o código suficiente para os testes passarem.

    Ou seja, só se colocaria o if se seu código desse barra vermelha, e como daria barra verde, não seria colocado nenhum if.

    Abs,,,

  3. April 26th, 2010 at 22:57 | #3

    Sou totalmente favorável a testes unitários, mas não acho que ele resolve isso não. Quem escreve esse tipo de código escreve porque acha que é necessário, então a pessoa escreveria mesmo antes de rodar o teste.

  1. No trackbacks yet.