Чи є хорошою ідеєю визначити одну велику приватну функцію в класі для підтримки дійсного стану, тобто оновлення членів даних об’єкта?

Хоча в наведеному нижче коді використовується проста покупка одного товару на веб-сайті електронної комерції, моє загальне питання стосується оновлення всіх членів даних, щоб постійно зберігати дані об’єкта у дійсному стані.

Я знайшов "послідовність" та "стан зло" як відповідні фрази, обговорювані тут: https://en.wikibooks.org/wiki/Object_Oriented_Programming#.22State.22_is_Evil.21

<?php

class CartItem {
  private $price = 0;
  private $shipping = 5; // default
  private $tax = 0;
  private $taxPC = 5; // fixed
  private $totalCost = 0;

  /* private function to update all relevant data members */
  private function updateAllDataMembers() {
    $this->tax =  $this->taxPC * 0.01 * $this->price;
    $this->totalCost = $this->price + $this->shipping + $this->tax;
  }

  public function setPrice($price) {
      $this->price = $price;
      $this->updateAllDataMembers(); /* data is now in valid state */
  }

  public function setShipping($shipping) {
    $this->shipping = $shipping;
    $this->updateAllDataMembers(); /* call this in every setter */
  }

  public function getPrice() {
    return $this->price;
  }
  public function getTaxAmt() {
    return $this->tax;
  }
  public function getShipping() {
    return $this->shipping;
  }
  public function getTotalCost() {
    return $this->totalCost;
  }
}
$i = new CartItem();
$i->setPrice(100);
$i->setShipping(20);
echo "Price = ".$i->getPrice(). 
  "<br>Shipping = ".$i->getShipping().
  "<br>Tax = ".$i->getTaxAmt().
  "<br>Total Cost = ".$i->getTotalCost();

Якісь недоліки чи, можливо, кращі способи це зробити?

Це повторювана проблема в реальних програмах, підкріплених реляційною базою даних, і якщо ви не широко використовуєте збережені процедури, щоб перенести всю перевірку в базу даних. Я думаю, що сховище даних повинно просто зберігати дані, тоді як код повинен виконувати всі роботи з підтримання стану виконання.

EDIT: це пов’язане питання, але не має рекомендації щодо найкращої практики щодо єдиної великої функції для підтримки дійсного стану: /programming/1122346/c-sharp-object-oriented-design-maintain- дійсний-об'єкт-стан

EDIT2: Хоча відповідь @ eignesheep найкраща, ця відповідь - /software//a/148109/208591 - це те, що заповнює рядки між відповіддю @ eigensheep і тим, що я хотів знати - код повинен лише обробляти, і глобальний стан повинен бути заміщений переходом стану між об'єктами з підтримкою DI.

За умови рівності ви повинні виразити своїх інваріантів у коді. У цьому випадку у вас є інваріант

$this->tax =  $this->taxPC * 0.01 * $this->price;

Щоб висловити це у своєму коді, видаліть змінну податкового члена та замініть getTaxAmt () на

public function getTaxAmt() {
  return $this->taxPC * 0.01 * $this->price;
}

Вам слід зробити щось подібне, щоб позбутися змінної загальної вартості учасників.

Виявлення ваших інваріантів у коді може допомогти уникнути помилок. У початковому коді загальна вартість є неправильною, якщо вона встановлена ​​перед тим, як встановити значення SetPrice або setShipping.

Будь-які недоліки [?]

Звичайно. Цей метод покладається на те, щоб кожен завжди пам’ятав щось робити. Будь-який метод, який покладається на всіх і завжди, завжди може вийти з ладу .

можливо кращі способи зробити це?

Один із способів уникнути тягаря церемонії запам'ятовування - це обчислення властивостей об'єкта, які залежать від інших властивостей, якщо це потрібно, як @eigensheep запропонував.

Інша - зробити предмет кошика незмінним та обчислити його за допомогою конструктора / фабрики. Зазвичай ви використовуєте метод "обчислити за потребою", навіть якщо ви зробили об'єкт непорушним. Але якщо розрахунок занадто трудомісткий і його читали б багато-багато разів; Ви можете вибрати варіант "обчислити під час створення".

$i = new CartItem();
$i->setPrice(100);
$i->setShipping(20);

Ви повинні запитати себе; Чи має сенс кошик без ціни? Чи може змінюватися ціна товару? Після його створення? Після обчислення її податку? Можливо, вам слід зробити CartItemнезмінним і призначити ціну та доставку в конструкторі:

$i = new CartItem(100, 20);

Чи має сенс візок без кошика, до якого він належить?

Якщо ні, то я б очікував $cart->addItem(100, 20)натомість.