Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> Ocena skryptu.
-arrtxp-
post
Post #1





Goście
Witam, czytając o MVC, stworzyłem własne rozwiązanie i chciałbym abyście oceni mój kod, gdzie mniej więcej przedstawiam system działania:

// struktura najważniejszych plików
[PHP] pobierz, plaintext 
  1. /a-control // folder gdzie mamy klasy
  2. + klasy
  3. /mod-view // folder z plikami html
  4. + pliki z html
  5.  
  6. index.php
[PHP] pobierz, plaintext


index.php 

[PHP] pobierz, plaintext 
  1. <?php
  2. 	ob_start();
  3. 	header('Content-type: text/html;charset=utf-8');
  4. 	date_default_timezone_set('Europe/Warsaw');
  5. 	session_start();
  6.  
  7. 	include('./a-control/aStart.php'); // klasa, która waliduję mi dane (np: aStart::spr_int($dane);) i w niej ustawiam połączenie z bazą.
  8. 	include('./a-control/aLoadClass.php'); // jak sama nazwa mówi
  9.  
  10. 	// ustawienia loadera class
  11. 	$firstLoader = new ClassLoader('i', './a-control/');
  12. 	$firstLoader->register();
  13.  
  14. 	// generowanie strony
  15. 	if(empty($_GET['v'])) { $_GET['v'] = 'start'; }
  16. 	$u = iPage::loadClass($_GET['v']); // wywołanie odpowiedniej kasy
  17. 	$u->view();
  18. 	ob_end_flush();
  19. ?>	
[PHP] pobierz, plaintext


iPage.php // ustala to co ma być wyświetlone
[PHP] pobierz, plaintext 
  1. <?php
  2. 	abstract class iPage
  3. 	{
  4. 		public static $_page = 'start';
  5. 		public $fd_www = '/www/'; // folder z strona
  6. 		public static $db_key = 'baza'; // baza danych - prefix
  7. 		public $_msg = '';		
  8.  
  9. 		// wyczytywanie odpowiedniej klasy	
  10. 		public static function loadClass($page) 
  11. 		{
  12. 			self::$_page = aStart::spr_txt($page);
  13. 			$i_class = array(
  14. 				'index' => 'iView_Index',
  15.  		         );
  16.  
  17. 		        if(!empty($i_class[self::$_page])) 
  18. 		        {
  19.  				 return new $i_class[self::$_page];
  20. 			} 
  21. 			else 
  22. 			{ 
  23. 				return new iStart; 
  24. 			}
  25. 		 }
  26.  
  27. 		// generowanie strony
  28. 		public function view() 
  29. 		{
  30. 		       $file = $this->fd_www.'/mod-view/'.self::$_page.'.php';
  31. 			if(file_exists($file)) 
  32. 			{			
  33. 				include($file);
  34. 			} 
  35. 			else 
  36. 			{
  37. 			      self::$_page = 'error';				
  38. 			      include($this->fd_www.'/mod-view/error.php');
  39.   			  }
  40.                  }
  41. 		 // koniec	
  42. 	}
  43. ?>	
[PHP] pobierz, plaintext


a-control/iView_Index.php 
[PHP] pobierz, plaintext 
  1. <?php
  2. 	class iStart extends iPage
  3. 	{
  4. 		public function __construct() {
  5. 			// tutaj sobie sprawdzam co ma być wywołane, np:     
  6.                        if(isset($_GET['zaloguj'])) { $this->zaloguj(); }  
  7. 		}
  8. 		public function zaloguj()
  9.                 {
  10.                      // zapytanie do bazy, czy coś w tym stylu
  11.                 }
  12.                 public function dom()
  13.                 {
  14.                        return 'Ale Twoja chata to ruina.';
  15.                 }
  16. 	}
  17. ?>
[PHP] pobierz, plaintext


mod-view/index.php - widok

[PHP] pobierz, plaintext 
  1. <html>
  2. <head>
  3. <title>Dom</tilte>
  4. <//head>
  5. <body>
  6.       <?php echo $this->dom(); ?>
  7. </body>
  8. </html>
[PHP] pobierz, plaintext


Teraz chciałbym dowiedzieć się czy takie rozwiązanie jest złe. Jakie stwarza problemy takie rozwiązanie?
Go to the top of the page
+Quote Post
Daimos
post
Post #2





Grupa: Zarejestrowani
Postów: 1 319
Pomógł: 118
Dołączył: 26.11.2003
Skąd: Lublin

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

