What Python learned from economics
lazypython.blogspot.com
lazypython.blogspot.com
Always remember the interest on Code Debt is brutal so the wise entrepreneur strives to avoid it.
A lot of what he says makes sense, but perhaps someone can explain why this is wrong:
def bad_append(new_item, a_list=[]):
a_list.append(new_item)
return a_list
and this is correct: def good_append(new_item, a_list=None):
if a_list is None:
a_list = []
a_list.append(new_item)
return a_list good_append(1) -> [1]
good_append(2) -> [2]
bad_append(1) -> [1]
bad_append(2) -> [1, 2]
Presumably, keeping an ever-growing shared list is not the desired behavior.It is by design that the default parameter to a function parameter is evaluated only once, as the alternative has bad performance implications.
It is by design that when you have a reference, that modifications to that value are still available on the same reference. (That is, Python has mutable objects with a couple of immutable exceptions.) This is obviously a fairly normal choice, very basic to the language design.
Now, take these two normal things together and you get that if you make your default function parameter a mutable object, it'll be the same mutable object every time. The relevant "import this" aphorism would be "Special cases aren't special enough to break the rules."
So, is this "by design"? Well, not directly, it's a natural consequence of other aspects of the design. All languages have corner cases of one sort or another. Making this an exception would itself be a wart, after all:
l = []
def f(param = l): pass
def g(param = []): pass
Shouldn't f and g have similar behavior? If you propose that the value be evaluated every time, they won't, and then people would complain about that, too.Do people actually use the results of long-running or side-effectful functions as default parameters? It seems the only time this issue comes up is when people get tripped up by mutating simple [] or {} default parameters.
I would expect a default expression to be simple and pure. If this expression should only be executed once, requiring a programmer to be explicit about the complexity wouldn't be a bad thing.
Lists have other quirks related to mutability; there are other obscure Pythonisms that use this. If you want a nice magic-free immutable container, tuples are a better bet.
foo = makeSomethingDifferent( foo )
And I need to catch myself when foo is a default param l = []
def f(param = l): pass
def g(param = []): pass
I would say f should be a syntax error, as languages shouldn't allow a default parameter to be an expression. If people wanted that behavour, they could write: def f(param =[]):
if param==[]: param = l
#...etc...Again, if you change this just for argument lists, you're looking at another wart.
I'm not disagreeing with you. Your logic makes sense in isolation. However, language design invariably requires this sort of tradeoff.
How far down does Ruby copy? (A default value could be deep.) How does the programer specify a different level (including no copying)?
Python's rule seems bad until you try to use the alternatives outside the single case that they were designed for. Python's rule handles all cases reasonably.
To do it the python way, you'd have to use a global variable to store the array - which is exactly what python is doing for you. Doesn't sound so handy when its put like that, huh?
>>> def hello(a=[]): a.append('hello'); return a;
...
>>> hello()
['hello']
>>> hello()
['hello', 'hello']
On post, what mindslight said.The empty list in the first one is only created at the time the 'def' is executed. All subsequent calls to 'bad_append' hold the reference to this list instead of a new empty one.
The 'good' one uses a sentinel value to indicate that you want to start with an empty list but also gives you the ability to pass in an existing one if needed.
Also, the approach sounds familiar: didn't Java have the same rationale? And of course Apple - to each their own, I guess (I don't like "the Apple way", but others love it - probably would be the same with Python).
That article is really putting me off learning python, to be honest.
You can do things in a million different ways, monkey patch everything under the sun but in the end, having just one way to do things makes code more reliable and readable.
If you don't like the way the framework is built, and its one preferred way of doing things, select a different framework or write your own. Trying to undermine the assumptions of an existing framework has a tendency to cause subtle hard to detect bugs. And trying to support all approaches tend to create stuff that is bloated, ugly and slow.
obvious is mandatory here.
Furthermore, language design by its very nature involves incentivizing some things and disincentivizing others. What this blog post is truly claiming, IMHO, is that Python did it willfully and well, whereas many other languages grew by accretion and without intentional thought as to what they were incentivizing.
So don't judge Python because it intentionally incentivizes some behaviors at the cost of others - every language does that - judge the things it incentivizes.
Is not a good thing. It leads to curse of choice, analysis paralysis, bikeshedding, and all the related drains on productivity.
I submit that "There should be one obvious way to do it" is the #1 reason devs are/feel more productive with Python.
Sadly Python is growing too many exceptions to that rule.
"from X import *" is for REPL usage, where it is damned handy, not to mention a 20-line script. The community strongly discourages its use in big programs.
This place I know was one of the earliest adopters of Python, and the code they barfed up must be the buggiest, ugliest (GUI), slowest in its class software I've ever seen. It's so buggy, it's insane: typos, calling methods that don't exist, assuming arbitrary user input does not have 'keywords' in them...
They couldn't write code in C, so they thought the problem was with the language and switched to Python.
I say, lousy programmers, if they don't want to look for another occupation, should stick with Java - at least the typos will be caught more frequently.