0

我似乎找不到问题所在。

// Check that someone from this IP didn't register a user within the last hour (DoS prevention)
$query = mysqli_query($dbc, "SELECT COUNT(id) FROM accounts WHERE registration_ip = '".$_SERVER['REMOTE_ADDR']."'  AND registered > ".(time() - 3600));

if (mysqli_num_rows($query) > 0) {
    $errors[] = 'To prevent registration flooding, at least an hour has to pass between registrations from the same IP. Sorry for the inconvenience.';
}

为什么无论如何这总是返回true?即使帐户表是空的。

4

3 回答 3

6

如果我没记错的话,您的数据将始终是表示计数值的 1 行(因为您使用的是计数)。

于 2012-11-11T21:01:45.427 回答
2

即使有 0 行符合您的条件,它也会简单地返回它。

+-----------+
| COUNT(id) |
+-----------+
|         0 |
+-----------+

因为你想要的count行。它的0. 因此有一排

这是你应该如何处理它。

$row = mysqli_fetch_row($query));
$count = intval($row[0]);
mysqli_free_result($query);

if ($count > 0)
....
于 2012-11-11T21:07:08.970 回答
1

您正在检查返回的行而不是值。所以像这样检查

$row = mysqli_fetch_row($query));
$count = $row[0];

if ($count > 0)
{

}
于 2012-11-12T00:28:07.163 回答