유효한 상태를 유지하기 위해, 즉 객체의 데이터 멤버를 업데이트하기 위해 클래스에 하나의 큰 개인 함수를 정의하는 것이 좋습니다.

아래 코드에서 전자 상거래 사이트의 간단한 단일 품목 구매가 사용되지만 일반적인 질문은 모든 데이터 멤버를 업데이트하여 객체의 데이터를 항상 유효한 상태로 유지하는 것입니다.

https://ko.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();

단점이 있거나 더 좋은 방법은 무엇입니까?

관계형 데이터베이스가 지원하는 실제 응용 프로그램에서 반복되는 문제이며, 저장 프로 시저를 광범위하게 사용하지 않으면 모든 유효성 검사를 데이터베이스로 푸시합니다. 코드는 모든 런타임 상태 유지 작업을 수행해야하지만 데이터 저장소는 데이터를 저장해야한다고 생각합니다.

편집 : 이것은 관련 질문이지만 유효한 상태를 유지하기위한 하나의 큰 기능에 관한 모범 사례 권장 사항이 없습니다 : /programming/1122346/c-sharp-object-oriented-design-maintaining- 유효한 객체 상태

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)대신 기대 합니다.