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

[ amitkic @ 23.08.2009. 22:00 ] @

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).

[ Mister_rap @ 24.08.2009. 01:58 ] @

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.

[ amitkic @ 24.08.2009. 14:25 ] @

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

[ Tudfa @ 24.08.2009. 18:16 ] @

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:

Code:

$forum_id= $_GET['forum_id'];

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

Code:

$forum_id= (int)$_GET['forum_id'];

[ AMD guy @ 30.08.2009. 15:11 ] @

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)

[ Mister_rap @ 30.08.2009. 15:25 ] @

Probaj da "razmisljas" objektno...

[ AMD guy @ 30.08.2009. 15:31 ] @

Hvala na savetu. Probacu

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

[ m.stojanov @ 02.09.2009. 01:59 ] @

Citat:

AMD guy: 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)

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

[ BigFoot @ 02.09.2009. 09:04 ] @

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.

[ zigizig @ 02.09.2009. 13:44 ] @

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 :)

[ AMD guy @ 02.09.2009. 18:13 ] @

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.

[ amitkic @ 02.09.2009. 19:25 ] @

malo odlutaste od teme, znachi treba da odgledate moj kod i kazete shta radim pogreshno, AMD GUY je dobio odgovor koji je trazio