Rating system problem

Selo

Hello!

I'm doing a normal rating system for my website and I've encountered a problem.
The system seems to work but even if I rate it 10 many times it never goes over 7.5~, and when it reaches 7.5 it starts going down.

Here's the code:

<?
include ('conect.php');
// variables
$id = $_GET[ 'song_id' ];
$rate = $_GET[ 'rate' ];  

//Query for information
$query = @mysql_db_query("selo_music","SELECT ID,link,rating,ratings,lastrater FROM canciones WHERE ID= '$id' ORDER BY ID") OR DIE('Algo falla en el query');
$row = @mysql_fetch_array($query);
$rating = $row['rating'];
$ratings = $row['ratings'];
$lastrater = $row['lastrater']; 
$ip = getenv('REMOTE_ADDR');
echo $ip;
echo $lastrater;
if ($ip==$lastrater)
{
$message='Sorry you have already rated this game';
echo $message;
}
//Else perform the vote checking if vote is between 0 and 10
if ($rate>='1' and $rate<='10' and "$ip"!="$lastrater")
{
$totalscore = $ratings*$rating+$rate;
$averages = $totalscore / ($ratings + 1);
$average = round($averages, 1);
$ratings = $ratings + 1;


echo 'Rating is';
echo $average;
//Update the tables
mysql_query("UPDATE canciones SET rating = '$average' WHERE id = '$id'") OR DIE ('Errorcito 1');
mysql_query("UPDATE canciones SET lastrater = 'whatever' WHERE id = '$id'") OR DIE ('Errorcito 2');
mysql_query("UPDATE canciones SET ratings = '$ratings' WHERE id = '$id'") OR DIE ('Errorcito 3');
$message='Thank you for rating this game!';
}
else{
echo 'problem';
}
?>

Thanks in advance!

bpat1434

Why are you multiplying in your total score? It should be just a simple addition. You have X ratings with a maximum of Z points possible (Z*X). The way a rating script typically runs is like so:

User chooses rating from 1 - 10.
User response is added to the total points for that item
The number of ratings received is incremented by 1
The Average is calculated (total points over votes cast)

So as an example:

30 people visit my site and give the following ratings:
8, 5, 7, 9, 2, 5, 1, 6, 2, 6, 1, 2, 6 ,1, 4, 7, 6, 1, 6, 5, 2, 5, 8, 10, 9, 3, 2, 8, 6, 10

My total ratings are: 30 (30 visitors cast a rate)
My total score is: 153
My average rating is: 153/30 = 5.1

Now, if you had 10 10s, it would look like this:
Total ratings: 10
Total score: 100
Average: 100/10 = 10

Understand?

Dada78

I am trying to set up a rating system like this that you can see below the picture.

http://mesquitechristmas.com/local/displays.php

It is just something simple where a visitor can vote by clicking on one of the numbers and then it displays the avg and how many voted.

I am not sure how to work this or where to begin. I have a column set up for ratings but I am not sure what else I need.

Any help would be great

-Thanks

bpat1434

If you're saving to a database, you need to store the number of votes and the sum of their votes.

So for each picture in your database, you have two more columns: sum and votes. You'd then take the (sum)/(votes) and return that as the average value.

Dada78

So how do I do that to where it enters the information to the correct row for that information being displayed? Each row has an ID that is auto-increment I just do not know how to use that to call the information for that row or insert information for that that ID row etc so that the vote is going to the right information being displayed. All the information on the page is being displayed from the database with the exception of the html of course. I am just stuck there because I am not sure how to display multiple information from different rows on different pages using the same template from calling the information from the row using the ID? Does that make sense? Sorry if I didn't explain it correctly, I am tired. I know what I need just not sure how to explain it.

benracer

ok i would...

1) have a column called total and another count
2) have a seperate table called lastip
2) have a seperate page called rate.php that contains something like...

<?php
// connect to table rater ip
// select * from lastip
// add returned data to array called row

$lastrater = $row['ip'];
$rating = $_GET['rating'];
$image = $_GET['image']; // this is how you would id the row
$ip = getenv('REMOTE_ADDR');

