Discussion :

[fred1599]

Salut !

Salut et désolé de répondre aussi tardivement,

@fred1599:
En ce qui concerne le découpage pourquoi pas mais le but de ce script c'est de transférer le contenu du presse-papier d'un ordi à un autre sans tenir compte de l'os et du réseau.


https://github.com/diablo76600/ClipboardToCloud

et donc ? Je ne comprend pas non plus la relation entre ma demande de découpage et la prise en compte de l'OS et du réseau, je veux bien que tu m'éclaires...

Il n'y a pas de réelles excuses à ne pas appliquer les bonnes pratiques, et cela quelque soit le projet. Le découpage est surtout lié au fait de rendre plus facile le debuggage de ton code. Si tu reviens deux ou trois années sur ton projet, tu souhaiteras sans doute gagner du temps à sa lecture. On pourrait très bien imaginer que ce projet en partie ou entièrement soit repris comme base sur un sujet plus complexe...

Ce qui est déjà très bien, c'est que tu es partagé cela sur une plateforme, ça peut éventuellement aidé à faire évoluer ton projet.

Après il y a beaucoup de choses à en dire, mais je ne sais pas si tu es vraiment près à faire du refactoring sur ton code

Si tu considères que c'est fonctionnel et que pas besoin de plus, je ne perdrai pas de temps à faire des remarques sur ton code, dans le cas contraire, je peux faire des propositions via GitHub.

[Diablo76]

Salut !

Il n'y a pas de réelles excuses à ne pas appliquer les bonnes pratiques, et cela quelque soit le projet. Le découpage est surtout lié au fait de rendre plus facile le debuggage de ton code. Si tu reviens deux ou trois années sur ton projet, tu souhaiteras sans doute gagner du temps à sa lecture. On pourrait très bien imaginer que ce projet en partie ou entièrement soit repris comme base sur un sujet plus complexe...

Ce qui est déjà très bien, c'est que tu es partagé cela sur une plateforme, ça peut éventuellement aidé à faire évoluer ton projet.

Après il y a beaucoup de choses à en dire, mais je ne sais pas si tu es vraiment près à faire du refactoring sur ton code

Si tu considères que c'est fonctionnel et que pas besoin de plus, je ne perdrai pas de temps à faire des remarques sur ton code, dans le cas contraire, je peux faire des propositions via GitHub.

Salut,

Si bien sur que j'accepte tes remarques et propositions notamment sur le découpage et le refactoring

[Arioch]

Sans avoir regardé le code, pour ma part voici mes remarques :

1. le dossier .vscode n'a rien à faire dans le repo. Un fichier .gitignore solutionnerait cela.
2. en l'état, le programme récupéré n'est pas utilisable. Il faudrait à minima un fichier requirements.txt contenant les librairies additionnelles afin d'éviter d'avoir ce qui suit au lancement du programme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

ModuleNotFoundError: No module named 'PyQt5'

3. un README.md plus étoffé serait un plus appréciable et apprécié je pense.

[fred1599]

1. le dossier .vscode n'a rien à faire dans le repo. Un fichier .gitignore solutionnerait cela.

Ça dépend des cas, ici sans doute, en équipe, le .vscode peut être partagé pour rendre uniforme le développement sur un IDE imposé (surtout VSCode, c'est du vécu).

Pour tout le reste je suis en effet de ton avis... mais c'est plus sympa de faire sa revue sur GitHub maintenant que le partage est fait

[Arioch]

Ok pour l'explication de la présence du dossier .vscode. J'ai pour habitude de le placer dans le .gitignore.

[Diablo76]

Sans avoir regardé le code, pour ma part voici mes remarques :

1. le dossier .vscode n'a rien à faire dans le repo. Un fichier .gitignore solutionnerait cela.
2. en l'état, le programme récupéré n'est pas utilisable. Il faudrait à minima un fichier requirements.txt contenant les librairies additionnelles afin d'éviter d'avoir ce qui suit au lancement du programme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

ModuleNotFoundError: No module named 'PyQt5'

3. un README.md plus étoffé serait un plus appréciable et apprécié je pense.

J'en prends note, a vrai dire c'est la première fois que j'utilise Github en collaboratif.

[Diablo76]

Salut,

Sur les conseilles et la bienveillance de @fred, j'ai revu ma façon de penser le code:

https://github.com/diablo76600/ClipboardToCloud

Bon, je ne suis pas sur d'avoir répondu à ce qui qui est le refactoring

[fred1599]

Nettement mieux ! Rien à voir avec ce que tu proposais au départ, c'est modulable et maintenable.

Quelques points,

Plutôt que le package Modules, on pourrait créer un package service et un package ui pour séparer l'utilisation et la présentation utilisateur.

et concernant ce morceau de code par exemple (regarder si pas d'autres cas comme celui-ci)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
def file_changed(self) -> None:
  if self.service_directory_file.file_is_changed is False:
      self.manager.paste_to_clipboard()
  self.service_directory_file.file_is_changed = False

en

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
def file_changed(self) -> None:
  if not self.service_directory_file.file_is_changed:
      self.manager.paste_to_clipboard()
  self.service_directory_file.file_is_changed = False