后端代码规范避免数组下标越界

后端代码规范避免数组下标越界

目录

抛出问题

回答问题

举个栗子

解题思路

解题实践

实践1:

实践2:

抛出问题

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

回答问题

肯定是有的

举个栗子

比如,我有一个工具性质的方法如下:

我怎么保证 $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; }

注意

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

以上就是后端代码规范避免数组下标越界的详细内容,更多关于数组下标越界的资料请关注易知道(ezd.cc)其它相关文章!

推荐阅读