CodeFest 2017, Сергей Крапивенский (Rambler Digital Solutions), Кто подставил Барбару Лисков, или кто кого SOLID

https://2017.codefest.ru/lecture/1166

MVVM, VIPER, MVC – очень разные подходы к проектированию архитектуры мобильных приложений. Тем не менее, все они построены на базе гораздо более фундаментальных принципов – SOLID. Зачастую применение таких масштабных паттернов может быть недостаточно оправдано, и следования набору простых и понятных правил будет достаточно для написания чистого и поддерживаемого кода.

Легче всего учиться на ошибках, поэтому все выступление построено на модели "Нарушение принципа -> Обсуждение -> Рефакторинг". В результате доклада у зрителей должно остаться четкое понимание, как принципы SOLID применяются в стандартных задачах iOS разработки.

Тезисы:

Как прокачка фундаментальных знаний помогает нам быстрее становиться сильными разработчиками

Как появился и зачем нужен SOLID

Примеры нарушения каждого из принципов SOLID в реальных проектах

Что делать в случае нарушения принципов и когда их вообще применять

Transcript

1. Кто подставил Барбару Лисков? Сергей Крапивенский, Rambler&Co или Кто кого SOLID?
2. “Do Not Learn Frameworks”
3. Фундаментальные знания
4. Фундаментальные знания Язык
5. Фундаментальные знания Язык Фреймворки
6. Фундаментальные знания Язык Фреймворки
7. RayWenderlich - driven development Senior ReactiveCocoa Developer Рамблер - секта VIPER
8. Фундаментальные знания Язык Фреймворки
9. Фундаментальные знания Язык Фреймворки
10. Фундаментальные знания Язык Фреймворки
11. Фундаментальные знания • Структуры данных • Алгоритмы • Паттерны • DRY, KISS, YAGNI • SOLID • И многое другое
13. Single responsibility principle Open-closed principle Liskov substitution principle Interface segregation principle Dependency inversion principle
14. Single Responsibility Principle “A class should have only one reason to change”
15. MassiveViewController 1.Ответственности сильно связаны 2.Класс перестает помещаться в голове 3.Тяжело поддерживать и тестировать
16. UITableViewController 1.Изменение логики работы с таблицей 2.Поменять иерархию вьюшек
17. UITableViewController Решение проблемы: не использовать его
18. Запуск приложения Push Notifications Quick Actions Уведомления о состояниях приложения Открытие по URL Фоновая загрузка данных AppDelegate
19. AppDelegate Запуск приложения Quick Actions Поиск Push Notifications Открытие URL Состояния приложения Загрузка в фоне Handoff Extensions
20. AppDelegate Запуск приложения Quick Actions Поиск Push Notifications Открытие URL Состояния приложения Загрузка в фоне Handoff Extensions https://github.com/rambler-digital-solutions/RamblerAppDelegateProxy
21. SRP о снижении сложности
22. Single responsibility principle Open-closed principle Liskov substitution principle Interface segregation principle Dependency inversion principle
23. Open-Closed Principle “Software entities (classes, modules, functions, etc.) should be open for extension, but closed for modification”
24. Простой и устойчивый дизайн
25. Новость Новость Новость Новость
26. Новость Новость Новость Новость Новость с фото Новость с фото
27. Новость Новость Новость Новость Новость с фото Новость с фото Реклама Реклама
28. } else ... return cell } func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell { let cell = tableView.dequeueReusableCell(withIdentifier: "SomeIdentifier")! as UITableViewCell let model = self.models[indexPath.row] if (model is News) { let newsCell: NewsCell? = (cell as? NewsCell) newsCell?.setup(withNews: model as! News) } else if (model is Advertisement) { let adCell: AdvertisementCell? = (cell as? AdvertisementCell) adCell?.setup(withAd: model as! Advertisement)
29. } else ... return cell } func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell { let cell = tableView.dequeueReusableCell(withIdentifier: "SomeIdentifier")! as UITableViewCell let model = self.models[indexPath.row] if (model is News) { let newsCell: NewsCell? = (cell as? NewsCell) newsCell?.setup(withNews: model as! News) } else if (model is Advertisement) { let adCell: AdvertisementCell? = (cell as? AdvertisementCell) adCell?.setup(withAd: model as! Advertisement)
30. } else ... return cell } func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell { let cell = tableView.dequeueReusableCell(withIdentifier: "SomeIdentifier")! as UITableViewCell let model = self.models[indexPath.row] if (model is News) { let newsCell: NewsCell? = (cell as? NewsCell) newsCell?.setup(withNews: model as! News) } else if (model is Advertisement) { let adCell: AdvertisementCell? = (cell as? AdvertisementCell) adCell?.setup(withAd: model as! Advertisement)
31. } else ... return cell } func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell { let cell = tableView.dequeueReusableCell(withIdentifier: "SomeIdentifier")! as UITableViewCell let model = self.models[indexPath.row] if (model is News) { let newsCell: NewsCell? = (cell as? NewsCell) newsCell?.setup(withNews: model as! News) } else if (model is Advertisement) { let adCell: AdvertisementCell? = (cell as? AdvertisementCell) adCell?.setup(withAd: model as! Advertisement)
32. } else ... return cell } func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell { let cell = tableView.dequeueReusableCell(withIdentifier: "SomeIdentifier")! as UITableViewCell let model = self.models[indexPath.row] if (model is News) { let newsCell: NewsCell? = (cell as? NewsCell) newsCell?.setup(withNews: model as! News) } else if (model is Advertisement) { let adCell: AdvertisementCell? = (cell as? AdvertisementCell) adCell?.setup(withAd: model as! Advertisement)
33. func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell { let model: CellObject = self.models[indexPath.row] as! CellObject let cell = tableView.dequeueReusableCell(withIdentifier: "CellIdentifier")! as! ConfigurableCell cell.configure(withObject: model) return cell as! UITableViewCell }
34. NewsCell PhotoCell AdCell <ConfigurableCell> func configure(withObject: CellObject) News Photo Advert <CellObject> func cellClass() -> AnyClass
35. Когда применять?
37. Single responsibility principle Open-closed principle Liskov substitution principle Interface segregation principle Dependency inversion principle
39. Liskov Substitution Principle “Subtypes must be substitutable for their base types”
40. Нужен для проверки корректности наследования
41. func hideViews() { UIView.animate(withDuration: 1.0, animations: { for view in self.animatableViews { view.alpha = 0.5 } }) }
42. UIView UIVisualEffectView Всё ок ! <UIVisualEffectView> is being asked to animate its opacity. This will cause the effect to appear broken until opacity returns to 1
43. ¯\_(ツ)_/¯
44. Single responsibility principle Open-closed principle Liskov substitution principle Interface segregation principle Dependency inversion principle
45. Interface Segregation Principle “Clients should not be forced to depend on methods that they do not use”
46. MailAPIClient - auth - findContact - getMailboxes - createMailbox - sendMail Auth Contacts Mailboxes Messages - auth - findContact - getMailboxes - createMailbox - sendMail - auth - findContact - getMailboxes - createMailbox - sendMail - auth - findContact - getMailboxes - createMailbox - sendMail - auth - findContact - getMailboxes - createMailbox - sendMail
47. AuthService MailboxService ContactService MessageServic e Auth Contacts Mailboxes Messages - auth - findContact - getMailboxes - createMailbox - sendMail - auth - findContact - getMailboxes - createMailbox - sendMail - auth - findContact - getMailboxes - createMailbox - sendMail - auth - findContact - getMailboxes - createMailbox - sendMail MailAPIClient
48. ISP - это не SRP
49. - sendSelfDestructMessage - deleteBothMessages ChatChannel - loadMessages - sendMessage - replyToMessage - forwardMessage Channel SecretChat Supergroup - banUser Group - loadMessages - sendMessage - replyToMessage - forwardMessage - sendSelfDestruct - deleteBoth - pinMessage - banUser - pinMessage - loadMessages - sendMessage - replyToMessage - forwardMessage - sendSelfDestruct - deleteBoth - pinMessage - banUser - loadMessages - sendMessage - replyToMessage - forwardMessage - sendSelfDestruct - deleteBoth - pinMessage - banUser - loadMessages - sendMessage - replyToMessage - forwardMessage - sendSelfDestruct - deleteBoth - pinMessage - banUser
50. Single responsibility principle Open-closed principle Liskov substitution principle Interface segregation principle Dependency inversion principle
51. Dependency Inversion Principle “A. High-level modules should not depend on low-level modules. Both should depend on abstractions”
52. Dependency Inversion Principle “B. Abstractions should not depend on details. Details should depend on abstractions”
53. func displayNews() { let newsPredicate = NSPredicate(...) let filteredNews = News.findAll() as! [News] // Отображаем новости } Зависимость от Core Data
54. NewsViewController MagicalRecord Realm
55. protocol NewsProvider { func obtainNewsForDate(date: NSDate) -> [News] } Зависимость от Core Data func displayNews() { let date = NSDate.init() let filteredNews = self.newsProvider?.obtainNewsForDate(date: date) // Отображаем новости }
56. protocol NewsProvider { func obtainNewsForDate(date: NSDate) -> [News] } Зависимость от Core Data func displayNews() { let date = NSDate.init() let filteredNews = self.newsProvider?.obtainNewsForDate(date: date) // Отображаем новости }
57. class ViewController: UIViewController { init(newsProvider : NewsProvider) { self.newsProvider = newsProvider super.init(nibName: "ViewController", bundle: nil) } } Зависимость от Core Data
58. NewsViewController <NewsProvider> MagicalRecord NewsProvider
59. Зависимости инвертированы #
60. Серебряной пули нет
61. - Оноре де Бальзак “Обстоятельства переменчивы, принципы - никогда”
62. Спасибо! serkrapiv sergey.krapivenskiy rambler-ios