Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [PHP]Prosta gra na klasie, Singleton, sesja
mimol
post
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
 
 Start new topic
Odpowiedzi
Crozin
post
Post #2





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ć. (IMG:style_emoticons/default/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

Posty w temacie
- mimol   [PHP]Prosta gra na klasie   24.02.2013, 12:00:28
- - CuteOne   Praktyczny przykład wykorzystania singletona [PHP...   24.02.2013, 12:17:56
- - mimol   Mógłbyś napisać jak to osiągnąć? W klasie game, ma...   24.02.2013, 12:21:29
- - CuteOne   Obiekt powinien zajmować się tylko jedną rzeczą, w...   24.02.2013, 12:26:05
- - mimol   Nie za bardzo rozumiem... Nie chcę tworzyć klasy s...   24.02.2013, 12:41:57
- - Crozin   1. Nie potrzebujesz korzystać z własnej klasy obsł...   24.02.2013, 13:00:09
- - mimol   CytatNie ma potrzeby ręcznie (de)serializować obie...   24.02.2013, 13:13:25
- - Crozin   1. W momencie wywołania funkcji session_start() na...   24.02.2013, 13:38:15

« Następny starszy · Przedszkole · Następny nowszy »
 

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

 

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

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

RSS Aktualny czas: 4.10.2025 - 16:54
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn