[Framework] TurFramework Preview 2, No to się dorobiłem własnego Frame'a... Opcje

[Turgon]

Witam!
Kiedyś postanowiłem, że napiszę sobie CMS'a. Po kilku dniach wziąłem się do pracy. Projekt porzuciłem po trzech tygodniach. Ostatnio go odgrzebałem. Pomyślałem i zrobiłem takie cuś. Podchodzi bardziej to do FrameWorka. Manuala nie udało mi się opracować, ale proszę o ocenienie kodu, wskazanie kierunku itp.
I teraz ostatecznie Frameworkiem się stało (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Paczuszka z SQLem:
Paczka Zip
Paczka Tar.gz

Zdravim
Łukasz

Ten post edytował Turgon 16.01.2007, 13:24:40

[Athlan]

Jak już się Prph doczepił tych helperów to i ja to zrobie. Przytoczę kawałek kodu

[PHP] pobierz, plaintext 
<?php
addPasswordInput($name,$cssId = null,$cssClass = null)
?>
[PHP] pobierz, plaintext 

nie przewidziałeś że np ktoś może sachceć dodać inne parametry, chociażby np style. Czyżby przeróbka funckji? Nie lepiej zrobić, żebym listę parametrów ładował w tablicy?

Funkcje addTextInput() i addPasswordInput() w zasadzie różnią się słowem text/password, nie cierpie proceduralnego kopiowania kodu...

Zabolała mnie jeszcze jedna rzecz. Mianowicie fetching zapytań masz w klasie turquery.class.php, wykonanie ich turquery.class.php, obsługę bazy w tursql.class.php. Powiem tak: SYF. Klasa odpowiedzialna za baze danych powinna stanowić całość. A jak użyja innej bazy danych niż MySQL to chyba komponent turquery.class.php nie pójdzie do kosza? używasz tam funkcji do obsługi mysql... radzę zrobić jedność. Aby połączyć się z bazą muszę także ręcznie wywołać metodę za to odpowiedzialną. Nie wiam jak u Ciebie, ale u mnie w modelu podajesz nazwę klasy odpowiedzilną za obsługę źródła danych, wówczas konfigi, połączenie itd wykonuje się samo (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

W dispatcher

[PHP] pobierz, plaintext 
<?php
private static $app;
?>
[PHP] pobierz, plaintext 

po co static?

Obsługa uzytkownika (o mój Boże). Znowu jestem uzależniony od mysql... klasa chyba powinna TYLKO przechowywać info o nim, zbieranie danych z bazy danych to Model (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
<?php
if(count($this->vars) > 0){
			foreach($this->vars as $name => $val){
				if(!is_array($val)){
					$render = preg_replace('/{'.$name.'}/i',$val,$render);
				}
			}
		}
?>
[PHP] pobierz, plaintext 

po co... a nie lepiej wsadzić w template <?= $sText ?> albo <?php= $sText ?> , nie ma to jak szybkość działania.

Rozumiesz co to zmienne statyczne?

[PHP] pobierz, plaintext 
<?php
public final function registry($name){
?>
[PHP] pobierz, plaintext 

Używa się ich zazwyczaj w metodach statycznych, a Ty kombinujesz je w normalnych metodach wymagających instancji obiektu.

Podsumowywując... nie jest najlepiej. Przynajmniej takie jest moje zdanie.

Pozdrawiam, Athlan (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

btw polecam zajżeć http://strzalek.net/blog/5/przyjazne-urle-piszemy-router (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował Athlan 16.01.2007, 15:53:16