[MySQL][PHP]Problem ze skryptem - głosowanie z MySQL Opcje

[imatix]

Może ktos wie czemu ten skrypt nie działa?? :

[PHP] pobierz, plaintext 
<?php 
 // Connects to your Database 
 mysql_connect("mysql1.yoyo.pl", "dbxxxxxx", "haslo") or die(mysql_error()); 
 mysql_select_db("dbxxxxxx") 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>"; 
 } 
 
 
 ?>
[PHP] pobierz, plaintext 

Wczesniej dodalem do bazy danych :

[SQL] pobierz, plaintext 
CREATE TABLE vote (id INT(4) NOT NULL AUTO_INCREMENT PRIMARY KEY, name VARCHAR(30), total INTEGER, votes INTEGER) 
[SQL] pobierz, plaintext 

oraz

[SQL] pobierz, plaintext 
INSERT INTO vote (name, total, votes) VALUES ( "First item", 45, 10 ), ( "Second item", 15, 4 ), ( "Third thing", 25, 7 ), ( "The Forth", 20, 5 ), ( "Fifth Thing", 0, 0 )
[SQL] pobierz, plaintext 

Na stronie wszystko sie wyswietla, ale skrypt nie liczy sredniej, po kliknieciu nic sie nie zmienia:
Adres do skryptu http://www.muzycznalista.yoyo.pl/

Prosze o pomoc.
Pozdro

Ten post edytował imatix 13.05.2010, 09:44:06

[croc]

Myślę, że chodzi o to, że skrypt jest staroświecki i używa register_globals.

Musiałbyś zamienić ten fragment:

