Spawn Framework, http://spawnframework.com/ Opcje

[Spawnm]

Witam,

proszę o opinie na temat:

1. strony www

2. frameworka

Co byście zmienili, czego wam brakuje.

Adres: http://spawnframework.com/

[cojack]

Mnie się podoba prostota frameworka ;] nie testowałem bo nie mam czasu, ale z tego co widać w manualku, to prostota rządzi, ja też coś tam swojego skrobie, tylko mój jest troszkę bardziej przykokszony, ale z ORM'a to ja tam nic nie widziałem, Spawn?

@EDIT a nie jednak jest, tylko o weryfikacji danych nie ma ani słowa na podstawie modeli, a mogłeś to zrobić ;]

Ten post edytował cojack 8.07.2010, 17:00:10

[erix]

1. • jak dla mnie trochę za jaskrawe kolory
   • czcionki szeryfowe raczej nie na ekran
2. widząc prefiks sf myślę o Symfony

@up: jest, tylko nie wiem, czemu w przykładach nie chwali się nim od razu (a powinien).

[Spawnm]

http://spawnframework.com/doc/21/sf_orm

dodam że w orm działają metody z
http://spawnframework.com/doc/20/sf_db
oraz w obydwu przypadkach metody z pdo .

Powód edycji: [Spawnm]:

[cojack]

Ale teraz to Ci się oberwie:

1) Po co komu te śmieci w pliku do ściągnięci? Nie wyczyściłeś kopi zapasowych plików, zacznij używać svn'a i nie będziesz miał z tym żadnego problemu.
2) Co to jest za licencja?
3) Autoload, po co? Na co? :/
4) View, dziwne to factory u Ciebie, pierwszy raz widzę takie coś. Bardziej niż na faktory to mi wygląda na wzorzec złożony.

@EDIT
to nie mogłeś w kodzie napisać co to za licencja, jak to się czyta wygląda jak GPL, a tu widzisz BSD ;]

Ten post edytował cojack 8.07.2010, 17:31:38

[Spawnm]

licencja:
http://www.opensource.org/licenses/bsd-license.php

[SHiP]

Hahahaha udostępniłeś ludziom hasła do swojej bazy...

Człowieku zrób porządek w paczce

EDIT: W wolnej chwili spojrzę dokładniej na całość. Ogólnie mi się nie podoba "bałagan", np w controlerze masz metodę redirect albo error404. Ja bym redirect wrzucił do routera a 404 wyrzucił w ogóle...

Ten post edytował SHiP 8.07.2010, 17:35:10

[Spawnm]

posprzątane , zupełnie o tym zapomniałem że te pliczki się robią ;p

[Crozin]

Dobra, zaczniemy od błędów/złych rozwiązań:

1. Brak możliwości zdefiniowania środowisk - albo przynajmniej dwóch podstawowych: production i development
2. index.php:
   • Już nagłówek Content-Type? Czyli co... pliku XML już nie będę mógł [b]normalnie[b] (tak wiem... nagłówki można nadpisywać) wyświetlić
   • Nie wiem czemu ludzie jak definiują sobie ścieżkę do katalogu to dodają jakiegoś slasha na końcu
   • Korzystając z set_include_path() powinieneś podać jeszcze jej starą zawartość: get_include_path()
