Witam serdecznie
Posiadam skrypt na generowanie rekordów z bazy danych:

[PHP] pobierz, plaintext 
<?php
 
 
include "sprawdzacz.php";
 
include('config.php');
 
 
$query1 = 'UPDATE token SET used_by = "'.session_id().'" WHERE used_by = 0 ORDER BY id ASC LIMIT 1;'; 
 mysql_query($query1);
 
 
$query = 'SELECT token FROM token WHERE used_by = "'.session_id().'" ORDER BY id DESC LIMIT 1;';
 mysql_query($query);
 
 
 
$result=mysql_query($query);
 
 
 
echo "<b><center>Wygenerowany kod:</center></b><br><br>";
 
 
{
 
$token=mysql_result($result,"token");
 
echo "$token";
 
}
 
?>
[PHP] pobierz, plaintext 

Przy strukturze bazy danych:

[SQL] pobierz, plaintext 
id   token   used_by
1   asewdsad   0
2   dsgsdgsds  0
3   sdfdsfsdfs   0
4   dsfsdfsghs  0
[SQL] pobierz, plaintext 

Podstawowo w used_by jest wartośc 0 lecz po wykonaniu zapytania zmienia się na identyfikator sesji...


Moje pytanie jest następujące:
Przy odświerzaniu strony zapytanie wykonuje się na nowo...
Jest jakiś sposób by wykonywał się TYLKO RAZ?
Nie wiem... COOKIES, SESJE, cokolwiek... byle by zabezpieczyć reszte kodów...
Zresztą zauważyłem, że można to zrobić tak:

1. wykonuje się zapytanie
2. zapisuje się ID Sesji w used_by
3. pokazuje się kod
4. zamiast po F5 (odświerzeniu) losował się nowy kod ustawić jakiś parametr by ten ID SESJI mógł zobaczyć kod, który uzyskał właśnie jego ID SESJI...

Takie coś chyba da się zrobić ale ja nie wiem jak

Prosze o pomoc.

za pomocą cookies.
"3600" <- czas trwania ciasteczka, ustawione na 1h.
ID SESSJI znajduje się w ciasteczku $_COOKIE["sesid"], jeżeli ciasteczko nie istnieje wtedy generuje nową wartość.

[PHP] pobierz, plaintext 
<?
	ob_start();
	include "sprawdzacz.php";
	include('config.php');
	if(!$_COOKIE["sesid"]){
		$ses = session_id();
		$query1 = 'UPDATE token SET used_by = "'.$ses.'" WHERE used_by = 0 ORDER BY id ASC LIMIT 1;';
		mysql_query($query1);
		setCookie("sesid", $ses, 3600);
	}
	$query = 'SELECT token FROM token WHERE used_by = "'.session_id().'" ORDER BY id DESC LIMIT 1;';
	mysql_query($query);
	$result=mysql_query($query);
	echo "<b><center>Wygenerowany kod:</center></b><br><br>";
	$token=mysql_result($result,"token");
	echo "$token";
	ob_end_flush();
?>
[PHP] pobierz, plaintext 

Widzę, że wiesz o co mi chodzi.
Ale właśnie sprawdziłem Twój sposób - nie działa ;/
Zapytanie i tak się wykonuje... Generowane są kolejne kody ;|
Wie ktoś w czym jest błąd lub ma inna propozycję?

Pomyśl tak.... Sprawdzanie istnienia wpisu o podanym session_id na samym starcie skryptu Wiem, że to dodatkowe zapytanie do bazy, ale pamiętaj, że SELECT będący po UPDATE jest warunkowy. O co mi chodzi? Ów SELECT następuje tylko gdy UPDATE faktycznie się dokonał, a więc gdy mysql_affected_rows zwróci coś innego niż 0. Można więc napisać to tak:
- robimy zapytanie wyciągające kod o podanym session_id,
- jeśli dostaniemy coś, to znaczy, że kod już dla tego session_id istnieje i po prostu go wyświetlamy,
- jeśli kod nie istnieje to robimy update rezerwując jeden z puli pozostałych,
- jeśli mysql_affected_rows zwróci coś innego niż 0 to podajemy ten kod
- jeśli dostaniemy z mysql_affected_rows 0 to wyświetlamy info o braku wolnych kodów

Chyba prosty algorytm?

