Принципы SOLID: глубокое погружение

// ── НАРУШЕНИЕ: две причины для изменения в одном классе ─────────

class OrderProcessor {

  void process(Order order) {

    // бизнес-логика

    order.setStatus(CONFIRMED);

    // сохранение — причина #2 для изменения

    db.save(order);

    // уведомление — причина #3 для изменения

    emailClient.send(order.getEmail(), buildTemplate(order));

  }

}

// ── ИСПРАВЛЕНИЕ: у каждого класса одна причина для изменения ────

class OrderConfirmationService {

  OrderConfirmationService(OrderRepository repo, OrderNotificationService notifier) { ... }

  void confirm(Order order) {

    order.confirm();          // только доменная логика

    repo.save(order);         // делегирует сохранение

    notifier.sendConfirmation(order); // делегирует уведомление

  }

}

class OrderRepository      { void save(Order o) { ... } }   // только БД

class OrderNotificationService { void sendConfirmation(Order o) { ... } } // только email

// ── НАРУШЕНИЕ: каждый новый способ оплаты требует модификации ───

class PaymentProcessor {

  void process(Payment p) {

    if (p.type() == CREDIT_CARD) chargeCard(p);

    else if (p.type() == PAYPAL)  chargePayPal(p);

    else if (p.type() == CRYPTO)  chargeCrypto(p);  // добавлено позже — изменён существующий класс

  }

}

// ── ИСПРАВЛЕНИЕ: закрыт для модификации, открыт для расширения ──

interface PaymentGateway { void charge(Payment p); }

class CreditCardGateway  implements PaymentGateway { ... }

class PayPalGateway       implements PaymentGateway { ... }

class CryptoGateway       implements PaymentGateway { ... }  // новый метод = только новый класс

class PaymentProcessor {

  private final Map<PaymentType, PaymentGateway> gateways;

  void process(Payment p) {

    gateways.get(p.type()).charge(p);  // никогда не изменяется

  }

}

// ── НАРУШЕНИЕ: Square нарушает поведенческий контракт Rectangle ─

class Rectangle {

  protected int width, height;

  void setWidth(int w)  { this.width = w; }

  void setHeight(int h) { this.height = h; }

  int area() { return width * height; }

}

class Square extends Rectangle {

  @Override void setWidth(int w)  { width = height = w; }  // нарушает контракт Rectangle

  @Override void setHeight(int h) { width = height = h; }  // нарушает контракт Rectangle

}

// Клиентский код ломается при подстановке Square вместо Rectangle:

// rect.setWidth(5); rect.setHeight(4); assert rect.area() == 20; // ПРОВАЛ для Square

// ── ИСПРАВЛЕНИЕ: моделируем реальное отношение — без наследования ─

interface Shape { int area(); }

record Rectangle(int width, int height) implements Shape {

  public int area() { return width * height; }

}

record Square(int side) implements Shape {

  public int area() { return side * side; }

}

// ── НАРУШЕНИЕ: толстый интерфейс принуждает к ненужным заглушкам ─

interface Worker {

  void work();

  void takeBreak();      // нерелевантно для Robot

  void receivePaycheck(); // нерелевантно для Robot

}

class Robot implements Worker {

  public void work() { ... }

  public void takeBreak() { /* Robot не отдыхает — принудительная заглушка */ }

  public void receivePaycheck() { /* Robot не получает зарплату — заглушка */ }

}

// ── ИСПРАВЛЕНИЕ: маленькие ролевые интерфейсы ───────────────────

interface Workable  { void work(); }

interface Breakable { void takeBreak(); }

interface Payable   { void receivePaycheck(); }

class Robot      implements Workable { ... }                        // только нужное

class HumanWorker implements Workable, Breakable, Payable { ... }   // полный набор

// Планировщик нуждается только в Workable — не знает о перерывах и оплате:

class ProductionScheduler { void schedule(List<Workable> workers) { ... } }

// ── НАРУШЕНИЕ: модуль верхнего уровня зависит от детали ─────────

class OrderService {

  private final MySqlOrderRepository repo = new MySqlOrderRepository();

  void confirm(Long id) {

    Order order = repo.findById(id);

    order.confirm();

    repo.save(order);

  }

}

// ── ИСПРАВЛЕНИЕ: оба зависят от абстракции ───────────────────────

interface OrderRepository { Order findById(Long id); void save(Order o); }

class OrderService {

  private final OrderRepository repo;  // зависит от абстракции

  OrderService(OrderRepository repo) { this.repo = repo; }  // DI через конструктор

  void confirm(Long id) { ... }  // логика не изменилась

}

// Низкоуровневая деталь зависит от абстракции (не наоборот):

class MySqlOrderRepository implements OrderRepository { ... }    // продакшен

class InMemoryOrderRepository implements OrderRepository { ... } // тесты