如何优化这段if else多层嵌套？

Posted 2023-03-21

一个仓库调拨的业务逻辑是这样的（如图），怎么能优化的减少一些嵌套？求大神给个思路

我喜欢下面的逻辑结构，供参考：

<?php

$errStr = "";//错误消息

$outStr = "";//执行结果

if ($in_deport==\'\') $errStr .= "错误1";

if ($out_deport==\'\') $errStr .= "错误2";

if ($in_deport==out_deport) $errStr .= "错误3";

if ($errStr==\'\')

$sql=\'..\';

if (!$res) $errStr .= "错误4";

if ($errStr==\'\')

$sql=\'..\';

if (...) $errStr .= "错误5:库存不足";

if ($errStr==\'\')

$sql=\'..\';

if (...) $errStr .= "错误6:不存在";

if ($errStr==\'\')

$sql=\'..\';

if (...) $errStr .= "错误7:写入数据库失败";

else $outStr .= "写入数据库成功";

echo<<<END

<div>$errStr</div>

<div>$outStr</div>

END;

参考技术A 如果前面发生异常，应该直接返回操作的。比如做提示
if(!$res)

echo '产品不存在...';
return '按你的返回需求';

$sql = "检查产品是否库存足够";
$res1 = '';
if(!$res1)

echo '库存不足';
return '按你的返回需求';

$sql = "检查产品是否有入库仓";
$res2 = '';
if(!$res2)

//todo insert

else

//todo update

阿昌之丑陋代码优化多层嵌套if-else

前言

平常我们在实际业务开发中，如果有多层条件判断的情况下，一般像阿昌我这种菜鸡，都会直接想到if-else的多层嵌套去完成这个业务的开发，但是如果业务的逻辑很复杂庞大，代码的可读性就会很差，这里在记录的小破站上学习到的一种方法，通过do-while的方式。

这里做记录，以免我这小脑袋忘记和方便回想起来。

情景模拟

• 这里模拟3层if-else的多层嵌套

@Test
void doHandler() 
    Boolean isHuman = true;
    Boolean isMan = true;
    Boolean isSixAge = true;
    Boolean flag = false;

    if (isHuman) 
        if (isMan) 
            if (isSixAge) 
                flag = true;
             else 
                System.out.println("他不是6岁");
            
         else 
            System.out.println("他不是男性");
        
     else 
        System.out.println("他不是人类");
    
    System.out.println(flag);

---

正文

上面就出现了3层的嵌套，这里就会让可读性很差。

那这里提供一种将多层的嵌套，变成一层的if-else

• 采用do-while

@Test
void test()
    Boolean isHuman = true;
    Boolean isMan = true;
    Boolean isSixAge = false;
    Boolean flag = false;

    do 
        if (!isHuman)
            System.out.println("他不是人类");
            break;
        
        if (!isMan)
            System.out.println("他不是男性");
            break;
        
        if (!isSixAge)
            System.out.println("他不是6岁");
            break;
        
        flag=true;
    while (false);
    System.out.println(flag);

上面这样子就可以解决多层的if-else多层嵌套的问题，重构成一层

---

就是这次记录的全部内容了，感谢你能看到这里!!! 小破站地址