Skip to content
New issue

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

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

Already on GitHub? # to your account

chore: refatorar estrutura do middleware para melhor legibilidade #120

Closed
wants to merge 1 commit into from

Conversation

k-vanio
Copy link

@k-vanio k-vanio commented Mar 10, 2025

As melhorias aplicadas nesta nova versão incluem:

Uso de uma Constante (Cors) em Vez de String Literal

  • Antes: "cors" era comparado diretamente.

  • Agora: Cors (provavelmente uma constante definida em outro lugar) melhora a legibilidade, evita erros de digitação e facilita a manutenção do código.

  • Remoção de strings.ToLower

  • Reduz operações desnecessárias, assumindo que Cors já está padronizado corretamente.

  • Uso de if Simples em Vez de switch

  • Código mais conciso e legível, mantendo a verificação do tipo (mwc, ok := mw.(func(http.Handler) http.Handler)).

  • Retorno Antecipado (early return)

  • Se nf estiver vazio ou não for "cors", a função retorna imediatamente, evitando verificações desnecessárias.

@jeffotoni
Copy link
Owner

@k-vanio primeiramente obrigado pela colaboração.
não aceitamos o seu pull request por que modificamos muita coisa do quick.go, em no dia que enviou tinhamos algumas versoes para serem subidas e subimos.
Então modificamos, documentamos e o cors foi um dos melhorados e corrigido.

Se for possível é claro, vc atualizar com a main atual e dar uma olhada e fazer suas observaçoes e melhorias fique a vontade ok?

Obrigado.🐧☺️

@jeffotoni jeffotoni closed this Mar 22, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants