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.