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

Adiciona Asyncio e AIOHttp para a execução de grande volume de requests em menor tempo. #3

Merged
merged 1 commit into from
Jul 26, 2024

Conversation

dan5e3s6ares
Copy link
Contributor

No description provided.

Copy link
Member

@ssjunior ssjunior left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deixo minhas sugestões aqui:

1 - Usar main:
https://docs.python.org/pt-br/3/library/__main__.html

2 - Usar um lint de código para seguir as recomendações PEP8 nos arquivos.

3 - Nomear arquivos, funções, classes de acordo com PEP8.

O PEP 8 é o guia de estilo de código para a linguagem Python. Em relação a nomeação de arquivos, ele recomenda seguir essas convenções:

Nomes de arquivos: Usar letras minúsculas separadas por underscores. Por exemplo: meu_arquivo.py, utils.py, teste_modulo.py.
Nomenclatura de módulos/pacotes: Também devem ser escritos em letras minúsculas, separados por underscores quando necessário para melhorar a legibilidade. Por exemplo: meu_pacote, meu_pacote.modulo.
Nomes de classes: Utilizar a convenção "CapWords", ou seja, cada palavra começa com uma letra maiúscula. Por exemplo: ClasseExemplo, MinhaClasse.
Nomes de funções e variáveis: Escrever em letras minúsculas, separando palavras com underscores quando necessário para melhorar a legibilidade. Por exemplo: funcao_exemplo, variavel_temporaria, contador.
Constantes: Escrever completamente em letras maiúsculas, separando palavras com underscores. Por exemplo: CONSTANTE_EXEMPLO, MAXIMO_TAMANHO.

@rafaelcalixto rafaelcalixto self-requested a review May 24, 2024 20:38
@dan5e3s6ares
Copy link
Contributor Author

Deixo minhas sugestões aqui:

1 - Usar main: https://docs.python.org/pt-br/3/library/__main__.html

2 - Usar um lint de código para seguir as recomendações PEP8 nos arquivos.

3 - Nomear arquivos, funções, classes de acordo com PEP8.

O PEP 8 é o guia de estilo de código para a linguagem Python. Em relação a nomeação de arquivos, ele recomenda seguir essas convenções:

Nomes de arquivos: Usar letras minúsculas separadas por underscores. Por exemplo: meu_arquivo.py, utils.py, teste_modulo.py. Nomenclatura de módulos/pacotes: Também devem ser escritos em letras minúsculas, separados por underscores quando necessário para melhorar a legibilidade. Por exemplo: meu_pacote, meu_pacote.modulo. Nomes de classes: Utilizar a convenção "CapWords", ou seja, cada palavra começa com uma letra maiúscula. Por exemplo: ClasseExemplo, MinhaClasse. Nomes de funções e variáveis: Escrever em letras minúsculas, separando palavras com underscores quando necessário para melhorar a legibilidade. Por exemplo: funcao_exemplo, variavel_temporaria, contador. Constantes: Escrever completamente em letras maiúsculas, separando palavras com underscores. Por exemplo: CONSTANTE_EXEMPLO, MAXIMO_TAMANHO.

Perfeito, concordo com as suas observações, mas eu segui o que já se encontrava no código, adicionando algumas idéias novas, afinal o que você observou não está no código original.
Mas realmente, o que você comentou é a forma correta, façamos assim, caso haja o merge desse PR para a master, seria interessante você colocar dentro do padrão por você sugerido.

@dan5e3s6ares dan5e3s6ares requested a review from ssjunior May 27, 2024 11:22
@ssjunior
Copy link
Member

@dan5e3s6ares bom dia. Sim já estava lá. Apenas dei uma sugestão de melhores práticas que deveriam ser padrão. 👍

Lembro que não é papel de quem faz review, corrigir ou ajustar o código que está em revisão. Quem faz o review tem por intenção apenas indicar o que pode ser melhorado.

O padrão que sugeri não é meu é da comunidade Python: https://peps.python.org/pep-0008/

Não me oponho ao merge, então qualquer pessoa com permissão pode aprovar o mesmo.

BTW dá uma olhada no niquests, passei a usar ele no lugar do requests. É assíncrono, segue a mesma linha do requests e tem sido atualizado constantemente.

@dan5e3s6ares
Copy link
Contributor Author

niquests

Eu não tinha visto o niquests, você falou e acredito que passarei a usar, os argumentos deles são bons.

@rafaelcalixto
Copy link
Contributor

Desculpe a demora para fazer o review!!! Ficou ótimo! Já testei aqui e está rodando 100%.

@rafaelcalixto rafaelcalixto merged commit c0baa90 into PythonFloripa:main Jul 26, 2024
@dan5e3s6ares dan5e3s6ares deleted the Multiple-Requests branch July 29, 2024 11:47
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

Successfully merging this pull request may close these issues.

3 participants