how to code review

44
@dylanninin ই؉অդᎱਭ

Upload: gong-haibing

Post on 09-Apr-2017

144 views

Category:

Presentations & Public Speaking


0 download

TRANSCRIPT

Page 1: How to Code Review

@dylanninin

Page 2: How to Code Review
Page 3: How to Code Review
Page 4: How to Code Review
Page 5: How to Code Review
Page 6: How to Code Review

OpenPlay 1.0 / - - - …

: - http://bit.ly/1TfZFpV

Page 7: How to Code Review

OpenPlay 1.0 - - - …

: - http://bit.ly/1Tk7zOX

Page 8: How to Code Review
Page 9: How to Code Review

- if else -

- http://bit.ly/1VUBI8f

Page 10: How to Code Review

Rating - rating/motm - rating/motm - …

- http://bit.ly/1UE2CCj - http://bit.ly/1PGTPaa

Page 11: How to Code Review
Page 12: How to Code Review

Cub ElasticSearch - - - - …

- http://bit.ly/1M25XI1 - http://bit.ly/1M25wxz

Page 13: How to Code Review
Page 14: How to Code Review

Redis pubsub - / - pubsub, list -

- http://bit.ly/1M26wSg

Page 15: How to Code Review
Page 16: How to Code Review

Admin Stats - 1.x (MongoDB) —> 2.x (ElasticSearch) - DownloadMixin - crontab -> queue -> queue/msg

- http://bit.ly/1VUBrSM - http://bit.ly/1PGTPaa

Page 17: How to Code Review
Page 18: How to Code Review
Page 19: How to Code Review
Page 20: How to Code Review
Page 21: How to Code Review
Page 22: How to Code Review
Page 23: How to Code Review

• • • • • •

Page 24: How to Code Review

Python

Page 25: How to Code Review

>>> import this The Zen of Python, by Tim Peters

Beautiful is better than ugly. Explicit is better than implicit. Simple is better than complex. Complex is better than complicated. Flat is better than nested. Sparse is better than dense. Readability counts. Special cases aren't special enough to break the rules. Although practicality beats purity. Errors should never pass silently. Unless explicitly silenced. In the face of ambiguity, refuse the temptation to guess. There should be one-- and preferably only one --obvious way to do it. Although that way may not be obvious at first unless you're Dutch. Now is better than never. Although never is often better than *right* now. If the implementation is hard to explain, it's a bad idea. If the implementation is easy to explain, it may be a good idea. Namespaces are one honking great idea -- let's do more of those!

Page 26: How to Code Review
Page 27: How to Code Review
Page 28: How to Code Review

Hello World

Page 29: How to Code Review
Page 30: How to Code Review
Page 31: How to Code Review
Page 32: How to Code Review
Page 33: How to Code Review
Page 34: How to Code Review
Page 35: How to Code Review

• • • GitHub + Pull Request + Issue • • •

Page 36: How to Code Review
Page 37: How to Code Review

-

• • • • • • • • • •

Page 38: How to Code Review

-

• • • •

Page 39: How to Code Review

-

• • • • • • • ‘TODO’

Page 40: How to Code Review

-

• • • • “ “ • API

Page 41: How to Code Review

-

Page 42: How to Code Review

Just Do It

Page 43: How to Code Review

Q & A