[PHP] Sesja - błędy w kodzie Opcje

[ebate]

Ot kod który ma działać (a powinien przynajmniej w zamyśle) następująco: losują się 2 zdjęcia, klikam na jakieś zdjęcie przez co dodaje wartość=1 do odpowiednich danych w bazie, a następnie losuje kolejne 2, itak w kólko.

Podany kod tak raczej nie działa, z sesjami bawię się od dziś więc jakby ktoś dojrzał jakieś błędy niech je napisze.

[PHP] pobierz, plaintext 
	require('conn.php');
 
           session_start();
 
 
		$_SESSION['zdj_first']=("$wykonaj = mysql_query("SELECT * FROM user WHERE plec='meżczyzna' ORDER BY rand() limit 1");
 
 
		while($wiersz = mysql_fetch_array($wykonaj))
 
		{
 
		echo '<a href=glosowaniem.php><img height="200" width="175" src="'.$wiersz['sciezka'].'"/></a>';
 
		}");
 
		$_SESSION['zdj_second']=("$wykonaj1 = mysql_query("SELECT * FROM user WHERE plec='meżczyzna' ORDER BY rand() limit 1");
 
 
		while($wiersz1 = mysql_fetch_array($wykonaj1))
 
		{
 
		echo '<a href=glosowaniem.php><img height="200" width="175" name="przycisk" src="'.$wiersz1['sciezka'].'"/></a>';
 
		}");
 
if($_SESSION['zm_sesji']) {
 
mysql_query("UPDATE user SET wygrana=wygrana+1 WHERE sciezka='.$wiersz['sciezka'].'");
 
mysql_query("UPDATE user SET przegrana=przegrana+1 WHERE sciezka='.$wiersz1['sciezka'].'");
 
}
 
if($_SESSION['zdj_second']) {
 
mysql_query("UPDATE user SET wygrana=wygrana+1 WHERE sciezka='.$wiersz['sciezka'].'");
 
mysql_query("UPDATE user SET przegrana=przegrana+1 WHERE sciezka='.$wiersz1['sciezka'].'");
 
}
 
 
session_destroy();
[PHP] pobierz, plaintext 

[kadlub]

Nie wiem jaki masz błąd bo nie napisałeś ale

[PHP] pobierz, plaintext 
 session_start();
[PHP] pobierz, plaintext 

daje się na samym początku pliku

[ebate]

Rozumiem, że session_start(); ma być przed require('conn.php');. Poprawiony kod wygląda następująco:

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

[kadlub]

działa
a jak nie to jakie błędy??

[ebate]

Niestety, ciągle wyskakiwał błąd:
Parse error: syntax error, unexpected T_STRING in C:\xampp\htdocs\PIOTR\glosowaniem.php

Więc pousuwałem przy zmiennych $_SESSION znaki (""), a następnie znowu wyskoczył błąd:

Parse error: syntax error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in C:\xampp\htdocs\PIOTR\glosowaniem.php on line 30

[kadlub]

[PHP] pobierz, plaintext 
 
mysql_query("UPDATE user SET wygrana=wygrana+1 WHERE sciezka='".$wiersz['sciezka']."'");
[PHP] pobierz, plaintext 

[ebate]

W końcu nie wywala błędów. Tylko, że dalej jest pewnie problem. W zamyśle po kliknięciu na jedno ze zdjęć dodaje 1 punkt do odpowiedniego pola bazie (wygrana lub przegrana). Niestety, nic nie dodaje. Czy błędem jest to, że aktualnie pole wygrana bądź przegrana nie mają żadnej wartości?

[kadlub]

pokaż kod po wszystkich poprawkach

[ebate]

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

[kadlub]

myślę że jak losujesz z bazy jeden rekord to pętle while nie są ci potrzebne w warunkach zapytania masz .$wiersz['sciezka']. a to chyba będzie dostępne tylko wewnątrz pętli a i jeszcze jedno nie wiem skąd skrypt ma wiedzieć kiedy został wciśnięty przycisk.

[ebate]

No właśnie nad tym głowie się już trochę. Może pomoże Ci to, co nospor napisał:

Przedewszystkim, to w sesji musisz zapisać wylosowane rekordy. Przecież jak user kliknie w obrazek, to ty po przeładowaniu strony znowu losujesz i wylosowane rekordy są już inne niż z paczki dwóch co koleś klikał.

Po kliknieciu sprawdzasz w sesji, czy dany klik jest i jak jest to juz wszystko masz co chciałeś. Jednemu dajesz wygrana a drugiem przegrana

[mortus]

1. Identyfikator użytkownika, którego zdjęcie zostało kliknięte musisz przekazać w adresie url:

[HTML] pobierz, plaintext 
<a href="glosowaniem.php?id=ID">...</a>
[HTML] pobierz, plaintext 

2. Do wylosowania dwóch unikalnych użytkowników posłuż się zapytaniem:

[SQL] pobierz, plaintext 
SELECT * FROM `user` WHERE `plec` = 'mężczyzna' ORDER BY RAND() LIMIT 2
[SQL] pobierz, plaintext 

Pamiętaj jednak, że przy dużej liczbie rekordów w bazie danych tego rodzaju zapytania są bardzo wolne.
3. W sesji zapisujesz identyfikator wyniku zapytania, a powinieneś raczej zapisać po prostu odpowiedni wiersz z bazy danych

[PHP] pobierz, plaintext 
// korzystając z powyższego zapytania
$wynik = mysql_query($zapytanie);
while($wiersz = myslq_fetch_assoc($wynik)) {
 $_SESSION['wylosowani_uzytkownicy'][$wiersz['id']] = $wiersz;
 // a tutaj wyświetlanie adresu url w odpowiedniej formie (wspomniałem wyżej) oraz zdjęcia
}
[PHP] pobierz, plaintext 

4. Teraz po kliknięciu na któreś ze zdjęć w tablicy $_GET['id'] powinieneś mieć identyfikator użytkownika, którego zdjęcie zostało kliknięte. Porównujesz ten identyfikator z danymi zapisanymi w sesji (a dokładniej w $_SESSION['wylosowani_uzytkownicy']) i wykonujesz odpowiednie UPDATE-y.
5. W zapytaniach UPDATE powinieneś się posługiwać raczej identyfikatorami (id), które są przechowywane w sesji, aniżeli ścieżką do zdjęcia/obrazka, jak to robisz teraz.

[kadlub]

Porównujesz ten identyfikator z danymi zapisanymi w sesji (a dokładniej w $_SESSION['wylosowani_uzytkownicy']) i wykonujesz odpowiednie UPDATE-y.

hmmm nie wiem czy to będzie działało bo po kliknięciu na link zostaną wylosowane nowe rekordy i przypisane do sessi a w get będzie wynik wcześniejszego losowania więc ten warunek nie będzie spełniony

[mortus]

hmmm nie wiem czy to będzie działało bo po kliknięciu na link zostaną wylosowane nowe rekordy i przypisane do sessi a w get będzie wynik wcześniejszego losowania więc ten warunek nie będzie spełniony

Przecież można najpierw porównać $_GET['id'] z zawartością sesji, a dopiero później wylosować nowe wartości, a można nawet i ich nie losować. Punkty nie oznaczały kolejności działań.

Ten post edytował mortus 5.11.2011, 15:06:21

[ebate]

A więc oto mój aktualny kod. Niestety, w dalszym ciągu nie dodaje punktów w bazie

[PHP] pobierz, plaintext 
<?php
 
session_start();
 
require('conn.php');
 
$wynik = mysql_query("SELECT * FROM `user` WHERE `plec` = 'mężczyzna' ORDER BY RAND() LIMIT 2");
 
while($wiersz = mysql_fetch_assoc($wynik))
 
{
 
$_SESSION['wylosowani_uzytkownicy'][$wiersz['id']] = $wiersz;
 
echo '<a href="glosowaniem.php?id=ID"><img height="200" width="175" name="przycisk" src="'.$wiersz['sciezka'].'"/></a>';
 
}
 
if($_SESSION['wylosowani_uzytkownicy']==$_GET['id'])
 
{
 
mysql_query("UPDATE user SET wygrana=wygrana+1 WHERE id='".$_GET['id']==$_GET['id']."'");
 
}
 
session_destroy();
 
?>
[PHP] pobierz, plaintext 

Zaktualizowałem kod

Ten post edytował ebate 5.11.2011, 18:08:10

[kadlub]

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

zobacz to tylko jest z tym jeden problem po kliknięciu odświeżenia strony zaktualizuje znów ten rekord

[ebate]

Powiem Ci że.... DZIAŁA!

Wszystko jest ok,tylko, jak wspomniałeś, Po kliknięciu rozdaje odpowiednio po punkcie i losuje inne zdjecia (jak powinno być), zaś przy ręcznym odświeżaniu dodaje punkty poprzednim zdjęciom, które dostało ostatnio punkty i losuje nowe, więc prawie wszystko jest OK!

I dzięki wszystkim za dotychczasowe poświęcenie mi waszego czasu.

Kurcze, nie rozumiem dlaczego tak się dzieje. Przecież przy odświeżaniu zaczyna się nowa sesja, więc nie powinno nic dodawać.