[php] System glosowania i błąd Opcje

[Mlodycompany]

Witam. Mam skrypt oceniania filmow na stronie.

[PHP] pobierz, plaintext 
<?
$glosy = mysql_query("SELECT * FROM `filmy_glosy` WHERE `ID` = '".$_GET['film']."'");
while($row = mysql_fetch_array($glosy)){
$wartosc = ($row[1] * 1) + ($row[2] * 2) + ($row[3] * 3) + ($row[4] * 4) + ($row[5] * 5) + ($row[6] * 5) + ($row[6] * 6) + ($row[7] * 7) + ($row[8] * 8) + ($row[9] * 9) + ($row[10] * 10);
$suma_glosow = $row[1] + $row[2] + $row[3]+ $row[4]+ $row[5]+ $row[6]+ $row[7]+ $row[8]+ $row[9]+ $row[10];
$srednia = $wartosc / $suma_glosow; // linia 39
if($srednia != 0){
echo('<tr class=naglowek3><th>Głosów</th><th>'.$suma_glosow.'</th></tr>');
echo('<tr class=naglowek2><th>Średnia Głósów</th><th>'.round ($srednia , 2).'</th></tr>');
}
}
echo('
<tr class=naglowek3>
	<th>Oceń Film</th>
	<th>
	<a href="index.php?film='.$_GET['film'].'&vote=1">1</a>
	<a href="index.php?film='.$_GET['film'].'&vote=2">2</a>
	<a href="index.php?film='.$_GET['film'].'&vote=3">3</a>
	<a href="index.php?film='.$_GET['film'].'&vote=4">4</a>
	<a href="index.php?film='.$_GET['film'].'&vote=5">5</a>
	<a href="index.php?film='.$_GET['film'].'&vote=6">6</a>
	<a href="index.php?film='.$_GET['film'].'&vote=7">7</a>
	<a href="index.php?film='.$_GET['film'].'&vote=8">8</a>
	<a href="index.php?film='.$_GET['film'].'&vote=9">9</a>
	<a href="index.php?film='.$_GET['film'].'&vote=10">10</a>
	</th>
</tr>
</table>
');
 
if($_GET['vote'] != ""){
$glosy = mysql_query("SELECT * FROM `filmy_glosy` WHERE `ID` = '".$_GET['film']."'");
$ilosc = mysql_num_rows($glosy);
while($glos = mysql_fetch_array($glosy)){
echo $glos[$_GET['vote']];
 
$glos_dodaj = $glos[$_GET['vote']] + 1;
$vote = mysql_query("UPDATE `filmy_glosy` SET `".$_GET['vote']."` = '".$glos_dodaj."' WHERE `ID` = '".$_GET['film']."'");
if($vote){
header("Location: index.php?film=".$_GET['film']."#");
}
}
}
?>
[PHP] pobierz, plaintext 

i jezeli w tabeli filmy_glosy mam wszystkie wartosci '0' oprocz id to wyskakuje mi taki bład:Warning: Division by zero in c:\usr\www\filmy\film.php on line 39. Od czego to zależy i jak sie tego pozbyc??

Powód edycji: Dodałem tag. /webdicepl

[nospor]

No pomysl troche.
$srednia = $wartosc / $suma_glosow;
Przeciez jesli masz same zera to dzielisz przez 0. Nie uczyli cie w szkole ze nie wolno dzielic przez zero?
Daj warunek na sume glosow ze jesli jest rowna zero to srednia tez jest zero juz bez wyliczania

[Mlodycompany]

no tez prawda, ale na zdrowy rozsadek wartosc powinna byc pokazana jako 0.

[webdice]

Przenoszę na przedszkole.

[nospor]

ale na zdrowy rozsadek wartosc powinna byc pokazana jako 0.

Nie, ponieważ nie wolno dzielic przez zero. Takie prawa matematyki. Kazdy jezyk programowania w takim wypadku wypluje ci bledem.

[Mlodycompany]

no dobrze. zrobilem tak jak mowisz, ale to tez nie jest idealne rozwiazanie poniewaz srednia nie jest pokazywana. pokazuje ilosc glosow, a srednia jest '0'

[PHP] pobierz, plaintext 
<?php
$glosy = mysql_query("SELECT * FROM `filmy_glosy` WHERE `ID` = '".$_GET['film']."'");
while($row = mysql_fetch_array($glosy)){
$wartosc = ($row[1] * 1) + ($row[2] * 2) + ($row[3] * 3) + ($row[4] * 4) + ($row[5] * 5) + ($row[6] * 5) + ($row[6] * 6) + ($row[7] * 7) + ($row[8] * 8) + ($row[9] * 9) + ($row[10] * 10);
$suma_glosow = $row[1] + $row[2] + $row[3]+ $row[4]+ $row[5]+ $row[6]+ $row[7]+ $row[8]+ $row[9]+ $row[10];
$srednia = $wartosc / $suma_glosow;
if($suma != 0 && $wartosc != 0){
$srednia = $wartosc / $suma_glosow;
}
echo('<tr class=naglowek3><th>Głosów</th><th>'.$suma_glosow.'</th></tr>');
echo('<tr class=naglowek2><th>Średnia Głósów</th><th>'.$srednia.'</th></tr>');
}
?>
[PHP] pobierz, plaintext 

[nospor]

Nie, zrobiles inaczej niz ci mowilem a na dodatek pomyliles nazwy zmiennych Jak można popelniac bledy przy takich pierdulach? Jak ci nie wychodzi to pomysl troche dluzej na przyszlosc a nie od razu na forum walisz. Jak sam dojdziesz gdzie masz blad to dla ciebie wyjdzie z wiekszym pozytkiem

[PHP] pobierz, plaintext 
<?php
$glosy = mysql_query("SELECT * FROM `filmy_glosy` WHERE `ID` = '".$_GET['film']."'");
while($row = mysql_fetch_array($glosy)){
$wartosc = ($row[1] * 1) + ($row[2] * 2) + ($row[3] * 3) + ($row[4] * 4) + ($row[5] * 5) + ($row[6] * 5) + ($row[6] * 6) + ($row[7] * 7) + ($row[8] * 8) + ($row[9] * 9) + ($row[10] * 10);
$suma_glosow = $row[1] + $row[2] + $row[3]+ $row[4]+ $row[5]+ $row[6]+ $row[7]+ $row[8]+ $row[9]+ $row[10];
if($suma_glosow != 0){
$srednia = $wartosc / $suma_glosow;
} else
$srednia = 0;
echo('<tr class=naglowek3><th>Głosów</th><th>'.$suma_glosow.'</th></tr>');
echo('<tr class=naglowek2><th>Średnia Głósów</th><th>'.$srednia.'</th></tr>');
}
?>
[PHP] pobierz, plaintext 

[Kicok]

Można dzielić przez zero. Wynikiem dzielenia przez zero jest nieskończoność

Niekoniecznie.



~Mlodycompany - nie lepiej byłoby ci używać takiej struktury tabeli filmy_glosy:

Kod

+------------+--------+
|  ID_filmu  |  glos  |
+------------+--------+
|     1      |   10   |
|     1      |    9   |
|     1      |    9   |
|     1      |    1   |
|     1      |   10   |
|     2      |    7   |
|     2      |    8   |
(...)

Dodawanie nowego głosu to zwykły INSERT, a obliczanie średniej jest jeszcze prostsze:

[PHP] pobierz, plaintext 
<?php
 
	// $_POST['ID_filmu'] = 1
	$query = 'SELECT AVG( glos ) AS srednia FROM filmy_glosy WHERE ID_filmu = ' . intval( $_POST['ID_filmu'] );
	$result = mysql_query( $query ) or die( mysql_error() );
 
	$row = mysql_fetch_row( $result );
	$srednia = $row['srednia'];
 
	echo $srednia; // 7,8
 
?>
[PHP] pobierz, plaintext