Próbowałem przyporządkować mysql_affected_rows do tego skryptu. Wyskakiwał błąd... nie pamiętam treści...
Nie do końca Ciebie rozumiem... Przed UPDATE mam zrobić jeszcze inne zapytanie czy jak?
Miło by było gdybyś podał jakiś przykład

Tak... Przed UPDATE sprawdź czy istnieje w bazie już rekord z określonym session_id. Jeśli tak, to znaczy, że kod już jest w bazie i nie musisz rezerwować go dla owego usera, Tak więc wtedy UPDATE możesz sobie darować. A chyba o to Ci chodziło z tego co wywnioskowałem Zauważ, że wtedy przy przeładowaniu już kod z podanym session_id znajdzie i nie zarezerwuje nowego. No chyba, że w międzyczasie zajdzie sytuacja z regeneracja id sesji. No ale to niezwykle rzadkie przypadki.

Pseudokod:

[PHP] pobierz, plaintext 
$res = query('SELECT kod FROM codes WHERE used by = \''.session_id().'\'');
if( $res AND sql_num_rows($res) > 0 ) {
  //wyświetl ów kod
} else {
  $nowy = query( 'UPDATE token SET used_by = \''.$sesion_id().'\' WHERE used_by = 0 ORDER BY id ASC LIMIT 1' );
  if( $nowy AND sql_affected_rows() > 0 ) {
    $kod = query( 'SELECT token FROM token WHERE used_by = \''.session_id().'\' ORDER BY id DESC LIMIT 1' );
    if( $kod AND sql_num_rows( $kod ) > 0 ) {
      //wyświetl kod
    } else {
      //komunikat błędu
    }
  } else {
    //komunikat błędu
  }
}
[PHP] pobierz, plaintext 

Rozumiem, ze w tym przypadku wszędzie zastosować muszę frazę if( $kod AND sql_num_rows( $kod ) > 0 ) zamiast. mysql_query($query1); ?
Bo zapytanie już chyba wiem jak zbudowac. A } else { chyba są tutaj zbędne ;0

Zapytanie przed UPDATE bedziewiec takie?:

[PHP] pobierz, plaintext 
$query2 = query('SELECT token FROM token WHERE used_by = \''.session_id().'\'');
if( $query2 AND sql_num_rows($query2) > 0 ) {
[PHP] pobierz, plaintext 

Jedynymi rzeczami jakie musisz zrobić to wstawienie odpowiedniej partii kodu w miejsca przeze mnie opisane, czyli wyświetlanie kodu lub błędu. No i oczywiście zastosowanie mssql, mysql, mysqli czy pgsql bądź innych w zależności od stosowanej bazy danych.

Przyjrzyj się strukturze pseudo-kodu i przeanalizuj co on robi a sam odpowiesz sobie na pytanie "Co muszę poprawić/dopisać?". To naprawdę nie jest trudne. Myślę, że sobie z tym powinieneś poradzić gdy po prostu spokojnie siądziesz i popatrzysz co on ma robić. To nie jest fizyka kwantowa tylko kilka prostych warunków, które w zależności od wyniku coś wykonują bądź nie.

Ok. Zrobiłem tak jak radziłeś...
Cały czas wyskakuje mi błąd w } else {

KOD:

[PHP] pobierz, plaintext 
<?php
 
 
include "sprawdzacz.php";
 
include('config.php');
 
 
$res = query('SELECT token FROM token WHERE used by = \''.session_id().'\'');
if( $res AND sql_num_rows($res) > 0 )
 
{
echo "<center>$token</center>";
} else {
 
$nowy = query( 'UPDATE token SET used_by = \''.session_id().'\' WHERE used_by = 0 ORDER BY id ASC LIMIT 1' );
if( $nowy AND sql_affected_rows() > 0 )
 
$kod = query( 'SELECT token FROM token WHERE used_by = \''.session_id().'\' ORDER BY id DESC LIMIT 1' );
if( $kod AND sql_num_rows( $kod ) > 0 )
 
$result=mysql_query($query);
 
 
 
echo "<b><center>Wygenerowany kod:</center></b><br><br>";
 
 
{
 
$token=mysql_result($result,"token");
 
echo "<center>$token</center>";
 
}
 
} else {
BLAD 1
}
} else {
BLAD 2
}
}
 
?>
[PHP] pobierz, plaintext 

