PHP if 判断怎么简写？

This topic created in 3729 days ago, the information mentioned may be changed or developed.

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

$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';

有没有什么黑科技呀？

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

Supplement 1 · Mar 26, 2016

目前比较好的方法：

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

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

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

29 replies • 2016-03-31 09:10:19 +08:00

imydou

Mar 26, 2016

if(true):

endif;

snailsir

Mar 26, 2016 via iPhone

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

snailsir

Mar 26, 2016 via iPhone

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

漏了个括号

iyaozhen

Mar 26, 2016

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

iyaozhen

Mar 26, 2016

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

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

abelyao

Mar 26, 2016

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

tuuuz

Mar 26, 2016

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

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

不喜勿喷😀

hellov22ex

Mar 26, 2016

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

iyaozhen

Mar 26, 2016 via Android

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

iyaozhen

Mar 26, 2016 via Android

@tuuuz 哈哈，这个赞同。

xi4oh4o

Mar 26, 2016

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

abelyao

Mar 26, 2016

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

Sunyanzi

Mar 26, 2016

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

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

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

iyaozhen

Mar 26, 2016

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

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

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