Kadobon

misschien zo?

mysql_query("INSERT INTO `[users]` SET `code`=`code`='".$code."' WHERE `login`='".$result->login."'");

Pff code=code='" . $code . "'.... Dat kan niet!!

code='" . $code . "' of code=code+"' . $code . "'

Zo iets heb ik ook eens gemaakt voor me criminals.
ik kan je het wel uitleggen maar geen scipt geven.

pb me maar leg ik je het uit hoe je dat doe.
een gift code

Die code gaat niet werken, omdat je geen while loop gebruikt. Zie daarvoor [func]while[/func].

Ook is je code niet echt netjes geschreven, zie mijn top 10:

Citaat

1. Geen onnodige code, gebruik zoveel mogelijk loops
2. Geen shorttags, altijd <?php en ?>.
3. Variabelen buiten de quotes (bijv. echo 'Hallo'.$naam; en NOOIT echo 'Hallo $naam';.
4. Kolomnamen in MySQL zonder ` typen (alleen als het niet anders kan zoals bij bijvoorbeeld een numeriek veldnaam ` gebruiken)
5. ALTIJD tabben, dus bij elke accolade een tab ('inspringen'.
6. Javascripts zoveel mogelijk in de head sectie plaatsen
7. Zoveel mogelijk commentaar plaatsen, een andere programmeur kan zo makkelijk iets begrijpen
8. Inputs altijd checken op bijvoorbeeld numeriek, alfanumeriek, een bepaalde reeks en bij tekst inputs ALTIJD strip_tags() en i.c.m. MySQL(i) mysql(i)_real_escape_string();
9. Geef bij een SELECT query altijd ALLE veldnamen op, tenzij het er meer dan 20 zijn en je ze allemaal nodig hebt, dan gebruik je een wildcard.
10. Wees gericht op de toekomst, gebruik dus geen functies die al depracted zijn of die er binnenkort worden uitgegooid.

Toon Meer

Vooral punt 4 en 9 zijn erg belangrijk voor jou :).
Ook raad ik je aan mysql(i)_fetch_array(); te gebruiken i.p.v. assoc, volgens mij werkt dit een stuk sneller (ook omdat je een wildcard gebruikt) en bovendien is assoc een beetje uit de "mode".

Veel succes ermee!

Citaat

Returns an associative array that corresponds to the fetched row and moves the internal data pointer ahead. mysql_fetch_assoc() is equivalent to calling mysql_fetch_array() with MYSQL_ASSOC for the optional second parameter. It only returns an associative array.

Maakt geen kloot uit welke van de 2 je gebruikt, en uit de mode? Sinds wanneer is PHP een mode wereld geworden en daarnaast wordt assoc nog net zo vaak als het niet vaker is gebruikt dan array, aangezien ik geen rede snap om een row aan te roepen met $row['1'] ipv. $row['id']

mysql_query("UPDATE `[users]` SET `code`=`code`='".$code."' WHERE `login`='".$result->login."'");

Je roept het met assoc op en gebruikt object ?

Goed opgemerkt Frank, dat kan dus inderdaad niet ;).