Nick Velten, dat geloof ik niet. Ik ga nu even kijken of ik wat op Sitedeals kan vinden. Overigens, game-projects was van G4G?

ALTER TABLE users ADD leeftijd TINYINT(3) UNSIGNED default '0';

Dat stel ik voor...

En post anders je hele edit.php maar.

Bedenk je wel: Een MSN invite moet je niet gebruiken wanneer je er geen verstand van hebt! Dat zal je host niet leuk vinden! MSN invite scripts laten namelijk vaak toe om tienduizenden mailtjes te versturen.

Je leeftijd moet geen VARCHAR zijn, maar een TINYINT (nog nooit iemand die ouder als 128 is ontmoet, dus TINYINT is genoeg).

Daarnaast hebben we meer code nodig om de fout te ontdekken, meer code uit edit.php. Bijvoorbeeld het formulier.

Daarnaast staat er een fout in je query, je hebt een komma tussen aanhalingstekens staan, maar die komma hoort daar niet.

Enkele tips voor je:

Gebruik wel quotes om de attributen van de HTML tags heen, anders is de HTML niet valid.
Deze code is niet functioneel als Captcha vervanging, omdat:
- Computers ook kunnen optellen en vermenigvuldigen.
- Het antwoord al in een hidden field staat, deze is voor een bod zo te lezen!

Verder ziet de code er redelijk uit, probeer het nog iets meer over regels te spreiden, en wees iets duidelijker in de naamgeving ($x en $y zegt niet zoveel, al zijn het standaard variabele die vaak een telling bevatten). Daarnaast zijn hoofdletters niet gebruikelijk in langauge constructors (echo(), print() en switch() zijn bijvoorbeeld langauge constructors, maar array() is er ook één).

Nieuwe reactie samengevoegd met originele reactie op 05.05.09 13:28:58:
HTML commentaar tags (<!-- Commentaar --> ) zijn niet nodig binnen een extern CSS bestand. Het is eigenlijk zelfs niet meer nodig binnen een HTML bestand zelf!

Edit:

Citaat

http://www.criminalspoint.com/ledenlijst/leden/pagina/23.html
Geeft geen resultaten, waarom is deze pagina er dan?

Het zou wel eens de bekende ceil() en floor() afrondfout kunnen zijn (staat wel op php.net). En anders klopt er iets niet helemaal.

Uitbreidingen zijn inderdaad nog mogelijk!

Niels: Ik stelde het zelf voor... Sorry, maar ik vind dit een beetje te kort door de bocht.

En om jou niet teleur te stellen: Ik heb het niet voor Koen gedaan, maar voor Criminalspoint...

Flatliner:
ADD `winkel` INT NOT NULL DEFAULT NULL

NOT NULL DEFAULT NULL... Analyseer en concludeer. Sorry, maar zoiets ga ik echt niet voorkauwen. Pak desnoods het woordenboek erbij, dit is echt niet lastig en zeer logisch.

Als je nu kunt vertellen van de fout is, dan wil ik je wel verder helpen...

Bedankt voor alle positieve reacties!

Durk: Die zoekfunctie is er al! (Was er altijd al, maar door een foutje functioneerde die niet)

De ordering moeten we het nog even over hebben. Maar wanneer je zoekt, sorteert hij in ieder geval wel op alfabet!

Als het Open Source zou zijn zou je er juist wel geld voor mogen vragen (zie de GNU licentie). Maar dit is geen opensource, dit is gewoon gejatte rotzooi.

Dat klopt. Wanneer je alle leden bekijkt, staan ze gerangschikt op het aantal posts.

Beste Criminalspoint leden,

Zojuist heb ik de ledenlijst voor Criminalspoint afgerond. Ik had aan Koen gevraagd of ik dit mocht doen, waarop ik ben begonnen.

De ledenlijst vervangt nu de oude ledenlijst, en daarnaast de pagina's leden online en crew.

Ik wil Wouter even bedanken voor het uploaden, en het meedenken en snel oplossen van kleine foutjes.

Nu kunnen jullie weer lekker de ledenlijst doornemen (leuk he:p). Graag hoor ik wat jullie er van vinden!

Groeten,

Killingdevil, die hoopt in de toekomst nog eens iets voor Criminalspoint te kunnen doen!

@game-projects: Lees mijn reactie eerst nog eens... Ik maak helemaal niet andermans werk zwart... Ik zeg er niks over dat bepaald werk slecht is. Dat maak jij er misschien van.

Hoe weinig verstand kunnen mensen hebben van de dingen waar ze mee bezig zijn...

Veldtype: INT, DATETIME, VARCHAR, TEXT, enz...
Lengte: Lengte, niet bij alle veldtypes nodig: Dat wat tussen haakjes staat...
Standaard: DEFAULT, in dit geval niet van toepassing.
Null: NULL of NOT NULL... Meestal NOT NULL (ook standaard)

De rest heb je niet nodig, en snap je waarschijnlijk toch niet...

ALTER TABLE tst
ADD(
        drank int(15) NOT NULL,
    zak int(15) NOT NULL,
        drankt datetime NOT NULL,
        xp int(15) NOT NULL default '10'
)

Het moet ADD zijn. En je had een type foutje: NUL in plaats van NULL.:) Dat de ALTER TABLE opdrachten tussen () konden wist ik niet! Weer wat geleerd!

Twee verschillende users.;)
Eéntje die wel toegang heeft tot database B (Backup database), en één niet. Normaal maak je dan connectie met een user die niet bij database B kan, alleen in de cron. Bij een beetje host moet dat kunnen.

En als je root toegang hebt: Gewoon de database bestanden kopiëren natuurlijk.

Geen problemen mee gekregen, maar dat neemt nog niet weg dat de banditi-source minstens net zo illegaal is als de G4G source.

Het jatten van andermans werk mag niet, ook (of zelfs zeker) niet de lay-out en andere grafische elementen. En daarnaast, eigenlijk ben je gewoon een onbeschofte klootzak als je met andermans source geld gaat lopen vangen. Maar goed, daar maken mensen hier zich niet druk om, he?

Dat script ziet er wel redelijk uit. Maar denk wel dat het vrij sloom is.

TRUNCATE test2.tst;
INSERT INTO test.tst SELECT * FROM test2.tst;

Even LIST TABLES doen, vervolgens dat voor alle tabellen uitvoeren. In het voorbeeld is test2 een exacte kopie van test, deze wordt dus steeds opnieuw gebackupt (om de week bijvoorbeeld).

Ik zal aanraden de hele database gewoon te kopiëren naar een andere database.

Wil je nu graag de database in een bestand hebben (een dump), zou ik alle gegevens van de tabellen in csv bestanden zetten...

Dan kan zeker wel. Wat wil je allemaal opslaan? Echt alles, of alleen de inhoud van de tabellen?