Forum PHP.pl

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

Paczuszka z SQLem:
Paczka Zip
Paczka Tar.gz

Zdravim
Łukasz

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

1."* Not allowed to use. This is only preview!"
to moge go uruchomic czy nie?

2.

[PHP] pobierz, plaintext 
<?php
$config['database']['host'] = '127.0.0.1';
?>
[PHP] pobierz, plaintext 

a da sie nr. portu skonfigurowac?

3. brak formatowania kodu:

[PHP] pobierz, plaintext 
<?php
class indexVendor extends TurVendor{
 
public function index(){
$this->view->loadView('index');
return $this->view->render();
}
public function another(){
}
}
?>
[PHP] pobierz, plaintext 

4. nie udalo mi sie go uruchomic na lighttpd (mod_rewrite)

5.

[PHP] pobierz, plaintext 
<?php
$var = preg_replace('/<script type="text/javascript">(.*)</script>/i','',$var);
?>
[PHP] pobierz, plaintext 

a co gdy podam:

[HTML] pobierz, plaintext 
<script>...</script>
<script              type="..." >...</script>
itd
[HTML] pobierz, plaintext 

?

6. co robi "class TurVendor" to juz zupelnie nie wiem
bloki, zmienne, uprawnienia -- wszystko w jendej klasie

1. Możesz, ale tylko do testów. Miałem na myśli użycie w projektach itp. ;]
2. Dodajesz po hoście porcik
3. Kate nie oferuje takiej opcji, a ja nie jestem przyzwyczajony wciskać co chwilę "Tab".
4. Działa bez mod_rewrite. Tam są tylko dwie testowe reguły, które uruchamiałem na zdalnym serwerze.
5. Powinno zablokować tylko właściwe ;] Czyli JavaScriptowe, choć do końca nie testowałem. Słabo go znam.
6. To podstawa dla każdego Vendora, ponieważ każdy Vendor nie będzię dziedziczył po 20 klasach , bo to trochę bez sensu. I te bloki, to nie działają bo wyrzuciłem klasy typu TurBlock. Odchudziłem aplikację, bo interesuje mnie głównie ocena jądra.

Jeśli chodzi o klasę TurKernel, to mi się nie podoba. Zawiera metody, które coś wypisują na ekran. Rozumiem, że to są testy, ale ładniej by to wyglądało, gdyby np. metody TurKernel::compileInfo(), albo TurKernel::listModules() wyszukiwały informacje, a potem je zwracały. Z samej tej klasy IMHO da się wydzielić co najmniej 5 różnych klas, bo teraz klasa ma zbyt dużą odpowiedzialność. W obecnym stanie rozwoju nie stanowi to problemu, ale jeśli zechcesz rozbudować, to napotkasz na mur. Będziesz musiał przekazywać coraz więcej parametrów do metod, a co za tym idzie, coraz więcej instrukcji warunkowych. Jeśli to jednak skończony projekt, to nie ma problemów. : )

Pozdrawiam.

Wiem, wiem Cysiaczku, dlatego obecnie przebudowywuję ją całą. Te metody zostaną w takim stanie, gdyż takie je lubię Zmienię też delikatnie strukturę jądra tzn. niektóre klasy będą domyślnie używane. Powstanie też autoloader , ponieważ to pozwoli na wybieranie tylko potrzebnych modułów podczas kompilacji. Jak skończę wrzucę poprawioną wersję na serwer
I zapomniałbym. Dorzucę też obsługę interfejsów Klasę TurView też czeka przebudowa.

Ten post edytował Turgon 15.01.2007, 12:05:59

Zmien edytor, chociazby Kwrite ma autowciecia. A co z eclipse?

Najpierw uruchomilem bez zadnych rewrittow --> undefined index 'vendor' itd.

Nie wiele do oceniania.
Dużo bałaganu widzę, mało czegoś pożytecznego. Np. tursql. Halo! Mamy 2007, mamy PDO, jeżeli ktoś nie lubi to Creole. No ok, chyba że ktoś naprawde chce coś jeszcze lepszego, czemu by nie, ale niech to mam jakąś funkcjonalność, a nie nakładka na kilka natywnych f-cji.



