Now located at https://github.com/JeffPaine/beautiful_idiomatic_python.
Github gists don't support Pull Requests or any notifications, which made it impossible for me to maintain this (surprisingly popular) gist with fixes, respond to comments and so on. In the interest of maintaining the quality of this resource for others, I've moved it to a proper repo. Cheers!
I added notes for python 3 users. Could you please pull this changes? https://gist.github.com/mongoose11235813/151a04156e1a709ad6e4
@mongoose11235813 Thanks! You're changes have been merged. Sorry for the delay. Some searching turned up that github doesn't have notifications for comments on gists :(
For my (or other's) future reference on how I merged in the changes from your fork (since gists don't support pull requests):
git remote add mongoose11235813 git@gist.github.com:151a04156e1a709ad6e4.git
git fetch mongoose11235813
git merge mongoose11235813/master
git push origin master
There is yet a better way of counting things: https://treyhunner.com/2015/11/counting-things-in-python/
Hi, under "Using decorators to factor-out administrative logic" can you @JeffPaine please fix the if statement?
Cool!
Nice to see this written. Please change dicitonary keys to dictionary keys.
It's so useful! Thanks
@robinchew: done, thanks! 😃
@DimitriPapadopoulos: Done, thanks! 🍰
All: glad everyone is enjoying and learning from this write up. Raymond Hettinger really gave a wonderful talk here!
Thanks for the notes @JeffPaine!
That was a great talk. The ratio content/time was huge. There's someone in the video comments (DarthChrisB) who time-stamped the video with each topic.
Thanks for sharing.
Interesting~
You should precise the library for defaultdict, influence, deque... Good job thank you !
Thanks for sharing.
this is awesome, thanks!
Typo on "Factor-out temporary contexts":
yield fieldobj
Thanks so much for writing this up! I actually had just started to do the same, but then decided I should at least check online to see if anyone else already had!
I see @ikem-krueger mentioned this but you have a typo with the redirect_stdout function. It should yield fileobj instead of 'fieldobj'.
You can fork my fix below:
https://gist.github.com/michaelbasca/0186c7c16326d76fa1764a8d09d6854b
cool!
Thanks for sharing. This really useful!
Hi!
This part is not working (even indentation fixed)
https://gist.github.com/JeffPaine/6213790#better-9
Consider updating from my fork:
https://gist.github.com/BartoszCki/fd918625972e6637dc41e59d0d822db6
# First we are shown this as the better way to reverse,
reversed(colors)
# And then this is shown for reversing sorted elements
sorted(colors, reversed=True)
# Why not this?
reversed(sorted(colors))
"""
I'm sure there's a perfectly good reason, like, "There's an algorithm with which sorting and reversing
can be done together, faster than they can be done apart.", I just think that it should be made explicitly apparent.
"""
Hey. Small typo: explicidly -> explicitly.
Thank you very much.
List Comprehensions and Generator Expressions
result = []
for i in range(10):
s = i ** 2
result.append(s)
print sum(result)
IndentationError: expected an indented block
Factor-out temporary contexts
@contextmanager
def redirect_stdout(fileobj):
oldstdout = sys.stdout
sys.stdout = fileobj
try:
yield fieldobj # 'fieldobj' may be 'fileobj'?
finally:
sys.stdout = oldstdout
I've addressed all comments here and moved this gist to a proper repo: https://github.com/JeffPaine/beautiful_idiomatic_python (so Pull Requests can be received and I get notifications, which isn't possible for gists).
If you have any other fixes / comments, please make them on that repo as I won't be checking back here again in the future. Thanks a lot!
@frodon1 Raymond Hettinger actually mentioned that this example can be simplified using an early return. See video. He said that for...else can be useful when a situation like this occurs "intermeshed with other more complex code in other operation so there is not a shortcut out".