Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
crivano opened this issue Mar 29, 2022 · 0 comments
Open

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

crivano opened this issue Mar 29, 2022 · 0 comments

Comments

@crivano
Copy link
Contributor

crivano commented Mar 29, 2022

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant