Forum PHP.pl

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??

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

Przenoszę na przedszkole.

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

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 

Nie, zrobiles inaczej niz ci mowilem a na dodatek pomyliles nazwy zmiennych (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) 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 

Niekoniecznie.



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

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