Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> [PHP]Prosta gra na klasie, Singleton, sesja
mimol
post 24.02.2013, 12:00:28
Post #1





Grupa: Zarejestrowani
Postów: 247
Pomógł: 5
Dołączył: 10.12.2007

Ostrzeżenie: (0%)
-----

Witam. Napisałem sobie klasę game
[PHP] pobierz, plaintext 
  1. <?php
  2. class Game
  3. {
  4.     private $number;
  5.     private $try;
  6.     private static $instance;    
  7.  
  8.     public static function getInstance() 
  9.     {
  10.         if(self::$instance === null) {
  11.             self::$instance = new Game();
  12.         }
  13.         return self::$instance;
  14.     }
  15.  
  16.     public function __construct() 
  17.     {
  18.         $this->number = rand(1, 100);
  19.         $this->try    = 1;
  20.     }
  21.  
  22.     public function type($type)
  23.     {
  24.         if ($type == $this->number){
  25.             echo 'Gratz. You need '.$this->try.' tries';
  26.         } elseif ($type < $this->number){
  27.             echo 'Select greater number';
  28.         } else {
  29.             echo 'Select smaller number';
  30.         }
  31.         $this->try++;      
  32.     }
  33. }
  34.  
  35. ?>
[PHP] pobierz, plaintext

