Organiser ses classes

Bonjour

Voici mon problème :

Avec le temps, mes classes sont de plus en plus grosses, on trouve toujours de nouvelles méthodes à ajouter.
J'essaye toujours d'avoir des fichiers plutôt petits. L'une des principales raisons est qu'avec SVN, cela permet de mieux voir si une modification est locale (on a juste retoucher une méthode) ou globale (on a complètement revu toutes les méthodes). En C, j'en arrivais parfois à un fichier par fonction.


Actuellement, j'ai un fichier par classe. Découper jusqu'à ce niveau est facile. Malgré cela, dans certain cas, je commence à trouver ces fichiers trop gros, donc je cherche comment séparer le contenu de cette classe sur plusieurs fichiers.

Mon premier essai, qui a échoué, consistait en gros à faire ceci :

Code:

---

1 2 3 4 5 6 7

fichier class_a_core.py class A: def __init__(self): print("A.__init__")   def m1(self): print("A.methode 1")

---

Code:

---

1 2 3 4

fichier class_a_tools.py class A: def m2(self): print("A.methode 2")

---

et un exemple d'utilisation:

Code:

---

1 2 3 4 5 6

from multi_fic.class_a_tools import * from multi_fic.class_a_core import *   a = A()   a.m2()

---

Bien sur cela ne marche pas.


Est-ce que quelqu'un a une piste pour que j'arrive à découper mon application en petit fichier.

Cordialement
Emmanuel

Salut, je ne saisis pas trop bien, quand tu importe avec "import*", tu importe deux classes du même nom, à savoir "A".

A la ligne a = A() comment Python s'y retrouve-t-il ?

J'utilise assez bien la séparation en fichier, ça n'a rien de compliqué.

Ex:

J' ai un fichier Otool.py, qui contient une série d'utilitaires classiques, conversion, algorythmes récurrents etc
Celui-ci commence comme ceci:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

#!/usr/bin/env python # -*- coding: utf-8 -*-   import os import etc   class Tools(): """Some tools wich are used by the application.       """ def __init__(self): ...do something     def valid_image(self, fp): 1ere méthode   def etc...

---

Ensuite dans le corps du programme principal;

Code:

---

1 2 3 4 5

from Otool import Tools   tool = Tools() ...

---

et lorsque j'ai besoin d'une méthode:

Code:

---

1 2	x = tool.valid_image(image)

---

simple, non ?

vincent

Citation:

---

Envoyé par VinsS

J' ai un fichier Otool.py, qui contient une série d'utilitaires classiques, conversion, algorythmes récurrents etc

---

OK, tu as un fichier Otool.py avec ta classe Tools().
Maintenant, tu fais grossir ta classe Tools, tu rajoutes des méthodes.
Si, pour des raisons que j'ai cité précédemment, tu finis par trouver le fichier Otool.py trop long et que tu souhaites diviser ce fichier en plusieurs fichiers, par exemple Otool_1.py et Otool_2.py, tu procèdes comment ? Voilà ma question.

Cordialement
Emmanuel

Bonjour,
Pour moi c'est une erreur que de vouloir fractionner une même classe et d'ailleur je sais pas si c'est possible en python.
Une classe est sensée regrouper une ensemble de fonction ayant attrait à un objet. Si ta classe est trop grande, regarde plutôt du côté de la conception. Peut être mets-tu trop de chose dans une même classe (trop procédural), et tu augmentes ainsi ton couplage.
Si ta classe est longue, mais que toutes les méthodes sont liées à l'objet, pour moi il n'y a pas de problème. D'ailleurs il est parfois bon de regrouper plusieurs classes dans un même fichier afin de regrouper les classes similaires, et permettre des imports plus cohérents
Bref, je répond pas à ta question d'un point de vue technique... :?

Bonjour,

Citation:

---

Envoyé par dahtah

Si ta classe est trop grande, regarde plutôt du côté de la conception. Peut être mets-tu trop de chose dans une même classe

---

Citation:

---

Envoyé par dahtah

Bref, je répond pas à ta question d'un point de vue technique... :?

---

Si, tu réponds très bien.
Si je ne me trompes pas, ton point de vu est que :
- une classe est dans un seul fichier. règle python inviolable.
- si la classe est trop grande, c'est une erreur de conception. Point de vu que j'accepte.

merci de ta réponse.

Est-ce que d'autre personne partage ce point de vu ?

Cordialement
Emmanuel

Citation:

---

Envoyé par emmanuel_dumas

- une classe est dans un seul fichier. règle python inviolable.

---

