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

[REFATORAÇÃO] SOLID e Clean Code em scheduler.py #31

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

Conversation

RCristyan
Copy link
Collaborator

O arquivo scheduler foi refatorado para aplicação de alguns princípios de Clean Code e SOLID. Resolve esta issue.

  • Exemplo de Single Responsability Principle:
    image
    Em vez de calcular as formas diferentes de se agendar uma tarefa no loop principal, foram criados métodos específicos para isso. Existem métodos públicos para agendar tarefas diariamente, semanalmente e mensalmente, e existem métodos privados (cujos nomes começam com dois underscore) que são chamados internalmente pela classe.

  • Exemplo de composição de métodos:
    image
    No loop principal da classe Scheduler, são chamados métodos especializados em cada tipo de agendamento.
    image
    A aplicação deste princípio de Clean Code facilita a manutenibilidade do código.

  • Exemplo de renomeação de variáveis:
    image
    O Clean Code tem um princípio que diz que nomes de variáveis devem expressar suas finalidades, por isso algumas variáveis foram renomeadas.

Houveram também outras modificações menores no código, porém as funcionalidades originais do código não foram afetadas.

guilhermedelyra and others added 30 commits October 5, 2020 18:07
Co-authored-by: Lyra <guilyra12@gmail.com>
#13 - Make Pydemic UI compatible with latest version of streamlit
Merge changes from upstream into master of fork
Samuel Borges and others added 3 commits November 24, 2020 11:01
@RCristyan RCristyan added documentation Improvements or additions to documentation enhancement New feature or request labels Nov 24, 2020
@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@f7c05d9). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##             master     #31   +/-   ##
========================================
  Coverage          ?   3.97%           
========================================
  Files             ?      76           
  Lines             ?    4548           
  Branches          ?     472           
========================================
  Hits              ?     181           
  Misses            ?    4366           
  Partials          ?       1           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f7c05d9...8bbd780. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants