od kilku dni intensywnie klepie w core. przepisalem od nowa dodawanie i edycje notek, zrobilem w miare porzadnie (choc nie idealnie) generowanie sie drzewka menu, czyszcze kod, konstruuje obszerne TODO (zeby lark jeszcze chcial to czytac :/ ) i takie tam rozne.
wywalilem generowanie kilku/kilkunastu, a w ekstremalnych warunkach nawet kilkadziesiat/kilkaset zapytan do bazy danych podczas generowanie sie drzewka menu, zastapilem to tylko jednym query, reszta to operacje na tablicy. nie sprawdzalem wartosci liczbowych, ale w niektorych przypadkach przyspieszenie powinno byc rzedu kilkuset procent.
w edycji i dodawaniu newsow (notek) poprawilem w duzym stopniu usability, jest teraz o niebo lepiej. forularz przestal sie resetowac przy podgladzie wpisu, komunikaty o bledach pojawiaja sie na tej samej stronie etc. trzeba jeszcze dopracowac, ale szkielet juz jest.
do czyszczenia kodu jest sporo - raz, ze lark pisal spora czesc tego juz dosc dawno, dwa ze widac miejscami brak zastanowienia sie nad niektorymi rzeczami ;) tak samo staram sie wykorzenic szerzacy sie faszyzm, i chyba w miare mi idzie ;)
jest tez wazna poprawka w kwestii bezpieczenstwa, ktora opisalem, i mam nadzieje ze lark ja zrobi, bo jak nie to sam bede musial… :P
tylko kurcze, czemu lark nie moze sie przekonac do stosowania opisowych tablic $_POST, $_GET etc ? wszedzie jest :
$a=$_POST['a'];i dalsze operacje na $a co jest ekstremalnie malo czytelne w kodzie na 500 linii, lub wiecej ;) musze sie po drodze zastanawiac, czy ta zmienna pochodzi z formularza, czy tez zostala wyciagnieta z bazy etc… ;)
Jak już skończysz te przeróbki i będzie można na to rzucić okiem, to daj znać. Chętnie spojrzę i pokomentuję, np. sprawy związane z tłumaczeniem na inne języki, z modyfikacją kodu i takich tam. Po grzebaniu w Mambo czy w Simple machines forum mam parę spostrzeżeń technicznych.
Adres bezpośredni: http://urzenia.net/163/core-2/#comment-144
mniej pisania mysz ;-) Przynajmniej jesli chodzi o zmienne tablicowe. Ot jedyny powód, tym bardziej, ze ich nazwy wydaje mi sie jednoznacznie sugeruja ich powolanie ;-)
Core nie mialo nigdy ujrzec swiatla dziennego ;-) ot, pisane to bylo jako prosty system blogowania, na ktorym wielu spraw sie uczylem. Zreszta sam wiesz jak bylo i jak szybko dodawana byla nowa funkcjonalnosc ;-)
todo czytam, tylko z czasem gorzej, bo mnie do grafikowania zmuszają :P W weekend posiedze. Przepisze to co w todo na klasy i z uzyciem singeltona, o ktorym zreszta nic nie napisales, a o co Cie prosilem :-) Masa niepotrzebnych instancji klasy do obslugi bazy danej zostala wywalona w kosmos.
Dobra, to sie wytlumaczyłem :-)) Spadam dłubać dalej prezentację pod directora, bo mnie klient molestuje ;-)
Adres bezpośredni: http://urzenia.net/163/core-2/#comment-145
@mikolaj: na pewno dam znac :)
@lark: mniej pisania jesli chodzi o _POST to slaby argument ;) poza tym, nie chodzi mi o to ze tak bylo, ale kiedys rozmawialismy nt tego, a ty jakos dalej tak pisales ;)
co do singletona, to nie wiem co pisac, bo nie uzywalem wczesniej, choc zaczelem teraz, przepisujac dodawanie newsow. korzysci namacalnych nie mam, nie widze jeszcze ;) moze za jakis czas ;)
Adres bezpośredni: http://urzenia.net/163/core-2/#comment-156