Rzut okiem na fragment kodu, wskazówki porady

Tutaj dyskutujemy o PHP oraz zgłaszamy i rozwiązujemy problemy dotyczące tego języka.
bielu000
Stały Bywalec
Stały Bywalec
Posty: 84
Dołączył(a): piątek, 8 maja 2009, 14:54

Rzut okiem na fragment kodu, wskazówki porady

Post przez bielu000 »

Cześć, piszę sobie od jakiegoś czasu w ramach ćwiczeń takiego małego uniwersalnego cms'a którego mógłbym użyć w przyszłości.
Generalnie póki co mam zrealizowane powiedzmy wczytywanie konkretnych modułów, menu itp, w odpowiednie miejsca poprzez parsowanie pliku html.
W pliku index.html, będącym plikiem szablonowym, kluczowe miejsca typu - położenie menu, miejsce głównego kontenera są otagowane specjalnymi znacznikami typu {content}. Chodzi o to, żeby była np w przyszłości możliwość stworzenia innego szablonowego pliku html, bez umieszczania w nim kodu PHP, ale tak by w konkretne miejsca zostały wczytane konkretne treści.

Głównym plikiem jest index.php. Po otworzeniu tego pliku uruchamia się skrypt, który pobiera z bazy danych listę elementów do "przerobienia" następnie pobiera zawartość pliku szablonowego(index.html), analizuje jego zawartość poszukując specjalnych tagów, które można by podmienić, a kiedy taki znajduje, odpala kolejną funkcję, która pobiera dane odpowiednio do zawartości tagu. Jeśli skrypt wykryje coś na wzór {menu.1}, sprawdza czy istnieje coś przypisanego do tego w bazie danych, jeśli tak to pobiera odpowiednie menu i wyświetla je w miejscu występowania tagu w pliku szablonowym. Jeśli trafi na tag {content}, sprawdza zawartość zmiennej predefiniowanej $_GET, a na jej podstawie wczytuje w to miejsce zawartość konkretnego pliku PHP, lub jeśli takowy nie istnieje sprawdza czy treść tej strony jest może umieszczona w odpowiedniej tabeli w bazie danych.

Generalnie póki co działa to, aczkolwiek zastanawiam się ogólnie nad tym rozwiązaniem. Nie mam doświadczenia w profesjonalnym programowaniu, więc nie wiem czy podejście, które reprezentuje w swoim kodzie jest odpowiednie, i czy nie jest on aby zbyt obciążony śmieciami.

Zawartość pliku szablonowego index.html: http://wklej.org/id/1885452/
Zawartość pliku głównego index.php http://wklej.org/id/1885458/
Zawartość pliku core.php http://wklej.org/id/1885463/
Zawartość pliku dbc_h.php http://wklej.org/id/1885467/

Na chwilę obecną, nie ma jeszcze wczytywania meta tagów, bo nie zdążyłem jeszcze takowej funkcji napisać.
Ogólnie zastanawiam się, co by można było zmienić, co wywalić, co jest bez sensu, a co jest nawet okej. Jasne mógłbym użyć tutaj pewnie jakiegoś systemu szablonów, ale zależy mi na tym, żeby mieć jak najwięcej swojego działającego kodu, bo dzięki temu pewnie najwięcej się nauczę ;)
Przydałoby mi się świeże spojrzenie na kod, właśnie przez kogoś kto się na tym zna ;) Chętnie przyjmę słowa krytyki ;)

//A dodam jeszcze, że oczywiście nie ma tutaj póki co żadnej obsługi błędów czy czegokolwiek w tym stylu. Na chwilę obecną nie jest mi to do końca potrzebne, aczkolwiek w dalszej fazie będzie to oparte o mechanizm wyjątków.
Morfidon
Administrator
Administrator
Posty: 1332
Dołączył(a): wtorek, 5 sierpnia 2008, 21:48
Contact:

Re: Rzut okiem na fragment kodu, wskazówki porady

Post przez Morfidon »

Hej,

Na pierwszy rzut oka wszystko OK :)

Staraj się zawsze zakańczać klamrę tak, żeby była równa w kolumnie :)

Pozdrawiam,

Arkadiusz
bielu000
Stały Bywalec
Stały Bywalec
Posty: 84
Dołączył(a): piątek, 8 maja 2009, 14:54

Re: Rzut okiem na fragment kodu, wskazówki porady

Post przez bielu000 »

A dziękuję ;)
Post Reply