search

Home  >  Q&A  >  body text

php - 一个函数里面用一个return 好 还是多个return 好?

<?php


 function dmeo1($num)
 {
    if($num>0)
    {
        return true;
    }
    else
    {
        return false;
    }
 }




 function dmeo2($num)
 {    
    $status = null;

    if($num>0)
    {
       $status  = true;
    }
    else
    {
       $status  = false;
    }

    return $status;
 }
巴扎黑巴扎黑2819 days ago878

reply all(21)I'll reply

  • 黄舟

    黄舟2017-04-10 15:26:26

    同意@城南往事的看法,如果一个函数在第一行就可以return,那么就没必要等到最后一行,效率先不说,可读性绝对是有差别的!

    reply
    0
  • ringa_lee

    ringa_lee2017-04-10 15:26:26

    使用哪种方式能带使代码更加清晰就使用哪种,不过通常情况下使用多个return是一个更佳的选择。
    具体可以参考《重构-改善代码既有设计》9.5节的介绍。

    reply
    0
  • 高洛峰

    高洛峰2017-04-10 15:26:26

    老师说啦,程序应该一个出口,避免使用多return,break

    reply
    0
  • 巴扎黑

    巴扎黑2017-04-10 15:26:26

    如果程序中有很多防御性设计,在出错之后程序就要退出,那么就要在每次出错的时候直接return,否则就这样了:
    如果程序就是一个算法,有确定的返回值类型,那么我认为还是一个return好:

    function getValue($x) {
        $y = 0;
        if ($x>0) {
            $y = $x*$x;
        } else if ($x == 0) {
            $y = -1;
        } else {
            $y = 1/$x;
        }
        return $y;
    }
    

    reply
    0
  • 黄舟

    黄舟2017-04-10 15:26:26

    http://stackoverflow.com/questions/36707/should-a-function-have-only-one-return-statement

    reply
    0
  • 伊谢尔伦

    伊谢尔伦2017-04-10 15:26:26

    我个人觉得第一种好,如果当一个方法很长时,用return可以看出在哪里直接退出,用第二种方法的话是不是得看到最后?

    reply
    0
  • ringa_lee

    ringa_lee2017-04-10 15:26:26

     function dmeo2($num)
     {    
        $status = false;
    
        if($num>0)
        {
           $status  = true;
        }
    
        return $status;
     }
    

    我比较习惯这种写法

    reply
    0
  • 黄舟

    黄舟2017-04-10 15:26:26

    还是看具体情况吧,如果一些前提判断,自然需要提前返回(也就是第一种)。第二种应该是那种逻辑的操作,最后给一个执行的状态。

    reply
    0
  • PHP中文网

    PHP中文网2017-04-10 15:26:26

    我更倾向于第二种,用变量去控制程序返回的结果,而不是在程序各个点直接return,程序控制,怎么会进行多余的流程呢

    reply
    0
  • 迷茫

    迷茫2017-04-10 15:26:26

    其实很简单,分支越少越好

    reply
    0
  • Cancelreply