Presentation in Bulgarian about SOLID design principles. Presented at PlovDev Conference 2014, ProxiAD office in Bulgaria (multiple times) and at an event for Woman Who Code in Sofia in 2017

1. kgolev.com@kotseto
Writing SOLID Code
Kostadin Golev
CTO @ Rewards Labs
Преразказ с елементи на разсъждение

2. kgolev.com@kotseto
Програмирам > 10 години
Професионалните ми интереси
включват Аgile, DevOps и Test
automation. Горд баща на две деца.
Обичам да пиша, блога ми се намира на
kgolev.com

3. kgolev.com@kotseto

4. kgolev.com@kotseto
S ingle Responsibility Principle
O pen-Closed Principle
L iskov Substitution Principle
I nterface Segregation Principle
D ependency Inversion Principle

5. kgolev.com@kotseto
Не са закони

6. kgolev.com@kotseto
Инструменти или идеи
Помагат да взимаме по-добри решения
Приложими са в много и различни ситуации
Помагат да комуникираме по-добре
Ако има нещо, което ни харесва или не ни харесва в
нашия дизайн, много често можем да намерим
принцип, с който да го обясним

7. kgolev.com@kotseto
Бай Иван, крушката и
вентилатора

8. kgolev.com@kotseto

9. kgolev.com@kotseto
Всяка част от вашия код
трябва да прави едно нещо и
да го прави добре

10. kgolev.com@kotseto
Кода трябва да има само една
причина да се промени

