PHP / MySQL - Нужна помощь в исправлении предупреждения PHP?

Question

PHP / MySQL - Нужна помощь в исправлении предупреждения PHP?

0

Я нашел этот script на about.com, о котором я пытаюсь узнать, о том, как создать рейтинговую систему, но script дает мне предупреждение, которое я перечислял ниже.

Мне было интересно, как я могу исправить эту проблему? И какую часть кода мне нужно изменить и где?

Ниже приведено предупреждение.

Warning: Division by zero on line 43

Ниже представлен script.

<?php
// Connects to your Database
mysql_connect("localhost", "root", "", "sitename") or die(mysql_error());
mysql_select_db("sitename") or die(mysql_error());



//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>";
}
?>

PeAk 11 дек. 2009, в 16:29

Источник

Теги:

php

mysql

2 ответа

2

Проблема здесь

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

Если голосов нет, вы делите число на ноль. И это плохо:)

Добавьте некоторую проверку, что $rated [votes] установлен, а не 0.

Vafliik 11 дек. 2009, в 13:42

Ещё вопросы

Atli · Accepted Answer · 2009-12-11T12-02-00.000Z

Вам нужно убедиться, что вы не делите с помощью 0. Если значения, которые вы получаете для total и votes из MySQL, равны 0, вы должны обойти разделение и установить фиксированное значение.

//This calculates the sites ranking and then outputs it - rounded to 1 decimal
if($ratings['total'] > 0 && $ratings['votes'] > 0) {
    $current = $ratings['total'] / $ratings['votes'];
}
else{
     $current = 0;
}

P.S.
Обратите внимание, как я процитировал элементы в массиве $ratings. Вы всегда должны это делать.

// This is INCORRECT. Causes error notices if you have error reporting on.
// and can have other consequences if you happen to use a `total` constant.
$ratings[total];

// It should be
$ratings['total']