首先祝我今天生日快乐!
有时候数据表里所存储的数据结构并不是我们想要输出到前端的数据格式。
所以我们需要在给前端输出数据之前重构一下我们的数据格式。
最近新接手一个laravel7写的一个项目(PHP7.3),在我一如既往的使用foreach()的时候却出现了一个问题
报错信息:Indirect modification of overloaded element of App\\Models\\User has no effect
我的错误代码是这样写的
if ($data['dataList']) {
$cates = (new RedpackCate())->get()->toArray();
foreach ($data['dataList'] as &$datum) {
if ($datum['games']) {
$games = json_decode($datum['games']);
foreach ($games as $game) {
foreach ($cates as $cate) {
if ($game == $cate['id']) {
$datum['game_names'][]= $cate['name'];
}
}
}
}
unset($datum['games']);
}
}
以前在用codeigniter框架的时候,我一直这么写,没有任何问题。是可以运行的,其实就是给下面这个数组添加一个
$data['dataList']
键名为
game_names
的数组,这个是根据数据表中game所存的序列化的字符串,json_decode后通过查询另一个表中数据map出来的,之前是没有的
{
"status_code":200,
"message":"请求成功",
"data":{
"count":13015,
"dataList":[
{
"id":1000013104,
"openid":"xxxxxx",
"baidu_openid":null,
"user_name":"Q2FnZUxlZQ==",
"avatar_url":"https:xxxxxxxxxxxxx",
"phone":"xxxxxxxxxxxx",
"levels_id":1,
"created_at":"2021-07-26 10:00:44",
"register_ip":"127.0.0.1",
"last_login_at":"2021-07-29 14:56:48",
"last_login_ip":"127.0.0.1",
"status":1,
"sign_days":1,
"score":95,
"last_sign_at":"2021-07-29",
"invit_code":"mYwvVx9",
"be_invit_code":"",
"updated_at":"2021-07-29 17:53:11",
"is_handsel":2,
"address":null,
"wallet":"0.00",
"games":"[\"1\",\"3\",\"5\",\"2\"]",
"game_names":[
"绝地求生",
"红包奖品"
],
"black":null
}
]
}
}
为什么会出现这个问题研究了几个小时,首先是根据报错信息
Indirect modification of overloaded element of App\\Models\\User has no effect
,报的是model里的错误,他说我
间接修改App\\Models\\User的重载元素无效
这句话我并不理解是什么意思。但是我本能的思考了一下是不是laravel7的一个机制,由于之前接触过mangento2框架,所以我对外国人设计框架时热衷配置,极端的封装编程思想有一点意识。我就去看了一下App\\Models\\User的这个文件。
我瞄上了$fillable这个属性
PS:突然接手项目比较急,我并没有仔细研究过laravel7的相关文档
/**
* The attributes that are mass assignable.
*
* @var array
*/
protected $fillable = [
这里填数据表里的字段
];
后来发现不是的,于是我开始百度,遇到了这个帖子
我当时很晚,所以粗略的看一下,对他里面所谓错误代码和回答者的修改代码进行了代码比对,看看是什么情况。我试了一下虽然不报错,但是并没有办法实现我的数据重构的逻辑,所以并不适用。
本来打算去laravel社区去提问在回答了提问前的20个问题之后我丧失了提问的激情,就开始回到代码中一步一步的打印。laravel报的错误其实我没看出来对我解决这个问题带来什么帮助。
可行的代码如下
if ($data['dataList']) {
$cates = (new RedpackCate())->get()->toArray();
foreach ($data['dataList'] as $datum) {
$datum['game_names'] = [];
if ($datum['games']) {
$games = json_decode($datum['games']);
foreach ($games as $game) {
foreach ($cates as $cate) {
if ($game == $cate['id']) {
$datum['game_names']= array_merge($datum['game_names'],[$cate['name']]);
}
}
}
}
unset($datum['games']);
}
}
总之,我的思路就是
如果取地址符&无法直接创造一个新的键我就在赋值之前,先定义这个键名
$datum['game_names'] = [];
然后再
$datum['game_names']= array_merge($datum['game_names'],[$cate['name']]);
就是这一步解决了这个问题,我并不了解这跟我之前写的
$datum['game_names'][]= $cate['name'];
有什么区别,因为我在这样写之前是在foreach里用的引用地址符的
foreach ($data['dataList'] as &$datum) {
虽然我通过以上先定义键名的方式解决了,但是我并不知道这是什么机制的导致的,是php7.3的缘故还是laravel7框架自身的缘故。
如果有哪位知道,还请在帖子下面不吝赐教!