No i oczywiście plik z formularzem.
[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();
  3. require 'game.php';
  4.  
  5. if (isset($_SESSION['game'])) {
  6.     $game = unserialize($_SESSION['game']);
  7. } else {
  8.     echo 'set';
  9.     $game = Game::getInstance();
  10.     $_SESSION['game'] = serialize($game);
  11. }
  12.  
  13. echo '<html><head><title>The Game</title></head><body>';
  14. echo '<form method="GET" enctype="text/plain">
  15.       Number: <input type="text" name="number"  pattern="\d*" />
  16.       <input type="submit" />
  17.       </form>';   
  18.  
  19. if (isset($_GET['number'])) {
  20.     $type = $_GET['number'];
  21.     $game->type($type);
  22. }   
  23.  
  24. ?>
  25. </body></html>
[PHP] pobierz, plaintext

Moje pytanie brzmi, czy da to się łatwiej zrobić? (bez wykorzystywania sesji, raz ustawić obiekt i potem cały czas z niego korzystać) Po co używać wzorca singleton. Przecież podczas pisania kodu, osoba, która go pisze wie czy utworzyła już dany obiekt czy nie...
Go to the top of the page
+Quote Post
CuteOne
post 24.02.2013, 12:17:56
Post #2





Grupa: Zarejestrowani
Postów: 2 958
Pomógł: 574
Dołączył: 23.09.2008
Skąd: wiesz, że tu jestem?

Ostrzeżenie: (0%)
-----

Praktyczny przykład wykorzystania singletona 
[PHP] pobierz, plaintext 
  1. class MojSingleton {}
  2.  
  3. class InnaKlasa{
  4.  
  5.     public function add() {
  6.         $mojSingleton = MojSingleton:: getInstance();
  7.     }
  8. }
[PHP] pobierz, plaintext



Właściwie po co ci ta klasa game? I w jakim celu zapisujesz ją do sesji zamiast stworzyć osobny obiekt dla sesji i wrzucać do niego dane z game?
Go to the top of the page
+Quote Post
mimol
post 24.02.2013, 12:21:29
Post #3





Grupa: Zarejestrowani
Postów: 247
Pomógł: 5
Dołączył: 10.12.2007

Ostrzeżenie: (0%)
-----

Mógłbyś napisać jak to osiągnąć?
W klasie game, mam tworzyć sesje, a w formularzu ją odczytywać?

Ten post edytował mimol 24.02.2013, 12:22:18
Go to the top of the page
+Quote Post
CuteOne
post 24.02.2013, 12:26:05
Post #4





Grupa: Zarejestrowani
Postów: 2 958
Pomógł: 574
Dołączył: 23.09.2008
Skąd: wiesz, że tu jestem?

Ostrzeżenie: (0%)
-----

Obiekt powinien zajmować się tylko jedną rzeczą, więc nie staraj się tworzyć klasy od wszystkiego.

- stwórz klasę przechowującą sesje (google session class example)
- niech game odpowiada tylko za siebie i za pomocą getterów pobieraj od niej dane do klasy obsługującej sesję
Go to the top of the page
+Quote Post
mimol
post 24.02.2013, 12:41:57
Post #5





Grupa: Zarejestrowani
Postów: 247
Pomógł: 5
Dołączył: 10.12.2007

Ostrzeżenie: (0%)
-----

Nie za bardzo rozumiem... Nie chcę tworzyć klasy sesji na siłę... czy to jest rzeczywiście konieczne?
Wsadzam obkiet Game do sesji ponieważ chcę skorzystać z funkcji type().
Czy według ciebie powinienem w sesji zapisywać number, try?? Co jeśli obiekt przechowuje więcej danych, wtedy będzie mały bałagan, za każdym razem odczytwać te dane...
Mógłbyś mi to troszkę jaśniej wytłumaczyć?

Najprostsza klasa sesji, którą udało mi się znaleźć TU czy jest ona wystarczająca? (Po przerobieniu na PHP5)
lub TU

Ten post edytował mimol 24.02.2013, 12:46:26
Go to the top of the page
+Quote Post
Crozin
post 24.02.2013, 13:00:09
Post #6





Grupa: Zarejestrowani
Postów: 6 476
Pomógł: 1306
Dołączył: 6.08.2006
Skąd: Kraków

Ostrzeżenie: (0%)
-----

1. Nie potrzebujesz korzystać z własnej klasy obsługi sesji. Domyślny mechanizm wbudowany w PHP w pełni tutaj wystarczy.
2. Nie ma potrzeby ręcznie (de)serializować obiektów "wrzucanych do sesji". Cała tablica $_SESSION i tak jest (de)serializowana przy zapisie/odczycie z pliku/pamięci.
3. Nie ma tutaj ani jednego powodu dla którego miałbyś korzystać z singletona, nie powinieneś tego robić. Przykład "praktycznego wykorzystania singletona" podany przez @CuteOne to jeden z paskudniejszych potworków jakie można umieścić w swoim kodzie - takiego czegoś również nie powinieneś praktykować. wink.gif
4. Odpowiadając na Twoje pytanie: musisz ręcznie umieścić obiekt w sesji i ręcznie go stamtąd odczytać. Nie mniej jednak możesz troszkę uprościć swój kod:
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. session_start();
  4.  
  5. if (!isset($_SESSION['game'])) {
  6.   $_SESSION['game'] = new Game();
  7. }
  8.  
  9. $game = $_SESSION['game'];
  10.  
  11. if (isset($_GET['type'])) {
  12.   $game->type($_GET['type']);
  13. }
  14.  
  15. // formularz
[PHP] pobierz, plaintext
5. Właściwość Game::try powinna nazywać się Game::tries, a komunikat o wygranej brzmieć: You needed n tries..
6. Metoda Game::type nie powinna raczej wyświetlać komunikatów tylko zwracać informację o wyniku swojego działania. Tutaj najlogiczniejszym wydaje się zwrócenie liczby mniejszej od zera w przypadku podania za małej liczby, większej od zera w przypadku za dużej i w końcu zera w przypadku trafienia. Dzięki temu Twoja klasa będzie bardziej uniwersalna - będziesz mógł ją wykorzystać w dowolny sposób, nie tylko w tym jednym konkretnym formularzu.

Ten post edytował Crozin 24.02.2013, 13:00:38
Go to the top of the page
+Quote Post
mimol
post 24.02.2013, 13:13:25
Post #7





Grupa: Zarejestrowani
Postów: 247
Pomógł: 5
Dołączył: 10.12.2007

Ostrzeżenie: (0%)
-----

Cytat
Nie ma potrzeby ręcznie (de)serializować obiektów "wrzucanych do sesji".
Też mi się tak wydaje
Jednak jeśli nie serializuje to dostaje error
[PHP] pobierz, plaintext 
  1. Fatal error: main(): The script tried to execute a method or access a property of an incomplete object. Please ensure that the class definition "Game" of the object you are trying to operate on was loaded _before_ unserialize() gets called or provide a __autoload() function to load the class definition in C:\xampp\htdocs\game\index.php on line 19
[PHP] pobierz, plaintext

Dlaczego w tym przypadku muszę recznie serializować ? Czy może jest inny sposób na pozbycie się tego błędu?


[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();
  3. require 'game.php';
  4.  
  5. if (!isset($_SESSION['game'])) {
  6.     $_SESSION['game'] = new Game();
  7. }
  8.  
  9. $game = $_SESSION['game'];
  10.  
  11. echo '<html><head><title>The Game</title></head><body>';
  12. echo '<form method="POST">
  13.       Number: <input type="text" name="number"  pattern="\d*" />
  14.       <input type="submit" />
  15.       </form>';   
  16.  
  17. if (isset($_POST['number'])) {
  18.     $type = $_POST['number'];
  19.     $game->type($type);
  20. }   
  21.  
  22. ?>
  23. </body></html>
[PHP] pobierz, plaintext


Pojawia się jeszcze jeden problem, którego nie przemyślałem... Z tym, że zawsze obiekt odczytuje z sesji, nigdy nie zwiększa się liczba prób, w tym wypadku można to rozwiązać na 2 sposoby.
a) Zapisywać liczbę prób w sesji.
cool.gif za każdym razem zapisywać obiekt do sesji.
Który lepszy?

Ten post edytował mimol 24.02.2013, 13:23:46
Go to the top of the page
+Quote Post
Crozin
post 24.02.2013, 13:38:15
Post #8





Grupa: Zarejestrowani
Postów: 6 476
Pomógł: 1306
Dołączył: 6.08.2006
Skąd: Kraków

Ostrzeżenie: (0%)
-----

1. W momencie wywołania funkcji session_start() następuje odczytanie i deserializacja danych z pliku (czy gdziekolwiek przechowujesz dane). W swoim kodzie nie zdefiniowałeś żadnego autoloadera, a klasę Game wczytujesz dopiero po wystartowaniu sesji, więc PHP nie ma pojęcia jak zdeserialiozwać obiekt typu Game, bo nie zna typu Game. Powinieneś skorzystać z autoloadera, albo przenieść wczytanie pliku game.php przed session_start().
2. Powinieneś raczej wrzucać cały obiekt do sesji. Gdy pojawią się nowe właściwości w grze nie będziesz musiał przynajmniej bawić się w modyfikowanie mechanizmu gry nigdzie poza samą klasą Game.
3. Jeżeli decydujesz się na wrzucenie obiektu do sesji powinieneś raczej implementować przy nim interfejs Serializable.
Go to the top of the page
+Quote Post
« Następny starszy · Przedszkole · Następny nowszy »
 

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Standardowy · Przełącz na: Linearny+ · Przełącz na: Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Wersja Lo-Fi Aktualny czas: 12.07.2025 - 22:05
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn