Posts by Darsstar

Je weet dat strlen() het aantal karakters in een string telt?
Ik denk dat jij intval() wilt gebruiken...
En haal die enkele quotes om die getallen weg!

Citaat van Reverb

Het kan nog steeds?

Ja, nu snap ik echt wat je bedoeld.
Kan alles nog? Kunnen links nog? Kan html nog gebruikt worden?
Wees eens iets duidelijker...

En code zou ook niet weg zijn...

1) strip_tags()
2) zie code:

$variable = preg_replace('{(?<=\b)((?:https?|ftp)://|www\.)[\w.]+[;#&/~=\w+()?.,:%-]*[;#&/~=\w+(-]}i', '', $variable);

Ik wil weten wie verantwoordelijk is voor die code!

Dat persoon mag zich namelijk heel stom gaan voelen en zijn hoofd een paar keer tegen iets hards aan stoten van mij...

stripslashes() maakt mysql_real_escape_string() zo goed als ongedaan (alleen de charecterset zij gewijzigd blijven lijkt mij)
Eerst beveilig je iets, en daarna maak je het weer onveilig...
Wie heeft die briljante logica ooit bedacht?

Ik weet nog steeds bijna zeker dat dat de oorzaak is hoor...
Een tweede mysql_real_escape_string() zou het namelijk weer veilig maken :p

"slogan = '$slogan',"

Waarom wordt daar geen mysql_real_escape_string() gebruikt zoals bij de rest van de variabelen?
Ik weet bijna zeker dat dat de oorzaak is...

Bij een include heb je geen ?key=value nodig...
Wat je doet is de code uit een ander bestand binnen de huidige code plaatsen.
En dus heeft die code toegang tot de $_GET van je index.php...

Pharox
isset() en empty() zorgen er juist voor dat je geen unidefined index/variable notices opduiken.

if ($_SESSION['uId']) // notice!


if(isset($_SESSIO['uId'])) // false

Als je regel 89 voor regel 87 zet zal de eerste notice niet meer voorkomen.
De tweede lijkt me een typo: $nFetch ipv $sFetch.

Het lijkt me erg vreemd als Criminolz/templateparser/src/class.template.php moet worden chmod naar 777...
Ik zou de map "compiled" (en inhoud) chmodden naar 777...

Citaat

Weet iemand waarom dit niet werkt?

Nee.

Het kan met alle soorten bestanden (voor zover ik weet), of je browser ze kan weergeven is een andere vraag...
Je moet alleen opzoeken wat de juiste content-type header voor het bestand is.

http://www.anyexample.com/prog…allery_(blob_storage).xml
*heeft gezocht op "blob", "image" en"mysql"*

simple uitleg:
get_file_contents() van de afbeelding opslaan.
Extensie opslaan.
De juiste headers verzenden als je de afbeelding wilt zien als afbeelding, en de inhoud van het blob veld echoën.

Ik zou eerder een I18n (klikje) class maken dan wat je nu hebt.
En dan simpel return i18n::message('User::logout()'); of hoe je het ook wilt laten werken.

Dan heb je ook nog wel bestanden vol met teksten nodig, maar de logica die de juiste kiest past mooi in de I18n class.

Verder zou ik ook een User class maken en mysql_fetch_object($query, 'User'); gebruiken om het meteen een mooie instantie van het user object te krijgen (en hier de logout() method in plaatsen in plaats van een Language class wat niets met een ingelogde gebruiker te maken heeft, niet dat het user object de optimale plaats hoeft te zijn)

Waar het argument van Patrick op neerkomt is het recyclen van code, en dus tijd besparen omdat je iets wat je al eens gescript hebt niet telkens opnieuw moet scripten.
En tijd === geld zoals we allemaal wel weten.

Verder is het idee achter OOP (ik maak even een grote stap) dat je op een manier script die meer lijkt op de echte wereld.
Een mens bestaat voor een groot deel uit organen, elk orgaan zou door een aparte class voorgesteld moeten worden omdat elk orgaan andere functies heeft en andere eigenschappen.
De buikholte is bijvoorbeeld te warm voor teelballen om goed te kunnen werken, en in de maag is het erg zuur enz.
Als zo'n systeem voor de natuur goed werkt, waarom dan niet voor programmeer talen?

