GabrielRF / RastreioBot

Telegram Bot @RastreioBot
https://rastreiobot.xyz
GNU General Public License v3.0
365 stars 56 forks source link

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

Closed teixeirazeus closed 2 years ago

teixeirazeus commented 2 years ago

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).

rougeth commented 2 years ago

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?

GabrielRF commented 2 years ago

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

rougeth commented 2 years ago

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