demoiselle / signer

Repositório que contém os componentes para facilitar a implementação de assinatura digital nos padrões da ICP-BRASIL
https://www.frameworkdemoiselle.gov.br/v3/signer/
GNU Lesser General Public License v3.0
142 stars 73 forks source link

CAdESSigner não deveria ter variáveis de instância #336

Open crivano opened 2 years ago

crivano commented 2 years ago

Isto é apenas uma sugestão, mas acredito que seja possível que essa classe tenha apenas métodos (possivelmente estáticos) que recebem todos os parâmetros necessários para produzir assinaturas de vários tipos.

Sem as variáveis de instância, o funcionamento da classe ficará mais previsível (pois não manterá estado).

Será mais simples para o usuário final, pois existirão vários métodos para escolher, e bastará chamar um deles passando todos os parâmetros necessários que a assinatura será produzida como desejado.

Por fim, até mesmo para criar testes unitários, acredito que ficará mais fácil.