UPDATE:改写一个问题,在这个类结构中是否存在太多静态方法(我意识到现在只有4个,但我最初是以2开始的)?如果是这样,有关如何重构这些类来使用某种Finder类的建议,以便我可以从Model类中移除静态函数?对于一个类,有多少个静态方法太多?
我有以下抽象类:
abstract class LP_Model_Abstract
{
protected static $_collectionClass = 'LP_Model_Collection';
protected $_row = null;
protected $_data = array();
public function __construct($row = null)
{
$this->_row = $row;
}
public function __get($key)
{
if(method_exists($this, '_get' . ucfirst($key)))
{
$method = '_get' . ucfirst($key);
return $this->$method();
}
elseif(isset($this->_row->$key))
{
return $this->_row->$key;
}
else
{
foreach($this->_data as $gateway)
{
if(isset($gateway->$key))
{
return $gateway->$key;
}
}
}
}
public function __set($key, $val)
{
if(method_exists($this, '_set' . ucfirst($key)))
{
$method = '_set' . ucfirst($key);
return $this->$method($val);
}
elseif(isset($this->_row->$key))
{
$this->_row->$key = $val;
return $this->_row->$key;
}
else
{
foreach($this->_data as $gateway)
{
if(isset($this->_data[$gateway]->$key))
{
$this->_data[$gateway]->$key = $val;
return $this->_data[$gateway]->$key;
}
}
}
}
public function __isset($key)
{
return isset($this->_row->$key);
}
public function save()
{
$this->_row->save();
}
abstract public static function get($params);
abstract public static function getCollection($params = null);
abstract public static function create($params);
}
然后这个类,它提供对类表继承方案的附加功能(其中,类型是重要的,在工厂的方式确定的附加功能) :
abstract class LP_Model_Factory_Abstract extends LP_Model_Abstract
{
protected static $_collectionClass = 'LP_Model_Collection_Factory';
abstract public static function factory($row);
}
这些最终导致以下类型的声明:
class Model_Artifact extends LP_Model_Factory_Abstract
{
protected static $_artifactGateway = 'Model_Table_Artifact';
public static function create($params)
{
}
public static function get($params)
{
$gateway = new self::$_artifactGateway();
$row = $gateway->fetchArtifact($params);
return self::factory($row);
}
public static function getCollection($params = null)
{
$gateway = new self::$_artifactGateway();
$rowset = $gateway->fetchArtifacts($params);
$data = array(
'data' => $rowset,
'modelClass' => __CLASS__
);
return new self::$_collectionClass($data);
}
public static function factory($row)
{
$class = 'Model_Artifact_' . $row->fileType;
}
}
你什么时候知道你在一个类中有太多的静态方法?你将如何重构现有的设计,以便静态方法可能封装在某种Finder类中?
微软笑话的提醒,技术上正确,但在这种情况下,不是非常有用。主要是因为这些方法已经不能修改对象的状态,他们只是返回对象或创建一个全新的对象。但无论如何thx答案。 ;-) – 2009-01-14 22:06:13
够公平的。我不熟悉php使用过去的概念信息在这种情况下=( – 2009-01-14 22:12:18
gabriel,你问“什么时候太多?”不看我的代码是这么多... – cgreeno 2009-01-14 22:14:52