Za pocetak najveca zamerka je sto nemas template. U istom fajlu ti je u obrada podataka preko php i grenerisanje html-a.
Onda nemas proveru da li ima post key , npr . $username = $_POST['username']; Napravi neku klasu za request koja ce raditi sve te provere.
Sve trpas u Engine. Ako je vezano za usera (singin, forgot pass, logout itd onda to ide u user klasu, ako je vezano za private message onda to ide u tu klasu.
Engine je tu da kooridnise izmedju klasa, requesta , outputa itd.
Onda imas LangSessionSR i LangSessionEN. To ne ide bas tako. Ako dodas novi jezik onda moras i nove sesije i nove metode? Malo fleksibilnije treba.
Ne mozes funkcijama da se vezes za neki jezik vec jezik treba da je parametar.
Vidim ovde puno rada, lepo napisano, funkcije dosta jasne na prvi pogled ali sve logicki dosta lose povezano i logicki lose organizovano.
To je prvi utisak.
Vidi se neko solidno osnovno znanje iz programiranja.
Pravljenjem svog fw neces nista novo nauciti vec ces samo ponoviti-utvrditi dosadasnje znanje ali kao sto sam rekao to je osnovno. Neces se pomaci sa mrtve tacke sem sto ces izfizikalisati brdo koda.
Moja preporuka je da predjes na neki fw i tu ces nauciti puno novih stvari.
Popravices logiku i organizaciju.
FW ce da ti omoguci da programiras i resavas probleme a ne da fizikalises nebitne stvari koje se ponavljaju i rutinski resavaju.
Dobices na kvalitetu projekata i to mnoooogo.
Napravices brze, bolji i lepsi projakt i laksi za odrzavanje i prosirivanje.
Sa ovim tvojim kodom ide lepo za pocetak ali ces kad-tad zaglaviti a da ne kazem ako se jos neko prikljuci on ce zaglaviti prvi dan.
[Ovu poruku je menjao VladaSu dana 14.06.2003. u 11:22 GMT+1]