优化下面的代码 if $lotteryType === 1 所有奖品随机roll给一名用户 $randUser = $this-randUser$unawardUserNum;
优化后的代码:
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));
}
优化思路:将每个分支的公共代码提取出来,避免重复执行。将分支中唯一不同的部分放在分支外部执行。
原文地址: https://www.cveoy.top/t/topic/fjYD 著作权归作者所有。请勿转载和采集!