1

我开发了一个推荐系统，登录的会员可以将推荐发送给那里的家人/朋友，以推荐他们注册。

一切正常，但昨天决定我将限制某人在 24 小时内可以发送的最大推荐。我将其限制为每天最多 3 次推荐。

我在下面发布了一些我似乎遇到问题的代码。我遇到的问题是，无论看起来如何，我都会收到一条错误消息，说我已经达到了今天的最大推荐次数。我不确定我在代码中做错了什么。

// referral query
$referral_limit = mysql_query("SELECT 'created_on' FROM 'user_referrals'
WHERE `referrer_uid` = $referrer_uid ") or die(mysql_error());

if(mysql_num_rows($referral_limit) > 0){
    while($row = mysql_fetch_assoc($referral_limit)){

            $db_time = $row['created_on'];

            if((time() - $db_time) > 86400){
                // is within 24 hours and has reached maximum daily referral allowance
                $error[] = "You have reached the maximum referrals for today.";
            }
    }
}

我确实尝试过回显 $db_time ，当我执行所有操作时，我得到的只是created_on的字段名称，而不是在这种情况下应该显示时间戳的实际值。数据库中的created_on字段包含进行推荐的时间戳，我检查它以确保推荐用户在过去 24 小时内没有进行推荐。

您还会注意到我没有添加将其限制为每天 3 个的额外位，但我不想添加该位，直到我可以先解决此问题。

数据库表如下所示：

CREATE TABLE IF NOT EXISTS `user_referrals` (
`id` int(11) NOT NULL AUTO_INCREMENT,
`referrer_uid` int(11) NOT NULL,
`recipient_username` varchar(15) NOT NULL,
`referrer_email` varchar(254) DEFAULT NULL,
`referred_id` char(32) NOT NULL,
`referred_email` varchar(254) NOT NULL,
`status` char(9) NOT NULL,
`created_on` int(11) NOT NULL,
`updated_on` int(11) DEFAULT NULL,
PRIMARY KEY (`id`),
UNIQUE KEY `referred_id` (`referred_id`),
KEY `referrer_uid` (`referrer_uid`)
) ENGINE=InnoDB  DEFAULT CHARSET=utf8 AUTO_INCREMENT=72 ;

编辑

经过一些帮助，这是我的代码。它仍然说在过去 24 小时内进行了推荐，即使没有。

我想我做错了错误检查。

$referral_limit = mysql_query("
        SELECT COUNT(*)
        FROM `user_referrals`
        WHERE `referrer_uid` = $referrer_uid
        AND `created_on` > UNIX_TIMESTAMP(DATE_SUB(NOW(), INTERVAL 1 DAY))") or die(mysql_error());

if($referral_limit > 0) {
    $error[] = "You have reached the maximum referrals for today.";
}
4

6 回答 6

9

当您也可以在 MySQL 中过滤它时，不要在 PHP 中过滤完整的 MySQL 结果集。在您的代码示例中，您可能从 MySQL 获取数千行，只是为了通过您的过滤循环运行它们。这是有史以来最难的性能杀手。而是使用更好的 SQL 语句：

SELECT COUNT(*) FROM 'referrals' WHERE `referrer_uid`=? AND created_on > UNIX_TIMESTAMP(DATE_SUB(NOW(), INTERVAL 1 DAY))

此语句仅返回给定引用者在过去 24 小时内的引用计数 (!)。

此外，您不应该直接将变量插入 MySQL WHERE name=$name，这会使您的应用程序面临无数 SQL 注入攻击。如果你不知道什么是 sql 注入，你现在就应该学习它。

于 2011-07-13T10:09:03.683 回答
2

您也可以使用 MySQL 来计算：

$referral_limit = mysql_query("SELECT COUNT(*)
                               FROM referrals
                               WHERE referrer_uid = $referrer_uid
                               AND created_on >= NOW() - INTERVAL 1 DAY
                              ") or die(mysql_error());

前提created_on是 adatetime或 a timestamp。为什么要宣布INT？

于 2011-07-13T10:09:13.340 回答
1

不要在您选择的字段名周围加上单引号，因为这将被解释为要返回的字符串。例如你需要

SELECT created_on ...

（除非您的代码中实际上有反引号，并且示例中包含的代码不正确）。

于 2011-07-13T10:07:02.317 回答
1

您的查询应该是

$referral_limit = mysql_query("SELECT `created_on` FROM `referrals`
 WHERE `referrer_uid` = $referrer_uid ") or die(mysql_error());

这是 created_on 的反引号，而不是引号。如果使用引号，则字段变为字符串，类似于SELECT 1 FROM table返回 1

于 2011-07-13T10:07:39.953 回答
1

created_onint(11) 非空， updated_onint(11) 默认空，

当然这些应该是日期时间字段？

于 2011-07-13T10:07:50.673 回答
0

更改您的查询

$referral_limit = mysql_query("SELECT 'created_on' FROM 'user_referrals'WHERE `referrer_uid` = $referrer_uid ") or die(mysql_error());

至

$referral_limit = mysql_query("SELECT `created_on` FROM `user_referrals` WHERE `referrer_uid` = $referrer_uid ") or die(mysql_error());
于 2011-07-13T10:33:23.790 回答