Citaat van Perrytje11

Wat vind je er fout aan? Het werkt gewoon...

Dat is de verkeerde instelling die ik vaak in de Banditi en weet ik veel welke sources topics zie staan. Het werkt toch? Ja, toevallig wel, maar goed is heel wat anders!

Je zult quotes om halen moeten zetten. Zet in die Banditi source de error reporting maar eens hoog, krijg je wel 100 errors per pagina. Snap niet dat jullie dat allemaal niet dwars zit en er niemand wat aan doet? De scripts worden er heus niet sneller op hoor, en zeker niet leesbaarder.

Nou, een user is nog lang geen connectie. Ik zou ongeveer als volgt de connectie class programmeren.

class DatabaseConnection {
 protected $user = 'Henk';
 protected $password = 'P@ssw0rd';
 protected $host = 'localhost';
 protected $database = 'db';


 protected $connection; //PDO connection


 private static $instance = null; //Instance of own class, singleton.


 /**
 * Private constructor, singleton
 */
 private function __construct {
  //Create the connection
  $this->connection = new PDO(...);
 }


 /**
 * Returns the singleton instance of this class.
 */
 public static function getInstance() {
  if(self::$instance == null)
   self::$instance = new self();


  return self::$instance;
 }


 public function getConnection() {
  return $this->connection;
 }
}

Zoals je kunt zien een singleton class. Ik denk dat de connection class ook niet echt iets is waar je mee moet beginnen. Het is namelijk erg moeilijk een connection class goed te ontwerpen, de meningen zijn verdeeld over hoe een goede connection/database class eruit ziet.

Dus al die namen worden dan namen van classes? Waarom zou je dat willen, tenslotte is DB_user toch geen DB_con? En DB_user is denk ik gewoon een string zoals 'Henk'? Hetzelfde geld voor user, host en denk ook select.

Ik denk dat je een class DatabaseConnection moet maken, die een connectie terug geeft (PDO, MySQLi), zo heb ik de vorige keer ook gewerkt, al was dat in Java.

Ik weet niet wat voor een namen dit zijn? Zijn dit de properties/eigenschappen van de class?

Citaat

Zelfde voor jou, brein aan! lees dat het al opgelost is.

Met alle respect, maar met een oplossing die complete onzin is zeker? Dat is nu precies waarom wij topics open houden, een oplossing is nog lang niet de beste of zelfs een goede oplossing.

Ik kan denk ik vrij duidelijk advies geven: Nee, start niet met een framework en nee, start niet met een designpattern.

Wanneer je met een framework bezig bent zijn de restricties waar je wat voor een klasse aan moet maken en wanneer zo duidelijk, dat je werkelijk niet leert object georiënteerd te programmeren. Je leert wat je framework wilt, en dan programmeer je nog lang niet per definitie goede object georiënteerde code.

Ik zou zeggen: Begin met het lezen van het voorbeeldje over de Auto en over het Voertuig, om er vervolgens geen fuck van te snappen. Onthoud waar het over ging, en ooit gaan het lampje branden wat ze nu werkelijk aan het vertellen waren.

Zorg daarnaast dat je weet wat een klasse is, wat methoden zijn, wat eigenschappen van een klasse zijn, wat visibility (dit wil je echt niet naar het Nederlands vertalen) van een methode of eigenschap is en wat voor een relaties klassen met elkaar kunnen hebben.

Dan kun je wel een voorzichtig begin maken, wat ik wel kan voorstellen is een Gastenboek. Denk dat het een goed idee is dat je een aangeeft welke klassen er in jou gastenboek script komen, dan kunnen wij hier beoordelen of je in de goede richting denkt.

Ik zit een beetje door je code heen te scannen, en toen zag ik de methode cleanWhiteSpaces() staan. Vervolgens scrolde ik naar beneden om deze reactie te schrijven, en werd ik nogmaals geholpen:

static public function execute($sText)
    {
        $oParser = new ubb($sText);
        $oParser->cleanWhiteSpace();
        $oParser->getPieces();
        $oParser->createTree();
        $oParser->validateTree();
        $oParser->parse();
        echo $oParser->sText;
        return noXSS($oParser->sText);        
    }

Ik heb niet nagezocht of dit ook werkelijk het probleem oplost. Echter, de kans lijkt me erg groot!

Succes.

[offtopic]Vrij goed stukje code moet ik zeggen... Waar komt het vandaan?[/offtopic]

Het is een heel slecht idee om met globale variabelen te gaan kloten. Ik snap dan ook niet wat het probleem is aan het meegeven van $data via een parameter?

function tekst($tekst, $data) {}

Maak a.u.b. niet meerdere topics over hetzelfde onderwerp aan en met dezelfde naam.

Topic gesloten.

@Perrytje11: Ja, natuurlijk staat zijn error reporting aan. Anders krijgt hij toch ook geen errors?

@Crimebrain: Bescherming? Wat voor bescherming waar tegen?

Ik denk dat je (onzichtbare) output voor de open-tag hebt staan. Kopieer de code eens, en plak het dan in een ander bestand en sla het weer op. Misschien scheelt dat.

Je zult het click() event (waarmee je neem ik aan een tabblad opent, door te klikken), kunnen triggeren.

Dat is zo moeilijk niet:

