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

challenge: feat/initializing tmp stress + fix #6

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

pafev
Copy link

@pafev pafev commented Feb 16, 2024

Alterações Realizadas:

  • Inicialização do valor de tmp, baseada na condição proposta no desafio
  • na geração de dataframe, utilização de concat, a fim de se evitar múltiplos merges
  • aparentemente, na versão mais recente do pandas, surgiu uma necessidade de tipagem em alteração de frames (provavelmente um bug), então ajustei isso na troca de valores de gender de um número para uma string (1 -> 'female')
  • esta foi detalhe, mas para evitar bugs, fiz uma checagem typesafe em count_stress, para evitar divisão por zero

Sugestões:

  • O código já está em um nível de desenvolvimento bastante avançado, então não seria ideal realizar refatorações densas de boas práticas por agr, mas futuramente cabe uma aplicação de princípios de legibilidade, como SOLID. Principalmente nos princípios de singularidade e inversão de dependências, enquanto que não vejo necessidade de aplicação do princípio de Liskov, tendo em vista que não encontrei hierarquia de classes
  • Acredito ser válida revisão da nomenclatura de algumas variáveis, além de existirem algumas declaradas e não utilizadas (em trechos pontuais do código)
  • No readme, acredito também ser válida declaração das versões de dependências utilizadas

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.

1 participant