To może najpierw zrób z tym pożądek, dopisz co trzeba żeby chciaż ten banalny przykład działa bonzowi

Strzałek: Wciąż na wielu serwerach nie ma PDO... Z resztą kilka drobnych klas będących na warsztacie stworzą wraz z TurSQL małego i prostego ORMa ;]
dr_bonzo: Dziwne... W Exception włącz trace i daj tutaj wynik


Dodałem poprawioną wersję. I chyba wiem o co chodzi Bonzo
Urle są w formacie : index.php?app=nazwaApp&vendor=nazwaVendor&action=nazwaAction .
Można zawsze zmienić nazwę każdej ze zmiennych. Jeśli którąkolwiek z nich pozostawimy pustą skrypt sam ją uzupełni domyślnymi danymi z kernelImage.php.

To ze sa w taki fromacie to sie domyslilem pouzupelnialem zmienne na jakies losowe wartosci i nie ruszylo. Zaraz sprawdze nowa wersje.


No wiem ze konfigu jeszcze nie ustawilem, ale przydala by sie inna informacja

Wchodze na "/" i undefined index, action, vendor, action.
http://php.dev.localhost/turgoncms/?vendor...mp;action=index
czyli defaultowe wartosci

dla vendor = 'turcms' tez nie dziala



Po co tyle kodu (jak zapomne to sie pewnie app sypnie, nie wyswietli czegos itd), nie latwiej przyjac ze akcja 'index' jest wyswietlana domyslnie widokiem 'index'? I po co zwracac wynik renderowania -- do zadanie FW, dostal info ktory widok uzyc, jakie ma parametry i sobie poradzi.

Wiem, wiem... Nad błędami muszę jeszcze pomyśleć
Kolejna sprawa app, to katalog z vendorami Powinien być turcms lub turpowerboard.
Rozumiesz ?

Co do widoków, to dobra postaram się coś pomyśleć , ale mi to nie przeszkadza Ale co ty byś proponował, bo do końca nie łapie ?

Ten post edytował Turgon 15.01.2007, 15:49:03

[PHP] pobierz, plaintext 
<?php
public function IndexAction()
{
   $this->text = "HelloWorld";
}
?>
[PHP] pobierz, plaintext 

i FW zajmuje sie reszta, zalacza domysly widok 'index' i odpala renderowanie, bo na razie w kazdej akcji musisz to recznie dopisywac, a to nie jest dobre



No w koncu uruchomilem

Masz dosyc nieintuicyjne nazwy:
jest katalog vendors/ i oczekuje ze w nim sa vendory czyli turcms itd.
a nie ze w vendors/ sa aplikacje

Wiem trochę napaprałem ;] Ale w configu można zawsze zmienić nazwę app na coś innego w linku To co proponujesz będzie wymagało przebudowy połowy FW, ale Ok ;] W sumie chyba muszę zmienić nazwę aplikacji i oznaczenie w temacie

Ten post edytował Turgon 15.01.2007, 16:17:09

FW ma chyba ulatwiac i uprzyjemniac pisanie aplikacji -- wiec chyba warto to raz a porzadnie zrobic niz w kazdej aplikacji sie meczyc.

Wiem, dlatego przebudowałem, ale coś nie działa hmmm... Na deskę kreślarską wraca Przebudowa zakończona Działa tak jak chciałeś ;] Teraz tylko trzeba troche zmienić rejestr systemowy, bo nie jest przystosowany do nowych warunków!

Ten post edytował Turgon 16.01.2007, 10:02:48

Witam,

Troche balaganiarski styl programowania. Pomine wciecia, bo nie wiem, czy naprawde tak piszesz, czy edytor zrobil Ci na zlosc i usunal tabulatory. Jednak nie trzymasz sie nazwenictwa - wszystko w FW w postaci turcoś.class.php, natomiast helpery nazwałeś ViewHelpers (dlaczego tu z duzych?).

