Open RCristyan opened 3 years ago
:exclamation: No coverage uploaded for pull request base (
master@f7c05d9
). Click here to learn what that means. The diff coverage isn/a
.
@@ 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.
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:
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:
No loop principal da classe Scheduler, são chamados métodos especializados em cada tipo de agendamento.
A aplicação deste princípio de Clean Code facilita a manutenibilidade do código.
Exemplo de renomeação de variáveis:
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.