Discussion :

[pfeuh]

Salut,

j'ai une classe qui prend en entrée un nom de fichier. Je voudrais qu'à la création d'une instance, si un problème comme un mauvais nom de fichier apparait, l'instance ne soit pas crée. Voici un exemple, ici, si on passe comme paramètre value=7, l'instance ne se crée pas. Je voudrais supprimer la fonction DUMMY intermédiaire, il doit y avoir moyen de faire ca dans DUMMY_CLASS.__init__, mais là je sèche.

A+

Pfeuh

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
def DUMMY(**kwds):
    obj = DUMMY_CLASS(**kwds)
    if obj.success:
        return obj
    else:
        return None
 
class DUMMY_CLASS():
    def __init__(self, **kwds):
        self.success = True
        if 'value' in kwds.keys():
            if kwds['value'] == 7:
                self.success = False
 
for index in range(10):
    print index, DUMMY(value=index)

Code :

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

1
2
3
4
5
6
7
8
9
10
0 <__main__.DUMMY_CLASS instance at 0x00B463F0>
1 <__main__.DUMMY_CLASS instance at 0x00B463F0>
2 <__main__.DUMMY_CLASS instance at 0x00B463F0>
3 <__main__.DUMMY_CLASS instance at 0x00B463F0>
4 <__main__.DUMMY_CLASS instance at 0x00B463F0>
5 <__main__.DUMMY_CLASS instance at 0x00B463F0>
6 <__main__.DUMMY_CLASS instance at 0x00B463F0>
7 None
8 <__main__.DUMMY_CLASS instance at 0x00B463F0>
9 <__main__.DUMMY_CLASS instance at 0x00B463F0>

[Rony22]

Salut,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
def DUMMY(**kwds):
    obj = DUMMY_CLASS(**kwds)
    if obj.success:
        return obj
    else:
        return None
 
class DUMMY_CLASS():
    def __init__(self, **kwds):
        self.success = True
        if 'value' in kwds.keys():
            if kwds['value'] == 7:
                self.success = False
 
for index in range(10):
    print index, DUMMY(value=index)

Hmm, je pense pas, à mon avis ce n'est pas possible que la classe dans la quelle tu te trouve soit détruite.

En plus, en voyant ton code, avec ta méthode ça ne fonction pas non plus, parce que l'instance est quand même crée !!
La seule chose que tu fait dans ta fonction est ne pas renvoyer l'instance, mais l'instance est bien la.
La preuve : ajoute un print 'Instance crée' dans l'init de la classe

Rony

[wiztricks]

Salut,

j'ai une classe qui prend en entrée un nom de fichier. Je voudrais qu'à la création d'une instance, si un problème comme un mauvais nom de fichier apparait, l'instance ne soit pas crée

Votre idée est un peu capillo-tractée: vous pourriez définir une fonction qui retourne None ou une nouvelle instance en fonction de la validité du nom de fichier - techniquement çà s'appelle une factory et elle n'a pas de raison d'être méthode de la classe "cible" -.
Ceci dit, Python permet quand même de faire çà en passant par __new__ qui est la factory "par défaut à canibaliser dans ce cas.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
class MyClass(object):
    def __new__(cls, *args, **kwds):
        value = kwds.get('value', None)
        if value == 7:
            return None
        return object.__new__(cls, *args, **kwds)
 
    def __init__(self, *args, **kwds):
         value = kwds.get('value', None)
         self.success = True #? ca sert plus a rien
 
 
 
for index in range(10):
    elt = MyClass(value=index)
    print 'index= %d, elt=%s, success=%s' % (
        index, elt, 'NA' if elt == None else elt.success)

- W

[pfeuh]

La seule chose que tu fait dans ta fonction est ne pas renvoyer l'instance, mais l'instance est bien la.

Elle n'est là que jusqu'au retour de la fonction qui l'a créé, puisque le garbage collector la détruit à la sortie de la fonction qui l'a créé, DUMMY, en l'occurrence.

vous pourriez définir une fonction qui retourne None ou une nouvelle instance en fonction de la validité du nom de fichier - techniquement çà s'appelle une factory

C'est ce que j'ai fait, DUMMY est la fonction factory. Merci pour le tuyau de __new__, je vais l'essayer.

Votre idée est un peu capillo-tractée

Pas tant que ça. En C, la fonction malloc retourne un NULL si elle n'a pas réussi à allouer une zone mémoire. La fonction fopen renvoie un pointeur sur une structure si elle a réussi à ouvrir un fichier et NULL si elle n'a pas réussi. Est-ce mal de souhaiter le même fonctionnement sur des instances de classes? D'ailleurs, je pense que s'il n'y a plus de mémoire disponible, c'est exactement ce que va faire python s'il ne peut pas instancier: retourner un None

