brunoocasali/qke

Remove logic inside report

Opened this issue · 2 comments

Remove part of logic from reports.

Em qual arquivo exatamente você quer remover a lógica?

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