Php 如何降低函数的圈复杂度，该函数返回的值依赖于3个布尔的笛卡尔乘积？

如何降低函数的圈复杂度，该函数返回的值依赖于3个布尔的笛卡尔乘积？如何使以下代码看起来更干净

这是一个学校项目，并不是作业本身的要求，但我通常发现自己编写的函数依赖于一个有时相当复杂的真值表。我认为这不是最好的方法

    public function getDiscount( $values ) {

    $res       = new stdClass();
    $res->code = 400;

    if ( ! is_bool( $values['new_customer'] ) || ! is_bool( $values['loyalty_card'] ) || ! is_bool( $values['coupon'] ) ) {
        $res->data = "Missing inputs";

        return $res;
    }

    if ( $values['new_customer'] == true && $values['loyalty_card'] == true && $values['coupon'] == true ) {
        $res->data = "Invalid input";

        return $res;
    }

    if ( $values['new_customer'] == true && $values['loyalty_card'] == true && $values['coupon'] == false ) {
        $res->data = "Invalid input";

        return $res;
    }

    $res->code = 200;

    if ( $values['new_customer'] == true && $values['loyalty_card'] == false && $values['coupon'] == true ) {
        $res->data = 20;

        return $res;
    }

    if ( $values['new_customer'] == true && $values['loyalty_card'] == false && $values['coupon'] == false ) {
        $res->data = 15;

        return $res;
    }

    if ( $values['new_customer'] == false && $values['loyalty_card'] == true && $values['coupon'] == true ) {
        $res->data = 30;

        return $res;
    }

    if ( $values['new_customer'] == false && $values['loyalty_card'] == true && $values['coupon'] == false ) {
        $res->data = 10;

        return $res;
    }

    if ( $values['new_customer'] == false && $values['loyalty_card'] == false && $values['coupon'] == true ) {
        $res->data = 20;

        return $res;
    }

    if ( $values['new_customer'] == false && $values['loyalty_card'] == false && $values['coupon'] == false ) {
        $res->data = 0;

        return $res;
    }

    $res->code = 400;
    $res->data = "Invalid input";
    return $res;

}

---

这是一种语言不可知范式，您会发现不同的语言将具有不同的内置函数和运算符，以便于处理真值表

我最喜欢的是C#7.0 switch语句中的新模式匹配

首先，确定你的真理表

CASE    New Customer    Loyalty     Coupon  Output
  1        Yes            Yes        Yes    'Invalid Input'
  2        Yes            Yes        No     'Invalid Input'             
  3        Yes            No         Yes      20
  4        Yes            No         No       15                
  5        No             Yes        Yes      30
  6        No             Yes        No       10                
  7        No             No         Yes      20
  8        No             No         No       0             

我们立即可以看到，情况1和2不依赖于优惠券条件，因此这是您的第一次优化。 第二个需要注意的条件是，没有新客户是真实的，并且他们有一张忠诚卡的情况，这是有意义的。 从这里开始，在代码和编程中最简单的方法就是使用嵌套分支，这样我们只对每个条件检查进行一次评估

public function getDiscount( $values ) {

    $res       = new stdClass();
    $res->code = 400;

    if ( ! is_bool( $values['new_customer'] ) || ! is_bool( $values['loyalty_card'] ) || ! is_bool( $values['coupon'] ) ) {
        $res->data = "Missing inputs";

        return $res;
    }

    if ( $values['new_customer'] == true && $values['loyalty_card'] == true ) {
        $res->data = "Invalid input";

        return $res;
    }

    $res->code = 200;

    // Check New Customer conditions
    if ( $values['new_customer'] == true) {
        if( $values['coupon'] == true ) 
            $res->data = 20;
        else
            $res->data = 15;
    }

    // Check existing customer conditions
    else {

        // Has Loyalty Card
        if ( $values['loyalty_card'] == true ) {
            // Has Coupon
            if( $values['coupon'] == true ) 
                $res->data = 30;
            else
                $res->data = 10;
        }

        // No Loyalty Card
        else {
            // Has Coupon
            if( $values['coupon'] == true ) 
                $res->data = 20;
            else
                $res->data = 0;
        }
    }

    // Don't need a default fail condition here, because we have covered all possible combinations     
    return $res;

}

使用加法可能更容易，真值表是表示下一个代码块的好方法：

CASE    New Customer    Loyalty     Coupon  Output
  1        Yes            Yes        Yes    'Invalid Input'
  2        Yes            Yes        No     'Invalid Input'             
  3        Yes  +15       No         Yes +5   =20
  4        Yes  +15       No         No       =15               
  5        No             Yes +10    Yes +20  =30
  6        No             Yes +10    No       =10               
  7        No             No         Yes +20  =20
  8        No             No         No       0             

public function getDiscount( $values ) {

    $res       = new stdClass();
    $res->code = 400;

    if ( ! is_bool( $values['new_customer'] ) || ! is_bool( $values['loyalty_card'] ) || ! is_bool( $values['coupon'] ) ) {
        $res->data = "Missing inputs";

        return $res;
    }

    if ( $values['new_customer'] == true && $values['loyalty_card'] == true ) {
        $res->data = "Invalid input";

        return $res;
    }

    $res->code = 200;
    $res->data = 0;

    // Check New Customer conditions
    if ( $values['new_customer'] == true) {
        $res->data += 15;
        if( $values['coupon'] == true ) 
            $res->data += 5;
    }

    // Check existing customer conditions
    else {

        // Has Loyalty Card
        if ( $values['loyalty_card'] == true )
            $res->data += 10;

        // Has Coupon
        if( $values['coupon'] == true ) 
            $res->data += 20;
    }

    // Don't need a default fail condition here, because we have covered all possible combinations     
    return $res;

}

---

有许多方法可以剥除这只猫的皮肤：）上面的代码示例强调了如何使用分支逻辑只对每个条件求值一次。在这种情况下，评估某些条件可能会对性能产生严重影响，因此您只需要评估一次。

您说得对，该代码是详细描述您需要的内容的一种方式，为我省去了一些麻烦，请编辑您的问题以包括您的真值表，这样我们就不会对您的逻辑产生错误的假设。谢谢您提供非常详细的答案：）