[wiztricks]

C'est ce que j'ai fait, DUMMY est la fonction factory. Merci pour le tuyau de __new__, je vais l'essayer.

Dans votre code, le test "object crée ou pas" se fait 'après' la création de l'objet - dans __init__ -.

Est-ce mal de souhaiter le même fonctionnement sur des instances de classes? D'ailleurs, je pense que s'il n'y a plus de mémoire disponible, c'est exactement ce que va faire python s'il ne peut pas instancier: retourner un None

Avec les langages de programmation, on ne souhaite pas! On essaie de comprendre ce qui n'a pas été écrit dans la documentation...
Et ... dans le meilleur des cas çàà va lever une exception et dans le pire, çà va attendre que GC ait libéré de la mémoire
- W

[pfeuh]

Dans votre code, le test "object crée ou pas" se fait 'après' la création de l'objet - dans __init__ -.

C'est donc la fonction factory qui doit faire les tests. Merci beaucoup pour ces précisions et le petit bout de code.

[wiztricks]

C'est donc la fonction factory qui doit faire les tests. Merci beaucoup pour ces précisions et le petit bout de code.

S'il s'agit de:

j'ai une classe qui prend en entrée un nom de fichier. Je voudrais qu'à la création d'une instance, si un problème comme un mauvais nom de fichier apparait, l'instance ne soit pas crée.

On peut s'assurer que le nom du fichier est 'valide' avant de créer l'instance.
et _avant_ peut être une factory method qui s'amuse à...
En fait, si le nom de fichier n'est pas valide, les méthodes genre open vont généralement lever une exception.... Et il n'y a pas le choix.

Dans ce cas, la factory ne sert qu'à transformer l'exception en quelque chose de plus 'soft', mais passer un nom de fichier 'valide' c'est quand même le boulot de l'appelant... ainsi que de préférer d'adoucir l'exception.

Ceci dit, s'il est nécessaire de passer par une factory method, il est parfois utile de se souvenir de l'existence de __new__ qui peut retourner une instance ou None.

- W

[nardo47]

Salut,

Ceci dit, s'il est nécessaire de passer par une factory method, il est parfois utile de se souvenir de l'existence de __new__ qui peut retourner une instance ou None.

J'utilise assez peu __new__ (à tort ?) et j'ai tendance à passer par une méthode de classe.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
class Spam(object):
 
    def __init__(self, filename):
        # Do something.
        pass
 
    @classmethod
    def safe_init(cls, filename):
        try:
            f = open(filename)
        except IOError:
            return
        self = cls(filename)
        return self

C'est bourré d'erreurs mais ça ne sert qu'à démontrer l'utilisation d'une méthode de classe pour faire une factory.
Qu'en penses-tu Wiztricks ?

[kango]

il m'arrive aussi d'utiliser des classmethod pour des factory (mais pas pour remplacer l'__init__ de la classe en question). Il m'est arrivé d'en voir dans quelques codes.

je crois que c'est comme tout, ce n'est pas gênant si ça a un sens.

[wiztricks]

Salut,

C'est bourré d'erreurs mais ça ne sert qu'à démontrer l'utilisation d'une méthode de classe pour faire une factory.
Qu'en penses-tu Wiztricks ?

Mmmm si çà fonctionne
Après on peut faire le jeu de massacre qu'est la revue de code, mais ce qui est important est que l'appelant aura une API similaire lorsqu'il écrit "instance = Spam(p1, p2, p3, ...)" que derrière çà passe par:

• __new__
• __init__
• __new__ ou __init__
• __call__
• les mêmes d'une metaclass.

Le seul intérêt dans cette histoire est de ne pas avoir à documenter la factory de Spam. D'autant qu'ici elle s'appelle "safe_init", pour l'autre classe ce sera, getInstance... puis on va faire un standard pour mettre tout le monde d'accord... Fatigue!!!

Après la question qui se pose est toujours la même: comment répartir ce qu'on fait dans __new__ et dans __init__. Ici par exemple, on ouvre le fichier... et on pourrait passer le f à __init__.
Note: les paramètres de __new__ étant identiques à ceux de __init__, la factory method autre que __new__ peut s'imposer dans ce cas. Ceci dit...

• __new__(cls, p1, p2, *args, **kwds), n'exclut pas:
• __init__(cls, p1, p2, p3, p4='abc', **kwds)

Mais si ca devient "sordide" un nom ad hoc sera préférable.
- W