Próbowałem wiele sposobów... wszystko przyporządkowałem poprawnie - w każdym razie tak sądzę.
Wiesz może co w tym kodzie powoduje ten błąd? Zawsze zapytanie zwracało błąd w wersach 20+ więc początek zrobiłem chyba poprawnie.

nic dziwnego że nie działa jak masz

[PHP] pobierz, plaintext 
if (true) {
 
} else {
 
} else {
 
}
[PHP] pobierz, plaintext 

Czyli tych }else{ nie powinno być... Dobrze rozumiem? Albo jakieś sugestie?

Popatrz na wcięcia u mnie w pseudokodzie. Specjalnie stosuje je, by inni widzieli jaki blok jest gdzie. Na jakim poziomie zagnieżdżenia się on zaczyna, kończy i znajduje. Źle pozagnieżdżałeś te IFy w sobie i stąd masz błędy.

Chodzi Tobie o brak { w 2 i 3 IFie (na końcu)? Fakt; tutaj chyba tez mam błąd
Co do rozmieszczenia to są chyba dobrze. Może sprecyzowalbys dokładniej w którym fragmencie... Zawezi to mój obszar poszukiwań...

to swoja drogą z tymi dwoma {, ale zauważ, ze potem masz otwarcie i zamknięcie bloku "z powietrza". Gdzie? Wokół
$token=mysql_result($result,"token");
echo "<center>$token</center>";
masz { i } które nie sa kompletnie z niczym związane. Popatrz na ten kod teraz:

[PHP] pobierz, plaintext 
<?php
include "sprawdzacz.php";
include('config.php');
$res = query('SELECT token FROM token WHERE used by = \''.session_id().'\'');
if( $res AND sql_num_rows($res) > 0 ) {
    //tu brak mysql_result dla tokena
    echo "<center>$token</center>";
} else {
    $nowy = query( 'UPDATE token SET used_by = \''.session_id().'\' WHERE used_by = 0 ORDER BY id ASC LIMIT 1' );
    if( $nowy AND sql_affected_rows() > 0 ) {
        $kod = query( 'SELECT token FROM token WHERE used_by = \''.session_id().'\' ORDER BY id DESC LIMIT 1' );
        if( $kod AND sql_num_rows( $kod ) > 0 ) {
            $result=mysql_query($query);
            echo "<b><center>Wygenerowany kod:</center></b><br><br>";
            $token=mysql_result($result,"token");
            echo "<center>$token</center>";
        } else {
            BLAD 1
        }
    } else {
        BLAD 2
    }
}
?>
[PHP] pobierz, plaintext 

i powiedz, który lepiej się czyta? Twój z wszystkim pociągniętym do marginesu lewego czy może mój z wcięciami?

Bez porównania Twój łatwiej się czyta

No dobrze. Poprawiłem na:

[PHP] pobierz, plaintext 
<?php
include "sprawdzacz.php";
include('config.php');
$res = mysql_query('SELECT token FROM token WHERE used by = \''.session_id().'\'');
if( $res AND sql_num_rows($res) > 0 ) {
    $token=mysql_result($result,"token");
    echo "<center>$token</center>";
} else {
    $nowy = mysql_query( 'UPDATE token SET used_by = \''.session_id().'\' WHERE used_by = 0 ORDER BY id ASC LIMIT 1' );
    if( $nowy AND sql_affected_rows() > 0 ) {
        $kod = mysql_query( 'SELECT token FROM token WHERE used_by = \''.session_id().'\' ORDER BY id DESC LIMIT 1' );
        if( $kod AND sql_num_rows( $kod ) > 0 ) {
            $result=mysql_query($query);
            echo "<b><center>Wygenerowany kod:</center></b><br><br>";
            $token=mysql_result($result,"token");
            echo "<center>$token</center>";
        } else {
            BLAD
        }
    } else {
        BLAD
    }
}
?>
[PHP] pobierz, plaintext 

I tym razem pokazuje się podobny error jak poprzednio...

[SQL] pobierz, plaintext 
Parse error: syntax error, unexpected '}' IN /home/modernwe/public_html/dotpay_kod_skrypty/main.php ON line 19
[SQL] pobierz, plaintext 

Wers 19 tyczy się } po pierwszym "BLAD" ;/
A i... zmieniłem na mysql_query
Co tym razem radzisz szefie ;>?

Linie w tym kodzie ostatniego posta: 5, 10, 12... Czy istnieje funkcja sql_num_rows lub sql_affected_rows? Przecież pisałem, że to pseudo-kod. dodaj przed nimi my by utworzyło mysql a powinno pójść.

Nie działa... TEN SAM BŁĄD:

[SQL] pobierz, plaintext 
Parse error: syntax error, unexpected '}' IN /home/modernwe/public_html/dotpay_kod_skrypty/main.php ON line 19
[SQL] pobierz, plaintext 

Ech... Bo nie można na pałe tam wpisać BLAD... PHP nie wie co to takiego. Wpisz zamiast obu BLAD:

[PHP] pobierz, plaintext 
echo 'Tu wyskoczy błąd, który muszę obsłużyć';
[PHP] pobierz, plaintext 

Dobra wiadomość... Poprzednia błąd zniknął ;D, a zła ze mam kolejny ;/

[SQL] pobierz, plaintext 
Warning: mysql_result(): supplied argument IS NOT a valid MySQL result resource IN /home/modernwe/public_html/dotpay_kod_skrypty/main.php ON line 15
[SQL] pobierz, plaintext 

Przecież mysql_result to prawidłowy argument MySQL :|

A chociaż popatrzyłeś co wysyłasz do mysql_result jako zmienna czy nie? Bo jak dla mnie ten fragment Ci bruździ.

[PHP] pobierz, plaintext 
if( $kod AND mysql_num_rows( $kod ) > 0 ) {
$result=mysql_query($query);
echo "<b><center>Wygenerowany kod:</center></b><br><br>";
$token=mysql_result($result,"token");
echo "<center>$token</center>";
}
[PHP] pobierz, plaintext 

A czemu? Skoro wynik zapytania masz w $kod, to skąd Ci się bierze nagle mysql_query($query)?
Tam masz mieć po prostu:

[PHP] pobierz, plaintext 
if( $kod AND mysql_num_rows( $kod ) > 0 ) {
echo "<b><center>Wygenerowany kod:</center></b><br><br>";
$token=mysql_result($kod,"token");//lub $token = mysql_result($kod, 0);
echo "<center>$token</center>";
}
[PHP] pobierz, plaintext 

. Bo tak masz zmienne w kodzie ni z gruchy ni z pietruchy I pomyśl trochę sam też, bo można pomoc jeden raz, drugi, trzeci, ale nie jest nikt kompilatorem, by Ci non stop poprawiać proste błędy i wskazywać gdzie są. Chcesz być programistą PHP choć w minimalnym stopniu? Naucz się to zauważać sam.

Nad tym teraz pomyślałem...:
Jakim cudem to może być złe umieszczone? mysql_result znowu mi tu szwankuje... Tym razem po "Wygenerowany kod".

[PHP] pobierz, plaintext 
echo "<b><center>Wygenerowany kod:</center></b><br><br>";            
            $token = mysql_result($result,"token");         
            echo "<center>$token</center>";
        } else {
[PHP] pobierz, plaintext 

Błąd:

[SQL] pobierz, plaintext 
Warning: mysql_result(): supplied argument IS NOT a valid MySQL result resource IN /home/modernwe/public_html/dotpay_kod_skrypty/main.php ON line 14
[SQL] pobierz, plaintext 

P.S.
Polecisz jakiej dobre tutoriale/kursy programowania w języku PHP?

Ech... Przyjrzałeś się choć dobrze temu co wkleilem i co napisałem? Czy odróżniasz $result od $kod ?

Co do tutoriali, kursów to ja się uczyłem z dokumentacji i manuala wspomaganych przez google, bo taki mam "nawyk", ale dla większości to bardzo ciężka lektura do przełknięcia zważywszy na ilość stron do choćby pobieżnego przejrzenia.

Faktycznie... $result = $kod. Teraz się wyświetla ale po odswiezeniu losuje się nowy kod ;/ punkt wyjścia...

Sprawdź więc co za zapytania idą do bazy. Dzięki temu namierzysz ewentualny błąd. Od strony logiki oznaczało by to bowiem, że co odświeżenie zmienia się u Ciebie session_id. A tak może być jedynie gdy sesja ma ustawiony wysoki współczynnik regeneracji. Tyle, że to nie jest defaultowe ustawienie, a wątpie, że cokolwiek zmieniałeś.

Nic nie zmienialem wiec musi to być błąd kodu...

[PHP] pobierz, plaintext 
<?php
include "sprawdzacz.php";
include('config.php');
$res = mysql_query('SELECT token FROM token WHERE used by = \''.session_id().'\'');
if( $res AND mysql_num_rows($res) > 0 ) {
    $token = mysql_result($result,"token");
    echo "<center>$token</center>";
} else {
    $nowy = mysql_query( 'UPDATE token SET used_by = \''.session_id().'\' WHERE used_by = 0 ORDER BY id ASC LIMIT 1' );
    if( $nowy AND mysql_affected_rows() > 0 ) {
        $kod = mysql_query( 'SELECT token FROM token WHERE used_by = \''.session_id().'\' ORDER BY id DESC LIMIT 1' );
        if( $kod AND mysql_num_rows( $kod ) > 0 ) {
            echo "<b><center>Wygenerowany kod:</center></b><br><br>";            
            $token = mysql_result($kod,"token");         
            echo "<center>$token</center>";
        } else {
           echo 'Tu wyskoczy b??d, który musz? obs?u?y?';
        }
    } else {
        echo 'Tu wyskoczy b??d, który musz? obs?u?y?';
    }
}
?>
[PHP] pobierz, plaintext 

Wydaje mi się ze błąd jest w kolejności zapytania... Albo nie wiem ;/

Pizę jeszcze raz... Wyświetl sobie JAK wyglądają zapytania idące do bazy. Poza tym myślałem, że po naprowadzeniu Cię z tym $kod przejrzysz całość ponownie. To nie jedyne bowiem miejsce gdzie tego samego babola walnąłeś. W szale poprawek poprawiłeś to w jednym miejscu źle, a w innym zapomniałeś to zrobić. Przejrzyj jeszcze raz zamiast co i rusz wklejać kod. Pisałem już, że forum i użytkownicy tutaj to nie "inteligentny interpreter php" który napisze Ci gdzie błąd i jak go poprawić. Naucz myśleć samodzielnie zanim będziemy musieli ten temat zamknąć bo zwyczajnie traktujesz forum teraz jak własne podwórko wrzucając co i rusz nowe wersje bez przemyślenia. Popatrz na błędy jakie Ci wypluwa interpreter i zacznij myśleć zanim po raz kolejny wciśniesz Dodaj Odpowiedź. Dostałeś niemal gotowca ode mnie, a i tak robisz proste błędy logiczne. To świadczy o naprawdę słabych podstawach PHP u Ciebie i powinieneś się za to wziąć jak najszybciej.

Faktycznie pominalem ten fragment o ile się nie mylę...

[PHP] pobierz, plaintext 
<?php
include "sprawdzacz.php";
include('config.php');
$res = mysql_query('SELECT token FROM token WHERE used by = \''.session_id().'\'');
if( $res AND mysql_num_rows($res) > 0 ) {
    $token1 = mysql_result($res,"token1");
    echo "<center>$token1</center>";
} else {
[PHP] pobierz, plaintext 

W kodzie błędów nie widzę. Warningi nie wyswietlaja się już ale mysql_num_rows jakby nie działa ;/

Właśnie dlatego mówiłem: "Sprawdzaj JAKIE zapytanie idzie do bazy." Jeśli idzie z jakimś bykiem to nigdy nie dowiesz się jakim, jeśli nie zobaczysz jak ono wygląda i nie sprawdzisz czy choćby session_id się nie zmienia. Pisałem o współczynniku regeneracji sesji. Poczytaj o tym, a sam zobaczysz, że takie coś MOŻE mieć miejsce. Trochę własnej inwencji w wyszukiwaniu błędów. Od strony kodu w zasadzie nie można się już przyczepić do czegoś konkretnego. Jeśli istnieje, to właśnie w wyniku jakichś drobiazgów. Jeśli bowiem po sprawdzeniu nie znajduje żadnych kodów z podanym session_id, to znaczy, że brak ich w bazie faktycznie lub... wcale sesja nie istnieje! Bo session_id() w przypadku braku sesji zwraca ciąg pusty. Dlatego możesz miec przez to głupoty w zapytaniu. I stąd ciągle się upraszam o sprawdzenie tego zapytania.

PROBLEM ROZWIĄZANY!!!
przy $res było used by zamiast used_by

Dziękuję za pomoc Thek. Znasz się na tym jak nikt inny ;D



TEMAT DO ZAMKNIĘCIA