$("#tab").click();

http://api.jquery.com/click/

Nee, $data bestaat niet. Tenminste, niet in de scope van de functie. Vreemd genoeg weten hier heel veel mensen niet dat functies in hun eigen scope werken. Daarom moet je ook parameters meegeven en daarom zijn ze OOK zo handig.

http://php.net/manual/en/language.variables.scope.php

Heel langzaam maar zeker komt er een beetje feedback binnen. Meer feedback is zeker welkom. Wanneer we een patroon zien in de feedback kunnen we er echt goed iets meer doen.

Dit is jullie kans om iets aan CP of de crew te veranderen!

Je zet nu feitelijk meer data in één kolom. Dat moet je nooit doen, want dan krijg je van dit soort ongein en ben je je database wel heel erg aan het denormaliseren.

Mocht je hiermee willen bereiken dat de data minder opslagruimte kost heb ik een beter idee voor je:

Een IP past precies in een integer: [func]ip2long[/func]

Dus gewoonweg twee velden aan maken, dat is veel effici?nter.

Citaat van Killingdevil

Sorry, maar ik ga toch eventjes ingrijpen.

Van dit topic was het de bedoeling grappige nutteloze berichten uit andere topics te quoten. Jullie zijn nu eigenlijk meer aan het chatten, en daarvoor nodig ik jullie uit in de chat (die werkt toch?).

Ik weet dat dit het offtopic gedeelte van de website is, maar graag toch een beetje meer ontopic. :cheer:

Ik denk dat jullie de laatste kans nu wel hebben gehad?

Volgende offtopic reactie wordt zwaar bestraft!

Even een update:
Tot nu toe heb ik maar liefst 0 PM berichten met feedback ontvangen.

Vraag me dan af: Waarvoor waren al die topics dan? En was toch zoveel aan te merken? En nu via een niet-publiekelijk kanaal hoeft het ineens niet meer? Betekent dat dat alle commentaar eigenlijk meer een schreeuw om aandacht was van een selectief groepje leden?

Jullie zijn nog steeds allemaal uitgenodigd om via PM feedback te geven. Positieve feedback is natuurlijk ook welkom. We horen ook graag wat er juist wel goed gaat.

Het is ook niet bijzonder. Maar ik vind het wel leuk gedaan.

Zijn we er met CP toch ook nog een klein beetje bij.

Ps, klein detail: Onze belg Koen heeft deze erop gezet. :p

Op aanvraag van TS sluit ik het topic. Zie er geen heil meer in...

Dit topic sluit ik. Ik verwijs jullie graag door:

http://www.criminalspoint.com/forum/topic/8760/Feedback.html

:slotje:

Beste Criminalspoint leden,

Het valt mij op (en zo vreemd is dat niet, waarschijnlijk valt het iedereen wel op), dat er de laatste tijd veel 'feedback' wordt gegeven op de crew door middel van allerlei topics. Langzaam maar zeker beginnen we deze topics toch wat irritant te vinden en we hebben dan ook het idee dat het eigenlijke idee van deze website (een gezellig forum met als hoofddoel elkaar te helpen) eronder begint te lijden.

Maar natuurlijk is goede feedback nog altijd welkom. Daarom nodig ik jullie vanaf nu uit alle feedback naar de crew, een individueel crewlid of naar criminalspoint via PM naar mij toe te sturen. Intern regel ik dan dat de feedback bij de betrokken personen terecht komt (de gehele crew, de supermoderatoren, de betrokken moderator, enz.). Hoe die PM eruit moet zien lees je hieronder.

Het is belangrijk dat je feedback geeft waar wij iets mee kunnen. Als je CP slecht vindt geloven wij dat wel, maar wij willen graag weten waarom. Feedback mag natuurlijk algemeen zijn, zoals: 'Moderator x deelt veel te snel banpunten uit', of: 'Criminalspoint is veel te chaotisch'. Maar ook feedback op een bepaalde actie van een crewlid is zeer nuttig voor ons.

Wanneer je feedback op een crewlid wilt geven aan de hand van een reactie die hij van je heeft verwijderd, vermeld dan de link naar het bericht waarin staat dat je reactie is verwijderd met de reden. Hetzelfde geldt voor wanneer je een ban die je hebt gekregen wilt aanhalen.

Stuur de PM berichten a.u.b. in het volgende formaat:

Onderwerp: Feedback

Bericht:

[b]Feedback op:[/b] Criminalspoint/Crew/Naam crewlid
[b]Met betrekking op:[/b] eventuele links naar PM berichten of forum berichten.


De concrete feedback in goed Nederlands, met een [b]duidelijke[/b] onderbouwing.

Zodra ik je feedback heb verwerkt zal je van mij een bedankje krijgen, en wanneer er voldoende feedback wordt gegeven zal ik later een topic starten waarin aangegeven staat wat voor een feedback we hebben ontvangen en wat we er mee gaan doen.

Alvast bedankt voor al jullie feedback en medewerking.

Dit betekent ook een einde aan alle feedback topics. (Vooral slecht onderbouwde) feedback in het openbare forum wordt bestraft. Of liever gezegd, commentaar.

Met vriendelijke groet,

Mede namens de rest van de crew,
Killingdevil
Tijdelijke projectleider Criminalspoint.