Sans vouloir être méchant, je ne vois pas l'intérêt d'un tel projet. A priori, ceux qui se servent de l'API Google News sont assez grands pour faire un data_new_df=pd.DataFrame(googlenews.results()) seuls.
Le code dans googlenews-bot.py est pas terrible. En général on fait au moins un if __name__ == '__main__' qui appelle un main. Dur de s'y retrouver dans ce code.
Je ne vois pas l'intérêt de ton code pylog vu l'existence du module logging. Le code n'est pas terrible non plus, où l'on a en gros un copié/collé de 5 fois la même classe avec juste le print qui change. L'héritage devrait être utilisé ... et encore. Une classe suffit, en passant les arguments qu'il faut au constructeur.
Tu ouvres un fichier self.file=open(name_file, 'a') sans jamais le fermer. Très mauvaise pratique.
Sans vouloir être méchant, je ne vois pas l'intérêt d'un tel projet. A priori, ceux qui se servent de l'API Google News sont assez grands pour faire un data_new_df=pd.DataFrame(googlenews.results()) seuls.
Le code dans googlenews-bot.py est pas terrible. En général on fait au moins un if __name__ == '__main__' qui appelle un main. Dur de s'y retrouver dans ce code.
Je ne vois pas l'intérêt de ton code pylog vu l'existence du module logging. Le code n'est pas terrible non plus, où l'on a en gros un copié/collé de 5 fois la même classe avec juste le print qui change. L'héritage devrait être utilisé ... et encore. Une classe suffit, en passant les arguments qu'il faut au constructeur.
Tu ouvres un fichier self.file=open(name_file, 'a') sans jamais le fermer. Très mauvaise pratique.
Bon courage.
Bonjour,
Ce programme peut servir à ceux qui font des statistiques mais qui ne savent pas forcément utiliser une API.
De plus, ce programme peut-être utilisé par ceux qui ne savent pas coder en Python.
Pour le
if __name__ == "__main__":
je ne suis pas d'accord. Il n'y a pas d'intérêt à créer une énorme class "main" alors que le code s'exécute très bien sans.
Au passage, je conseil d'utiliser un `crontab` pour récupérer des news, par exemple toutes les heures.
Evidemment le but n'est pas de tout foutre dans une fonction main. Il faut découper encore, notamment le parsage des arguments argparser qui peuvent être placés dans une autre fonction.
Evidemment le but n'est pas de tout foutre dans une fonction main. Il faut découper encore, notamment le parsage des arguments argparser qui peuvent être placés dans une autre fonction.
Oui, je me suis aussi dit ça. Le code est trop chargé. Je vous invite donc à suivre l'évolution des versions sur GitHub.
Outre l'aspect utile de l'application : je ne l'utiliserai pas personnellement. Je pense qu'il y a un manque de pratique concernant l'utilisation de l'objet et des principes SOLID très liés. Si tu n'en fait pas, c'est le moment de te lancer via ce projet somme toute assez simple.
En python, est-ce normal de nommer des variables non constantes en majuscules ? Comme la variable APP_ARG_PARSER ? Selon la PEP8, seules les constantes sont nommées comme cela.
En réalité, ce que tu utilises dans tes fonctions pourraient être des classes. Simplement pour la maintenance et la reutilisabilité. Exemple : Notifier qui permet de créer ces Notifications, DataframeBuilder , Application (ou Console) qui prends un tableau d'arguments, les initialises dans un tableau et les met à disposition, DataProvider qui permet d'obtenir de nouvelles données en retournant un DataFrame, NewsPrinter pour afficher les news recherchées, etc.
Comme je le dis souvent : tant que ton code fonctionne, c'est que ça fait le taf. Mais essaye tout de même de "corriger" et/ou de faire ça en POO. Ca change pas mal la vie :)
Et j'ajoute que si tu es chaud bouillant, tu peux regarder du coté de https://refactoring.guru pour les design pattern.
× Après avoir cliqué sur "Répondre" vous serez invité à vous connecter pour que votre message soit publié.
× Attention, ce sujet est très ancien. Le déterrer n'est pas forcément approprié. Nous te conseillons de créer un nouveau sujet pour poser ta question.
Zonovum - mes projets: Google news bot
Zonovum - mes projets: Google news bot
Zonovum - mes projets: Google news bot
Zonovum - mes projets: Google news bot
Zonovum - mes projets: Google news bot
Site personnel : Julien Gidel - AutoMate - PHPresentation