3. PHP 5.3 już jest od jakiegoś czasu - mogłeś skorzystać z przestrzeni nazw - szczególnie, że sf jest już "zarezerwowane"
4. Mieszanie angielskiego i polskiego (sf_request::_filtr())
5. Utrudnianie rozszerzania FW stosując metody prywatne, tam gdzie powinny być chronione (np. sf_request::_filtr())
6. Czemu takie katalogi jak config czy logs są wrzucone wewnątrz katalogu library?
7. Brak jakiejś konsekwencji w komentarzach... raz phpDoc, raz "coś", raz ich kompletny brak
8. sf_router::location() - wtf? Od kiedy to router ma takie rzeczy wykonywać
9. Metody potworki... spawn::create() - kompletnie nietrafiona nazwa, wykonuje za wiele czynności: konfiguruje środowisko, określa i wywołuje kontroler, zajmuje się obsługą błędów
10. Czy mi sie wydaje czy router polegnie, gdy będę chciał: /my-awesome-product-123.html przekierować na kontroler: product, akcje: index z parametrem id = 123?
11. Po co w __call() sprawdzasz czy metoda istnieje? Przecież oczywiste jest, że nie, skoro __call zostało wywołane.
12. Jaki jest sens parametrów (2-gi i 3-ci) w sf_db::orWhere? orWhere('...', 'AND', 'AND')?
13. Jeżeli jakiś model ma w swojej nazwie "orm" to już nie jest poprawnym modelem
14. sf_jquery - ehkem... wtf?
15. sf_cookie/sf_math - masz tam błędy składni
16. Co w sf_math robią metody typu rbg2hex? Co to ma wspólnego z matematyką (na co wskazuje nazwa)? sf_math::dec2hex... http://php.net/manual/en/function.dechex.php
17. W ogóle to widzę tu więcej funkcji, które dublują już istniejące
18. sf_mail... po co to w ogóle jest "klasą"? Poza tym tylko utrudnia życie - poproszę o maila z załącznikiem do 100 osób
19. sf_html::rss() - a jak będę chciał dodać atrybut title? To samo z ::css() - a media mogę podać?
20. sf_view::factory() - czym to się różni od bezpośredniego wywołania konstruktora?

Takich kilka luźnych uwag/spostrzeżeń na pierwszy rzut oka patrząc po kilku plikach. Ogólnie - słabo... bardzo słabo.

PS. Popracuj nad angielskim.
PPS.

(...)and use the MVC architecture(...)

Nie, on nie jest oparty o MVC...

Ten post edytował Crozin 8.07.2010, 18:14:47

[Spawnm]

łojej nie sądziłem że tego aż tyle będzie ...

uż nagłówek Content-Type? Czyli co... pliku XML już nie będę mógł [b]normalnie[b] (tak wiem... nagłówki można nadpisywać) wyświetlić

inne frameworki też tak robią , np. kohana;)

Mieszanie angielskiego i polskiego (sf_request::_filtr())

jedno się trafiła, jeszcze gdzieś znajdziesz to pisz , wydawało mi się że tylko angielskie nazwy są...

Czemu takie katalogi jak config czy logs są wrzucone wewnątrz katalogu library?

uznałem że rupiecie danego frameworka powinny być w jego folderze aby się nie gryzły z innymi libami.

sf_router::location() - wtf? Od kiedy to router ma takie rzeczy wykonywać

jeśli chcę aby po wpisaniu nazwa.pl/forum
przeniosło mnie np. na fora.pl to po co mam do tego pisać specjalnie akcję skoro mogę ustawić przekierowanie w routerze

W ogóle to widzę tu więcej funkcji, które dublują już istniejące

- np?

aki jest sens parametrów (2-gi i 3-ci) w sf_db::orWhere? orWhere('...', 'AND', 'AND')?

ja widzę orWhere($where, $valWhere='OR' ,$sep='OR')

sf_cookie/sf_math - masz tam błędy składni

k... poprawiałem to... już działa...

Powód edycji: [Spawnm]: [Spawnm]:

[cepa]

k... poprawiałem to... już działa...

od tego testy sa wlasnie ;p

[Crozin]

od tego testy sa wlasnie ;p

Raczej IDE, które podkreśli na czerwono byki w składni, czy po prostu odpalenie pliku

uznałem że rupiecie danego frameworka powinny być w jego folderze aby się nie gryzły z innymi libami.

Ale to nie są rupiecie FW, tylko normalne dane, które nie powinny być w katalogu na kod źródłowy bibliotek itp. Poza tym FW to taki rupieć nadrzędny względem innych i ma prawo sobie śmiecić

- np?