if ($rating>='1' and $rating=<'10' and "$ip"!="$lastrater")
{
// update table set count=count +1 where id = $image
// update table set total = total + $rating where id = $image
// update lastip set ip=$ip

header("location:index.php");

}else{
echo "Sorry An Error Occured";
}
?>

3) On the main page where the rating is to be chosen have...

<?php

// connect to  lastip
// select * from last ip
// add results to array called row

$ip = $row['ip'];
$current_ip = getenv('REMOTE_ADDR');

if($ip == $current_ip){
// select * from table where id = $image
// add results to array called final

echo "Total Number Of Votes: ";
echo $final['count'];

echo "Average Score: ";
echo $final['total'] / $final['count'];
}else{
// html for voting here
}

You could also make this more accurate to prevent one person voting more than once by storing a session/cookie and checking this or even a user login but this gets more complicates... also there are ways to getround all blocks that we can try to use, a ip address can be changed, session can be cleared by closing the page, cookies can be deleted ect...

Dada78

In my database I have two columns rating and votes where rating will be the actual rating and votes will be the number of people who have voted. It only rates on scale from 1-5 as seen on the page. I am not really concerned with people voting more then once so that is ok if they do.

Here is a page where the rating system will be used.

http://mesquitechristmas.com/local/displays.php?id=2

I have the ID issue figured out. Here is the code for the display file.

<?php

include ('db_connect.php');

if (isset($_GET['id'])) {
  $id = mysql_real_escape_string($_GET['id']);
  $sql = "SELECT * FROM users WHERE id = $id;";
  if ($result = mysql_query($sql)) {
    if (mysql_num_rows($result)) {
      $row = mysql_fetch_array($result);
      $displayname = $row["displayname"];
      $displaytype = $row["displaytype"];
      $address = $row["address"];
      $city = $row["city"];
      $state = $row["state"]; 
      $postal = $row["postal"];
      $country = $row["country"];
      $rating = $row['rating'];
      $votes = $row['votes'];
    } else {
      die("No user found");
    }
  } else {
    die(mysql_error());
  }
}

?>

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
<head>
<meta http-equiv="content-type" content="text/html; charset=utf-8" />
<meta name="description" content="Mesquite Texas Country Christmas" />
<meta name="keywords" content="Mesquite, Texas, Country Christmas" />
<meta name="author" content="NA" />
<link rel="stylesheet" type="text/css" href="/stylesheet.css" media="screen" title="FBC" />
<script type="text/javascript" src="drop_down.js"></script>
<title>A Mesquite Country Christmas</title>
</head>
<body>

 <div id="wrap">

<a href="/index.html">
<img id="frontphoto" src="/images/header.png" width="760" height="237" alt="Mesquite Country Christmas" border="0"></a>

<div id="menu">

<h2 class="hide">Menu:</h2>

<ul id="avmenu">
<li><a href="/index.html">Home</a></li>
<li><a href="/christmasstory.html">The Christmas Story</a></li>
<li><a href="/directions.html">Directions</a></li>
<li><a href="#">Information</a><ul>
      <li><a href="/information.html">Display Facts & Info</a></li>
      <li><a href="/faq.html">FAQ</a></li>
      <li><a href="/playlist.html">2008 Playlist</a></li>
      <li><a href="#">Christmas History</a></li>
  </ul></li>
<li><a href="#">Photos</a>
  <ul>
      <li><a href="/2007photos.html">2007</a></li>
  </ul></li>
<li><a href="#">Videos</a>
  <ul>
      <li><a href="/2007videos.html">2007</a></li>
  </ul></li>
<li><a href="/guestbook.php">Guestbook</a></li>
<li><a href="/webcam.html">Web Cam</a></li>
<li><a href="/webradio.html">Internet Radio</a></li>
<li><a href="http://www.noradsanta.org/" TARGET="_blank">Track Santa</a></li>
<li><a href="/projects.html">Projects & How Tos</a></li>
<li><a href="/links.html">Links</a></li>
<li><a href="/contact_us.html">Contact Us</a></li>
</ul>

<center><a href="http://www.toysfortots.org/" TARGET="_blank"><img src="/images/toys_for_tots.jpg" border="0" width="110" height="153" vspace="10"></a></center>

