Writing SOLID Code
0 likes209 views
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
Writing SOLID Code
- 1. kgolev.com@kotseto Writing SOLID Code Kostadin Golev CTO @ Rewards Labs Преразказ с елементи на разсъждение
- 2. kgolev.com@kotseto Програмирам > 10 години Професионалните ми интереси включват Аgile, DevOps и Test automation. Горд баща на две деца. Обичам да пиша, блога ми се намира на kgolev.com
- 4. kgolev.com@kotseto S ingle Responsibility Principle O pen-Closed Principle L iskov Substitution Principle I nterface Segregation Principle D ependency Inversion Principle
- 6. kgolev.com@kotseto Инструменти или идеи Помагат да взимаме по-добри решения Приложими са в много и различни ситуации Помагат да комуникираме по-добре Ако има нещо, което ни харесва или не ни харесва в нашия дизайн, много често можем да намерим принцип, с който да го обясним
- 7. 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 • Кода ви е дълъг и прави много неща • Трудно ви е да обясните какво прави едно парче код • Името на метода/класа който пишете няма общо с кода, който разработвате
- 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 Салата с козе сирене
- 28. kgolev.com@kotseto Всеки клас, който приема X като параметър, трябва да може да работи с всеки подклас на X
- 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 • Замяната на типа на един клас променя поведението му • Налага се да проверяваме типа, за да избегнем нежелано поведение
- 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(); … }
- 46. kgolev.com@kotseto • Голям интерфейс с много методи • Празна имплементация на метод от интерфейс, който използваме
- 47. kgolev.com@kotseto The Open Source Library
- 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