Python: Unexpected behavior using contextmanager on class method [duplicate]

Question

I'm trying to use the with..as contruct in Python to make writing "reversible computing" code easier. However, using @contextmanager on a class method seems to change the default initialization of future class instances. Python 2.6 and 3.1 have the same behavior. Here is a simple example exhibiting this behavior:

#!/usr/bin/env python

import contextlib

class SymList:
    def __init__(self, L=[]):
        self.L = L

    @contextlib.contextmanager
    def SymAdd(self, a):
        self.L.append(a)
        yield
        self.L.append(a)

SL = SymList()
with SL.SymAdd(3):
    SL.L.append(5)
print(SL.L) # Expect and see [3, 5, 3]
SL2 = SymList()
print(SL2.L) # Expect [] and see [3, 5, 3]

Why isn't SL2 a fresh instance of SymList?
How is the SL2.L data member referring to the SL.L data member?

Community · Accepted Answer · 2017-05-23 12:31:52Z

15

This behavior is due to how mutable default arguments work in Python.

Try changing SymList.__init__() to the following:

    def __init__(self, L=None):
        if L is None:
             self.L = []
        else:
             self.L = L

As you modify self.L in one instance you are also modifying the L that is passed into SymList.__init__(), so the result with your code is that all instances would share the same L attribute when the instance is first initialized.

edited May 23, 2017 at 12:31

CommunityBot

11 silver badge

answered Aug 7, 2012 at 16:09

Andrew Clark

210k36 gold badges286 silver badges310 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

user1582421 Over a year ago

That makes sense. I was blinded thinking it was something fancier with decorators. Thanks!

Collectives™ on Stack Overflow

Python: Unexpected behavior using contextmanager on class method [duplicate]

1 Answer 1

1 Comment

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Linked

Related