Missbrauch von Eigentum reservierten Wort

Question

So habe ich einigen Gott verlassen Legacy-Code, der das reservierte Wort falsch property, um verwendet. In einer Basisklasse, die vererbt wird, wurden sie grundsätzlich implementiert.

class TestClass(object): 

    def __init__(self, property): 
     self._property = property 

    @property 
    def property(self): 
     return self._property 


test = TestClass('test property') 
print(test.property) 

Welche läuft ohne Fehler. Wenn Sie eine andere Methode, die unten hinzufügen, dass Sie erhalten,

class TestClass2(object): 

    def __init__(self, property): 
     self._property = property 

    @property 
    def property(self): 
     return self._property 

    @property 
    def other_property(self): 
     return 'test other property' 


test = TestClass2('test property') 
print(test.property) 
print(test.other_property) 

Welche wirft:

---> 10  @property 
    11  def other_property(self): 
    12   print('test other property') 

TypeError: 'property' object is not callable 

Weil Sie wissen, dass Sie property im lokalen Namespace überschrieben haben.

class TestClass3(object): 

    def __init__(self, property): 
     self._property = property 

    @property 
    def other_property(self): 
     return 'test other property' 

    @property 
    def property(self): 
     return self._property 


test = TestClass3('test property') 
print(test.property) 
print(test.other_property) 

Sie können dieses Problem umgehen, wenn Sie immer Ihre property Überschreiben am Ende der Klasse definieren. Wenn die property-Methode nur für die Basisklasse definiert ist, die Sie von Dingen erben, funktioniert das auch, weil Namespaces.

class TestClass4(TestClass): 

    def __init__(self, property): 
     super(TestClass4, self).__init__(property) 

    @property 
    def other_property(self): 
     return 'test other property' 


test = TestClass4('test property') 
print(test.property) 
print(test.other_property) 

Meine Entrüstung sagt, dass wir diese Variablennamen in der riesigen Menge von Legacy-Code aktualisieren muss, weil GAAAAH, aber anders als neue Methoden zu erinnern, die über der Definition von property Definition in der selten modifizierte Base hinzufügen Klasse, das bricht eigentlich nichts richtig?

Answer 1

nicht Schatten builtins ... fast ohne Refactoring auf alles, was Sie ganz BuiltIn Abschattung vermeiden

Verwendung __getattr__ statt @property Ihr _property Mitglied zurückzukehren ...

class TestClass(object): 
    def __init__(self): 
     self._property = 12 

    def __getattr__(self,item): 
     if item == "property": 
      #do your original getter code for `property` here ... 
      # now you have not overwritten the property keyword at all 
      return getattr(self,"_property") # just return the variable 
class TestClass2(TestClass): 
    def __init__(self): 
     self._property = 67 

print TestClass2().property 

class MySubClass(TestClass): 
    @property 
    def a_property(self): 
     return 5 

print MySubClass().property 
print MySubClass().a_property 

wirklich nebenbei bemerkt gibt es keinen guten Grund, in Python zu verwenden. alles was dazu führt ist, andere Programmierer später zu verwirren und die Tatsache zu verdecken, dass Sie tatsächlich eine Funktion aufrufen. Ich habe es regelmäßig getan ... Ich vermeide es jetzt, es sei denn, ich habe einen sehr sehr zwingenden Grund, nicht zu

Answer 2

Sie könnten immer property auf einen anderen Namen neu zuordnen. Solange Sie einen Namen wählen, der nicht mit Ihren anderen Klassenattributen übereinstimmt, und er wird nicht in der externen Schnittstelle für die Klasse verfügbar gemacht, so ist es nicht wirklich wichtig, wie Sie ihn benennen. bricht

tproperty = property 

class Test(...) 

    @tproperty 
    def property(self): 
     .... 

Answer 3

Ja, wenn Sie immer über die Definition des property Methode nichts neue Methoden hinzufügen. Setzen Sie also im Code einen schönen, großen Kommentar zu diesem Effekt. Hoffentlich jemand außer Kraft zu setzen property in einer abgeleiteten Klasse will zunächst an der Basisklasse suchen und Ihren Kommentar sehen ...

BTW, die property arg auf die __init__ Methode auch Schatten property, aber ich denke, das ist kein Problem.

Idealerweise sollte jemand reinigen dieses Chaos, aber ich verstehe, dass es nicht kosteneffektiv sein kann, das zu tun.

Auch bin ich etwas verwirrt, warum der ursprüngliche Codierer property ein @property in erster Linie gemacht.Dieses Konstrukt sollte nur verwendet werden, wenn der Wert des Attributs dynamisch berechnet werden muss und nicht einfach ein statisches Attribut zurückgegeben wird. Vielleicht waren sie neu in Python und ihnen wurde gesagt, dass das die Art ist, in Python getter zu machen ...