[英]How can I optimize this 'lottery' function in PHP?
早些时候,我在Matlab中为此类彩票功能编写了代码,只是为了测试是否可行。 但是,实际上我在PHP中需要它,因此我只是重写了代码,它的确起作用了,但是由于涉及很多循环,因此我想确保自己尽可能高效地执行它。
代码的作用是:
您可以调用函数$lotto -> type($users,$difficulty)
,它将返回两个数字。 这就是解释, $users
是在网站上注册的用户数,即可能购买票的人。 $difficulty
是介于1和10之间的数字,其中5是正常值,1是简单值,而10是困难值。 这里的困难是指匹配彩票中的所有数字有多么困难。
那么函数返回的数字是多少? 那将是$n
和$r
。 $n
是彩票上的数字数量, $r
是您可以从彩票中选择的数字数量。 例如,在英国,如果您选择6,则国家彩票有49个号码,即$n = 49
和$r = 6
。
函数如何计算这两个数字? 在英国国家彩票中,有13,983,816种可能的彩票组合。 如果我运行$lotto -> type(13983816,1)
,它将返回array(49,6)
。 基本上,它试图做到这一点,因此,票证组合与注册用户一样多。
tl; dr,代码如下:
<?php
class lotto {
public function type($users,$difficulty){
$current_r = $r = 2;
$current_n = 0;
$difficulty = ($difficulty + 5) / 10; // sliding scale from 1 - 10
$last_tickets_sold = 200; // tickets sold in last lotto
$last_users = 100; // how many users there were in the last lotto
$last_factor = $last_tickets_sold / $last_users; // tickets per user
$factor = $last_factor * $difficulty;
$users *= $factor;
while($r <= 10){
$u = 0;
$n = $r;
while($u < $users && $n < 50){
$u = $this -> nCr(++$n,$r);
}
if($r == 2){
$current_n = $n;
} elseif(abs($this -> nCr($n,$r) - $users) < abs($this -> nCr($current_n,$current_r) - $users)){
// this is a better match so update current n and r
$current_r = $r;
$current_n = $n;
}
$r++;
}
return array($current_n,$current_r);
}
private function nCr($n,$r){
return $this -> factorial($n) / (
$this -> factorial($r) * $this -> factorial($n - $r)
);
}
private function factorial($x){
$f = $x;
while(--$x){
$f *= $x;
}
return $f;
}
}
$lotto = new lotto;
print_r($lotto -> type(1000,5));
?>
我进行了快速扫描,发现了一些可以进一步优化的地方。
组合
您的算法是蛮力的,可以进一步优化
private function nCr($n,$r){
return $this -> factorial($n) / (
$this->factorial($r) * $this->factorial($n - $r)
);
}
至
function nCr($n,$r) {
$top = 1;
$sub = 1;
for($i = $r+1; $i <= $n; $i++)
$top *= $i;
$n -= $r;
for($i = 2; $i <= $n; $i++)
$sub *= $i;
return $top / $sub;
}
组合计算过多
计算组合是昂贵的。
$u = 0;
$n = $r;
while($u < $users && $n < 50){
$u = $this -> nCr(++$n,$r);
}
至
$n = $r + 1;
$u = nCr($n, $r);
while ($u < $users && $n < 50) {
$n++;
$u *= $n;
$u /= ($n - $r);
}
立即观察到您有可能被0除以错误
$last_factor = $last_tickets_sold / $last_users;
可以通过在其周围放一个简单的if语句来解决
$last_factor = ($last_users == 0) ? 0 : $last_tickets_sold / $last_users;
不管对代码进行详细检查,您确定循环都不需要继续或中断吗?
您的算法中factorial()的范围是[0,50],那么为什么不静态地对此进行预计算呢?
private static $factorial=array(1);
private static genFactorial($max) {
if( count( self::$factorial ) > $max ) return;
foreach ( range(count(self::$factorial), $max) as $n ) {
self::$factorial[$n] = $i*self::$factorial[$n-1];
}
}
现在添加一个self::genFactorial(50);
__construct()
或type()
并通过self::$factorial[$n]
替换对$this -> factorial($n)
的引用。
这只是一个快速的代码转储。 甚至没有检查过编译,因此不能原谅任何错别字等。但这是通过数组元素提取来替换函数调用(包括while循环)。
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.