login  Naam:   Wachtwoord: 
Registreer je!
 Forum

Update

Offline Martijn1989 - 07/05/2005 12:26 (laatste wijziging 07/05/2005 12:28)
Avatar van Martijn1989PHP ver gevorderde wat is hier fout aan :

Ik wil :
-Dat in de kolom, huis de waarde word veranderd in $huis

php code - Bekijk de code zonder highlighting - Klap code in 
  1. $insert2 = "UPDATE leden SET huis=$huis WHERE gebruikersnaam = '$MEMBER->gebruikersnaam'";
  2.    mysql_query($insert2) or die (mysql_error());


in dit stukje
php code - Bekijk de code zonder highlighting - Klap code in 
  1. SET huis=$huis
zit dus "de" fout volgens mij!

5 antwoorden

Gesponsorde links
Offline b4nkr0bz0r - 07/05/2005 12:28
Avatar van b4nkr0bz0r PHP gevorderde $insert2 = "UPDATE leden SET huis='" . $huis . "' WHERE gebruikersnaam = '" . $MEMBER->gebruikersnaam . '";
mysql_query($insert2) or die ('Query mislukt');
Offline Martijn1989 - 07/05/2005 12:30
Avatar van Martijn1989 PHP ver gevorderde (k)  
Offline Thomas - 07/05/2005 13:16
Avatar van Thomas Moderator Mja... Alleen de oplossing is niet zo interessant he, misschien snappen de mensen dan niet waarom het werkt.

De "huis" kolom in de tabel "leden" heeft waarschijnlijk een text-type (VARCHAR, TEXT, ...). Rond text moeten altijd quotes, en die ontbraken in de eerste query. Om numerieke velden hoeven trouwens géén quotes, terwijl men dit toch vaak doet.
Offline Legolas - 07/05/2005 13:20
Avatar van Legolas Onbekend kzou ook nog add- en stripslashes toevoegen. anders gaat ie mss query voortijdig afbreken, wat niet de bedoeling is:!:
Offline Shifty - 07/05/2005 15:51 (laatste wijziging 07/05/2005 15:52)
Avatar van Shifty Onbekend Ik ben het volledig met legolas eens. Zoals je script er nu uitziet kan je een zogenoemde "mysql injection" uitvoeren (als je nog een oude versie van php hebt). Probeer dus addslashes() te gebruiken. Deze kan je meteen in je query zetten (wel buiten quotes).

Probeer ook de namen van je queries wat beter vorm te geven. $query2 zegt niet zoveel, probeer daar iets als $wijzig_huis van te maken, of liever $qWijzig_huis. Zo zie je meteen dat je met een query te maken hebt (hungarian methode).
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2025 Sitemasters.be - Regels - Laadtijd: 0.202s