PHP 中如何优雅的处理未定义变量

[fbzl]

没有使用框架的老程序中经常会有获取$_GET['id']，而id确不存在，开启报错时，都会看到好多报错。

一种最原始的手动判断（一大坨，影响逻辑的代码）

写一个判断变量是否存在的函数（用的时候感觉怪怪的）

另一种情况是未知成员对象引用也会报错


PS：开发环境拒绝关闭报错

[shiny]

最低等：isset
中级： filter_input
终极：开发框架集成自动处理

[AlloVince]

$params = array_merge(['id' => ''], $_GET);

if ($params['id'])
...

[zhengkai]

$_GET += [
'id' => 0,
'page' => 1,
];

[zhengkai]

这种方法一般不用在 _GET _POST 上，因为网页参数要做很多处理，没有特别声明的整数要禁止负数，字符串要确保是 utf-8 的，等等

一般用在参数上， += 不仅仅是比 array_merge() 简短，还能避免一个坑。如果你的 key 恰好是 0,1,2 这种的，+= 是覆盖，array_merge 的话会给你搞出 3,4,5，3 个 key 变成 6 个 key ……

[MntCw]

@shiny 调用函数也分低高级，也是醉了。

业务逻辑理清楚，在用到相应变量时提前做处理。
三元操作符还是挺好用的。

[fbzl]

@shiny

isset 有一个坑是在赋值为 null 时 会返回 false

@zhengkai

数字key无法合并，好坑

+= 在php 5.4才开始可以用，开始用5.3.29测试 报错（囧

---

前期先用楼上的方法顶着，后期会重构自动处理，每次传值看起来都不爽

[shiny]

@MntCw 这个自己写个几年体会咯

[moro]

@

[b821025551b]

empty配合三目

[em70]

error_reporting(0)
世界不就清静了

[zhengkai]

@fbzl 5.4 才能用的是 array() 可以写成 []，+= 应该是一直能用的

[yangmls]

https://github.com/laravel/framework/blob/5.0/src/Illuminate/Support/Arr.php#L232

https://github.com/laravel/framework/blob/5.0/src/Illuminate/Support/helpers.php#L526

这两个函数是我一直用的更好的办法

[fbzl]

@zhengkai 手册上写的

array

(PHP 4, PHP 5)
array — Create an array

---

As of PHP 5.4.x you can now use 'short syntax arrays' which eliminates the need of this function.

Example #1 'short syntax array'
<?php
$a = [1, 2, 3, 4];
print_r($a);
?>

---



@b821025551b

empty()会提示'PHP Notice: Undefined index: id'，就是为了避免这个问题的，我才问的


@em70

问题描述中特别指明了，不能关闭报错

[fbzl]

@yangmls

本来准备去成熟框架中找的，但是没时间去找，之后也准备用`Laravel`重构

代码中

return value($default);

value 是自定义的吗？PHP中好像没有这个函数

[yangmls]

@fbzl 也是 laravel 的，一个处理 Closure 的函数

https://github.com/laravel/framework/blob/5.0/src/Illuminate/Support/helpers.php#L782

[yangmls]

楼上的 merge 方案无法处理多重数组的情况，而 laravel 能够简单地用 array_get($arr, 'foo.bar') 来获取

楼上的 isset 问题也考虑到了，所以用的 array_key_exists

[fbzl]

@yangmls

难道我搜索姿势不对，在Github上搜了195个结果,看了四五页就看不下去了

[yangmls]

@fbzl 搜索什么？

[micate]

自己实现一个类似 laravel 的 array_get() 方法是比较方便的，而且也不费电呀。

[geeglo]

$id = empty($_GET['id']) ? 1 : intval($_GET['id']);

这样写不会报错的呀~你是不是记错了？

[fbzl]

@geeglo 前后顺序写错了，汗

[lyragosa]

我是直接禁用所有报错信息，眼不见为净。

历史遗留没那么好修的

[fbzl]

@lyragosa
原来是5.2版本的，存在攻击漏洞，该升级了

顺便为了整理下逻辑，为自己开发方便点

看看有什么潜在漏洞后门之类的

[tcsky]

写一个全局的 array_get函数,用来处理数组.

[skydiver]

@fbzl 你记错了，empty不会报notice

[skydiver]

优雅的方法就是不要直接读取全局变量$_GET $_POST，而是用框架提供的方法。

[fbzl]

@skydiver

嗯，我逻辑写错了，直接替换的isset

现在的系统框架不完善，下一步就是重构，过渡期先找一下替代方案，还有一些小项目（随手写的）也可以用