PHP OOP最佳做法指定要调整的属性

我想使用一个类，确定薪水和人数与薪水，OOP是新的我，我已经通过大量的教程和示例，但我不知道如果我使用以下最佳实践：PHP OOP最佳做法指定要调整的属性

class quickCalc 
{ 
    private $salary = 0; 
    private $person = 0; 
    private $person_salary = 0; 

    private animal = 0; 
    private animal_salary = 0; 

    public function addCost($type, $salary) 
    { 
     $this->$type++; 

     $type_two = $type.'_salary'; 

     $this->$type_two += $salary; 
    } 

    public function getCount($type) 
    { 
     return $this->$type; 
    } 

    public function getCost($type) 
    { 
     $type = $type.'_salary'; 
     return $this->$type; 
    } 

} 

$test = new quickCalc(); 
$test->addCost('person', 15000); 

echo $test->getCount('person'); //1 
echo $test->getCost('person'); //10000 
echo $test->getCount('animal'); //0

因此可以看出，为我添加了一个人，或动物等，由1中的相关属性的增加，而且薪水值添加到现有值

我不确定的是，如果最好的做法是通过对象传递属性名称，如图所示：

public function addCost($type, $salary) 
{ 
    $this->$type++; 
}

或应每个属性都有其自身的功能，比如这个：

public function addPerson($salary) 
{ 
    $this->person++; 
    $this->person_salary++; 
}

来源

2016-04-26 JimmyBanks

这个问题似乎更适合http://codereview.stackexchange.com – JimL

你可以或者仍然始终_call_同样的方法，但使用'开关'语句，然后为所述类型运行关联的方法。 – Brett

每个属性都应该有自己的功能。拥有这样的“动态”属性会让人觉得奇怪。

然而，你可以检查如何property overload在PHP中工作。例子很有趣。

我必须补充说，它在我看来你的任务应该涉及一个数组（或Iterators）。您可以添加工资，然后用现有方法计算或计算平均值等。

来源

2016-04-26 19:44:09 soyuka

另一种解决方案是，而不是单独的private属性使用数组来存储单个类型和相应的薪水值。这里有一个例子：

class quickCalc{ 

    private $type = array('person' => 0, 'animal' => 0); 
    private $salary = array('person' => 0, 'animal' => 0); 

    public function addCost($type, $salary){ 
     $this->type[$type]++; 
     $this->salary[$type] += $salary; 
    } 

    public function getCount($type){ 
     return $this->type[$type]; 
    } 

    public function getCost($type){ 
     return $this->salary[$type]; 
    } 
} 

$test = new quickCalc(); 
$test->addCost('person', 15000); 
echo $test->getCount('person') . "<br />"; // 1 
echo $test->getCost('person') . "<br />"; // 15000 

$test->addCost('animal', 20000); 
$test->addCost('animal', 20000); 
echo $test->getCount('animal') . "<br />"; // 2 
echo $test->getCost('animal') . "<br />"; // 40000

输出：

来源

2016-04-26 19:51:18

PHP OOP最佳做法指定要调整的属性

回答

相关问题