数组下标越界

时间:2022-06-18 王中阳Go 人气:0

抛出问题

数组下标越界真的是开发过程中的痛，除了在开发过程中各种判断是否设置，是否为空，还有其他优雅的办法解决吗？

回答问题

肯定是有的

举个栗子

比如，我有一个工具性质的方法如下：

我怎么保证 $batchUserCover[$userid]['pickedFootprint'] 和 $batchFootprintList[$userid]['list'] 不会有下标越界的问题呢？

    //批量获得图片故事 优先精选故事 不足三个拼接最近发布的故事
    public static function batchImageFootprint($userIds, $batchUserCover, $batchFootprintList)
    {
        $ret = [];
        foreach ($userIds as $userid) {
            $ret[$userid] = array_slice(array_merge($batchUserCover[$userid]['pickedFootprint'], array_slice($batchFootprintList[$userid]['list'], 0, 3)), 0, 3);
        }
        return $ret;
    }

解题思路

1.在方法外判断是否设置值

2.在方法外保证已经设置值，确保 $batchUserCover[$userid]['pickedFootprint'] 这类参数一定是存在的，已经设置了的.

我认为思路2更好

解题实践

实践1：

在传入数据之前，设置好传入值，保证传入值的key必须存在，值可以为null，或者空数组。核心代码如下：

    public static function batchFormatCoverAndPickedFootprint($userListInfo)
    {
        foreach ($userListInfo as &$element) {
            $retData[$element['userid']] = [
                'pickedFootprint' => [],
                'coverFootprint' => [],
            ];
        }
        .
        .
        .
        return $retData;
    }

传入的数组的key必然符合[$userid]['pickedFootprint']，不会存在数组下标越界

    $batchUserCover = batchFormatCoverAndPickedFootprint(xxx);
    self::batchImageFootprint($userIds, $batchUserCover);

实践2：

和实践1的底层思路是一致的，区别在于实践1是在函数内首先定义了符合规范的初始值

实践2是先处理业务逻辑，在return之前定义了符合规范的初始值

（下面代码段写了注释，重点看后半部分；联合查询那部分代码质量也不错，没省略掉，看能不能抛转引玉。）

    public static function batchFootprintList($userIds, $pageCount = 21, $batchPickedFootprints = [], $select = 'userid,id,mid,image,text,ST_Astext(picgeom) as "picgeom",poi,poiid,city,province,country,pictime')
    {
        .
        .
        .
        //联合查询
        $union = self::query()->selectRaw($select)->where('userid', $userIds[0])
            ->where('status', self::TYPE_STATUS_NORMAL)
            ->whereNotIn('mid', $batchPickedFootprints[$userIds[0]])
            ->orderBy('id', 'desc')
            ->limit($pageCount);
        //避免重复查询第一条数据
        unset($userIds[0]);
        foreach ($userIds as $userId) {
            $unionItem = self::query()->selectRaw($select)->where('userid', $userId)
                ->where('status', self::TYPE_STATUS_NORMAL)
                ->whereNotIn('mid', $batchPickedFootprints[$userId])
                ->orderBy('id', 'desc')
                ->limit($pageCount);
            $union->unionAll($unionItem);
        }
        $allUserFootprints = $union->get()->toArray();
        $res = [];
        $chunkFootprintByUserid = self::_chunkFootprintByUserid($allUserFootprints);
        // 重点在这里
        foreach ($allUserIds as $userId) {
            $list = $chunkFootprintByUserid[$userId] ?? [];
            $count = count($list);
            //以此保证不会出现数据下标越界的问题
            $res[$userId]['list'] = $list;
            $res[$userId]['more'] = $count < $pageCount ? 0 : 1;
            $res[$userId]['track'] = $count > 0 ? (string)$list[$count - 1]['id'] : '';
        }
        return $res;
    }

注意

为了行文紧凑，代码段中省略了和文章无关的代码，用竖着的三个.省略。

加载全部内容