<center><a href="http://christmas.bronners.com/2007/house/534.html"><img src="http://christmas.bronners.com/voteforme/vote.jpg" border="0" width="110" height="153" alt="christmas decorations" vspace="10"></a></center>

</div>

<div id="content">


 <div class="fadebox">

<h2><? echo $row['displayname']; ?></h2>

<hr />

<br>

<table border="0" cellpadding="4" cellspacing="0" width="100%">

<tbody>
     <tr>
        <td align="center"><img src="http://www.christmaslightfinder.com/images/submitted/11283740-large.jpg" class="border"></td>

<p></p></td></tr><tr><td><center>Rated 0 from 0 people | <b>Rate This</b>: <a href="displays.php?id=<?php echo $row['id']; ?>&amp;rate=1">1</a> | <a href="displays.php?id=<?php echo $row['id']; ?>&amp;rate=2">2</a> | <a href="displays.php?id=<?php echo $row['id']; ?>&amp;rate=3">3</a> | <a href="displays.php?id=<?php echo $row['id']; ?>&amp;rate=4">4</a> | <a href="displays.php?id=<?php echo $row['id']; ?>&amp;rate=5">5</a>
</td><td>&nbsp;</td></tr></tbody></table>

<table align="center" style="border-top: 1px solid rgb(51, 102, 51);" border="0" cellpadding="4" cellspacing="0" width="100%">

<p><a name="info"><span class="title">About this Display...</span>
<tbody><tr><td class="bg2" width="100"><b>Display Name</b>:</td><td class="bg2"><? echo $row['displayname']; ?></td></tr><tr><td class="bg1"><b>Display Type</b>:</td><td class="bg1"><? echo $row['displaytype']; ?></td></tr><tr><td class="bg2"><b>Description</b>:</td><td class="bg2"><? echo $row['description']; ?></td></tr><tr><td class="bg1"><b>Address</b>:</td><td class="bg1"><? echo $row['address']; ?><br><? echo $row['city']. "," . $row['state']. " " . $row['postal']; ?><br><? echo $row['country']; ?><br><a href="http://www.mapquest.com/maps/map.adp?searchtype=address&amp;formtype=search&amp;countryid=US&amp;addtohistory=&amp;country=US&amp;address=<? echo $row['address']; ?>&amp;city=<? echo $row['city']; ?>&amp;zipcode=<? echo $row['postal']; ?>&amp;historyid=&amp;submit=Get+Map" target="_NEW"><b>Get Directions</b></a></td></tr><tr><td class="bg2"><b>Website</b>:</td><td class="bg2"><a href="<? echo $row['website']; ?>" target="_blank"><? echo $row['website']; ?></a></td></tr><tr><td class="bg1"><b>Sponsor Links</b>:</td><td class="bg1">
<script type="text/javascript"><!--
google_ad_client = "pub-8048181801684156";
//displays
google_ad_slot = "4239948836";
google_ad_width = 234;
google_ad_height = 60;
//--></script>
<script type="text/javascript"
src="http://pagead2.googlesyndication.com/pagead/show_ads.js">
</script>
		</td></tr></tbody></table></p>


</div>



   </div>
 </div>

<div id="footer">
&copy; 2007 Mesquite Country Christmas

<br />
<br />

<script type="text/javascript"><!--
google_ad_client = "pub-8048181801684156";
//468x60, created 1/8/08
google_ad_slot = "0360766123";
google_ad_width = 468;
google_ad_height = 60;
//--></script>
<script type="text/javascript"
src="http://pagead2.googlesyndication.com/pagead/show_ads.js">
</script>


</div>

</div>
</body>
</html>

I noticed in your code above you had lines commented out, is that suppose to be that way. I am not sure where the columns come into play that you suggested. In my case it will be rating and votes. Can all this be processed on the same page without another page then return back to the same page after they place their vote?

-Thanks

Dada78

Anyone?

bpat1434

Have you read my article on here? Part 1
Part 2

It's a bit more advanced, but uses the same technique for storing "ratings" and calculating the output. You could use that to guide you on how to do it.

Yes, you can combine the logic into one file; however, when you get to the point in time when you need to update or upgrade your coding, it can be really really difficult. I'd suggest you keep it in separate files.