Человек IT

Такой рефакторинг применяется, когда методу нужна дополнительная информация от вызывающего его клиента.

php
 
<?php
// ..... было
public function getProducts()
{
  // забираем продукты из базы
  return $this->orm->select('products');
}
 

Мы живем в изменяющемся мире! Предположим, что бизнес стал развиваться и товаров стало большe, тогда нам приедтся их категоризировать. После этого возникнет естественное желание иметь возможность просматривать товары только одной категории, однако старый функционал неплохо бы сохранить.

php
 
public function getProducts($category_id=null)
{
  $cid = intval($category_id);
  if($cid)
  {
    $this->orm->setCriteriaEqual('category_id',$cid);
  }
  // забираем продукты из базы
  return $this->orm->select('products');
}
 

Обратите внимание, что в данном случае рефакторинг выполнен безопасно, т.е. старый функционал не пострадал — если клиент вызовет метод без параметров, ему вернутся все продукты.

Комменты

• 05.10.2010 01:46:38 tenshi:

  это случай простой. но бывают и сложные, когда модифицируя метод можно незаметно внести ошибку. поэтому лучше не переписывать метод оставляя большую часть кода неизменной. а создать новый метод, а старого даже не касаться.
  
  а то такими темпами в getProducts появится неимоверное число параметров и сложная логика внутри.
  
  а вообще, возможно стоит сделать дсл в духе смалтолка
  
  function _getProducts()
  function _getProducts_category( $catId )
  function _getProducts_name( $name )
  function _getProducts_category_name( $catId, $name )
  
  $obj->getProducts( array( 'category' => 'xxx' ) )