zegildo / DataMining

Data Mining Projects
1 stars 27 forks source link

Melhoria do código #92

Open zegildo opened 5 years ago

zegildo commented 5 years ago

@Jcarllos olhei o teu trabalho. Em primeiro lugar gostaria de te dar os parabéns pelo esforço e pela tentativa em ir além. Minhas considerações são:

Sobre o código em python: Para um trabalho excelente é muito importante seguir as recomendações de estilo sugeridas pelo google para que seu código fique limpo e elegante.

Sobre a solução: Observe a tua função _getcard e as inúmeras sequências que você criou. Perceba que todas as classes contém windowbg . Será que daria para resolver tudo com um único xpath?

Meus parabéns por utilizar o método .extend um detalhe que geralmente passa despercebido diante dos programadores.

Sobre o print: Ao inves de imprimir: print(subjects +', ' + authors +', ' + replies.strip() +', ' + views.strip()) é mais elegante e útil utilizar o .format.

Um abraço, José.