[PHP] pobierz, plaintext 
//We only run this code if the user has just clicked a voting link
 if ( $mode=="vote") 
 { 
[PHP] pobierz, plaintext 

Na taki:

[PHP] pobierz, plaintext 
//We only run this code if the user has just clicked a voting link
 if ( $_GET['mode']=="vote") 
 {
    $voted = $_GET['voted'];
    $id= $_GET['id'];
[PHP] pobierz, plaintext 

[imatix]

OK dzieki dziala
Tylko jest cos zle jeszcze z cookie, bo ich nie przechwytuje i mozna glosowac ile wlezie, wiesz jak to zrobic?

[croc]

Jedna instrukcja wkradła się do komentarza. Poza tym w instrukcji setcookie lepiej podać wartość ze zmiennej, czyli zrób tak:

zamień to:

[PHP] pobierz, plaintext 
 //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); 
[PHP] pobierz, plaintext 

zamień na:

[PHP] pobierz, plaintext 
 //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($cookie, 'Voted', $month); 
[PHP] pobierz, plaintext 

[imatix]

Nom dzieki bardzo
A jak zrobić żeby rekordy pokazywalo zgodnie ze srednia a nie ID ?

[croc]

To znaczy masz na myśli sortowanie, tak?

Ja bym zrobił tak:

zamiast tego:

[PHP] pobierz, plaintext 
 //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>"; 
[PHP] pobierz, plaintext 

to:

[PHP] pobierz, plaintext 
 //Puts SQL Data into an array
 $data = mysql_query("SELECT *, total / votes AS average FROM vote ORDER BY average DESC") 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['average']; 
 Echo "Current Rating: " . round($current, 1) . "<br>"; 
[PHP] pobierz, plaintext 

[imatix]

Nom dzieki za pomoc

Jeszcze jedno pytanie, bo chciałem, aby zeby wybieralo tylko z tabeli vote i z kolumny "polskie" gdzie "nie" jak to trzeba wpisac ?
Bo tak na pewno nie:

[PHP] pobierz, plaintext 
 $data = mysql_query("SELECT *, total / votes AS average FROM vote ORDER BY average DESC," . "SELECT * FROM vote WHERE polskie=nie ") or die(mysql_error()); 
[PHP] pobierz, plaintext 

I jak bym chcial dodac jeszcze 1 warunek np: kolumna "gatunek" gdzie "pop"
Do bazy dopisalem za pomoca funkcji set np: wartosc: 'tak','nie'

Dobra juz mam ^^:

[PHP] pobierz, plaintext 
 $data = mysql_query("SELECT *, total / votes AS average FROM vote WHERE polskie='nie' ORDER BY average DESC") or die(mysql_error()); 
[PHP] pobierz, plaintext 

Wie ktos jak zrobic zeby przy kazdej zapetlonej pozycji pojawiaja sie kolejna liczba np 1,2,3,4?

Haloooo
Moze ktos pomoc??

Ten post edytował imatix 13.05.2010, 15:14:54

[croc]

Tak, ładnie znalazłeś rozwiązanie problemu kryteriów wyszukiwania Jak chcesz dodać kolejne kryteria, to łączysz je słowem AND (lub OR, zależy co chcesz osiągnąć - poczytaj o tym).

Żeby zrobić kolejne liczby, możesz zrobić np. tak:

[PHP] pobierz, plaintext 
$data = mysql_query("SELECT *, total / votes AS average FROM vote WHERE polskie='nie' ORDER BY average DESC") or die(mysql_error());
for($i = 1; $row = mysql_fetch_assoc($data); ++$i) {
   // i tutaj masz liczbę w zmiennej $i, co wiersz większą o 1
}
[PHP] pobierz, plaintext 

Jeszcze jedna uwaga - zamiast polskie = 'tak' / polskie = 'nie' lepiej zrobić typ pola na char(1) i wpisywać do niego tylko t/n lub typ liczbowy i 0/1.

[imatix]

[PHP] pobierz, plaintext 
//Wstawienie informacji z MySQL
$data = mysql_query("SELECT *, total / votes AS average FROM vote ORDER BY average DESC") or die(mysql_error());
 
for($i = 1; $row = mysql_fetch_assoc($data); ++$i) {
   // i tutaj masz liczbę w zmiennej $i, co wiersz większą o 1
}
 //Zapętlenie skryptu 
 while($ratings = mysql_fetch_array( $data )) 
 { 
 
 //Wykonawca 
 Echo "<div id=lista><TABLE border=0 cellspacing=0 cellpadding=0><TR><TD><IMG SRC=img/nuta.jpg ALT=nuta></TD>
 <TD width=300px;>" .$ratings['wykonawca']." - "; 
 
 //Tytuł piosenki
 Echo "" .$ratings['tytul']."</TD>"; 
 
 //Link do YouTube  
 Echo "<TD width=50px;><a href=" .$ratings['adres']." target=_blank>Play</a></TD>";
 
 //Wstawienie średniej arytmetycznej 
 $current = $ratings['average']; 
 Echo "<TD width=75px;>Ocena: " . round($current, 1) . "</TD>"; 
[PHP] pobierz, plaintext 

Cos mi sie strona nie chce wlaczac dalej

[croc]

Pętla for może być użyta tu ZAMIAST while.

[PHP] pobierz, plaintext 
//Wstawienie informacji z MySQL
$data = mysql_query("SELECT *, total / votes AS average FROM vote ORDER BY average DESC") or die(mysql_error());
 
 //Zapętlenie skryptu
 for($i = 1; $ratings = mysql_fetch_array( $data ); ++$i)
 {
 
 //Pozycja i wykonawca 
 Echo "<div id=lista><TABLE border=0 cellspacing=0 cellpadding=0>
<TR>
   <TD>" . $i . "</TD>
   <TD><IMG SRC=img/nuta.jpg ALT=nuta></TD>
 <TD width=300px;>" .$ratings['wykonawca']." - ";
 
 //Tytuł piosenki
 Echo "" .$ratings['tytul']."</TD>";
 
 //Link do YouTube  
 Echo "<TD width=50px;><a href=" .$ratings['adres']." target=_blank>Play</a></TD>";
 
 //Wstawienie średniej arytmetycznej 
 $current = $ratings['average']; 
 Echo "<TD width=75px;>Ocena: " . round($current, 1) . "</TD>"; 
[PHP] pobierz, plaintext 

[imatix]

Mam jeszcze problem ze skryptem komentarzy.
Mam na 1 stronie 2 skrypty komentarzy i jak naciskam wyslij to oba sie wlaczaja chociaz sa skierowane do innych tabel, jak to naprawic?

[PHP] pobierz, plaintext 
<?
				        mysql_connect("mysql3.yoyo.pl", "xxxx", "xxxx") or
						     die ("Nie można połączyć się z MySQL");
				        mysql_select_db ("xxxxxx") or
						     die ("Nie można połączyć się z bazą cwphp");
 
					    $wynik = mysql_query ("SELECT * FROM komentarze WHERE ok=0 ".
						                      "ORDER BY nr DESC LIMIT 0,3");
						while ($wynik && $rekord = mysql_fetch_assoc ($wynik)) {
							print "<P><DIV id=news><DIV id=news_title><B>".$rekord['nick']. "</B></DIV><BR>"
							.$rekord['tresc']."</P>";
							print "<P ALIGN=RIGHT></I>".$rekord['data']."</I></P></DIV>\n";
						}
					    ?>
<?
 
$nick = addslashes(htmlspecialchars ($_POST['nick']));
$tresc = addslashes(nl2br(htmlspecialchars ($_POST['tresc'])));
 
if ($nick && $tresc) {
	mysql_connect("mysql3.yoyo.pl", "xxxxx", "xxxxxx") or
	die ("Nie można połączyć się z MySQL");
	mysql_select_db ("xxxxx") or
	die ("Nie można połączyć się z bazą cwphp");
	$query = "INSERT INTO komentarze (nick, tresc, "." data) VALUES ('$nick', '$tresc', "." now());";
	$wynik = mysql_query ($query);
	print "Nowy post został dodany ";
 
} else {
	print "<H3>Dodaj news:</H3>";
	print "<FORM METHOD=POST><B>Dodaj nowy komentarz: </B><BR>";
	print "<INPUT TYPE=\"text\" NAME=\"nick\" VALUE=\"$nick\" ";
	print "<B>Treść:</B><BR><TEXTAREA NAME=\"tresc\" ";
	print "ROWS=6 COLS=60>$tresc</TEXTAREA><BR>";
	print "<INPUT TYPE=\"submit\" VALUE=\"Wyslij\">";
	print "</FORM>";
}
 
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
 <?
 
$nick = addslashes(htmlspecialchars ($_POST['nick']));
$tresc = addslashes(nl2br(htmlspecialchars ($_POST['tresc'])));
 
if ($nick && $tresc) {
	mysql_connect("mysql3.yoyo.pl", "xxxxx", "xxxxx") or
		die ("Nie można połączyć się z MySQL");
	mysql_select_db ("xxxxxx") or
		die ("Nie można połączyć się z bazą");
	$query = "INSERT INTO shoutbox (nick, tresc, "." data) VALUES ('$nick', '$tresc', "." now());";
	$wynik = mysql_query ($query);
	print "Nowy komentarz został dodany ";
 
} else {
	print "<IMG SRC=img/shoutbox.png>";
	print "<FORM METHOD=POST><B>Nick: </B><BR>";
	print "<INPUT TYPE=\"text\" NAME=\"nick\" VALUE=\"$nick\" ";
	print "<BR><B>Treść:</B><BR><TEXTAREA NAME=\"tresc\" ";
	print "ROWS=3 COLS=20>$tresc</TEXTAREA><BR>";
	print "<INPUT TYPE=\"submit\" VALUE=\"Wyslij\">";
	print "</FORM>";
}
 
?>
 
                        <?
				        mysql_connect("mysql3.yoyo.pl", "xxxx", "xxxx") or
						     die ("Nie można połączyć się z MySQL");
				        mysql_select_db ("xxxxx") or
						     die ("Nie można połączyć się z bazą cwphp");
 
					    $wynik = mysql_query ("SELECT * FROM shoutbox WHERE ok=0 ".
						                      "ORDER BY id DESC LIMIT 0,10");
						while ($wynik && $rekord = mysql_fetch_assoc ($wynik)) {
							print "<DIV id=shoutbox2><DIV id=shoutbox-tytul><B>".$rekord['nick']. "</B>, ".$rekord['data']."</DIV>&nbsp;"
							.$rekord['tresc']."";
							print "</DIV>\n";
						}
                        ?>
[PHP] pobierz, plaintext