Gebruik van criminolz

Het gebruiken van criminolz is totaal niet erg dat wil ik als eerste even voorop zetten ik heb deze source alleen gekozen omdat hij sowieso al onder GNU staat dus ik aan mag passen wat ik wil.

Ik zat een beetje rond te kijken en zat me te verbazen hoe rommelig het allemaal inelkaar steekt.

Punt 1:
Jullie hebben een template parser wel ingebouwd zitten maar als nog gebruiken jullie HTML in je php bestanden, waarom?

Punt 2:
Waarom kom ik steeds bestanden tegen die SQL injectie gevoelig zijn. Vooral bij de inlog pagina is dat belangrijk dat die er niet inzit.

Na wat rond kijken dacht ik, ik laat even zien wat ik bedoel met rommelig:

Hier de versie die het eerst was:
(Het zijn 2 .php bestanden)

http://pastebin.com/m6624a8c7

Ik heb letterlijk overgekopieert en even een kleine opmerking tabben is niet zo heel moeilijk en behoud het overzicht en WHY OWH WHY gebruiken jullie bij "inc.login.php" 2x een login functie 1x met md5 hash en 1x zonder md5 hash.

ten eerste die zonder md5 hash snap ik sowieso al niet waarom hij erin zit ten tweede snap ik niet waarom het er 2x in zit. Als je dan over gaat naar md5 verwijder dan de andere inlog mogelijkheid.

(en ja ik weet er zijn updates geweest, geef dan een converter tool mee die alle wachtwoorden in md5 zetten)

Daarnaast login.php is vrijwel een html document WAAROM gebruik je geen template parse daarvoor zijn die dingen gemaakt en hij zit er nog wel is ingebouwd bij jullie.

Het gebruik van @ moet je als het ook maar kan vermijden in dit geval is het makkelijk te vermijden door met een "if" statement te werken en dan kan je dat ook achterwege laten.

En hoe ik het heb aangepast:
(Zijn ook 2 bestanden)
http://pastebin.com/m1917c436

Het gebruik van tabben gebruik ik zelf ook en ik hou een structuur aan die ik altijd gebruik. Ik zeg niet dat het de beste programmateur is die er bestaat maar ik wil wel zeggen dat bepaalde handelingen tegenwoordig nodig zijn om geen hackers in je game te krijgen.

Zie dit ook niet als commetaar om commetaar te geven maar opbouwende kritiek die gebruikt kan worden bij de volgende update's van criminolz (als die nog komen)

Greetz,
Patrick

Nog een puntje:

het onnodig aanmaken van php bestanden:
/outgame/crew.php:

<?php
require("ingame/crew.php");
?>

verplaats het script dan gewoon?

Het kan dan wel van te leren, maar dan moet je juist de sql injectie's etc eruit halen en leren met een template parser om te gaan (zo kan je er miss later zelf 1 gebruiken)

Daarnaast ik zelf ben ook geen proffesionele scripter, ik ben zelf ook maar bezig met het aanpassen maar dan kan je jezelf nog altijd wel een nette manier van programmeren aanleren en dat is het belangrijkste vind ik.

En als ik niks te doen heb verander ik zulke scripts altijd, al is het voor mezelf al is het voor andere gebruikers

Ik heb weer niks te doen op school en zit dus verder bestanden te veranderen naar overzichtelijke programmatie.

Alles heb ik omgezet in samenwerking met de template parser. En uit eindelijk wil ik de bestanden vrijgeven als ik de volledige outgame heb gedaan en daarna als ik zin heb ga ik de ingame doen.

Tevens ga ik de optie van de FAQ maken, zodat je via het administratie paneel zelf categorieen en artikelen in de FAQ kan zetten zodat je niet in het bestand zelf moet zitten om het te veranderen.

(Deze optie had ik reeds voor een andere game gemaakt alleen die heb ik nooit afgemaakt)

Daarnaast ben ik van plan de indeling van de crew status te veranderen het gebruik van de velden "admin" en "moderator" ga ik eruit halen en ik ga het veld "level" terug brengen. Dit keer alleen ga ik een makkelijk overzicht er voor maken met de volgende levels:

4 = Game administrator
3 = Super game operator
2 = Game operator
1 = Gebruiker

en eventueel een level 5:
5 = Game development

(Deze groep is gelimiteerd tot 1 gebruiker voor veiligheid)

daarnaast is de contact script beveiligd met een check of het email adres wel geldig is en daarnaast moet ik er nog een captcha inbouwen hiervoor gebruik ik recaptcha (simpel maar zeel doeltreffend)

Voor enige tips, opmerkingen, toevoegingen, aanpassingen kan je het altijd vragen hier