PHP code not writing data into MySQL

Question

For some reason my php code isnt writing my variables into my mysql database.

Everything works up until the comment "dies here"

<?php
$name = ucwords($_POST['name']);
$sex = ucwords($_POST['sex']);
$age = intval($_POST['age']);
$email = $_POST['email'];

//DB ACCESS
$db = mysql_connect("localhost", "root", "root");
mysql_select_db("namedb", $db);
//DB ACCESS

if ($sex != 'M' && $sex != 'F') {

echo "Please go back and enter either M or F for 'Sex' <br />";
echo "<a href='index.html'>Back</a>";
die;
}

if (is_int($age) != yes) {
echo "Please enter a number for your age. <br />";
echo "<a href='index.html'>Back</a>";
die;
}

$query = "INSERT INTO people (age, name, email, sex) VALUES($age, $name, $email, $sex)";
mysql_query($query) or die ("Error Updating DB"); //DIES HERE
echo "Thanks $name, we've added you to our database.";

?>

My database is all set up, I have no idea why it isn't sending the data to the database. Here's a picture of the mySQL mysql

Thanks for the help.

Don't use mysql_ functions. Use PDO or mysqli with parameterized queries. That would also fix the serious SQL injection problems you currently have. — webbiedave
– webbiedave, Commented May 29, 2012 at 16:44
Could you give me a tutorial on how tO use PDO. Im not fimiar with it — user100051
– user100051, Commented May 29, 2012 at 17:26

Dejan Marjanović · Accepted Answer · 2012-05-29 16:35:43Z

3

$query = "INSERT INTO people (age, name, email, sex) VALUES('$age', '$name', '$email', '$sex')";

I don't support mysql_*

answered May 29, 2012 at 16:35

Dejan Marjanović

19.4k7 gold badges54 silver badges67 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Dejan Marjanović Over a year ago

You're welcome. Please consider using mysqli_* or PDO, mysql_* is in deprecation process, just so you know.

user1378430 · Accepted Answer · 2012-05-29 16:46:31Z

0

You should use this

  if (!is_int($age)){
       echo "Please enter a number for your age. <br />";
       echo "<a href='index.html'>Back</a>";
       die;
}

It's cleaner and probably more effective since is_int() returns a true or false statement and the if (is_int) basically means if (is_int($foo == true).

answered May 29, 2012 at 16:46

user1378430

Collectives™ on Stack Overflow

PHP code not writing data into MySQL

2 Answers 2

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related