Techniquement, je sais pas si tu peux scinder une classe en plusieurs fichiers en python. Je sais que c'est possible en C#. Notament dans les éditeurs d'IHM, où le code généré automatiquement est séparé du code programmeur. Pourtant les deux fichiers forment une même classe (mais bon, c'est un besoin spécifique).

Citation:

---

Envoyé par emmanuel_dumas

- si la classe est trop grande, c'est une erreur de conception. Point de vu que j'accepte.

---

Je trouve que sur un gros projet, un des points difficiles de la POO est la granularité des objets. De très gros objets qui font tout sont difficilement maintenables. Maintenant, de très petits objets, obligent à faire beaucoup de code "ciment" entre les objets, et le code est difficile à lire, car les appels aux autres objets obligent le programmeur à aller voir d'autres fichiers, puis d'autres, puis d'autres... Tu obtiens une structure "horizontale" dans un schéma UML.

Pour être pratique, je dirais que 200-300 lignes par objet sont acceptables. Après, c'est surtout une histoire de fonctionnalités communes. Cet avis bien sur n'engage que moi.

Bonjour,

Citation:

---

Envoyé par emmanuel_dumas

Si je ne me trompes pas, ton point de vu est que :
- une classe est dans un seul fichier. règle python inviolable.
- si la classe est trop grande, c'est une erreur de conception. Point de vu que j'accepte.

merci de ta réponse.

Est-ce que d'autre personne partage ce point de vu ?

---

Oui, scinder une classe en plusieurs fichiers ma paraît être une erreur, a priori. Ça peut vite devenir le bordel. De plus, est-ce possible en Python? Ça m’étonnerait.

Par contre, une classe trop grande, je ne sais pas trop ce que ça veut dire. Combien de lignes font tes classes trop grandes?

Quelques centaines de lignes, c’est correct. Mes classes ne dépassent guère 500 lignes et font souvent beaucoup moins.

Citation:

---

Envoyé par MorganStern

Par contre, une classe trop grande, je ne sais pas trop ce que ça veut dire. Combien de lignes font tes classes trop grandes

---

La classe qui m'a poussé à poster ce message fait actuellement 700 lignes. je vois rapidement un triplement de la taille de cette classe.

Cordialement
Emmanuel

Bonjour,

Je crois aussi qu'on ne peut pas scinder une classe sur plusieurs pages, parce que l'importation d'un module nécessite son exécution.

Par contre, on peut faire en sorte de placer dans un autre module à importer, le code de certaines méthodes longues sous forme de fonction.

Cependant, cela pose peut-être plus le problème de l'outil de développement que la longueur effective de la page. Avec eclipse+pydev, par exemple, une grande classe peut être affichée avec uniquement les titres des méthodes. Il suffit alors de cliquer sur le '+' dans la marge pour que le code de la méthode apparaisse. Et un nouveau clic sur le '-' dans la marge pour que soit retiré l'affichage de tout le code de la méthode. Cela permet d'avoir une vue d'ensemble de la classe, même si elle est très grande.

Tyrtamos

Si tu tiens vraiment à découper ta classe en plusieurs fichiers, pourquoi ne pas utiliser l'héritage ?
Une classe mère qui implémente une partie des méthodes, une classe fille qui hérite de la classe mère (et donc de ses méthodes) et qui implémentet celles qui restent.
Attention ensuite à bien implémenter des objets du type de la classe fille pour avoir accès à toutes les méthodes (mais un problème du genre se serait posé aussi en découpant la classe comme tu as tenté de le faire au début : en n'important que le module "core" il t'aurait manqué des méthodes).
Je dis une bêtise ?

edit, petite illustration dans l'interpréteur :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

Python 2.6.4 (r264:75706, Oct 27 2009, 06:16:59) [GCC 4.4.1] on linux2 Type "help", "copyright", "credits" or "license" for more information. >>> class A_mere: ... def affiche(self): ... print "ceci est une méthode de héritée de A_mere" ... >>> class A(A_mere): ... def encore(self): ... print "ceci est une méthode implémentée par la fille" ... >>> x = A() >>> x.affiche() ceci est une méthode de héritée de A_mere >>> x.encore() ceci est une méthode implémentée par la fille >>>

---

Rien ne t'empêche de mettre ces deux classes dans deux fichiers différents.

Abordons le problème différemment.
Une classe définit une entité et les méthodes associées sont à priori limitées au opérations associés.

Si vous avez des classes trop grosses, çà signifie - peut être - qu'elles empilent ou mélangent trop d'entités. Dans ce cas vous avez peut être un problème de conception et les possibilités de découper votre classe en plusieurs fichiers n'est peut être de toutes façons pas une solution.

Note: Si vous voulez faire çà, rien ne vous empêche de lire les fichiers dans un StringIO et de faire un exec pour l'importer.

Comment faire?
Personnellement, j'expérimente une méthode qui permet de masquer les détails en sous-classes ou en objets accessibles comme sous classes.

Exemple: Une entité Tutu peut avoir des méthodes globales, et des entités liées comme par exemple Tutu.Historique Tutu.Planning ayant des methodes spécifiques mais liées à la nature de Tutu.
Dans ce cas Historique peut être une sous-classe ou une classe de même niveau dans un autre fichier. Le lien entre Historique et Tutu est de la forme 'parent". A l'initialisation (__init__) de Tutu, j'ai une instruction du type:
self.historique = Historique (parent=self)
Dans __init__ d'Historique, je mémorise parent:
self._parent = kwds.get('parent')
et on peut à partir de là accéder à certains attributs/méthodes du père.

Cela me permet de tester Historique indépendamment de Tutu, rendre ou pas Historique "public",... Et donc éventuellement de placer Tutu et Historique dans des fichiers différents.
- W

Il doit être possible de scinder une classe dans plusieurs fichiers en utilisant ensuite la fonction setattr pour ajouter les fonctions à ta classe.

exemple:

Code:

---

1 2 3 4

## fichier file1.py class toto(object): def __init__(self, a): self.a = a

---

Code:

---

1 2 3 4 5 6

## fichier file2.py import file1   def affiche(self, nombre): print self.a, nombre   setattr(file1.toto, "affiche", affiche)

---

Code:

---

1 2 3 4 5 6

## fichier file3.py import file2 import file1   test = file1.toto(5) test.affiche(10) ## 5, 10

---

Comme cité plus haut, le choix d'un éditeur permettrait de produire fichiers "normaux" snas nécessiter plusieurs pirouettes fastidieuse ou entachant la lisibilité... J'ai en tête l'éditeur leo. Très bon, mais nécéssitant une petit investissement en temps. il fonctionne sur un principe d'outliner, cet outliner permet de "diviser" une clasee ou ce que l'on veux en plusieurs nodes, tout en produisant des fichiers de sortie "normaux", cad sans que cette "division" soit visible, ça ne l'est qu'au sein de l'éditeur... après c'est une question de goûts.