Inserting into a database, HELP PLEASE

[internetguy]

I am having trouble inserting data into a database. The connection is good because there is no error reported in the mysql_connect(). Here is my code, let me know if there is anthing wrong.

<?php
$name = $_POST['name'];
$email = $_POST['email'];
$url = $_POST['url'];

$db = mysql_connect('localhost','root');
if (!$db)
{
echo 'Failure to connect to database, please try later.';
}
mysql_select_db('users');
$query = &quot;INSERT INTO info VALUES ('&quot;.$name.&quot;', '&quot;.$url.&quot;', '&quot;.$email.&quot;')&quot;;
$result = mysql_query($query);
if ($result)
echo 'your information has been submited';
?>

 

[DRJ478]

Add some error checking to the statement. MySQL actually tells you what's going wrong.

$result = mysql_query($query) OR die(mysql_error());

BTW, using 'root' is not advisable especially if there's no password. Set a root password immediately (as recommended by the MySQL installation instructions) and create a different user who is used to connect.

 

[DRJ478]

Also add:

mysql_select_db('users') or die(mysql_error());

 

[internetguy]

ok, it says that the column count doesn't match value count at row 1. I have 4 columns in my MySQL database, ID, name, url, and email. I made the ID row as an auto-incrementing integer so that it assigns each user an ID. I am not sure if this has anything to do with it or not.

 

[aidz98]

$query = &quot;INSERT INTO info VALUES ('&quot;.$name.&quot;', '&quot;.$url.&quot;', '&quot;.$email.&quot;')&quot;;

is just plain wrong, then. you have four fields to consider inserting to. you could rewrite that in two ways.

(1)
$query = &quot;INSERT INTO info (name, url, email) VALUES ('&quot;.$name.&quot;', '&quot;.$url.&quot;', '&quot;.$email.&quot;')&quot;;

or (2)
$query = &quot;INSERT INTO info VALUES ('', '&quot;.$name.&quot;', '&quot;.$url.&quot;', '&quot;.$email.&quot;')&quot;;

note the two single quotes at the start of your list to accommodate the auto_increment ID (assuming that's how you ordered your fields)... specify a blank field for auto_increment fields to have the DB automatically assign it

glad to be of help.

 

[internetguy]

Wow thank you so much, that line got it to work perfectly. I really appreciate it, I have been so mad cause I couldn't figure it out.