Remove logic inside report
Opened this issue · 2 comments
brunoocasali commented
Remove part of logic from reports.
jcserracampos commented
Em qual arquivo exatamente você quer remover a lógica?
brunoocasali commented
Existem alguns arquivos da camada de report
lib report que fazem a impressão do conteúdo na tela, e esses arquivos não estão tão legais assim, pois eles compartilham muita coisa, https://github.com/brunoocasali/quake_log_parser/blob/master/lib/reports/simple_format_reporter.rb#L15-L37 não me parecem ser classes específicamente criadas para montar uma impressão entende? Ao meu ver cada classe de report deveria ter um método que montaria um layout e só. Enfim, a ideeia é mais ou menos essa sinta-se livre para sugerir! :D