[es] - moj dosadashnji rad- potrebne kritike(i pozitivne i negativne)

amitkic
student
zvechan

Član broj: 186610
Poruke: 42
82.114.95.*

Profil

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

^{23.08.2009. u 22:00 - pre 178 meseci}

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

Prikačeni fajlovi

kosovski portal.rar - 8.55k

Odgovor na temu

Mister_rap
SE at Viacom

Član broj: 8822
Poruke: 2540
95.155.2.*

Jabber: mister_rap@jabber.com

+21 Profil

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

^{24.08.2009. u 01:58 - pre 178 meseci}

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.

Odgovor na temu

amitkic
student
zvechan

Član broj: 186610
Poruke: 42
82.114.95.*

Profil

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

^{24.08.2009. u 14:25 - pre 178 meseci}

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

Odgovor na temu

Tudfa
Jovicevic Vladimir

Član broj: 152699
Poruke: 384
*.dynamic.sbb.rs.

+3 Profil

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

^{24.08.2009. u 18:16 - pre 178 meseci}

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'];

Prikačeni fajlovi

sql_injection_result.jpg - 119.83k

Odgovor na temu

AMD guy
Miroslav
.NET developer

Član broj: 128930
Poruke: 1007

Sajt: www.its.edu.rs

+38 Profil

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

^{30.08.2009. u 15:11 - pre 178 meseci}

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)

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

Prikačeni fajlovi

Prodavnica.rar - 15.04k

Odgovor na temu

Mister_rap
SE at Viacom

Član broj: 8822
Poruke: 2540
*.crnagora.net.

Jabber: mister_rap@jabber.com

+21 Profil

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

^{30.08.2009. u 15:25 - pre 178 meseci}

Probaj da "razmisljas" objektno...

Odgovor na temu

AMD guy
Miroslav
.NET developer

Član broj: 128930
Poruke: 1007

Sajt: www.its.edu.rs

+38 Profil

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

^{30.08.2009. u 15:31 - pre 178 meseci}

Hvala na savetu. Probacu

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

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

Odgovor na temu

m.stojanov
Test Test

Član broj: 126777
Poruke: 231
85.222.223.*

+3 Profil

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

^{02.09.2009. u 01:59 - pre 178 meseci}

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?

Odgovor na temu

BigFoot
Boban Jovanović
Arilje

Član broj: 1098
Poruke: 991
91.150.111.*

+35 Profil

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

^{02.09.2009. u 09:04 - pre 178 meseci}

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.

Two beer or not two beer...

Odgovor na temu

zigizig

Član broj: 42968
Poruke: 167
82.117.198.*

+10 Profil

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

^{02.09.2009. u 13:44 - pre 178 meseci}

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

Odgovor na temu

AMD guy
Miroslav
.NET developer

Član broj: 128930
Poruke: 1007

Sajt: www.its.edu.rs

+38 Profil

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

^{02.09.2009. u 18:13 - pre 178 meseci}

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.

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

Odgovor na temu

amitkic
student
zvechan

Član broj: 186610
Poruke: 42
82.114.95.*

Profil

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

^{02.09.2009. u 19:25 - pre 178 meseci}

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

Odgovor na temu