Sebbene nel codice riportato di seguito venga utilizzato un semplice acquisto di un singolo articolo in un sito di e-commerce, la mia domanda generale riguarda l'aggiornamento di tutti i membri dei dati per mantenere sempre validi i dati di un oggetto.
Ho trovato "coerenza" e "stato è il male" come frasi pertinenti, discusse qui: 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();
Svantaggi o forse modi migliori per farlo?
Questo è un problema ricorrente nelle applicazioni del mondo reale supportate da un database relazionale e se non si utilizzano estesamente le procedure memorizzate per inviare tutta la convalida nel database. Penso che l'archivio dati dovrebbe semplicemente archiviare i dati, mentre il codice dovrebbe fare tutto lo stato di runtime mantenendo il lavoro.
EDIT: questa è una domanda correlata ma non ha una raccomandazione di buone pratiche riguardo a un'unica grande funzione per mantenere lo stato valido: /programming/1122346/c-sharp-object-oriented-design-maintain- valido-oggetto-state
EDIT2: sebbene la risposta di @ eignesheep sia la migliore, questa risposta - /software//a/148109/208591 - è ciò che riempie le linee tra la risposta di @ eigensheep e quello che volevo sapere - il codice dovrebbe solo elaborare, e lo stato globale dovrebbe essere sostituito dal passaggio di stato abilitato DI tra oggetti.