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

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. 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

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