Discussion :

[shadowsam]

Après avoir écrit quelques uns de mes opérateurs pour certaines classes, je me pose la question de savoir si je n'ai pas écris une usine à gas.
A savoir que ces opérateurs vont être utilisé par PyQT car se sont les opérateur d'un DataModel, et donc appelé vaaaaaaaaaachement souvent.

Voici la facon dont je les écrit aujourd'hui:

Code :

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

1
2
3
4
5
def __eq__(self, other):
    if isinstance(other, int):
        return self.revision() == other
    else:
        return self.revision() == other.revision()

En fait j'ai un peu peur que isinstance() soit légèrement... prohibitif en terme de temps d'exécution

Limite je me demandais si passer sur quelque chose comme ca:

Code :

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

1
2
3
4
try:
    return self.revision() == other.revision()
except AttributError, e:
    return self.revision() == other

ne serait pas plus efficace. Qu'en pensez vous ?

[kango]

bonsoir,

après avoir profilé un script j'ai été en effet assez surpris par le temps passé dans isinstance mais je ne peux pas dire que j'ai trouvé ça prohibitif.

voici un petit example:

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
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
import time
 
def title1(var):
 
    if type(var) is str:
        var=var.title()
    else:
        var="No title"
 
def title2(var):
 
    if isinstance(var,str):
        var=var.title()
    else:
        var="No title"
 
def title3(var):
 
    try: var=var.title()
    except AttributeError: var="No title"
 
def main(f):
 
    print time.clock()
    for i in xrange(10000000):
        f("toto")
        f(3.14)
    print time.clock()
 
if __name__ == "__main__":
 
    main(title1)

l'utilisation de title1 prend 9.57sec.
l'utilisation de title2 prend 10.57sec.
l'utilisation de title3 prend 37.41sec.

on aurait donc tendance à dire que le try est moins performant.

maintenant si dans ma fonction main, je retire f(3.14):

l'utilisation de title1 prend 6.43sec.
l'utilisation de title2 prend 5.66sec.
l'utilisation de title3 prend 4.36sec.

là forcément le try est plus performant dans la mesure où on ne rentre jamais dans l'instruction except.

il est marrant de constater que la tendance is/isinstance s'inverse. on dirait que isinstance est plus rapide si elle retourne "Vrai".

donc je dirais que la réponse dépend du nombre de fois où dans ton cas other sera un int ou du même type que self.

[shadowsam]

Merci pour les chiffres, kango, c'est vrai que je cherchais une réponse basée sur les algos des différentes méthodes et, suivant mon idée, je n'ai même pas pensé à faire des tests ==> Shame on me !

En tout cas ton analyse m'aura permis de comprendre comment marche le isinstance. Je vais réfléchir à tout ça et voir si je dois changer mon code.