PHP if 判断怎么简写？

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

这是一个创建于 3162 天前的主题，其中的信息可能已经有所发展或是发生改变。

最近经常写这样的代码，感觉很啰嗦：

$array = [];

$string = "0...balabala";
// $start 和 $length 变量值不固定
$str = substr($string, 1, 10);
if ($str === false) {
    $str = '123';
}

$array[] = $str;

三目运算？

$str = substr($string, 1, 10) === false ? '123' : substr($string, 10, 10);

substr 两次影响效率？

($str = substr($string, 1, 10)) === false ? $str = '123' : null;

这样？（错误）

$str = substr($string, 0, 1) ?: '123';

有没有什么黑科技呀？

业务都还没写完，天天想这些。感觉我这是病，得治。

第 1 条附言 · 2016-03-26 23:27:08 +08:00

目前比较好的方法：

($str = substr($string, $start, $length)) !== false || $str = '123';

// 条件：$start >= 0 $length > 0 符合一般的应用场景，推荐
$array[] = isset( $string[$start] ) ? substr( $string, $start, $length ) : '123';

然后如果是公司内部协作开发的话还是「不要标新立异」。

29 条回复 • 2016-03-31 09:10:19 +08:00

imydou

2016-03-26 20:45:18 +08:00

if(true):

endif;

snailsir

2016-03-26 20:58:56 +08:00 via iPhone

$str = substr($string, 21, 10)) || $str = '123';

snailsir

2016-03-26 21:00:14 +08:00 via iPhone

（$str = substr($string, 21, 10)) || $str = '123';

漏了个括号

iyaozhen

2016-03-26 21:01:27 +08:00

@snailsir $str 最终有可能等于 true

iyaozhen

2016-03-26 21:05:46 +08:00

@snailsir 貌似不行，和 $str = substr($string, 0, 1) ?: '123'; 一样有弊端（ 0 == false ）

还是要这样：($str = substr($string, 0, 1)) !== false || $str = '123';

abelyao

2016-03-26 21:23:49 +08:00

$array[] = substr($string, $start, $length) or $array[] = '123';
这样呢？

tuuuz

2016-03-26 21:46:04 +08:00

我觉得就按啰嗦的那种方式写吧～

感觉可读性会比较好，如果以后有同事需要维护你的代码，他会感谢你的

不喜勿喷😀

hellov22ex

2016-03-26 21:49:32 +08:00

最正式或者说最通俗易懂的写法

iyaozhen

2016-03-26 21:53:45 +08:00 via Android

@abelyao 这样有个坑。$array 有可能增加两个。😂 中间变量还是要的

iyaozhen

2016-03-26 21:57:42 +08:00 via Android

@tuuuz 哈哈，这个赞同。

xi4oh4o

2016-03-26 22:00:19 +08:00

简单返回值的话用 expr1 ?: expr3
剩下的 if else 不要少，只有 if 也不好

abelyao

2016-03-26 22:50:40 +08:00

@iyaozhen 我晕… 我就是帮你简化到最简单的程度，你增加一个中间变量也不影响这个写法成不成立嘛，那就改成： $str = substr($string, $start, $length) or $str = '123';
关键是这写法有没有其它坑，不懂你的实际应用情况~

Sunyanzi

2016-03-26 23:02:54 +08:00

为什么依赖 substr 的返回值 ..? 在 $start 非负 $length 恒正的情况下我会这么写 ...

$array[] = isset( $string[1] ) ? substr( $string, 1, 10 ) : '123';

对的我就是能不使用额外变量就不使用额外变量的三元狂魔 ...

iyaozhen

2016-03-26 23:04:14 +08:00

@abelyao 额，不好意思。我没别的意思。只是我也那边试过，发现不行。

嗯这个目前看来是最优的，也便于阅读。但还是有点小问题，#5 楼。

substr($string, $start, $length)有可能返回 0 ，布尔值是 false 。