How to write python code so people would love you (or not)

Transcript

1. Никита Соболев github.com/sobolevn 1
2. Как писать на питоне так, чтобы все вас ненавидели любили !2
3. Ни одной м***ы !3
5. def recommended_books(books): book_names = [ book.name for book in books ] good_books = [ book_name for book_name in book_names if book_name not in ('Twilight', 'Learning Go Lang') ] recommended_books = [ find_similar(book_name) for book_name in book_names ] return recommended_books !5
6. def recommended_books(books): book_names = [ book.name for book in books ] good_books = [ book_name for book_name in book_names if book_name not in ('Twilight', 'Learning Go Lang') ] recommended_books = [ find_similar(book_name) for book_name in book_names ] return recommended_books !6
7. def recommended_books(books): book_names = [ book.name for book in books ] good_books = [ book_name for book_name in book_names if book_name not in ('Twilight', 'Learning Go Lang') ] recommended_books = [ find_similar(book_name) for book_name in book_names ] return recommended_books !7
8. def recommended_books(books): book_names = [ book.name for book in books ] good_books = [ book_name for book_name in book_names if book_name not in ('Twilight', 'Learning Go Lang') ] recommended_books = [ find_similar(book_name) for book_name in book_names ] return recommended_books !8
9. def recommended_books(books): book_names = [ book.name for book in books ] good_books = [ book_name for book_name in book_names if book_name not in ('Twilight', 'Learning Go Lang') ] recommended_books = [ find_similar(book_name) for book_name in book_names ] return recommended_books !9
10. Три действия = одна функция !10
11. Композиция !11
12. Три действия = одна функция !12
14. Абстракция !14
15. 3 = 1* * при соблюдении уровня абстракции !15
16. Композиция и абстракция !16
17. def recommended_books(books): book_names = [ book.name for book in books ] good_books = [ book_name for book_name in book_names if book_name not in ('Twilight', 'Learning Go Lang') ] recommended_books = [ find_similar(book_name) for book_name in book_names ] return recommended_books !17
18. def recommended_books(books): recommended_books = recommend(filter(transform(books))) return recommended_books !18
19. Три разных слота !19
21. transform = db lookup filter = api call recommend = cache lookup !21
22. def recommended_books(books): book_names = book_repo.names_by_ids(books) good_books = filter_api.by_preferences(book_names) recommended_books = cache.not_shown(good_books) return recommended_books !22
23. def recommended_books(books): book_names = book_repo.names_by_ids(books) good_books = filter_api.by_preferences(book_names) recommended_books = cache.not_shown(good_books) return recommended_books !23
24. def recommended_books(books): recommended_books = recommend(filter(transform(books))) return recommended_books !24
25. Нам срочно нужно переиспользовать! !25
26. Может быть как параметры? !26
27. def recommended_books(recommend, filter, transform, books): recommended_books = recommend(filter(transform(books))) return recommended_books !27
28. Некрасиво! !28
29. Может в два захода? !29
30. def recommended_books(recommend, filter, transform): def inner(books): return recommend(filter(transform(books))) return inner !30
31. Две вложенные функции можно записать понятнее! !31
32. class RecommendedBooks(object): def __init__(self, book_repo, filter_api, cache): self._repo = book_repo self._api = filter_api self._cache = cache def __call__(self, books): book_names = self._repo.names_by_ids(books) good_books = self._api.by_preferences(book_names) recommended = self._cache.not_shown(good_books) return recommended !32
33. Но мы все равно вынуждены таскать за собой параметры ВЕЗДЕ! !33
34. Может нормальный класс напишем? !34
36. Stop writing classes youtube.com/watch?v=o9pEzgHorH0 36
37. Что понять ООП и писать нормально, нам нужно: !37
38. Что понять ООП и писать нормально, нам нужно: • Прочитать уйму умных книг !37
39. Что понять ООП и писать нормально, нам нужно: • Прочитать уйму умных книг • Посмотреть на ужас Java и C++ !37
40. Что понять ООП и писать нормально, нам нужно: • Прочитать уйму умных книг • Посмотреть на ужас Java и C++ • Понять прототипы из JS, протоколы из TypeScript !37
41. Что понять ООП и писать нормально, нам нужно: • Прочитать уйму умных книг • Посмотреть на ужас Java и C++ • Понять прототипы из JS, протоколы из TypeScript • Выучить SmallTalk !37
42. Что понять ООП и писать нормально, нам нужно: • Прочитать уйму умных книг • Посмотреть на ужас Java и C++ • Понять прототипы из JS, протоколы из TypeScript • Выучить SmallTalk • Поработать с акторами и Erlang. Выслушать тонну критики !37
43. Что понять ООП и писать нормально, нам нужно: • Прочитать уйму умных книг • Посмотреть на ужас Java и C++ • Понять прототипы из JS, протоколы из TypeScript • Выучить SmallTalk • Поработать с акторами и Erlang. Выслушать тонну критики • Понять, что ООП нужно только для очень высокого уровня абстракции в некоторых типах систем !37
45. FP
46. OOP FP
47. OOP FP WTF ?!
48. Вот бы можно было вставить нужное поведение... !39
49. class RecommendedBooks(object): def __init__(self, book_repo, filter_api, cache): self._repo = book_repo self._api = filter_api self._cache = cache def __call__(self, books): book_names = self._repo.names_by_ids(books) good_books = self._api.by_preferences(book_names) recommended = self._cache.not_shown(good_books) return recommended !40
50. Но нам нужно понять ограничения... !41
51. Может быть типы? !42
52. НО В ПИТОНЕ НЕТ ТИПОВ !43
53. int + int = int !44
54. В ПИТОНЕ ЕСТЬ ТИПЫ !45
55. Типы курильщика Типы здорового человека uint double &mut str uint16_t std::boxed::Box<dyn Any + 'static> List[int] data Bool = False | True float Callable[[int], int] IO[Result[User, Exception]]
56. book_names = self._repo.names_by_ids(books) !47
57. class RecommendedBooks(object): def __init__( self, book_repo, filter_api, cache, ): self._repo = book_repo self._api = filter_api self._cache = cache def __call__(self, books): book_names = self._repo(books) good_books = self._api(book_names) recommended = self._cache(good_books) return recommended !48
58. class RecommendedBooks(object): def __init__( self, book_repo: Callable[[List[int]], List[str]], filter_api, cache, ) -> None: self._repo = book_repo self._api = filter_api self._cache = cache def __call__(self, books: List[int]): book_names = self._repo(books) good_books = self._api(book_names) recommended = self._cache(good_books) return recommended !49
59. class RecommendedBooks(object): def __init__( self, book_repo: Callable[[List[int]], List[str]], filter_api: Callable[[List[str]], List[Book]], cache, ) -> None: self._repo = book_repo self._api = filter_api self._cache = cache def __call__(self, books: List[int]): book_names = self._repo(books) good_books = self._api(book_names) recommended = self._cache(good_books) return recommended !50
60. class RecommendedBooks(object): def __init__( self, book_repo: Callable[[List[int], List[str]], filter_api: Callable[[List[str]], List[Book]], cache: Callable[[List[Book]], List[Book]], ) -> None: self._repo = book_repo self._api = filter_api self._cache = cache def __call__(self, books: List[int]) -> List[Book]: book_names = self._repo(books) good_books = self._api(book_names) recommended = self._cache(good_books) return recommended !51
61. Но класс - не функция! !52
62. @final @attr.dataclass(frozen=True, slots=True) class RecommendedBooks(object): _repo: Callable[[List[int]], List[str]] _api: Callable[[List[str]], List[Book]] _cache: Callable[[List[Book]], List[Book]] def __call__(self, books): book_names = self._repo(books) good_books = self._api(book_names) recommended_books = self._cache(good_books) return recommended_books !53
63. SRP Callable class !54
64. pip install punq !55
65. !56
66. import punq # Types: BookNamesByIds = Callable[[List[int]], List[str]] FilterPreferedBooks = Callable[[List[str]], List[Book]] NotShownBooks = Callable[[List[Book]], List[Book]] !56
67. import punq # Types: BookNamesByIds = Callable[[List[int]], List[str]] FilterPreferedBooks = Callable[[List[str]], List[Book]] NotShownBooks = Callable[[List[Book]], List[Book]] # DI: container = punq.Container() container.register(BookNamesByIds, my_book_repo) container.register(FilterPreferedBooks, my_api) container.register(NotShownBooks, my_cache) !56
68. import punq # Types: BookNamesByIds = Callable[[List[int]], List[str]] FilterPreferedBooks = Callable[[List[str]], List[Book]] NotShownBooks = Callable[[List[Book]], List[Book]] # DI: container = punq.Container() container.register(BookNamesByIds, my_book_repo) container.register(FilterPreferedBooks, my_api) container.register(NotShownBooks, my_cache) # Somewhere: container.resolve(RecommendedBooks)([1, 2, 3]) !56
69. Ключевой вывод Композиция позволяет писать сложную реализацию, абстракция оставляет ее простой для использования 57
70. А где тут dry- python? !58
71. Callable[[List[int]], List[str]] !59
72. Callable[[List[int]], List[str]] !60
73. Callable[[List[int]], List[str]] • Выбросит ли она исключение? !60
74. Callable[[List[int]], List[str]] • Выбросит ли она исключение? • Будет ли добиться в базу или http? !60
75. dry-python/returns Делаем неявное – явным 61
76. Callable[[List[int]], List[str]] !62
77. Callable[[List[int]], List[str]] • Выбросит ли она исключение? Да? Result[List[int], Exception] !62
78. Callable[[List[int]], List[str]] • Выбросит ли она исключение? Да? Result[List[int], Exception] • Будет ли добиться в базу или http? Да? IO[List[int]] !62
79. Callable[[List[int]], List[str]] • Выбросит ли она исключение? Да? Result[List[int], Exception] • Будет ли добиться в базу или http? Да? IO[List[int]] • Все вместе? Конечно! IO[Result[List[int], Exception]] !62
80. И более удобная композиция compose, pipe, box, pipeline, lift 63
81. class FetchUserProfile(object): def __call__(self, user: int) -> IO[Result['User', Error]]: return pipe( user_id, self._make_request, IO.lift(box(self._parse_json)), ) @impure @safe def _make_request(self, user_id: int) -> Response: ... @safe def _parse_json(self, response: Response) -> 'User': ... !64
83. Версия 0.12 !66
84. !67
85. Legacy-first !68
86. github.com/life4/flakehell !69
87. opencollective.com/ wemake-python-styleguide !70
88. opencollective.com/ wemake-python-styleguide • 4 человека !70
89. opencollective.com/ wemake-python-styleguide • 4 человека • 3 компании: wemake.services, dry-labs, ПАО "Ростелеком" !70
90. opencollective.com/ wemake-python-styleguide • 4 человека • 3 компании: wemake.services, dry-labs, ПАО "Ростелеком" • Вы! !70
91. Сегодня мы многое поняли
92. Полезные ссылки • https://guide.elm-lang.org/architecture/ • https://blog.ploeh.dk/2016/03/18/functional- architecture-is-ports-and-adapters/ • https://fsharpforfunandprofit.com/rop/ • https://www.destroyallsoftware.com/screencasts/ catalog/functional-core-imperative-shell • https://www.youtube.com/watch?v=o9pEzgHorH0 • https://sobolevn.me/ !72
93. tlg.name/ opensource_findings !73
94. Вопросы? github.com/sobolevn sobolevn.me 74