sf_math::dec2hex(), sf_math::hex2dec (dechex/hexdec), sf_date::add (datetime::add), sf_date::w3cDate (to nie jest przypadkiem date() z formatem "c" czy http://pl.php.net/manual/en/class.datetime...e.constants.w3c ), sf_date::isDate (checkdate, datetime), sf_csv (nie chce mi się sprawdzać, ale czy to robić coś poza tym co robi fgetcvs?),

ja widzę orWhere($where, $valWhere='OR' ,$sep='OR')

Dlatego się zastanawiam po co są te parametry skoro ta funkcja ma robić za "or" - sama jej nazwa na to wskazuje

[skowron-line]

[PHP] pobierz, plaintext 
class sf_url{
 
	/**
	*return controller and action
	*
	*@param - string - core - if true - return base spawn config
	*@return - string - /{controller}/{action} 
	*/
	public static function base($core=false){		
		if($core==true){
			$par=sf_config::get('spawn');
			print_r($par); //<------------------------------------------------- WTF 
			return '/'.$par['controller'].'/'.$par['action']; 
		}
		return '/'.sf_request::instance()->param(0).'/'.sf_request::instance()->param(1);
	}
[PHP] pobierz, plaintext 

Napisz co cię skłoniło do napisania własnego FW, czego brakowało lub czego było w istniejących rozwiązaniach.

Ten post edytował skowron-line 8.07.2010, 20:00:35

[Spawnm]

Raczej IDE, które podkreśli na czerwono byki w składni, czy po prostu odpalenie pliku winksmiley.jpg

Jak pisałem poprawiałem to i testowałem , nie wiem czemu w paczce były stare wersje ...

co do czemu rupiecie takie jak config trzymam w sf - nie jestem sam i akurat tego nie planuję zmieniać w strukturze swoje frameworka

@skowron-line - wywalone już.

Napisz co cię skłoniło do napisania własnego FW, czego brakowało lub czego było w istniejących rozwiązaniach.

Brakowało mi prostoty struktury fw , brakowało mi fw do małych projektów, fw który by nie zajmował kilka MB. Widząc w jakim kierunku rozwija się np. kohana w której pracowałem uznałem że ja w tym czymś pracować dalej nie będe.

Powód edycji: [Spawnm]:

[cepa]

Raczej IDE, które podkreśli na czerwono byki w składni, czy po prostu odpalenie pliku

jak masz 20 tys plikow w projekcie to bedziesz kazdy z osobna otwieral i uruchamial ?

[Spawnm]

jak masz 20 tys plikow w projekcie to bedziesz kazdy z osobna otwieral i uruchamial ?

A ty piszesz najpierw 20tyś plików i dopiero później zaczynasz wyszukiwać gdzie zgubiłeś ';' ?
Czy robisz to etapami ?

[skowron-line]

[PHP] pobierz, plaintext 
class sf_date{
 
	/**
	*add X day to date
	*
	*@param - string - date
	*@param - string - day 
	*@return - string - new date
	*/
	public static function addDate($date,$day){
		$sum = strtotime(date('Y-m-d', strtotime($date)) . ' +'.$day.' days');
		$dateTo=date('Y-m-d',$sum);
		return $dateTo;
	}
[PHP] pobierz, plaintext 

Lat, miesięcy już nie dodajesz i wydaje mi się że lepiej by było jak by zwracał w sekundach, no ale to może tylko moje odczucie. Ogólnie niezły śmietnik w kodzie, mam dziwne wrażenie że pisany na kolanie i byle szybciej.

[Spawnm]

Ogólnie niezły śmietnik w kodzie

Domyślam się że wszystko cudowne tam nie jest ale
to co wyżej uważasz za śmietnik? Przesadzasz

mam dziwne wrażenie że pisany na kolanie i byle szybciej.

Może nie byle szybciej ale na kolanie chwilami tak

[cepa]

A ty piszesz najpierw 20tyś plików i dopiero później zaczynasz wyszukiwać gdzie zgubiłeś ';' ?
Czy robisz to etapami ?

eh nie kminisz..

powiedzmy ze twoj framework sie rozbudowal, i masz spora chierarchie obiektow (patrz Zend, Symnfony, etc), no i jakas twoja klasa jest uzywana np: w 20 miejscach w kodzie...
zmieniles cos w tej klasie i co teraz? bedziesz sprawdzal recznie czy caly pozostaly kod dziala? na tym wlasnie polega testowanie, zauwaz ze gigantyczne frameworki rownierz maja gigantyczna mase testow a zaloze sie ze ich stabilnosc jest na tyle wysoka ze na oko mozna by powiedziec ze testy sa wcale nie potrzebne

pozatym z komercyjnego punktu widzenia, nie moge wykorzystac twojego kodu na powaznie, bo nie ma on zadnej gwarancji dzialania, a testy sa taka wlasnie gwarancja

[skowron-line]

to co wyżej uważasz za śmietnik? Przesadzasz

Broń boże to co podałem to nie przykład śmietnika, takie małe spostrzeżenie, uwaga na to jak można by tą metodę dopracować. Mało czytelny jest kod, brakuje spacji enterów, bardziej rozbudowanych komentarzy do metod