为什么 PHP 对象数组中的变量没有变化？

Question

我正在尝试从 PHP 中的各种来源组装array 的对象（我是该语言的新手）。问题是我无法将数据存储在我正在尝试编写的foreach 循环中的$bandwidthData array 中。

private function _getData($startDay, $totalDays)
{
    $devices = ubntDevices::getDevices();
    $data = [];
    $bandwidthData = [];
    $count = 0;

    foreach ($devices as $device) {
        $bandwidthData[$count]['device'] = $device;
        $bandwidthData[$count]['bandwidth'] = new ubntModel($device->file);
        $bandwidthData[$count]['bandwidth']->getMonthData();
        $bandwidthData[$count]['tree'] = new graphTree($device->hostid);
        $bandwidthData[$count]['graphid'] = ubntGraph::getGraphByHostId($device->hostid);
        $bandwidthData[$count]['hostname'] = $device->host_name;
        $count++;
    }
    return $bandwidthData;
}

如果我从foreach 循环中返回，我会得到正确的输出（但显然只针对第一个设备）。我已经测试了所有其他 function 来源，它们似乎返回了正确的数据。知道我做错了什么吗？提前谢谢你。

Answer 1

您的 PHP 错误日志应该指出出了什么问题。强烈推荐 XDebug。
然而，现在更常见的是使用这样的关联数组：

private function _getData($startDay, $totalDays)
{
    $devices = ubntDevices::getDevices();
    $bandwidthData = [];

    foreach ($devices as $device) {
        $ubntModel = new ubntModel($device->file);
        $deviceData = array('device' => $device,
                            'ubntModel' => $ubntModel,
                            'bandwidth' => $ubntModel->getMonthData(),
                            'tree' => new graphTree($device->hostid),
                            'graphid' => ubntGraph::getGraphByHostId($device->hostid),
                            'hostname' => $device->host_name);
        $bandwidthData[] = $deviceData;
    }
    return $bandwidthData;
}

Answer 2

我看到的一些东西：

这个变量在使用吗？

$data = [];

此外，此分配运行但不鼓励使用，因为此时 $bandwidthData[$count] 不存在：¹

$bandwidthData[$count]['device'] = $device;

可以转换为：

$bandwidthData[$count] = [ 'device' => $device ];

此外，这只是一个无处返回的吸气剂。不是吗？

$bandwidthData[$count]['bandwidth']->getMonthData();

为了进一步学习 PHP，我可以为代码 sn-p 建议这种更简洁的方法，只是为了提高可读性：

private function _getData( $startDay, $totalDays ) {
    $bandwidthData = [];

    foreach ( ubntDevices::getDevices() as $device ) {
        $bandwidthData[] = [
            'device'    => $device,
            'bandwidth' => new ubntModel( $device->file ),
            'tree'      => new graphTree( $device->hostid ),
            'graphid'   => ubntGraph::getGraphByHostId( $device->hostid ),
            'hostname'  => $device->host_name,
        ];
    }

    return $bandwidthData;
}

无论如何，您必须学习如何调试仅查看服务器日志的简单小代码块，或者在您怀疑的行中使用懒惰的var_dump( $bandwidthData[ $count ] )（不要为这种调试方法争吵：它非常愚蠢，但非常戏剧化简单有效，对新手友好 - 如果您无法设置调试器，因为服务器可能不是您的等）或设置调试器。

---

¹来自https://www.php.net/manual/en/language.types.array

如果 $arr 还不存在，它将被创建，所以这也是创建数组的另一种方式。然而，这种做法是不鼓励的，因为如果 $arr 已经包含一些值（例如来自请求变量的字符串），那么这个值将保留在该位置并且 [] 实际上可能代表字符串访问运算符。通过直接赋值来初始化变量总是更好