11. kgolev.com@kotseto
class TaxiService {
void orderTaxi(String phone, String address) {
if (phone.length() < 7) {
throw new BadPhoneNumberException
(“Phone not valid!");
}
taxiPool.sendTaxi(address);
smsClient.sendMessage(phone,
"Your taxi is on the way!");
}

12. kgolev.com@kotseto
class TaxiService_V2 {
void orderTaxi(String phone, String address) {
if (phone.length() < 7
&& phone.length() > 12) { // <-- mistake
throw new BadPhoneNumberException
(“Phone not valid!");
}
taxiPool.sendTaxi(address);
smsClient.sendMessage(phone,
“Your taxi is on the way!");
}

13. kgolev.com@kotseto
boolean phoneIsNotValid(String phone) {
return phone.length() < 7
|| phone.length() > 12;
}

14. kgolev.com@kotseto
public class … {
public boolean phoneIsNotValid(String phone) {
return phone.length() < 7
|| phone.length() > 12;
}
}

15. kgolev.com@kotseto
class TaxiService_V3 {
void orderTaxi(String phone, String address) {
if (smsService.phoneIsNotValid(phone)) {
throw new BadPhoneNumberException
("Phone not valid!");
}
taxiPool.sendTaxi(address);
smsService.sendMessage(phone,
“Your taxi is on the way!");
}

16. kgolev.com@kotseto
• Кода ви е дълъг и прави много неща
• Трудно ви е да обясните какво прави едно
парче код
• Името на метода/класа който пишете няма
общо с кода, който разработвате

17. kgolev.com@kotseto
Стария телевизор

18. kgolev.com@kotseto

19. kgolev.com@kotseto
Софтуера трябва да бъде
затворен за модификация, но
отворен за надграждане

20. kgolev.com@kotseto
Customer silverCustomer = new Customer(SILVER);
getDiscountAmount(100, silverCustomer); // 10
Customer goldenCustomer = new Customer(GOLD);
getDiscountAmount(100, silverCustomer); // 20

21. kgolev.com@kotseto
double getDiscountAmount (int purchaseTotal,
Customer customer) {
double discount = 0;
switch (customer.type()) {
case SILVER:
discount = 0.1;
break;
case GOLD:
discount = 0.2;
break;
}
return discount * purchaseTotal;
}

22. kgolev.com@kotseto
class Customer {
double getDiscount() {
return 0;
}
}

23. kgolev.com@kotseto
class SilverCustomer extends Customer {
@Override
double getDiscount() {
return 0.1;
}
}
class GoldenCustomer extends Customer {
@Override
double getDiscount() {
return 0.2;
}
}

24. kgolev.com@kotseto
double getDiscountAmount_V2 (int purchaseTotal,
Customer customer) {
return customer.getDiscount() * purchaseTotal;
}

25. kgolev.com@kotseto
Добавянето на нова функционалност изисква
множество промени в стар код, вместо предимно
писане на нов

26. kgolev.com@kotseto
Салата с козе сирене

27. kgolev.com@kotseto

28. kgolev.com@kotseto
Всеки клас, който приема X като
параметър, трябва да може да
работи с всеки подклас на X

29. kgolev.com@kotseto

30. kgolev.com@kotseto
interface Duck {
void quack();
}

31. kgolev.com@kotseto
class RealDuck implements Duck {
@Override
public void quack() {
//quack code goes here!
}
}

32. kgolev.com@kotseto
Duck realDuck = new RealDuck();
realDuck.quack(); //works!

33. kgolev.com@kotseto
Duck duck = new ElectricDuck();
duck.quack(); //doesn't work?

34. kgolev.com@kotseto
class ElectricDuck implements Duck {
boolean turnedOn = false;
public void quack() {
if (turnedOn) {
//quack code goes here!
}
}
public void turnOn() { … }
}

35. kgolev.com@kotseto
if (duck instanceof ElectricDuck) {
((ElectricDuck)duck).turnOn();
}
duck.quack(); //now works!

36. kgolev.com@kotseto
class ElectricDuck_V2 implements Duck {
boolean turnedOn = false;
public void quack() {
if (!turnedOn) {
turnOn();
}
//quack code goes here!
}
public void turnOn() { … }
}

37. kgolev.com@kotseto
Duck anyDuck = new AnyKindOfDuck();
anyDuck.quack(); //always works

38. kgolev.com@kotseto
Накратко: не трябва кода да ви
изненада

39. kgolev.com@kotseto
• Замяната на типа на един клас променя
поведението му
• Налага се да проверяваме типа, за да избегнем
нежелано поведение

40. kgolev.com@kotseto
Групата по интереси

41. kgolev.com@kotseto

42. kgolev.com@kotseto
Не принуждавайте никой и нищо да зависи от
интерфейс който не му е нужен

43. kgolev.com@kotseto
interface Parent {
void takeCareOfChildren();
}
interface Programmer {
void code();
}
interface Driver {
void drive();
}

44. kgolev.com@kotseto
class Kostadin implements Parent, Programmer, … {
void takeCareOfChildren();
void code();
…
}

45. kgolev.com@kotseto

46. kgolev.com@kotseto
• Голям интерфейс с много методи
• Празна имплементация на метод от интерфейс,
който използваме

47. kgolev.com@kotseto
The Open Source Library

48. kgolev.com@kotseto

49. kgolev.com@kotseto
Смяната на двигателя не трябва да води до смяна
на автомобила
Depend upon abstractions
Do not depend on concretions

50. kgolev.com@kotseto
class OpelDiesel {
OpelDieselEngine engine;
OpelDiesel() {
engine = new OpelDieselEngine();
}
void start() {
engine.ignition();
}
}

51. kgolev.com@kotseto
class OpelGasoline {
OpelGasolineEngine engine;
OpelGasoline() {
engine = new OpelGasolineEngine();
}
void start() {
engine.ignition();
}
}

52. kgolev.com@kotseto
OpelDiesel car = new OpelDiesel();
car.start();
OpelGasoline carGasoline = new OpelGasoline();
carGasoline.start();

53. kgolev.com@kotseto
class Opel {
Engine engine;
Opel(Engine engine) {
this.engine = engine;
}
void start() {
engine.ignition();
}
}

54. kgolev.com@kotseto
class OpelDieselEngine implements Engine {
@Override
public void ignition() {
}
}

55. kgolev.com@kotseto
Opel dieselCar = new Opel(new OpelDieselEngine());
dieselCar.start();
Opel gasolineCar = new Opel(new OpelGasolineEngine());
dieselCar.start();

56. kgolev.com@kotseto
Opel oldOpel = new Opel(new VolgaEngine(),
new MoskvitchBrakes());

57. kgolev.com@kotseto
• Промяната или добавяне на модул води до
промени на модулите, които седят над него
(смяна на колата заради нов акумулатор)
• Модули на по-високо ниво (кола), зависят от
точно определени модули на ниско ниво
(двигател/акумулатор)

58. kgolev.com@kotseto
Software should be soft!
By Definition

59. kgolev.com@kotseto
Въпроси?
@kotseto
kgolev.com/talks/solid