Verder kunnen twee instanties van de zelfde class drastisch verschillen door dat bepaalde eigenschappen verschillen.
Een bepaalde functie aanroepen op instantie A en B met precies de zelfde parameters kunnen hele andere resultaten hebben, een zelfde soort werking met alleen functies is niet of heel moeilijk te maken.

Het lijkt me dat er nog meer zijn, maar ik zou het op dit moment niet meer weten...

Ja, maar dan kan je maar met een database verbinden...
Misschien heb je dat nu niet nodig, maar als je deze class wilt hergebruiken zou ik er alvast rekening mee houden...

http://www.php.net/manual/en/language.oop5.basic.php
Daar zie je ook geen "public $this->var;" maar "public $var" staan...

Waarom is $database verdwenen uit de paremeter lijst van Database::__construct()?

Ook wil ik nu echt eens van je horen waar volgens jou $RespondsHandler vandaan komt...
Want die variabele bestaat NIET binnen die methods...
En zet eens een "return FALSE;" na elke keer dat $RespondsHandler iets met de fouten doet, dat is ietsjes mooier...

En maak van van de else in Database::Value() die controleert of $this-Value (dat moet natuurlijk $Value worden) een array is eens een elseif die controleert of $Value een string is.
Dan maak je er nog eens als bij die mysql_fetch_assoc($Query) returned.

Kunnen we weer een beetje meer ontopic gaan?
Voor zover dat kan binnen de offtopic categorie dan...

Niet dat ik weet, maar je kunt beginnen met de velden waar het probleem optreed, dus waar mensen in de min komen te staan...

Plaats voortaan niet alles als een Request aub.
We hebben een server-side categorie waar het beter past.
Maar het past nog beter in de categorie voor Banditi!

Wil je dat het niet meer mogelijk is om in de min te gaan?
Regel dit in de database: maak alle int velden UNSIGNED (onder de attribuut select box in phpmyadmin)

Citaat van Kabling

Darsstar het was ook niet echt op jou gericht. Ik bedoelde het meer in het algemeen. Want mijn post count is helemaal zeer laag vergeleken met de rest.
En ik vind eerlijk gezegd Joshua ook helemaal niet te jong. Want hij weet wel precies hoe hij zich moet gedragen tegen over de rest.

Ik bedoelde te zeggen dat ik het zo ongeveer het zelfde doe, dus als ze daarom op jou zouden zeuren, dan mogen ze ook op mij gaan zeuren...
Eens zien of ze dat gaan doen XD

Citaat van Kabling

En ik help al wel mensen, ik probeer al dingen op te lossen voor bepaalde mensen en let goed op het forum. Jullie zien geen reacties maar dat wil niet zeggen dat ik niet gekeken heb naar het forum.

Vergelijk mijn postcount eens met dat van anderen (niet crew) leden.
Dat ik niet post in topics die ik niet interessant vind wil toch ook niet zeggen dat ik ze niet bekijk :p

Verder heb ik zelf geen commentaar over de leeftijd van Joshua omdat ik zelf ook ergens op mijn veertiende bij de crew kwam (ik was destijds ook gevraagd als mod voor het Ogame forum, maar was 2 jaar te jong (geen 16 jaar), de engige houvast die ik heb aan hoelang ik in de crew zit :p)

*begint heel eerlijk te zijn*
Joshua en Rick zouden als als ik een lijstje op had moeten stellen (zonder gesprekken etc.) onder het "twijfel" kopje terecht zijn gekomen, en geen van de drie mod's stond daar. (wat betekend dat de twijfel gevallen niet zijn gekozen)
Maar ze hebben nog 8 dagen me fout te bewijzen, als ik ze dat niet al gedaan hebben... (dit ga ik niet verklappen)

Ok, daar komt het weer:

