Function til db-protection SMART

Lækkert med et dejligt beskrivende og kreativt indlæg :-)

Nu er jeg ikke selv PHP bruger, men udfra hvad du skriver, lyder det bestemt brugervenligt og anvendeligt!

Så 'thumbs up' herfra..

Jeg kan godt se at selve clear functionen er smart i dit system dog vil jeg stadig fraråde folk at bruge netop denne løsning.

Problemet med løsningen er at du faktisk bare gør det mere kompliceret!
Når du indsætter dine værdier i en tabel vil du altid skulle lave en forespørgsel og i denne forspørgsel kan du ligeså godt kalde mysql_real_escape_string eller mysqli_real_escape_string alt efter hvad du vil bruge, istedet for at kalde en anden "hjemmelavet" function.

Problemet med hjemmelavede functioner gør ofte kode mere uoverskueligt for 3. part! derfor som regl brug de functionsnavne som php har givet istedet for at lave synonymer som dette er! (Efter min mening!)

#2 - Har utrolig svært ved at følge dig, mener faktisk at hele din post ikke argumentere for at man ikke bør bruge den, men der imod blot er en opremsning af hvorfor du ikke gider bruge den.

1: kalde mysqli_real_escape_string i sql. Jo det kunne man men sig mig hvad er forskellen på at gøre det ved hver eneste POST variable og så bruge min funktion og skrive clean($_POST); ?? i mine øjne er min metode blot tidssparende.

2: Bør altid bruge php's egen funktioner ? ohm, lille ven det er netop lige det jeg gør !!! ?? har bare gjort brugen af den tidssparende !

3: uoverskueligt for 3. part, tjoh det er måske rigtigt hvis man ikke forstår at sætte de fornødne kommentarer.

Jeg vil gerne påstå at min teknik er 100% lige så god og bruger venlig som at skrive det hele som #2 foreslår. Forskellen ligger i mine øjne KUN i tidsmængden du vælger at bruge på at "clean" dine POST/GET variabler !

#3 Hvorfor begynde at kalde man lilel ven?
Nå men ud over det vil jeg give #2 ret, det ser max besværligt ud og uoverskueligt.

Det andet er jeg kan ikke lige se hvor jeg skulle få brug for dette, og det nok der det det bliver uoverskueligt.. Men det er sikkert effektivt. - Har aldrig modtaget mine POST/GET i et array, så det er nok grunden til jeg ikke kan se hvor jeg skal bruge det :)

#2 - 3. part har intet at gøre i din kode aligevel, og den vil være lige så uoverskuelig som var det ikke brugt, så det er dårligt aggument for det. Når du er færdig og har en kode på 1000+ linjer er den sq uoverskuelig lige meget hvad du bruger, for en 3. part.

Men anyways, så ser det meget nice ud, hvis man har brug for det :)

#4 > Jeg må give #3 ret i at $_POST/$_GET jo er et array når du får den ind så det #3 tænker er at du kan kalde: $params = clean($_POST); og så ikke tænke mere over sikkerheden.
Dog mener jeg stadig at det argument med at koden bliver alt for uoverskueligt alene er nok til ikke at bruge denne stump kode.

#5 > det er selvfølgelig rigtig nok at det er et array, det havde jeg ikke lige tænkte på. Men grunden er at man altid smider den over i en local variable med det samme, eller det er meget normalt at gøre det. :)

Så konklusionen er at den funktion ikke vil blive brugt =)

Først undskyld for den lille ven kommentar, det var ikke for at fornærme nogen, ved ikke lige hvad den lavede der.

Jeg ved ikke om det er fordi jeg selv har lavet koden eller fordi den bid kode jeg smidt her ind er dårlig opstillet, men jeg syntes bestemt ikke at den er uoverskuelig, måske er det min egen blindhed :P ..

#6 ja det er da meget normalt at gøre, men tænk over det her, hvis du har bygget dit system op via MVC metoden så kan du bruge funktionen clean allerede i dine controllers hvilke betyder at i dine models kan du kalde alle variabler allá $params['KEY'] hvilket i mine øjne gør dine models meget mere overskueligt plus at din clean funktion jo selv vil ligge som en model modul og derfor vil den ikke blive set i den overordnet kode.

Igen vil jeg gerne give jordz en undskyldning for den total barnlige kommentar, håber ikke at jeg har stødt jordz med den kommentar :D

Nu når vi har kommet til den konklution at koden ikke er brugbar, er der så nogen som har nogen forslag for muligvis optimering af den ?


[EDIT]
Vil gerne sige at hvis man fjerner alle kommentarene, skulle koden ikke virke uoverskueligt for nogen da den bare består af et loop og en php function, tilføjede de kommentar fordi jeg troede de ville forklarer hvad funktionen gjorde til dem som måske ikke har arbejdet særlig meget med php. My bad ;)
[EDIT]