php code controleren of het goed is ja/ nee

[wilma]

Hoi iedereen,

Ik overweeg om met een scripter in zee te gaan die een paar scripts voor mij zal gaan schrijven. Ik heb een stukje php code gevraagd wat hij geschreven heeft, maar kan hiervan zelf de kwaliteit niet beoordelen of het goed/ veilig geschreven is ja of de nee.

Kunnen jullie hierover op/ aanmerkingen geven of het goed geschreven is ja of de nee?

Als het goed geschreven is krijgt hij de opdrachten van mij.

http://pastebin.com/m3d565331

alvast bedankt voor de moeite.

[Mikey]

Uit dit stukje kun je niet ophalen of iemand wel of niet goed is. Vrij standaard.

[Thunderbite]

Het lijkt mij inderdaad ook niet dat je uit zon klein algemeen stukje kan/gaat beoordelen of iemand wel of niet geschikt is voor een opdracht.

Kan je om referentie contact of iets dergelijks vragen als je aan de kwaliteiten twijfelt?

[marsipulami]

Ik ga mee in de bovenstaande antwoorden.

Het wordt interresanter wanneer er intensieve berekeningen worden gedaan(dan kan je kijken of hij/zij effecient script) en wanneer er data over en weer wordt verstuurd via forms oid(om te kijken of hij/zij ook veilig script)

Ik zou meer code opvragen.

P.s. KLIK HIER?? Dit wordt over het algemeen als ongewenst en niet gebruikersvriendelijk aanschouwd.

[Robin L]

Is inderdaad lastig om uit de code op te maken of ik goed kan scripten of niet, momenteel heb ik niet echt complete systemen hier liggen. (Ja ik ben de scripter waar het in dit topic over gaat) Doorgaans werk ik vrijwel alleen voor klanten die aanpassingen willen omdat ik zelf geen tot weinig tijd heb om een project voor een klant te starten. Wel heb ik momenteel zelf 2 projecten lopen, welke ook volledig in de dergelijke code en in een genormaliseerde / geoptimaliseerde database geschreven zullen worden, hier kan ik helaas nog geen voorbeelden van tonen totdat ik het idee aangetekend en in een gesloten envelop naar mij heb toegestuurd, het is namelijk een uniek concept wat ik media mei zal lanceren en ik liever niet gestolen zie worden.

Referenties kunt u verkrijgen via de mail ;-)

EDIT:
Om in te gaan op de reactie hierboven, die tekst was enkel voor een klant van mij, www.spirituelegids.eu . Klant heeft mij opgelicht voor 135 euro en deze oplichter staat nationaal bekend, ook in TROS Opgelicht geweest, maar hier zal ik verder niet op ingaan, heb reeds zijn website offline kunnen halen na contact met de hoster (hosted.nl)

Ik zal even kijken naar een contact form die ik hier wellicht kan plaatsen.

[SF-Jeroen]

Ik zou zeggen laat eens een stukje met een paar ingewikkelde queries en berekeningen zien, dan is iedereen daarna blij (hoop ik).

[Robin L]

Ik zou zeggen laat eens een stukje met een paar ingewikkelde queries en berekeningen zien, dan is iedereen daarna blij (hoop ik).

Hierbij in ieder geval al vast een reactie systeem wat ik had geschreven voor het downloadsysteem van een eigen site van mij.

http://pastebin.com/m4f46818d

[systemdeveloper]

Ow leuk, even zeuren over andermans code

Je hebt html en php door elkaar en geen commentaar in de funktie. Dat kan straks wel eens lastig (duur) worden om te onderhouden. (Bij de meer ingewikkelder funkties dus, hehe).

Bij een try() catch() komt er meestal ook wel een throw exception bij. Hier alleen een return "", waardoor de aanroepende code dus zou moeten controleren of het resultaat de tekst 'Database error' bevat, om er überhaupt achter te komen of er een fout is opgetreden. Dit is 'fout' omdat je hierdoor een eventuele hoger gelegen error handling teniet doet. (Denk aan geneste try/catch constructies).

De naamgeving van de funktie kan beter: in principe is het een displayAllLinkPartners() en geen GetAllLinkPartners(). De funktie toont de records namelijk direct via een echo en 'get' dus eigenlijk niets. Als de deze 'getAllLinkPartners()' een returnwaarde heeft, dan zijn het geen linkpartners maar is het altijd een error (reg. 22). Fout? Niet syntactisch maar wel logisch.

De links die in de divs gebruikt worden zijn niet speciaal seo-vriendelijk. Dit zegt niets over de code uiteraard, maar maybe toch een punt om aan te denken.

Volgens 'de boekjes' is er wel meer over die code te zeuren maar voor de rest is hier niks uit te halen of het echt goed of fout is. Het blijft maar 1 funktie zonder context en die is 'goed genoeg'.

