V2EX = way to explore
V2EX 是一个关于分享和探索的地方
Sign Up Now
For Existing Member  Sign In
darasion
V2EX  ›  PHP

两种代码风格, 你喜欢哪一种?

  •  
  •   darasion · Nov 6, 2012 · 6998 views
    This topic created in 4927 days ago, the information mentioned may be changed or developed.
    一、蛇吞大象
    if (is_array($params)) {
    foreach ($params as $param) {
    //...
    }
    }

    二 、蚂蚁啃骨头
    $params = is_array($params) ? $params : array();
    foreach ($params as $param) {
    //...
    }
    17 replies    1970-01-01 08:00:00 +08:00
    enj0y
        1
    enj0y  
       Nov 6, 2012
    see it used for what and think about which.
    azoon
        2
    azoon  
       Nov 6, 2012
    我用第一种。第二种有人这样写?
    sethverlo
        3
    sethverlo  
       Nov 6, 2012
    第一种我喜欢把 {} 都另起一行……
    TheOnly92
        4
    TheOnly92  
       Nov 6, 2012
    有些时候我会这么写
    foreach ((array) $params as $param) { ... }

    但是当你必须这么做代码很容易出问题,
    最好能保证 $params 肯定是 array()
    tux
        5
    tux  
       Nov 6, 2012
    if (is_array($params))
    {
    foreach ($params as $param)
    {
    //...
    }
    }
    我这样算什么啊
    enj0y
        6
    enj0y  
       Nov 6, 2012
    第一种如果后面没有else逻辑,会只有$params为数组时才执行。第二种会将非数组的$params重置为空数组,有可能会。。。。
    Sunyanzi
        7
    Sunyanzi  
       Nov 6, 2012
    我的方式 ... 和第一种差不多是一个道理 ...

    if ( is_array( $params ) ) foreach ( $params as $param ) {

    } else {

    }

    顾及简洁以及减少性能浪费 ... 避免毫无意义的赋值和 foreach ...
    imcj
        8
    imcj  
       Nov 7, 2012
    var i = 0, size = params.length;
    for ( ; i < size; i++ ) {
    }
    fmfsaisai
        9
    fmfsaisai  
       Nov 7, 2012
    对于数组这种大数据尽量减少只用三元式:
    http://www.laruence.com/2011/11/04/2258.html
    binjoo
        10
    binjoo  
       Nov 7, 2012
    有时候的简洁,看着更加蛋疼。
    darklowly
        11
    darklowly  
       Nov 22, 2012
    @Sunyanzi 这个是玩小聪明的技巧。一般是适得其反。
    mygoare
        12
    mygoare  
       Nov 23, 2012
    第二种 可能foreach空数组?
    mygoare
        13
    mygoare  
       Nov 23, 2012
    好吧,我二了。。。sorry
    aveline
        14
    aveline  
       Nov 23, 2012
    以前很喜欢第一种 ... 不过看起来是很漂亮 ... 但是执行起来 ... 只能呵呵了 ...

    简单写了个测试脚本 ... 每种调用 200000 次, 结果如下:

    test_1: 1.2074031829834
    test_2: 1.2558000087738

    明显第一种性能更好 ...

    http://gist.github.com/4132644
    aveline
        15
    aveline  
       Nov 23, 2012
    忘了补充下 ... 测试环境是

    "PHP 5.3.15 with Suhosin-Patch (cli) (built: Aug 24 2012 17:45:44)
    Copyright (c) 1997-2012 The PHP Group
    Zend Engine v2.3.0, Copyright (c) 1998-2012 Zend Technologies
    with Xdebug v2.1.3, Copyright (c) 2002-2012, by Derick Rethans"
    chisj
        16
    chisj  
       Nov 23, 2012
    我习惯这样的,因为感觉降低了一点复杂度,缩进太多的代码我看不习惯。。。。还有我喜欢用if处理异常而不是正常逻辑^.^,有和我一样的筒靴吗?
    if (!is_array($params)) {
    return;
    }
    foreach ($params as $param) {
    //...
    }
    IFoon
        17
    IFoon  
       Dec 20, 2012
    第一种比第二种性能高。
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Solana   ·   2912 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 82ms · UTC 06:44 · PVG 14:44 · LAX 23:44 · JFK 02:44
    ♥ Do have faith in what you're doing.