Сергей Крапивенский
•
1 like•820 views
Сергей Крапивенский Rambler Digital Solutions Кто подставил Барбару Лисков, или кто кого SOLID
![} 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)](https://image.slidesharecdn.com/11beingsolid-170405045530/85/-28-320.jpg)
![} 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)](https://image.slidesharecdn.com/11beingsolid-170405045530/85/-29-320.jpg)
![} 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)](https://image.slidesharecdn.com/11beingsolid-170405045530/85/-30-320.jpg)
![} 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)](https://image.slidesharecdn.com/11beingsolid-170405045530/85/-31-320.jpg)
![} 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)](https://image.slidesharecdn.com/11beingsolid-170405045530/85/-32-320.jpg)
![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
}](https://image.slidesharecdn.com/11beingsolid-170405045530/85/-33-320.jpg)
![func displayNews() {
let newsPredicate = NSPredicate(...)
let filteredNews = News.findAll() as! [News]
// Отображаем новости
}
Зависимость от Core Data](https://image.slidesharecdn.com/11beingsolid-170405045530/85/-53-320.jpg)
![protocol NewsProvider {
func obtainNewsForDate(date: NSDate) -> [News]
}
Зависимость от Core Data
func displayNews() {
let date = NSDate.init()
let filteredNews = self.newsProvider?.obtainNewsForDate(date:
date)
// Отображаем новости
}](https://image.slidesharecdn.com/11beingsolid-170405045530/85/-55-320.jpg)
![protocol NewsProvider {
func obtainNewsForDate(date: NSDate) -> [News]
}
Зависимость от Core Data
func displayNews() {
let date = NSDate.init()
let filteredNews = self.newsProvider?.obtainNewsForDate(date:
date)
// Отображаем новости
}](https://image.slidesharecdn.com/11beingsolid-170405045530/85/-56-320.jpg)
Сергей Крапивенский
- 1. Кто подставил Барбару Лисков? Сергей Крапивенский, Rambler&Co или Кто кого SOLID?
- 2. “Do Not Learn ɴǰ”
- 7. RayWenderlich - driven development Senior ReactiveCocoa Developer Рамблер - секта VIPER
- 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”
- 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. AuthServiceMailboxServiceContactService 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 ChannelSecretChat 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
- 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
- 60. Серебряной пули нет 😢
- 61. - Оноре де Бальзак “Обстоятельства переменчивы, принципы - никогда”