Pérola arquitetônica

Aqui está um código que é uma verdadeira obra arquitetônica!


A montagem eu mesmo que fiz :-)
Bom, mas quais são os problemas com esse código?

Este código está aí para permitir que campos possuam máscara de dados. Até aí, é um requisito normal de qualquer sistema. O problema é a forma como está sendo feito. Eu verifiquei e existem várias máscaras que não estão sendo utilizadas. Um primeiro problema aqui é desempenho, pois esse script executa em toda página que é carregada, acessando elementos HTML pelo jQuery. É um código desnecessário e sem justificativa.

Agora conceitualmente, essas inúmeras máscaras, qual o significado delas dentro do sistema? Qual tipo de campo é formatado por 19 números sequenciais? Não há motivo para esse código existir. Seria necessário a associação com alguma regra do sistema para que elas fizessem sentido. Máscaras que têm sentido são, por exemplo, CPF (999.999.999-99) ou CEP (99999-999).

[]'s

Comentários

Postagens mais visitadas deste blog

Trocando configurações padrão do Live TIM

Uma proposta de Clean Architecure com Modelo de Atores

Testes automatizados em sistemas autenticados com certificados digitais, usando Selenium e PhantomJS