search

concretesolutions / canarinho

Utilitários Android para padrões Brasileiros
Apache License 2.0
178 stars 42 forks source link

Update ValidadorCPF.java #10

Closed andersonb closed 7 years ago

andersonb commented 7 years ago

Corrigir o modificador de acesso ao validador de CPF

cs-victor-nascimento commented 7 years ago

Hey @andersonb ! Obrigado pela contribuição!

No entanto, nossa idéia era deixar os validadores públicos apenas pelas constantes da própria classe Validador. Nesse caso, use: Validador.CPF. Isso para que você não tenha que criar várias instâncias dos validadores.

Se tiver mais dúvidas pode deixar comentários aqui.

andersonb commented 7 years ago

@cs-victor-nascimento O Singleton não resolveu esse problema de ter várias instâncias dos validadores?

cs-victor-nascimento commented 7 years ago

Pois é @andersonb, ele resolve sim. Eu pensei em uma coisa e escrevi outra...

Mas a idéia é que os validadores da lib fiquem centralizados na própria interface. Caso sejam incluídos outros validadores podemos até acabar movendo os mesmos de pacotes para organizar melhor. Hoje se você quiser saber quais validadores tem na Canarinho é só ver esta interface.

Isso pode estar confuso agora, mas todo mundo usa pela mesma interface e evita de usar métodos específicos de um validador. Talvez num major release (2.0.0) isso seja alterado já que está gerando confusão no uso. Nesse caso atualizaríamos todos os validadores.