1. Gdzie przekazujesz zmienne do widoku? Bo chyba o tym zapomniałeś.
2. Jak przekażesz z kontrolera np. tytuł strony do szablonu?
3. Jedna z metod Twojego kontrolera zwraca string, dlaczego, po co?
4. MVC, gdzie model? (IMG:style_emoticons/default/smile.gif)
5.
Cytat
/a-control // folder gdzie mamy klasy
nie klasy tylko klasy kontrolera, bo w katalogu np. a-model będziesz trzymał też "klasy", ale chodzi o modele.
6.
Cytat
[PHP] pobierz, plaintext 
  1. $i_class = array(
  2. 				'index' => 'iView_Index',
  3.  		         );
  4.  
  5. 		        if(!empty($i_class[self::$_page])) 
  6. 		        {
  7.  				 return new $i_class[self::$_page];
  8. 			}
[PHP] pobierz, plaintext

Czyli patrząc na to, każdy kontroler musisz zapisać tutaj w tablicy. Inaczej go nie uruchomisz. Po co w każdym kontrolerze, ładować pełną listę, np. 50-100 kontrolerów ?
Ustal nazewnictwo plików, a następnie sprawdzaj, czy kontroler istnieje jako plik. Później sprawdź czy jest tam wywoływana klasa / metoda. Do tego musisz napisać jakąś "klasę", która zajmie się wszystkim. Nie rób wszystkiego w KAŻDYM kontrolerze.

Ten post edytował Daimos 29.11.2014, 16:40:19
Go to the top of the page
+Quote Post
-arrtxp-
post
Post #3





Goście
ad:
1. Nie muszę, wszystko dzieje się w obiekcje, po dane sięgam poprzez - $this->coś
5. Dlaczego nie mogę trzymać wszystkiego w jednym miejscu ?
6. Z tym 1 masz rację.

[PHP] pobierz, plaintext 
  1. Do tego musisz napisać jakąś "klasę", która zajmie się wszystkim. Nie rób wszystkiego w KAŻDYM kontrolerze.
[PHP] pobierz, plaintext


klasa abstrakcyjna o nazwie iPage robi wszystko, czyli ustala " moduł " i " widok ".

Ogólnie to działa w ten sposób, np: strona.pl?v=news&nowy

poprzez iPage::loadClass($_GET['v']); następuje wyczytanie klasy i utworzenie obiektu.
A w tej klasie np: News poprzez __construct() następuje załadowanie odpowiednich danych/ wywołanie metod (przez $_GET['news']), które są ładowanie do widoku.
To wszystko tworzy całość.

Czy te rozwiązanie ogranicza mnie w jakiś sposób ? wg mnie, nie (albo nie mam o tym pojęcia, dlatego się pytam). Tworzę widok, klasa obsługująca i mam dodane rozszerzenie. I pytam się czy takie rozwiązanie mnie ogranicza i czy jest złe ? Jak tak to dlaczego ?

Ten post edytował arrtxp 29.11.2014, 18:07:37
Go to the top of the page
+Quote Post
Daimos
post
Post #4





Grupa: Zarejestrowani
Postów: 1 319
Pomógł: 118
Dołączył: 26.11.2003
Skąd: Lublin

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

Zacznijmy od tego, czym jest widok, kontroler i model.
Jeden kontroler nie służy do obsługi całej aplikacji. Powinieneś mieć jakiegoś bootstrapa, klasę która odpowiada za request, routing itp. Najlepiej jakbyś podejrzał jak to jest rozwiązane w innych frameworkach.
Kontroler powinien posiadać jak najmniej kodu, każda akcja powinna odpowiadać za konkretny request. Nie faszeruj kontrolera całym szkieletem aplikacji. Kontroler nie odpowiada za to, jaki kontroler uruchomić, zresztą czytając to na głos, słychać jawnie problem logiczny (IMG:style_emoticons/default/smile.gif)

Analizując Twój kod, to Twoja klasa iPage działa jak bootstrap, obsługa żądania, routing. Wybierasz tam jaki kontroler uruchomić, jaką metodę. Więc zupełnie zbędne jest, aby każdy kontroler po tym dziedziczył.

Podsumowując, koniecznie przyjrzyj się jak rozwiązują te problemy inne FW
Go to the top of the page
+Quote Post
Spawnm
post
Post #5





Grupa: Moderatorzy
Postów: 4 069
Pomógł: 497
Dołączył: 11.05.2007
Skąd: Warszawa




 
[PHP] pobierz, plaintext 
  1. iPage::loadClass
  2. aStart::spr_txt()
  3. return new iStart; 
[PHP] pobierz, plaintext


Bardzo intuicyjne nazwy.
Samego kodu klas w pełni nie udostępniłeś, więc trudno ocenić.
Poczytaj o mvc i naucz się jakiegoś fw zanim zajmiesz się dalszym rozwojem swojego.
Go to the top of the page
+Quote Post
« Następny starszy · Object-oriented programming · 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: 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 Aktualny czas: 22.08.2025 - 15:13
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn