RastreioBot icon indicating copy to clipboard operation
RastreioBot copied to clipboard

Refatoração evitando varias chamadas de .lower()

Open teixeirazeus opened this issue 2 years ago • 3 comments

Pequena otimização na análise de situação do pacote. É melhor guardar o resultado do metodo .lower(), do que ficar processando a string a cada if. O método .lower() deve percorrer toda a string, trata-se de uma complexidade de O(n).

teixeirazeus avatar Sep 21 '22 19:09 teixeirazeus

Valeu @teixeirazeus!! PRs que diminuem a complexidade são sempre muito bem vindos!

Uma sugestão, adiciona o .lower() direto na linha 68, no momento que a variável é criada:

situacao = evento['descricao']

Dessa forma a gente nem precise do sufixo _lower no nome. O que acha?

rougeth avatar Sep 22 '22 09:09 rougeth

@rougeth Mas fazendo assim vai alterar o texto das mensagens também, não?

GabrielRF avatar Sep 22 '22 11:09 GabrielRF

Ah verdade, ela usada pra montar a mensagem. Desconsidera o que eu falei então.

rougeth avatar Sep 22 '22 12:09 rougeth