Bij de constructor heb je (misschien onbewust) iets heel slims gedaan, NULL als de standaard waarde gezet.
Zelf had ik geen standaard waarde neergezet en dus vier verplichte parameters.
Maar mysql_connect(); zonder enige parameter kan gewoon werken (het hangt van de php.ini af of er een werkende verbinding wordt gemaakt).
Maar mysql_connect(NULL, NULL, NULL, NULL); is precies het zelfde als mysql_connect(); (heb het getest)
Gefeliciteerd, je hebt (misschien onbewust) een feature toegevoegd die niet voorgesteld was :p

De fouten:

# Create standard variables for class
    private $this->identifier;
    private $this->SqlLine;
    private $this->Value;

Het 'this->' gedeelte moet weg, dat geeft fouten, we zitten dan wel binnen de class, maar nog niet binnen een method.
Verder lijk je Value en SqlLine niet voor elke method beschrikbaar te stellen, ik zou ook niet weten waarom, maar die mogen dus weg...
$this->identifier zou ik gebruiken voor de optionele 2e parameter voor mysql_query(), voegt weer een feature toe.

Database::__construct()
De constructor nog even public maken aub.
$RespondseHandler bestaat niet binnen de methods, je zult eerst $RespondseHandler = new RespondseHandler; of iets dergelijks binnen de method moeten plaatsen...

Database::Value()

if ( !$SqlLine ) OR ( !$Value )

") OR (" die haaktjes horen daar niet.

if ( !$SqlLine OR !$Value )

Dat is correct
Je sloot de if, dan kwam OR wat niet zomaar ergens hoort te staan en daarna open je een haakje waar PHP ook niets van snapt.

De rest van de Database::Value() method laat ik maar even zitten omdat er toch niets veranderd is...

Database::Rows()
Controleer ook nog even of $Query wel een string is of niet.

Omdat Database::SqlQuery niet statisch is kun je die niet zo aanroepen (de naamgeving van methods is altijd alsof een method statisch is, mocht het je nog niet opgevallen zijn).
Omdat het niet statisch is heb je een object van de Database class nodig waar je de method op kan aanroepen, omdat we al binnen een niet-statische method zitten is dat al gebeurd en representeert $this dat object.
$this->SqlQuery($Query)

self::identifier = mysql_connect($hostname, $username, $password);

Die regel zal altijd fout zijn, puur om de reden dat er een dollarteken mist.

self::$identifier = mysql_connect($hostname, $username, $password);

Die regel zou goed kunnen zijn, mist $identifier een statische variable is, maar in dit geval is dat niet zo.

$Value = array('id', 'username', 'password');
$Value = array_flip($Value); // $Value = array('id' => 0, 'username' => 1, 'password' => 2);


$assoc = mysql_fetch_assoc($query); // we hebben alle velden geslecteerd, verzin zelf maar net zoveel velden als je wilt naar 'id', 'username' en 'password'


// de eerste parameter van array_intersect (ik was een c vergeten in mijn eerste post kom ik net achter)
// is de master array met de waardes die je uiteindelijk wilt hebben
// de tweede parameter is een array zodat je de keys van beide arrays kan vergelijken
// de return value is een array die alle waardes (de keys zijn onaangetast) bevat voor de keys die in beide arrays voorkomen
// array_flip is dus nodig om de waardes van $Value de keys te laten worden.
$array_return = array_intersect_key($assoc, $Value); // $array_return = array('id' => (een id), 'username' => (een username), 'password' => (een password))

hopelijk snap je die regel nu wel?
Het is precies wat jij wilt, alleen dan is er geen foreach loop nodig.

Edit: Ik zag net dat $database = NULL ook nog een parameter voor de constructor is...
Ik zou dat juist vooraan zetten en die verplicht maken (geen standaard waarde)
dan kun je $db = new Database('criminalspoint'); gebruiken ipv $db = new Database(NULL, NULL, NULL, 'criminalspoint'); :p

Nieuwe reactie samengevoegd met originele reactie op 18.01.10 22:15:25:
Trouwens, "if so run the if otherwise the else" als je zulke comments neerzet kun je net zo goed een heel PHP boek in commentaar voor elke regel zetten... :p