moj dosadashnji rad- potrebne kritike(i pozitivne i negativne)

Radim na jednom projektu chisto zbog shto boljeg savladavanja php nego shto mi to treba, dole imate zipovani moj rad do sada, sve radi onako kako sam i hteo ali josh uvek daleko od zavrshetka, ono shto ja hocu jeste da ukoliko ima likova dobre volje pogledaju kod i kazu mi gde greshim da ne bi ponavljao ponovo iste greshke, ovo mi je prvi oop pokushaj pa mi zato i trebaju vashe kritike(i pozitivne i negativne), hvala unapred.

P.S.-za kod nisam koristio nikakve gotove skripte ni framework-ove, niti templat-e, ceo kod sam sam napisao(upitanju je jedino php i mysql).

Evo par mojih predloga premda sam ti samo na brzinu pregeldao kod.

Kao prvo, treba ti neka naming konvenicja, recimo Moja_Klasa.class.php za klase i sl.
Izbegavanje public varijabli, jasno definisanje f-ja u smislu private, protected, public.

U nekim slucajevima prenosenje instanci preko konstruktora klase recimo, db da bude singleton.
Pokusaj da primenis mvc, i napravis neku modularnu strukturu kako bi sebi olaksao zivot i povecao rejuzabiliti.

hvala brate, ovo trazim, ochekujem josh komentara i od ostalih

Imas sql injection... Ako pozoves ovakav url:

kosovski portal/public/index.php?page=forum_topics&forum_id=1 UNION SELECT id,id,username,password,id,id,id FROM users --

Mogu da se dobiju sve informacije o korisnicima...

Nisam se skorz udubio u kod, ali mislim da bi ovo izbegao time sto bi samo stavio na stranici forum_topics umesto:

to isto uz dodavanje konverzije tog forum_id u integer, ovako:

Da ne otvaram slicnu(istu) temu evo i mog rada(seminarski za skolu) nije jos gotov fali Order.php koji ce obradjivati porudzbine.
Koriscen PHP i Mysql(WAMP)

Potpis

http://i.imgur.com/V3feW.jpg
http://on.wsj.com/H9yjz6 -- India Graduates Millions, but Too Few Are Fit to Hire

Probaj da "razmisljas" objektno...

Hvala na savetu. Probacu

Object Oriented PHP for Beginners
http://www.killerphp.com/tutorials/object-oriented-php/

Potpis

http://i.imgur.com/V3feW.jpg
http://on.wsj.com/H9yjz6 -- India Graduates Millions, but Too Few Are Fit to Hire

Bas me interesuje...Da li je ovo tvoje? Jesi li ti ovo osmislio?

Naravno da nije. To se vidi već na prvi pogled. Isto tako ljudi uzmu neki CMS (npr. Joomla!) instaliraju, izaberu temu, dodaju neke module i kažu oni napravili. Jedno je prilagodjavanje postojećeg rešenja, drugo korišćenje biblioteka ili framework-a, a treće pisanje kompletno od nule sa samo svojim kodom.

Potpis

Two beer or not two beer...

Potpuno si u pravu.
Ali ne postoji drugi nacin sem uzeti nesto sto vec radi i poceti sa ucenjem i sitnim prepravkama. Vremenom se nauci pa se moze napraviti i sopstveno resenje.
Inace mogu toplo da preporucim Eclipse IDE za php. Ima i code complete :)

Nisam ni rekao da je moje.
Tacno, moram od necega poceti, ja sam izabrao ovaj nacin.

Profesoricu ni ne interesuje odakle mi to, vazno samo da radi i da ja razumem sta sam uradio.

Potpis

http://i.imgur.com/V3feW.jpg
http://on.wsj.com/H9yjz6 -- India Graduates Millions, but Too Few Are Fit to Hire