此时此刻,我有一些想法:
/**
* Fields, which are selected from database for collections of acreditations
*
* @var array
* @access private
*/
private $selectFields = [ 'firstname', 'lastname', 'additional_info' ];
/**
* @return array
*/
private function get_selectFields() {
return $this->selectFields;
}
接下来在方法中我做了ex。这个:
public function myFunction() {
var_dump( $this->get_selectFields() );
}
这是好习惯还是我应该改变一些想法?
答案 0 :(得分:2)
具有私有字段和私有方法的代码只返回该字段根本没有意义。通常,private
修饰符用于限制从另一个类访问一个类的内部属性。在您的情况下,您只允许在内部从此类启动的对象访问$selectFields
。获取$selectFields
没有公共接口。因此,$ this-> get_selectFields()和$ this-> selectFields在逻辑上是相等的,因为它们都返回存储在$selectFields
中的数据。
结论:get_selectFields
多余,应删除,直到公开或在返回之前开始处理/修改$selectFields
。
答案 1 :(得分:0)
如果你只是询问好的做法,我认为你是: 它完全取决于它如何扩展,并得到发展。但是,您所呈现的内容并没有任何适当的低效率,假设您需要这些方法单独工作。