优化后的代码:

if ($lotteryType === 1) {
    //所有奖品随机roll给一名用户
    $randUser = $this->randUser($unawardUserNum);
    $unawardUserNum = [$randUser];
}

$rollUserNum = array_fill(0, min(count($unsentItemData), count($unawardUserNum)), 1);

if ($lotteryType !== 2) {
    $rollUserNum = number_avg(count($unsentItemData), count($unawardUserNum));
}

优化思路:将每个分支的公共代码提取出来,避免重复执行。将分支中唯一不同的部分放在分支外部执行。

优化下面的代码 if $lotteryType === 1 所有奖品随机roll给一名用户 $randUser = $this-randUser$unawardUserNum;

原文地址: https://www.cveoy.top/t/topic/fjYD 著作权归作者所有。请勿转载和采集!

免费AI点我,无需注册和登录