Inserting data in mySQL

AJPayne

hey

Well I have a form, and I need to insert the data the user entered iton its corresponding field in the DB.

basically, here is what I have so far:

<?php

$username = "username";
$password = "pass";
$database = "db";

mysql_connect(localhost,$username,$password);
@mysql_select_db($database) or die( "Unable to select database");

$name = $_POST['company_name'];

if($_GET['action'] == "add")
	{

	$query = "INSERT INTO host ('host_id','name') VALUES ('','$name')";
	mysql_query($query);
}



mysql_close();

?>

but it doesn't work.

What am I doing wrong?

thanks

Shrike

Does $_GET['action'] actually equal 'add' at the relevant point? If so then use mysql_error() to see if there's an SQL error. Name is quite possibly a reserved word although the quotes might get around that.

AJPayne

Hi,

Thanks for your reply.

When a user submits the form, the url changes to form.php?action=add

and that what the $_GET['action'] == "add" does.

tsinka

Hi,

reading your post I assume that you have something like

Try to change it to:

<form action="form.php" method="POST">
<input type="hidden" name="action" value="add" />

Then use $POST['action'] instead of $GET['action'].

If the form has several submit buttons (e.g. one for add and one for delete) the do something like:

<form action="form.php" method="POST">
....some form elements....
<input type="submit" name="delete" value="delete stuff..." />
<input type="submit" name="add" value="add stuff...." />
</form>

// in form.php
if (isset($_POST['add'])) {
  // add data
} else if (isset($_POST['delete'])) {
  // delete data
}

If the method of the form is GET you need to use $GET again instead of $POST.

Additionally, add mysql_error to the mysql_query line like suggested by Shrike, e.g.

mysql_query($query) or die("SQL error: ".mysql_error());

Thomas

AJPayne

well this is the error I get:

http://www.findagoodhost.com/host_signup.php

I tried everything, but doesn't want to work.

And no there isn't any delete function as you suggested.

Thanks.

tsinka

Hi,

post the code that causes that error.
I suggest that you use $_POST["something"] directly in a string or maybe a whitespace where it shouldn't be.

Thomas

AJPayne

yea sure.

Line 18 is:

$query = "INSERT INTO host (host_id,name) VALUES ('','$_POST['company_name']')";

However, when I put the code as follow:

if($_GET['action'] == "add")
	{
		mysql_query("INSERT INTO host SET name='{$_POST['company_name']}'");
	}

then it works fine.

But I really wanted to use the first code I posted on the 1st post, so that I can insert all the values the user entered in the database in 1 statement instead of repeating the mysql_query for all the fields in the form.

Thanks again

tsinka

$query = "INSERT INTO host (host_id,name) VALUES ('','{$_POST['company_name']}')";

$query = "INSERT INTO host (host_id,name) VALUES ('','".$_POST['company_name']."')";

Thomas

AJPayne

thank you soo much Thomas, it works perferctly 🙂

Thank you.