Forum PHP.pl > [MySQL][PHP] Głosowanie zalogowanego usera

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [MySQL][PHP] Głosowanie zalogowanego usera

LUCU

5.10.2012, 00:20:36

Witam!

PHP/mySQL działa tylko i wyłącznie lokalnie więc nie ma mowy o cookies i ip.
Mam problem ze skryptem, który będzie pozwalał tylko raz głosować zalogowanemu użytkownikowi.
Do rzeczy - 3 tabele :
Users, Utwory, Głosy

W tabeli Głosy mamy pola:
id_oceny
id_utworu
id_usera
ocena

W jaki sposób sprawdzić czy user głosował już wcześniej na utwór? Teoretycznie to proste bo wystarczy wysłać zapytanie czy id_usera oraz id_utworu znajduje się w tabeli Głosy ale słaby jestem i nie wiem w jaki sposób napisać kod, z góry dzięki za pomoc!

b4rt3kk

5.10.2012, 01:08:34

[SQL] pobierz, plaintext 
SELECT COUNT(id_oceny) FROM glosy WHERE id_utworu="$id' AND id_usera='$user'
[SQL] pobierz, plaintext

[PHP] pobierz, plaintext 
if ($wynik_zapytania>0) echo 'juz glosowales'; else {
  // dodaj glos do bazy
} 
[PHP] pobierz, plaintext

O to chodzi?

szalek01

5.10.2012, 01:10:06

Kod

SELECT id_oceny FROM oceny WHERE id_utworu = 10 AND id_usera = 13;

Jak coś znajdzie znaczy że user już głosował.

//ah byłem 2

LUCU

8.10.2012, 20:47:08

Dzięki za pomoc

Jest jeden motyw , który mi spokoju nie daje - mimo tego , że tabela 'głosy' jest pusta zawsze jest echo 'juz glosowales'

[PHP] pobierz, plaintext 
 
session_start();
$id_usera = $_SESSION['id_usera'];
$id_utworu = $_POST['id_utworu'];
 
$glosowal = mysql_query("SELECT COUNT(id_oceny) FROM glosy WHERE id_utworu='$id_utworu' AND id_usera='$id_usera'");
 
if ($glosowal>0) 
	{
	echo 'już głosowałeś';
	}
else 
	{
 
if ($_POST['lubi'])
	{ 
	mysql_query("UPDATE `utwory` SET `lubi` = `lubi` + 1 WHERE id_utworu='$id_utworu'");
	mysql_query("INSERT INTO glosy VALUES ('','$id_usera','$id_utworu')");
 
	}
	else if ($_POST['nielubi'])
	{
 
	mysql_query("UPDATE `utwory` SET `nielubi` = `nielubi` + 1 WHERE id_utworu='$id_utworu'");
	mysql_query("INSERT INTO glosy VALUES ('','$id_usera','$id_utworu')");
	}
header ("location: glosowanie.php");
	}
 
?>
[PHP] pobierz, plaintext

bostaf

9.10.2012, 08:42:04

Cytat(LUCU @ 8.10.2012, 21:47:08 )

mimo tego , że tabela 'głosy' jest pusta zawsze jest echo 'juz glosowales'

[PHP] pobierz, plaintext 
$glosowal = mysql_query("SELECT COUNT(id_oceny) FROM glosy WHERE id_utworu='$id_utworu' AND id_usera='$id_usera'");
if ($glosowal>0) 
[PHP] pobierz, plaintext

Zobacz w manualu, jaki rezultat zwraca funkcja mysql_query: "For SELECT [...] mysql_query() returns a resource on success, or FALSE on error." - zwraca zasób. Porównywanie do integera zawsze zwróci fałsz.
Zanim zaczniesz porównywać, musisz ten zasób obrobić, na przykład funkcją mysql_fetch_assoc:

[PHP] pobierz, plaintext 
$glosowal = mysql_query("SELECT COUNT(id_oceny) FROM glosy WHERE id_utworu='$id_utworu' AND id_usera='$id_usera'");
if(FALSE !== $glosowal)
    $wiersz_rezultatu = mysql_fetch_assoc($glosowal);
$ile_ocen = (int) $wiersz_rezultatu['count(id_oceny)'];
if ($ile_ocen > 0) {
[PHP] pobierz, plaintext

Proponowałbym nieco prostszą metodę sprawdzenia w bazie, czy user głosował, bez COUNT - wtedy wynik zapytania zwróci 0 albo więcej wierszy i sprawdzenie sprowadzi się do policzenia ilości wierszy funkcją mysql_num_rows:

[PHP] pobierz, plaintext 
 
$q = mysql_query("SELECT id_oceny FROM glosy WHERE id_utworu='$id_utworu' AND id_usera='$id_usera'");
$ile_ocen = mysql_num_rows($q);
if($ile_ocen > 0) {
[PHP] pobierz, plaintext

No i na koniec najważniejsze. Widzisz te czerwone ramki w opisach każdej funkcji mysql_ w manualu?
"Sugerowane alternatywy
Nie zaleca się stosowania tego rozszerzenia [mysql]. W zamian powinno używać się rozszerzenia MySQLi lub PDO_MySQL"
To nie jest czyjaś subiektywna opinia. To jest zalecenie twórców PHP.

b4rt3kk

9.10.2012, 12:51:44

num_rows jest mniej wydajny od COUNT, bo w pierwszym przypadku zapytanie pobiera wszystkie wyniki, w drugim są jedynie zliczane. W celu tylko sprawdzenia ilości wyników, bez konieczności ich przetwarzania nie należy stosować num_rows.

[PHP] pobierz, plaintext 
$row=mysql_fetch_array(mysql_query("SELECT COUNT(id_oceny) as glosowal FROM glosy WHERE id_utworu='$id_utworu' AND id_usera='$id_usera'"));
$glosowal = $row['glosowal'];
[PHP] pobierz, plaintext

bostaf

9.10.2012, 16:40:49

Cytat(b4rt3kk @ 9.10.2012, 13:51:44 )

num_rows jest mniej wydajny od COUNT

Zdecydowanie. Chociaż subiektywnie objawia się to przy dużej ilości wierszy.
Dla przykładu wyniki eksperymentu z tabelą z 20 000 wierszy i z tabelą z 2 000 000 wierszy (tylko dwie takie same kolumny w obu):

20 tys. wierszy
- Liczenie z "COUNT(*)": około 0,0002 sek.
- Liczenie z "SELECT id": około 0,0400 sek.
Czyli COUNT(*) o dwa rzędy wielkości szybciej, ale różnica dla odbiorcy subiektywnie niezauważalna.
2 mln. wierszy
- Liczenie z "COUNT(*)": około 0,0002 sek.
- Liczenie z "SELECT id": około 4,8000 sek.
Różnica jest już powalająca.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.