duanbanfei2875 2018-06-07 03:53
浏览 20
已采纳

PHP对象数组要求我克隆每个对象,否则它将持续更改

I have a class that has an array of objects. I have a function that loops over the array, and formats a value, and then just puts these formatted values into a separate array... and then returns them.

Here is relevant code:

class MoneyThing {
    public $taxes = array();

    public function formatTaxes(){
        // Pretend the $taxes array is populated as such when called, with a single LeviedTax object
        //  Array
        //  (
        //      [0] => LeviedTax Object
        //      (
        //          [name] => HST
        //          [rate] => 13
        //          [total] => 17.55
        //          [harmonized] => 1
        //      )
        //  )

        echo "<pre>".print_r($this->taxes, true)."</pre>";

        $formattedTaxes = $this->taxes;

        foreach($formattedTaxes as $tax){
            // Make a clone of the object.  PHP object are copied to other vars by reference by default.
            // $tax = clone $tax;

            // Just adds a dollar sign
            $tax->total = money_format($tax->total);
            $formattedTaxes[] = $tax;
        }

        echo "<pre>".print_r($this->taxes, true)."</pre>";

        // Will print the 'total' with a dollar sign. Why???:
        //  Array
        //  (
        //      [0] => LeviedTax Object
        //      (
        //          [name] => HST
        //          [rate] => 13
        //          [total] => $17.55
        //          [harmonized] => 1
        //      )
        //  )

        return $formattedTaxes;
    }
}

I have printed the class's array before and after the foreach loop. I am seeing the array keep these changes, as if I made them directly by reference within the foreach. I am expecting to have the same array before and after the loop as I'm not even working with said array.

It is my understanding that Arrays are copied plain and simple, whereas Objects are copied by reference. However, since I am copying an array of objects I would expect to not have to use clone because it's not by reference... right?

How come I have to uncomment $tax = clone $tax; in order to not actually persist the formatted values?

  • 写回答

1条回答 默认 最新

  • doukong9982 2018-06-07 04:15
    关注

    Objects are always passed by reference. Technically speaking array of objects in PHP is array of references to objects.

    From design perspective you are trying to solve the wrong problem.

    Fairly speaking you don't need this method. The calls like money_format() should be implemented at the moment of tax output, for example when you will interpolate this into HTML or write into CSV.

    If you are really need to output this stuff multiple time, for example you have to print the formatted tax value into dozens different file formats, it is worth to introduce little overhead in the form of formattedTotal property on the LeviedTax class. In this case you will have a setter like

    class LeviedTax {
      public function setTotal($total) {
        $this->total = $total;
        $this->formattedTotal = money_format($total);
      }
    }
    
    本回答被题主选为最佳回答 , 对您是否有帮助呢?
    评论

报告相同问题?

悬赏问题

  • ¥66 定制开发肯德基自动化网站下单软件
  • ¥20 vscode虚拟环境依赖包未安装
  • ¥15 odoo17关于owl开发js代码问题
  • ¥15 光纤中多普勒频移公式的推导
  • ¥15 怎么制作一个人脸识别门禁系统
  • ¥20 大华dss监控平台网络关闭登不进去
  • ¥15 请使用蚁群算法解决下列问题,并给出我完整的代码
  • ¥20 关于php录入完成后,批量更新数据库
  • ¥15 请教往复密封润滑问题
  • ¥15 cocos creator发布ios包