:ecshop中红包编码生成规则分析和改进!我们知道在ecshop后台的促销管理里面,有红包的功能。的功能无非就是可以直接抵消现金的作用,ecshop红包有两类型,一种是直接发送到个人帐户上去的。就是存储在ecs_user_bonus表中的,这个一旦发送到客户的帐户上去。
那么意味着别人拿到了这个编码,也不能使用这个编码进行抵扣。另外一种ecshop红包就是线下发送的。当你得到这个编码,在红包的有效期内,可以使用该红包。
我们必须知道,ecshop的红包编码是有问题的。我们看flow.php
if (((!empty($bonus) && $bonus['user_id'] == $_SESSION['user_id']) || ($bonus['type_money'] > 0 && empty($bonus['user_id']))) && $bonus['order_id'] <= 0)
这一句表示,就算是该编码没被使用。只要是发到个人帐户去之后。即使你登陆了,也不能通过这个编码来进行使用,所以这就是一大问题。
其次,我们看ecshop红包的编码生成机制。
$num = $db->getOne("SELECT MAX(bonus_sn) FROM ". $ecs->table('user_bonus'));
$num = $num ? floor($num / 10000) : 100000;
for ($i = 0, $j = 0; $i < $bonus_sum; $i++)
{
$bonus_sn = ($num + $i) . str_pad(mt_rand(0, 9999), 4, '0', STR_PAD_LEFT);
$db->query("INSERT INTO ".$ecs->table('user_bonus')." (bonus_type_id, bonus_sn) VALUES('$bonus_typeid', '$bonus_sn')");
$j++;
}
首先我们看清楚,当你的红包编码达到了几百完以上,就容易产生重复的编码。其次,在录入红包编码的时候,根本没判断是否有重复的红包,这就造成了很大的问题,如果红包编码重复,那将是非常严重的问题
$db->query("INSERT INTO ".$ecs->table('user_bonus')." (bonus_type_id, bonus_sn) VALUES('$bonus_typeid', '$bonus_sn')");
为了解决这个问题,首先我们必须保证在录入编码之前,确保该编码没重复,所以必须判断数据库中是否有该编码。那么我们就必须对bonus_sn 来进行扫描判断,那么多数据如何提高扫描速度呢。
我们必须在bonus_sn上建立索引。其次,我们可以修改红包编码的规则。既然是十位,我们就可以用我们自己的办法来得到。
static function GenSecret($len=6, $type=self::CHAR_WORD)
{
$secret = '';
for ($i = 0; $i < $len; $i++) {
if ( self::CHAR_NUM==$type ){
if (0==$i) {
$secret .= chr(rand(49, 57));
} else {
$secret .= chr(rand(48, 57));
}
}else if ( self::CHAR_WORD==$type ){
$secret .= chr(rand(65, 90));
}else{
if ( 0==$i ){
$secret .= chr(rand(65, 90));
} else {
$secret .= (0==rand(0,1))?chr(rand(65, 90)):chr(rand(48,57));
}
}
}
return $secret;
}
这个方法,要强过ecshop红包默认的编码规则。