[PHP]Sesja - dziwne zachowanie Opcje

[qrzysztof]

Witam,

Mam stronę o następującym szkielecie:

[PHP] pobierz, plaintext

1. <?php
3. session_start();
5. if($_POST['costam'])
6. {
7. $obiekt= new Obiekt;
8. #inne nieistotne instrukcje
9. $_SESSION['obiekt']=$obiekt;
10. }
11. if($_POST['inne_costam'])
12. {
13. $obiekt=$_SESSION['obiekt']
14. $obiekt->zrobCos();
15. #inne nieistotne instrukcje
16. }
19. ?>

[PHP] pobierz, plaintext

Lokalnie wszystko działa dobrze. Jedyny problem jest taki, że wywala w 14. linii jakoby funkcja zrobCos() była wykonywana na non-object, w następujących okolicznościach (wszystkie muszą zajść równocześnie):

1) na serwerze,
2) w przeglądarce firefox,
3) tuż po włączeniu tej przeglądarki (tzn nie było wcześniej ani jednego jej okna, przechodzimy od razu na moją stronę),
4) klikamy szybko (nie jakoś błyskawicznie, ale po prostu szybko żeby wywołać część tworzącą obiekt i część z niego korzystającą.

Rozumiem, że obiekt w pierwszej części kodu nie zdążył jeszcze powstać, kiedy wywoływana już jest w drugiej części funkcja na obiekcie. Tyle, że jest to dla mnie dziwne - takie opóźnienia. Ciekawe, że inne przeglądarki tego problemu nie mają. Da się to jakoś wyprostować? Czy może mam łapać wyjątek z komunikatem "Spróbuj jeszcze raz", "Klikaj wolniej" albo "Nie tak szybko mistrzu"?

Ten post edytował qrzysztof 7.01.2010, 11:41:27

[Meares]

[PHP] pobierz, plaintext 
if(isset($_POST['inne_costam'])) {
  if(!is_object($_SESSION['obiekt'])) $obiekt = new $_SESSION['obiekt']; else $obiekt = $_SESSION['obiekt'];
  $obiekt->zrobCos();
}
[PHP] pobierz, plaintext 

PS. Używaj isset() i empty().

Ten post edytował Meares 7.01.2010, 11:44:51

[qrzysztof]

Jest to na pewno jakieś rozwiązanie, ale uświadomiło mi jedynie, że za mało szczegółów podałem w przykładzie. Tak więc jeszcze raz:

[PHP] pobierz, plaintext 
<?php
 
  session_start();
 
  if(isset($_POST['wazna_dana']))
  {
  $obiekt= new Obiekt;
  $obiekt->zaktualizuj($_POST['wazna_dana']);
  #inne nieistotne instrukcje
  $_SESSION['obiekt']=$obiekt;
  }
 
  if($_POST['inne_costam'])
  {
  $obiekt=$_SESSION['obiekt']
  $obiekt->zrobCos();
  #inne nieistotne instrukcje
  }
 
?>
[PHP] pobierz, plaintext 

Obiekt niestety nie bez przyczyny jest tworzony w pierwszej części kodu. Są na nim tam wykonywane operacje. Bez nich funkcja zrobCos() nie da oczekiwanych rezultatów.

Dodam jeszcze, że POST['wazna_dana'] i POST['inne_costam'] pochodzą z różnych formularzy, umieszczonych na różnych stronach. Także obydwie te zmienne nie mogą zostać przesłane do skryptu równocześnie. Najpierw jest przesyłana jedna, a potem druga.

Ten post edytował qrzysztof 7.01.2010, 11:59:27

[Meares]

A nie możesz zapisać $_POST['wazna_data'] w $_SESSION['wazna_data']? Wtedy wystarczyło by "rozszerzyć" to co podałem wyżej:

[PHP] pobierz, plaintext 
<?php
session_start();
 
if(isset($_POST['wazna_dana'])) {
  $obiekt = new Obiekt;
  $obiekt->zaktualizuj($_POST['wazna_dana']);
  $_SESSION['obiekt'] = $obiekt;
  $_SESSION['wazna_data'] = $_POST['wazna_data'];
}
 
if(!isset($_POST['inne_costam'])) {
  if(!is_object($_SESSION['obiekt'])) {
    $obiekt = new $_SESSION['obiekt'];
    $obiekt->zaktualizuj($_SESSION['wazna_dana']);
  } else $obiekt = $_SESSION['obiekt'];
  $obiekt->zrobCos();
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Meares 7.01.2010, 12:17:06

[qrzysztof]

Owszem, mogę tak zrobić. Ale nie bardzo mi się podoba to rozwiązanie. Uszczegółowię już do końca mój kod

[PHP] pobierz, plaintext 
<?php 
 
      session_start();
 
      if(isset($_POST['wazna_dana']))
      {
      $obiekt= new Obiekt();
      $obiekt->zaktualizuj($_POST['wazna_dana']);
      $obiekt->pobierzZBazy();
      $_SESSION['obiekt']=$obiekt;
 
      $ekran=new Strona1($obiekt);
      $ekran->wyswietlStrone();
      }
 
 
      if($_POST['inne_costam'])
      {
        $obiekt=$_SESSION['obiekt'];
        $obiekt->zrobCos();
 
        $ekran= new Strona2($obiekt);
        $ekran->wyswietlStrone();
      }
?>
[PHP] pobierz, plaintext 

Mogę zachować zmienną "wazna_dana" w sesji a następnie przy odbraniu POST['inne_costam'] zaktualizować obiekt, pobrać dane z bazy i dopiero później wykonać funkcję zrobCos().

Odkryłem jednak coś dziwnego. Polecenie

[PHP] pobierz, plaintext 
$ekran=new Strona1($obiekt);
$ekran->wyswietlStrone();
[PHP] pobierz, plaintext 

wykonuje się bezbłędnie. A to znaczy, że obiekt w tym momencie był w pełni funkcjonalny. Problematyczna zatem jest część:

[PHP] pobierz, plaintext 
$obiekt=$_SESSION['obiekt'];
$obiekt->zrobCos();
[PHP] pobierz, plaintext 

Zanim obiekt zostanie rozpakowany i przypisany do zmiennej - wywoływana jest już funkcja zrobCos(). W tym układzie zastosowałem lepsze obejście. Umieściłem to przypisanie w pętli.

[PHP] pobierz, plaintext 
do
{            
} while (!$obiekt=$_SESSION['obiekt']);
[PHP] pobierz, plaintext 

I działa.

Tyle, że to nie jest rozwiązanie tylko właśnie obejście. Bo nadal nie jestem ani trochę mądrzejszy. Jeśli jest faktycznie tak, jak napisałem to czemu, do diabła, problem występował tylko w firefoxie? Na czuja coś mi podpowiada, że tu chodzi o jakieś parametr, który wpływałby na czas dostępu sesji do obiektu i pewnie jest bardziej eleganckie rozwiązanie tego problemu.

[Meares]

Może masz wyłączone cookies w FF? Sesje przechowują swoje dane w plikach tekstowych (w połączeniu z $_GET), lub właśnie w cookies.

[qrzysztof]

Nie. Tak jak pisałem - wszystko zależy od szybkości klikania. Przy szybszym są te błędy, przy wolniejszym nie ma.

A niestety moje obejście rozwiązuje problem tylko w jednym miejscu.

W innym gdzie kod zależy od tego czy istnieje SESSION['obiekt'] czy nie, już tego nie da się zastosować.

Jeśli klika się szybko to nigdy nie istnieje.

Jeśli wolniej to wtedy dopiero może istnieć bądź nie. Bo sprawdzania czy istnieje w pętli już nie zrobię. Bez sensu - sprawdza czy istnieje dopóki nie zaistnieje?

Dziwny ten firefox.