Znalazłem mały błąd, ktory może powodować w przyszłości złe wyświetlanie widoku:

[PHP] pobierz, plaintext 
<?php
$render = ob_get_contents();
ob_end_clean();
 
// Zamiast tego, uzyj:
 
$render = ob_get_clean();
?>
[PHP] pobierz, plaintext 

Pozdrawiam, Adrian.

Prph: Trzymam się, bo Helpery to cuś innego ;] dla mnie. Dzięki za tą funkcję, nie wiedziałem, że taka jest Co do wcięć ? Po co ? Edytor ich nie dodaje, to co będę na siłę je robił

Więc porobiłem wam te wcięcia, a wy zajmijcie się oceną kodu... Nowy Preview wyszedł ;] Zmiany są znaczące

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

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

[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

btw polecam zajżeć http://strzalek.net/blog/5/przyjazne-urle-piszemy-router

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

Odpada ten sposób! PATH_INFO jest blee i tyle.



Może, mi tego nie potrzeba, ale może jak będę miał trochę wolnego czasu to napiszę, to.


O jedności za przeproszeniem gówno prawda... http://www.codeigniter.com/user_guide/database/examples.html Popisałeś się też logiką w modelu podajesz nazwę klasy odpowiedzialną za obsługę źródła danych. Model to ma robić ! Jakie tutaj configi (przez "c" piszemy w takiej formie jak coś). Może czasem nie potrzebuje się łączyć z bazą danych...


Nie podoba się ? Wyłączasz. Mam inne upodobania.


Potrzebuje w funkcji getVendor() info o app. Bo nie mam ochoty tego wpisywać za każdym razem...

Zdravim
Łukasz

Ten post edytował Turgon 16.01.2007, 16:43:27

Niezłe uzasadnienie, powiedziało mi bardzo dużo Tutaj masz router napisany przeze mnie, który dokonale obrazuje, że jednak ten sposób nie jest fe
http://framework.vgroup.pl/expose-da42a823...1a0c8c2612b.htm


Chcesz kolejnego warna od mike_mech? (przy okazji pozdrawiam)

Ale nie "gówno prawda", napisałeś w pewnym momencie:

Dzięki, to komplement? Jeżeli nie chcesz się łączyć to chyba nie uzywasz w modelu połączenia z bazą danych nie? U Ciebie jednak nawet jeżeli połączenia nie potrzebuję, muszę się łączyć. zacytuję tutaj linie kodu index.php żeby nie było nieporozumień:

[PHP] pobierz, plaintext 
<?php
try{
	TurConfig::loadFile('./kernelImage.php');
	$kernel = new TurKernel();
	$sql = new TurSQL();
	$sql->connect();
	$sql->selectDb();
	TurView::setStyle(TurConfig::get('defaultStyle'));
	$user = TurKernel::register('TurUser',new TurUser());
	$session = TurKernel::register('TurSession',new TurSession());
	echo $kernel->runVendor();
}
?>
[PHP] pobierz, plaintext 

Tutaj powini wypowiedzieć się ludzie odpowiadający za gramatykę. Nie napiszesz Configuracja tylko konfiguracja, tak samo nie będziesz pisał Controller tylko Kontroler, nie Computer tylko Komputer. To są swego rodzaju neologizmy wprowadzone (już na stałe) do przeciętnego użytku języka polskiego przez programistów.

Pozdrawiam, Athlan

-- [edit ] --

O widzę, że zedytowałeś post i dorzuciłeś linka do Manuala CI. Najpierw pomyśl, potem rzucaj linkami. Jawet w CI fetching, querying i połączenie odnoszą się do tego samego komponentu ładowanego poprzez $this->db-> ...

Pozdrawiam

Ten post edytował Athlan 16.01.2007, 16:55:03

Athlan, nie mam ochoty pisać kolejnego klona ZF, co ty zrobiłeś. I co nie łaska jak nie potrzebujesz wywalić connect ?