desafio dev jr #1
Conversation
| maxi = 0 | ||
| mini = 1000 | ||
| qmini = 0 | ||
| qmaxi = 0 |
There was a problem hiding this comment.
[suggestion] O que você acha de utilizar nomes de variáveis mais descritivos?
| with open('resumo_dos_jogos.xlsx', 'a+') as arquivo: | ||
| arquivo = df |
There was a problem hiding this comment.
[question] Aqui o arquivo é aberto dentro de um contexto como a variável arquivo e depois atribuído a variável df. Não seria melhor trabalhar só com uma variável?
| if pontuacao < mini: | ||
| mini = pontuacao | ||
| qmini += 1 | ||
| lista = [cont, pontuacao, mini, maxi, qmini, qmaxi] |
There was a problem hiding this comment.
[suggestion] Ao invés de trabalhar com uma lista (que pode ter muitas variações nas posições dos dados) o que você acha de utilizar um dicionário? Assim a estrutura de dados ficaria bem mais definida, evitando assim dúvidas!
| @@ -0,0 +1,85 @@ | |||
| import pandas as pd | |||
There was a problem hiding this comment.
[suggestion] O que você acha de colocar uma docstring no início do arquivo explicando como este módulo funciona?
| resumo.append(lista) | ||
|
|
||
| pastaUsuario = path.join(path.expanduser('~')) | ||
| # print('#Gera a planilha a partir dos dados obtidos usando um dataframe e depois convertendo em excel') |
There was a problem hiding this comment.
[suggestion] Já que está trabalhando com git, evite adicionar comentários de trechos de código em seus commits!
| lista = [cont, pontuacao, mini, maxi, qmini, qmaxi] | ||
| resumo.append(lista) | ||
|
|
||
| pastaUsuario = path.join(path.expanduser('~')) |
There was a problem hiding this comment.
[suggestion] Em python, atribuímos os nomes de variáveis no estilo snake_cased
| @@ -0,0 +1,85 @@ | |||
| import pandas as pd | |||
There was a problem hiding this comment.
[suggestion] O uso do pandas aqui é realmente necessário? Esta é uma biblioteca bem poderosa, mas para apenas escrever arquivos, utilizar os módulos nativos do python é uma melhor opção!
| 4: 'Quebra de recorde min.', | ||
| 5: 'Quebra de recorde máx' | ||
| }, inplace=True) | ||
| df.to_excel(pastaUsuario + '\\resumo_dos_jogos.xlsx', index=False) |
There was a problem hiding this comment.
[suggestion] O que você acha de ao invés de utilizar um arquivo .xlsx ( que apenas o office consegue trabalhar) utilizar um arquivo .csv?
| anexo = pastaUsuario + '\\resumo_dos_jogos.xlsx' | ||
| # "C:/Users/amand/resumo_dos_jogos.xlsx" | ||
| # configurar as informações do seu e-mail | ||
| email.To = "[email protected]" |
There was a problem hiding this comment.
[question] E se outra pessoa utilizar este módulo? O email deve continuar sendo enviado para você ou não?
| outlook = win32.Dispatch('outlook.application') | ||
| # criar um email | ||
| email = outlook.CreateItem(0) | ||
| jogador = input('Digite o nome do jogador: ') | ||
| assinatura = input('Digite o nome do responsável pelo envio desse email: ') | ||
| anexo = pastaUsuario + '\\resumo_dos_jogos.xlsx' | ||
| # "C:/Users/amand/resumo_dos_jogos.xlsx" | ||
| # configurar as informações do seu e-mail | ||
| email.To = "[email protected]" |
There was a problem hiding this comment.
[suggestion] Uma outra alternaiva seria utilizar a biblioteca nativa do python smtplib. Pois caso não possua o Outlook em sua máquina, o código não irá funcionar.
Aqui tem um exemplo de como enviar emails com a lib nativa do python.
Resolução do exercício.