Question

我有一个评级系统，它使用以下公式生成平均评级：

<（>（旧评级*旧时间金额）+新评级）/新评级金额

然而，如果目前的评级是3，并且它被评为一次，当我评价为3时，它表示新的评级为2.5

这里有什么错误？这是完整的代码。

<?php
session_start();
include("lib/db.php");
$db = new DBConnect;
if(isset($_POST['rating']) && is_numeric($_POST['rating']) && is_numeric($_POST['story']))
{
    if($_POST['rating'] > 5 || $_POST['rating'] < 1){die("INVALID RATING");}
    $rating = mysql_real_escape_string($_POST['rating']);
    $story = mysql_real_escape_string($_POST['story']);
    $c = $db->query("SELECT * FROM cdb_stories WHERE id=$story");
    $c = mysql_fetch_array($c);
    $u_name = mysql_real_escape_string($_SESSION['logged_in']);
    $uid = $db->query("SELECT id FROM cdb_users WHERE username='{$u_name}'");
    if(mysql_num_rows($uid) < 1){die("NOT LOGGED IN");}
    $uid = mysql_fetch_array($uid);
    $ratingd = $db->query("SELECT * FROM cdb_ratings WHERE userid='{$uid['id']}'");
    if(mysql_num_rows($ratingd) > 0)
    {
        $ratingd = mysql_fetch_array($ratingd);
        $new_rate = (($c['rating']*$c['rating_amt'])-$ratingd['rating']+$rating)/$c['rating_amt'];
        $db->query("UPDATE cdb_stories SET rating={$new_rate} WHERE id={$story}");
        $db->query("UPDATE cdb_ratings SET rating={$rating} WHERE userid='{$uid['id']}'");
        die();
    }
    $new_num = $c['rating_amt']+1;
    $new_rate = (($c['rating']*$c['rating_amt'])+$rating)/$new_num;
    $db->query("UPDATE cdb_stories SET rating_amt={$new_num}, rating={$new_rate} WHERE id={$story}");
    $db->query("INSERT INTO cdb_ratings VALUES({$uid['id']},{$rating},{$story})");
}
else
{
    die("INVALID FIELDS");
}
?>

Answer 1

   ((Rating * Times) + New) / (Times + 1)

为了你的价值观：

   ((3 * 1) + 3) / (1 + 1)
=  (   3    + 3) / 2
=              6 / 2
=              3

所以程序看起来在数学上是正确的。

我建议你把计算放到它自己的参数函数中，这样你就不会被那批你的其他代码烦恼了。这将使您更容易调试：

function new_rate($rating, $times, $new)
{
    return (($rating * $times) + $new) / ($times + 1);
}

然后，您可以在代码中更轻松地使用它。此外，如果其他原因是错误的原因，您可以通过测试裸功能找到答案。如果它的行为正确，您就知道错误放在其他地方。

希望这有帮助。

评级系统无法正常平均

1 个答案: