多个组合逻辑判断是 if else 更合理还是 goto 最后输出之前更合理?

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

比方有如下两种例子

第一种写法

function a () {
    if ($is_pass_1 ) {
        if ($is_pass_2 ) {
            if ($is_pass_3 ) {
                //...
                //业务代码处理
                return result;
            }
        }
    }
}

第二种写法

function b () {
    if (!$is_pass_1 ) {
        //逻辑处理
        goto xxxxx;
    }
    if (!$is_pass_2 ) {
        //逻辑处理
        goto xxxxx;
    }
    if (!$is_pass_3 ) {
        //逻辑处理
        goto xxxxx;
    }
    //业务代码处理

    xxxxxx: #goto 到的位置
    // result =
    return result 
}

Supplement 1 · Sep 1, 2015

之前我可能描述的不清楚, 现在补充下

$is_pass_1, $is_pass_2, $is_pass_3 等不是平级的，自然是不适用 switch
每个判断之后可能都会有部分逻辑处理
if 之后同样存在 else 语句
目前遵循同一个方法，只有最后一个出口
业务复杂后，虽然提取了公共的方法，但依然存在很多判断组合

我调整后的代码结构

第一种

function a() {
    if($is_pass_1) {
        //部分逻辑
        if($is_pass_2) {
            //部分逻辑
            if($is_pass_3) {
                //...
                //业务代码处理
                result ＝ 'balbalbalbal';
            } else {
                //else 逻辑
            }
        } else {
            //else 逻辑
        }
    } else {
        //else 逻辑
    }
    return result;
}

Supplement 2 · Sep 1, 2015

调整后的第二种写法

function b() {
    if(!$is_pass_1) {
        //逻辑处理
        goto xxxxx;
    }
    //逻辑处理
    if(!$is_pass_2) {
        //逻辑处理
        goto xxxxx;
    }
    //逻辑处理
    if(!$is_pass_3) {
        //逻辑处理
        goto xxxxx;
    }
    //逻辑处理

    //业务代码处理

    xxxxxx: #goto 到的位置
    // result =
    return result 
}

61 replies • 2015-09-03 13:41:24 +08:00

est

Sep 1, 2015

php 里难道没有 else if ？

niinaranpo

Sep 1, 2015

为什么不用 switch

feiyuanqiu

Sep 1, 2015

逻辑运算符哭晕在厕所...
function a () {
if ($is_pass_1 && $is_pass_2 && $is_pass_3 ) {

}
}

luban

Sep 1, 2015

这 2 段代码不等价，实现功能不一样

michaelzlies

Sep 1, 2015

l1905

Sep 1, 2015

@est 这些不是平级的 if else

l1905

Sep 1, 2015

@feiyuanqiu 每个判断条件后都会有部分逻辑处理，上面我可能写的不太清楚

huijiewei

Sep 1, 2015

一个很方便的准则就是

尽早返回。。。

业务逻辑再理一理吧。 goto 这种东西不用也罢

poorguy

Sep 1, 2015

不清楚你具体业务，不过应该可以错误的先退出
```php
function b () {
if (!$is_pass_1 ) {
//不成立的逻辑处理
return;
}
//成立的逻辑处理
```

morefreeze

Sep 1, 2015

你可以用

```php
do{
if (!$cond1 ) break;
if (!$cond2 ) break;
while (0 );
```

来避免掉 goto ，虽然你现在用 goto 都只是往下跳的，但你不保证之后别人修改你代码会怎么样理解这段呢

l1905

Sep 1, 2015

@huijiewei 呃呃呃。。 goto 到最后应该就是遵循的尽早返回。没找到 goto 在目前逻辑下太不合理的地方

l1905

Sep 1, 2015

@poorguy 目前遵循的是复杂逻辑下只有一个出口

otakustay

Sep 1, 2015

公用逻辑抽成另一个函数在各分支里调用就行，别用 goto

qian19876025

Sep 1, 2015

合适就行为嘛一定要与趋势作对

alphonsez

Sep 1, 2015

goto 挺好的，尤其 C 错误处理的时候。

ether

Sep 1, 2015

ibremn

Sep 1, 2015

举个 C 的例子吧。。
下面的函数是把图片数据解压成 iOS 可以直接显示的 Bitmap 。。。

第一种是遵循了"尽早返回"的原则。。。

第二种是用了 goto 。。。

在某些情况下，适当地用 goto 还是不错的，只要注意别出现 goto fail; goto fail; 这种 bug 就好蛤蛤蛤。。。

pathletboy

Sep 1, 2015

可以用 do while (0 );就别用 goto 了。

flowfire

Sep 1, 2015

@pathletboy do while (0 ) 什么鬼

ljbha007

Sep 1, 2015

switch case 就是为这种情况准备的啊

FrankFang128

Sep 1, 2015

你应该优化逻辑，不要出现这么多 if

hitmanx

Sep 1, 2015

想起了 Dijkstra 大神的那篇可能是最著名的文章<Go To Statement Considered Harmful>

zacard

Sep 1, 2015

优化逻辑，优先返回。

kobe1941

Sep 1, 2015

从开始学编程，所有的人包括老师和书籍都不让用 goto

msg7086

Sep 1, 2015

@flowfire do while 0 给你机会用 break 。

另外 Ruby 里推荐的方式是用 throw/catch 来解决。 C++/Java/PHP 的话也可以考虑用这种方法，比较干净。
当然最好的方法肯定是拆函数，只有实在不适合拆函数的时候才考虑 goto 。

narcotics

Sep 1, 2015

@msg7086 利用异常机制表达逻辑或许不是个好主意？

akagi

Sep 1, 2015

个人觉得可以试试表驱动？

21grams

Sep 1, 2015

goto 也可以适当的用一下，怕啥，不就是只恐龙嘛。

