[PHP] Błąd w kodzie, problem z sesjami, po kliknięciu odświeżenia strony aktualizuje wcześniejszy rekord Opcje

[ebate]

Mam kod, który pobiera 2 losowe dane z bazy MySQL (zdjęcia). Po kliknięciu na jedno z nich kliknięte dodaje 1 punkt do bazy w polu wygrana, zaś niekliknięte zdjęcie dostaje 1 punkt do bazy w polu przegrana. po tym wszystkim losują się dwa kolejne zdjęcia i tak w kółko.

Problemem jest to, że jeśli kliknę na któreś zdjęcie (doda punkty jak wspomniałem powyżej) i wylosują się 2 następne zdjęcie, to po ręcznym odświeżeniu strony kliknięte wcześniej zdjęcie dostaje kolejny punkt a niekliknięte również dostaje punkt (powtarza sie dodawanie punktów)

Czy da sie zrobic, aby nic nie dodawało przy odświeżaniu tylko po prostu losowało 2 kolejne zdjęcia?

Oto kod:

[PHP] pobierz, plaintext 
<?php
 
session_start();
 
	require('conn.php');
 
		$wykonaj=mysql_query("SELECT * FROM user WHERE plec='meżczyzna' ORDER BY rand() limit 1")or die(mysql_error());
 
		$wiersz = mysql_fetch_array($wykonaj);
 
 
 
		$wykonaj1=mysql_query("SELECT * FROM user WHERE plec='meżczyzna' ORDER BY rand() limit 1")or die(mysql_error());
 
		$wiersz1 = mysql_fetch_array($wykonaj1);
 
 
 
		echo '<a href=glosowaniem.php?pierwszy=1&wyg='.$wiersz['sciezka'].'&prze='.$wiersz1['sciezka'].'><img height="200" width="175" src="'.$wiersz['sciezka'].'"/></a>';
 
		echo '<a href=glosowaniem.php?drugi=1&wyg='.$wiersz['sciezka'].'&prze='.$wiersz1['sciezka'].'><img height="200" width="175" name="przycisk" src="'.$wiersz1['sciezka'].'"/></a>';
 
	if(isset($_GET['pierwszy'])) 
 
		{
 
			mysql_query("UPDATE user SET wygrana=wygrana+1 WHERE sciezka='".$_GET['wyg']."'")or die(mysql_error());
 
			mysql_query("UPDATE user SET przegrana=przegrana+1 WHERE sciezka='".$_GET['prze']."'")or die(mysql_error());
 
		}
 
	if(isset($_GET['drugi'])) 
 
		{
 
			mysql_query("UPDATE user SET wygrana=wygrana+1 WHERE sciezka='".$_GET['prze']."'") or die(mysql_error());
 
			mysql_query("UPDATE user SET przegrana=przegrana+1 WHERE sciezka='".$_GET['wyg']."'")or die(mysql_error());
 
		}
 
session_destroy();
 
?>
[PHP] pobierz, plaintext 

Odświeżam. Nie ma nikt na to pomysłu?

[Sephirus]

Powiem tak - ogólnie podejście masz średnie do tematu ale da się to naprawić bez zmieniania twojego kodu za mocno.

Co musisz zrobić? proste -

Na początku pliku po includowaniu conn.php wylosuj randomową zmienną do sesji przypisując starą jeżeli istnieje:

[PHP] pobierz, plaintext 
require('conn.php');
 
$oldR = isset($_SESSION['zmienna_do_zdjec']) ? $_SESSION['zmienna_do_zdjec'] : '';
$_SESSION['zmienna_do_zdjec'] = time() + mt_rand(1000000,9999999);
 
(...)
 
[PHP] pobierz, plaintext 

Teraz tam gdzie masz generowanie linków dodaj nową zmienną do getów:

[PHP] pobierz, plaintext 
echo '<a href=glosowaniem.php?pierwszy=1&wyg='.$wiersz['sciezka'].'&prze='.$wiersz1['sciezka'].'&r='.$_SESSION['zmienna_do_zdjec'].'><img height="200" width="175" src="'.$wiersz['sciezka'].'"/></a>';
 
echo '<a href=glosowaniem.php?drugi=1&wyg='.$wiersz['sciezka'].'&prze='.$wiersz1['sciezka'].'&r='.$_SESSION['zmienna_do_zdjec'].'><img height="200" width="175" name="przycisk" src="'.$wiersz1['sciezka'].'"/></a>';
[PHP] pobierz, plaintext 

Nastepnie przy sprawdzaju czy dodawać/odejmować punkty daj dodatkowy warunek:

[PHP] pobierz, plaintext 
if(isset($_GET['pierwszy']) && isset($_GET['r']) && $_GET['r'] == $oldR)  (...)
 
(...)
 
if(isset($_GET['drugi']) && isset($_GET['r']) && $_GET['r'] == $oldR) (...)
[PHP] pobierz, plaintext 

Skasuj z końca "session_destroy()".

Gotowe

Prosta idea - przy każdym odświeżeniu generowana jest nowa zmienna. po przejściu na link jest sprawdzana i się zgadza, po odświeżeniu już się nie zgadza bo została wygenerowana nowa

HTH!

[ebate]

Tak więc zrobiłem jak Ty mi zasugerowałeś i problem zniknął. Wszystko działa w jak najlepszym porządku. Wielkie dzięki za pomoc i poświęcony mi czas.