Post
#1
|
|
|
Grupa: Zarejestrowani Postów: 280 Pomógł: 46 Dołączył: 23.03.2010 Ostrzeżenie: (0%)
|
Tworzę klasę do obsługi szablonów. Mógłbym dać ten temat do działu Oceny, ale nie warto chyba o taką rzecz.
Chciałbym się dowiedzieć czy mój kod jest: optymalny, napisany prawidłowo z zasadami OOP i czy się nadaje na strone. Dodatkowo możecie napisać czy dobre mam nazewnictwo klasy, funkcji, zmiennych. Jest tam jedna funkcja (get_dir()) której będę najwyżej używał poza klasą, a wewnątrz używam zmiennej dir. Jeszcze myślę nad tym aby dać private funkcji load() ponieważ jest ona używana tylko wewnątrz klasy. Kod: https://www.dropbox.com/sh/eddsygq5kwhv3ty/FRDwEvOfGJ Live: http://evinek.ugu.pl/template/ Z góry dziękuje. Pozdrawiam, Paweł. |
|
|
|
![]() |
Post
#2
|
|
|
Grupa: Zarejestrowani Postów: 98 Pomógł: 33 Dołączył: 10.05.2011 Skąd: Krak Ostrzeżenie: (0%)
|
Co do dziedziczenia, dobrze rozumiesz. Musisz tylko w szablonie potomnym zaznaczyć, który szablon rozszerzasz np:
W kodzie, który podałeś zmieniłbym jeszcze system nazewnictwa metod na CamelCase np. blockStart zamiast block_start. W Twoim pomyśle jest ta wada, że mimo przechowywania nagłówków, menu itp. w osobnych sablonach dublujesz kod. Jeśli chciałbyś zmienić położenie menu, musiałbyś edytować każdy szblon. |
|
|
|
Evinek [PHP]Klasa szablonu - czy się nada? 11.07.2012, 20:05:52
peter13135 Jest optymalne, ale... to raczej nie jest system s... 11.07.2012, 21:24:30
Mephistofeles [PHP] pobierz, plaintext public function load... 12.07.2012, 09:33:44
Evinek Na początek funkcja load(). Jakimś cudem potrzebow... 12.07.2012, 20:10:20
Mephistofeles Tak. Brakuje funkcji takich jak escape (pisanie za... 12.07.2012, 23:21:17
Evinek Escape dodam. Helpery te dodać jako zwykłe funkcje... 14.07.2012, 14:48:59
Mephistofeles http://symfony.com/doc/current/book/templa...nce-a... 14.07.2012, 17:57:05
droslaw Do helperów możesz użyć metody __call. Pozwoli ona... 15.07.2012, 14:21:07
Evinek Nad helperami jeszcze pomyśle jak zrobić.
Co do d... 15.07.2012, 19:50:14
droslaw Wyobraź sobie taką sytuację. Masz dwa szablony: la... 16.07.2012, 09:38:33
Evinek Czyli, jeśli dobrze rozumiem, to będę miał na przy... 16.07.2012, 11:18:41
Evinek To jakim sposobem miał bym np. włączyć menu tylko ... 16.07.2012, 18:58:56
droslaw Jeśli menu chcesz dołączyć tylko z szablonem news,... 16.07.2012, 19:14:30
Evinek Dobra, coś stworzyłem.
Kod: https://www.dropbox.co... 19.07.2012, 20:06:43
droslaw Nazwę klasy pisze się z dużej litery.
U mnie konst... 22.07.2012, 19:26:25 ![]() ![]() |
|
Aktualny czas: 8.08.2026 - 18:48 |