2013-12-07 3 views
-1

Я пытаюсь создать систему рейтинга на своем сайте, но когда я ввел следующий код, он дает мне предупреждение «Предупреждение: Деление на ноль в/home/*/public_html/* * в строке 38"Предупреждение: Деление на нулевое сообщение

<?php 
// Connects to your Database 
    include ('config.php'); 

    //We only run this code if the user has just clicked a voting link 
if ($mode=="vote") 
{ 

//If the user has already voted on the particular thing, we do not allow them to vote again $cookie = "Mysite$id"; 
    if(isset($_COOKIE[$cookie])) 
     { 
     Echo "Sorry You have already ranked that site <p>"; 
     } 

//Otherwise, we set a cooking telling us they have now voted 
    else 
     { 
     $month = 2592000 + time(); 
     setcookie(Mysite.$id, Voted, $month); 

     //Then we update the voting information by adding 1 to the total votes and adding their vote (1,2,3,etc) to the total rating 
mysql_query ("UPDATE vote SET total = total+$voted, votes = votes+1 WHERE id = $id"); 
     Echo "Your vote has been cast <p>"; 
     } 
} 

//Puts SQL Data into an array 
$data = mysql_query("SELECT * FROM vote") or die(mysql_error()); 

//Now we loop through all the data 
while($ratings = mysql_fetch_array($data)) 
{ 

//This outputs the sites name 
Echo "Name: " .$ratings['name']."<br>"; 

//This calculates the sites ranking and then outputs it - rounded to 1 decimal 
$current = $ratings[total]/$ratings[votes]; 
Echo "Current Rating: " . round($current, 1) . "<br>"; 

//This creates 5 links to vote a 1, 2, 3, 4, or 5 rating for each particular item 
Echo "Rank Me: "; 
Echo "<a href=".$_SERVER['PHP_SELF']."?mode=vote&voted=1&id=".$ratings[id].">Vote 1</a> | "; 
Echo "<a href=".$_SERVER['PHP_SELF']."?mode=vote&voted=2&id=".$ratings[id].">Vote 2</a> | "; 
Echo "<a href=".$_SERVER['PHP_SELF']."?mode=vote&voted=3&id=".$ratings[id].">Vote 3</a> | "; 
Echo "<a href=".$_SERVER['PHP_SELF']."?mode=vote&voted=4&id=".$ratings[id].">Vote 4</a> | "; 
Echo "<a href=".$_SERVER['PHP_SELF']."?mode=vote&voted=5&id=".$ratings[id].">Vote 5</a><p>"; 
} 
?> 

PS Как изменить проверку с помощью файлов cookie, в базу данных mysql. Я хочу, чтобы он проверял базу данных, чтобы узнать, только что человек оценил.

+0

Вы не можете просто изменить куки хранения данных MySQL. Вам нужно каким-то образом связать пользователей с их данными db (например, системой входа). –

ответ

1

В строке 38 у вас есть $ratings[total]/$ratings[votes]. Проблема здесь в том, что если количество голосов в массиве оценок равно 0, вы делите на ноль.

$current = $ratings['votes'] > 0 ? $ratings['total']/$ratings['votes'] : 0;

+0

Спасибо @Navarr – user3078799

+0

Если вам это нравится, @ user3078799, вы должны отметить его как ответ. В вашем коде есть некоторые другие проблемы, включая потенциальную инъекцию SQL и другие проблемы с безопасностью. Я бы рекомендовал прочитать [PHP The Right Way] (http://www.phptherightway.com/), если вам это интересно, и смотрите на любую карьеру в этой области. – Navarr

3

Линия 38:

$current = $ratings[total]/$ratings[votes];

, вероятно, следует

$current = $ratings['total']/$ratings['votes'];

не говоря уже о других проблемах в коде.

Смежные вопросы