msg7086

Sep 1, 2015

@narcotics Ruby 里的异常是 raise/rescue 。 throw/catch 就是用来抛球接球的。

另外其实这种结构也算是异常。

undef404

Sep 1, 2015

@msg7086

c++里面不推荐用异常来做逻辑控制

l1905

Sep 1, 2015

@kobe1941 我也"从开始学编程，所有的人包括老师和书籍都不让用 goto" ，但遇到具体业务逻辑的用它的时候，却没想到强有力不用它的理由，或者说是列不出用 goto 有哪些不能容忍的缺点

hitmanx

Sep 1, 2015

@l1905 如果功能简单的话确实没什么。不过假如未来这个函数的功能随着迭代更新变得越来越长，越来越复杂，加之如果后续维护的人并不是当初编写者时，就很容易出各种各样的问题，因为 goto 语句破坏了结构性。即使你现在很清楚你在干什么，也不保证后续模仿你的风格继续增加 goto 语句的人也像你这样清晰，一点小看法。

leoe

Sep 1, 2015

我觉的这个逻辑用 goto 处理很好啊，代码更简洁、更易读。 linux 内核里有很多 goto 啊，尤其是逻辑块多，函数返回前还需要一些处理时，用 goto 很方便。

computeramber

Sep 1, 2015

测试 crawl ＝＝！

cxbig

Sep 1, 2015

goto 在 php 手册里已经明确标注不推荐使用了。
http://php.net/manual/control-structures.goto.php

clearPHP:
https://github.com/dseguy/clearPHP/blob/master/rules/no-goto.md

twi3325831

Sep 1, 2015

楼主试过这种写法没

function b () {
do {
if (!$is_pass_1 ) {
//逻辑处理
break;
}
//逻辑处理
if (!$is_pass_2 ) {
//逻辑处理
break;
}
//逻辑处理
if (!$is_pass_3 ) {
//逻辑处理
break;
}
//逻辑处理

//业务代码处理
} while (0 );

// 错误处理
// result =
return result;
}

dorentus

Sep 1, 2015

说句题外话， Swift 2 为了支持尽早返回，提供了 guard 关键字来尽早返回，然后提供了 defer 关键字来解决返回导致的无法集中释放资源的问题。

asj

Sep 1, 2015 via iPad

有些时候，看起来似乎今天一个程序员为软件写了 10 行代码，
其实他写了-1000 行代码

2015813

Sep 1, 2015 via Android

大师云:果断 if ，抛弃 goto ，它会让你晕头转向。

raincious

Sep 1, 2015

看了楼主的 Append 之后再看原来的，觉得一二两种方案的区别基本上就是第二种会被打更惨而已……

重构成这样应该就好多了：
https://gist.github.com/raincious/c2845e414d62832db861

cpper

Sep 1, 2015

对于 if else 这种判断的，请进行读表形式处理。代码大全有类似介绍

flowfire

Sep 2, 2015

@msg7086 不是应该是 do while 1 么、、、

msg7086

Sep 2, 2015

@flowfire do while 1 就变成死循环了。

vibbow

Sep 2, 2015

try
{
if ( ! aaaa )
{
throw new Exception ('aaa');
}

if ( ! bbb )
{
throw new Exception ('bbb');
}
}
catch (Exception $e )
{

}

flowfire

Sep 2, 2015

@msg7086
do while 0
就是只执行一次没什么卵用
do while 1
就是无限循环知道满足条件用 break 跳出。。
有哪里不对么。。

zhczhy

Sep 2, 2015

允许用 goto 就 goto ，不允许时大部分情况下都可以 do { ... } while (0 )
V2 上还是得多一些这样的纯技术讨论帖

bdnet

Sep 2, 2015

有个叫 switch case

更好的方式可以用字典，如果所用编程语言支持委托的话。

goto 很容易造成死循环

msg7086

Sep 2, 2015

@flowfire 你自己好好动脑筋想想吧……不想多吐槽了。

romisanic

Sep 2, 2015

@ibremn
不过不管公司要求还是自己写代码，还是尽量避免在一个方法中有过多的 return 。。。

mcfog

Sep 2, 2015

别说 goto 了，我连 else 都不想看到，尤其是一眼看不完的()里的条件或超过 3 行的{}里的代码。战术上提前 return ，抽函数，查表什么的方法有的是，战略上说穿了就是自己其实压根没想清楚就在写代码了，自然写的一手好面条

Mysdes

Sep 2, 2015

在我还是个程序猿的时候，老师千叮咛万嘱咐，如果没人把刀架你脖子上，能别 goto 就别 goto ，哈哈哈

lizhiqing1996

Sep 2, 2015

@flowfire do while 0 只执行一次，满足一定条件可以退出 while 代码块，从而实现了 goto 的功能。如果一直不退出 while 代码块，则 while 代码块里的语句只会执行一次，效果就和没用 while 一样，不会影响代码运行

@msg7086 不知道我说得对不

lizhiqing1996

Sep 2, 2015

我怎么看都感觉你的两串代码实现的是不一样的功能，不管是修改前或修改后...

mornlight

Sep 2, 2015

@romisanic 这个很奇怪，许多地方是推荐多用 return 少用嵌套 if else 来明确表示方法结束和返回值，代码可读性更高。

myv2ex

Sep 2, 2015

尝试利用面向对象的思想， 3 个条件非平级，则必定存在引用包含关系，那么可通过定义基本模型关系，再通过接口注入模型对象到处理模型的操作对象，操作对象中根据业务逻辑定义处理各模型的不同逻辑完成继承关系，通过覆写操作取代过程化的代码处理机制，后期随着业务模型对象的增加，也仅仅是完善模型接口定义及方法而已，对今后代码发展具有积极作用。