我似乎是codeblind为我自己的代码,所以我想问一些额外的眼睛和一些代码的建议,如果写几周前。
守则(缩短)
class TrackingCostCollection {
private $available=0;
private $ordered=0;
private $differenceNumKits=0;
public function fill ($row) {
...
}
/**
* @var TrackingCostCollection $collection
*/
public function merge ($collection) {
$this->available += $collection->available;
$this->ordered += $collection->ordered;
$this->differenceNumKits= $this->available-$this->ordered;
}
}
的问题
当迭代和由数据字段我遇到存储内的两个不同的两个不同的类的实例的问题之一分组数据数组包含相同的数据,虽然我不明白为什么?当我使用一个额外的“克隆”的问题消失,但我想了解为什么我的代码的行为方式是这样:
foreach ($arrValues as $row) {
...
$this->calculateStatistics($row);
}
private function calculateStatistics ($row) {
$fieldValue= $row['country'];
$collection = new TrackingCostCollection();
$collection->fill($row);
if (!isset($this->arrStatistics[$fieldValue])) {
$this->arrStatistics[$fieldValue] = $collection2;
} else {
/* @var TrackingCostCollection $previousCollection2 */
$previousCollection = $this->arrStatistics[$fieldValue];
$previousCollection->merge($collection2);
$this->arrStatistics[$fieldValue] = $previousCollection;
unset($previousCollection);
}
// using $collection instead of $collection2 will cause
// two array entries containing the same data
$collection2= clone $collection;
if (!isset($this->arrTotals[$fieldValue])) {
$this->arrTotals[$fieldValue] = $collection2;
} else {
/* @var TrackingCostCollection $previousCollection2 */
$previousCollection = $this->arrTotals[$fieldValue];
$previousCollection->merge($collection2);
$this->arrTotals[$fieldValue] = $previousCollection;
unset($previousCollection);
}
}
如果更多的代码是必要的诊断问题的原因只是让我知道我会添加更多。
非常感谢您的时间!
,克隆返回$收集的副本,并将其分配给$ collection2 – demonking
默认对象由PHP参考定义。 – PeeHaa
使用__clone()回调来解决此问题;就像其他评论者提到的那样 –