Ik ben voor school bezig met een systeempje, nu ben ik al niet geweldig met php en mysql , maar ik krijg niet uitgevogeld wat er mis is met deze query(s).

Als ik m uit probeer te voeren:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'met test test met test test met test test met test test metdafdsfasdf test test ' at line 1 ( dit gaat om de 2de query )

Hopenlijk kan iemand hier mij helpen.

Code:

 
$query="DELETE FROM beelden WHERE ID=".$_POST["id"]."";
($query,$conn) or die (mysql_error()); 
 
 
} elseif($_POST["verwijderen"] == FALSE ){
 
$query="UPDATE beelden SET TITEL=".$_POST["titel"].", OMSCHRIJVING=".$_POST["omschrijving"].", TAG1=".$_POST["tag1"].", TAG2=".$_POST["tag2"].", WHERE ID = ".$_POST["id"]."";
mysql_query($query,$conn) or die (mysql_error());

Code:

$query = "DELETE FROM beelden WHERE ID = ".$_POST["id"].""; mysql_query($query,$conn) or die (mysql_error());   } elseif($_POST["verwijderen"] == FALSE ){ $query = "UPDATE beelden SET TITEL = '".$_POST["titel"]."', OMSCHRIJVING = '".$_POST["omschrijving"]."', TAG1 = '".$_POST["tag1"]."', TAG2 = '".$_POST["tag2"]."' WHERE ID = ".$_POST["id"].""; mysql_query($query, $conn) or die (mysql_error());

De quotes om de post vars ontbraken.

Heel erg bedankt :)

Vergeet overigens niet om mysql_real_escape_string() te gebruiken voor user input.

een methode die ik altijd gebruik en overigens heel handig is:

$input1=$_POST["id"]

en dan in je query:

delete from bla where ID='$input1'

Origineel gepost door Harings Rob

een methode die ik altijd gebruik en overigens heel handig is:


en dan in je query:

Als je de waarde maar een keer gebruikt, dan zie ik er geen voordeel in. Alleen een nadeel door onnodig variabelen aan te maken (geheugen).

Je moet ook sowieso mysql_real_escape_string gebruiken zoals al eerder is gezegt. De huidige query zit vol met SQL Injections.

Of je stapt gewoon van mysql_* af en gaat PDO gebruiken.

Origineel gepost door J. Buitenhuis

Of je stapt gewoon van mysql_* af en gaat PDO gebruiken.

PDO kan ik aanraden, het is eenvoudig te gebruiken.