Jump to content

[SOLVED] Getting mixed up with opening and closing strings


eddy556

Recommended Posts

I have the following line:

 

mysql_query("INSERT INTO results 
(user, movie, input, prediction, error)
VALUES('" . $_POST['email'] '" , "' . $Movies[$m]['title'] . '","' . $Movies[$m]["rating"] . '","' . $result . '","' . $err . "' ) ") 
or die(mysql_error()); 

 

However I'm getting mixed up with the opening and closing of "'", can you please have a look?

You missed a dot that all mate.

 

. $_POST['email']  // .  << was missing ok '"

<?php
mysql_query("INSERT INTO results(user, movie, input, prediction, error)
VALUES('" . $_POST['email'] .'" , "' . $Movies[$m]['title'] . '","' . $Movies[$m]["rating"] . '","' . $result . '","' . $err . "' ) ")
or die(mysql_error()); 
?>

mysql_query("INSERT INTO `results`

(`user`, `movie`, `input`, `prediction`, `error`)

VALUES('" . $_POST['email'] ."' , '" . $Movies[$m]['title'] . "','" . $Movies[$m]["rating"] . "','" . $result . "','" . $err . "' )")

or die(mysql_error());

Here a safer code for you, with database protection in place.

<?php
mysql_query("INSERT INTO results(user, movie, input, prediction, error)
VALUES('" . mysql_real_escape_string($_POST['email']) .'" , "' .mysql_real_escape_string($Movies[$m]['title']) . '","' . mysql_real_escape_string($Movies[$m]["rating"]) . '","' . mysql_real_escape_string($result) . '","' . mysql_real_escape_string($err) . "' ) ")
or die("Database insert error\n".mysql_error()); 
?>

Redarrow, yours still have a little mistake (the single quote is in a wrong place):

 

Fixed here:

 

<?php
mysql_query("INSERT INTO results(user, movie, input, prediction, error)
VALUES('" . mysql_real_escape_string($_POST['email']) ."' , '" .mysql_real_esape_string($Movies[$m]['title']) . "','" . mysql_real_esape_string($Movies[$m]["rating"]) . "','" . mysql_real_escape_string($result) . "','" . mysql_real_esape_string($err) . "' ) ")
or die(mysql_error()); 
?>

Archived

This topic is now archived and is closed to further replies.

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.