Maar het is uiteindelijk wel de klant zijn budget dat bepaald hoe goed iets is; deze funktie is in 5 minuten gemaakt, een hele nette en goede funktie duurt 10-15 minuten. Drie keer zo lang, drie keer zo duur.

[Robin L]

Om in te gaan om de reactie van 'systemdeveloper'.

Commentaar zit inderdaad niet in dit script, wél in bovenstaand (laatste) script dat ik nog heb geplaatst. Voorgaande functie was enkel één van mijn werken, daardoor kan natuurlijk ook niet al mijn ervaring beoordeeld worden neem ik aan.

Over het SEO vriendelijk zijn van de URL's, deze zijn natuurlijk altijd nog te rewriten met gebruik van de RewriteRule in htaccess, hierover valt dus geen uitspraak te doen.

Over die return functie, ik begrijp wat je ermee bedoelt, een duidelijkere afhandeling in het geval dat er error's ontstaan. Over de naam van de functie zal ik verder niet op in gaan, dit verschilt natuurlijk per scripter wat zijn/haar voorkeur hiervoor is.

[MMaI]

in je tweede code heb je dan weer geen direct gebruik gemaakt van exceptionhandling, maar pas je gewoon Query or Die tactieken toe.

Verder is er over het geheel niet veel te zeggen, uiteindelijk zal de TS gewoon goed vast moeten leggen wat zij aan code etc wil krijgen, en tegen welk budget. Achteraf kan er dan gecontroleerd worden of er voldaan is aan deze voorwaarden.

[Mikey]

Iedereen is nooit te oud om wat te leren:

 $query          =       "INSERT INTO reactiesysteem (id, did, uid, naam, bericht, ip, datum, tijd) VALUES ('', '". $_GET['id'] ."', '". $userid ."', '". mysql_real_escape_string($_POST['naam']) ."', '". mysql_real_escape_string($text) ."', '". $_SERVER['REMOTE_ADDR'] ."', NOW(), ". $tijd .")"; 

Is er een reden voor dat je $_GET['id'] niet controleert op input ? Zoals het er nu staat is het vrij eenvoudig om andere dingen te doen dan bedoeld zijn Verder prefereer ik liever

if( ..)
{
     if( ..)
     {

     }
}
else
{ 

} 

Ipv delen op dezelfde lijn, via deze weg kun je heel snel zien waar je een } vergeten zou kunnen zijn. Het laat het verhaal wat overzichtelijker. Maar dat is een persoonlijk puntje

[marsipulami]

ik snap niet waarom je in je laatste code:

"INSERT INTO reactiesysteem (id, did, uid, naam, bericht, ip, datum, tijd) VALUES ('', '". $_GET['id'] ."', '". $userid ."', '". mysql_real_escape_string($_POST['naam']) ."', '". mysql_real_escape_string($text) ."', '". $_SERVER['REMOTE_ADDR'] ."', NOW(), ". $tijd .")";

De $_POST wel met mysql_real_escape_string doet, en de $_GET niet. Blijft onveilig op deze manier.

[Robin L]

Zie het nu inderdaad dat die GET variabele nog niet beveiligd was, code is ook al oud overigens (was voor roboo.nl welke niet meer online komt en verkocht gaat worden). Gewoon vergeten dus ;-)

[marsipulami]

Wil ik je er toch even op wijzen dat TS onder andere aan de hand van reacties hier bepaalt of je wel mag scripten voor TS of niet. Dingen vergeten kan, maar om dan zulke code hier te plaatsen ..... ook al is de code oud.

[systemdeveloper]

Om in te gaan om de reactie van 'systemdeveloper'.

Over het SEO vriendelijk zijn van de URL's, deze zijn natuurlijk altijd nog te rewriten met gebruik van de RewriteRule in htaccess, hierover valt dus geen uitspraak te doen.

Over de naam van de functie zal ik verder niet op in gaan, dit verschilt natuurlijk per scripter wat zijn/haar voorkeur hiervoor is.

Mja, ook voor naamgeving heb je standaarden en een 'get' funktie is toch wel bedoeld om ook een returnwaarde te leveren. De programmeur die jouw werk moet overnemen als je ziek wordt oid moet het ook snel kunnen oppakken.

Over die SEO... als jij (met echo) 'blabla.php' in je webpage output, dan wil ik nog zien hoe je die met mod_rewrite gaat omzetten naar seo vriendelijke urls...
Andersom lijkt me redelijk standaard... maar met mod_rewrite een 'blabla.php' omzetten naar bv "/blabla/" is pretty pointless, aangezien je hem dan met mod_rewrite weer terug moet zetten naar 'blabla.php' en dus een loop zou krijgen.