Tek-Tips is the largest IT community on the Internet today!

Members share and learn making Tek-Tips Forums the best source of peer-reviewed technical information on the Internet!

Register Log in
  • Congratulations strongm on being selected by the Tek-Tips community for having the most helpful posts in the forums last week. Way to Go!

Cannot enter record into DB 1

Status
Not open for further replies.
raydona

raydona

Programmer
May 12, 2005
27
GB
Hi, I’m new to PHP so please bear with me. I’m trying to send user input from a HTML form to a MySQL database. The database has one table with four fields (varchar(15), varchar(15), varchar(10), longtext). The HTML form looks like:

Code: 
<html> <body>
<form name="index" action="Z.php" method="get">
Type your first name: 
<input type="text" name="1stname"><br><br>
Type your last name: 
<input type="text" name="2ndname"><br><br>

Select type of car:
<select name="kindofcar">
<option value="volvo">Volvo</option>
<option value="saab">Saab</option>
<option value="fiat">Fiat</option>
<option value="audi">Audi</option>
</select><br><br>
Write your comments here:
<TEXTAREA NAME="comments" COLS=40 ROWS=6></TEXTAREA><br><br>
<input type="submit" value="Submit">
</form> 
</body> </html>

Z.php forms looks like:

<html> <body>
<?php
  $con = mysql_connect("mydatabase","username","password");
  if(!$con)
  { die('Could not connect: ' . mysql_error());
  }
  mysql_select_db("nameofdb", $con);
  $sql="INSERT INTO NameofTable(FirstName, LastName,   TypeOfCar,
 Comments)
  VALUES('$_POST[1stname]','$_POST[2ndname]','$_POST[kindofcar]',
'$_POST[comments]')";
  if(!mysql_query($sql,$con))
  { die('Error: ' . mysql_error());
  }
  echo "1 record added";
  mysql_close($con);
  ?>
</body> </html>

Every time I run the above I get the following error:
Parse error: syntax error, unexpected T_STRING, expecting ']' in /…../…../…/…/…/…../html/Z.php on line 12
Nothing is added to the database. How can I rectify this? I would be very grateful for all help.
 
Sort by date Sort by votes
Hi Feherke,
Thanks very much for your reply. I followed your suggestion concerning using mysql_real_escape_string(). I got the following code off the internet and made use of it:
<html> <body>
<?php
function check_input($value)
{// Stripslashes
if(get_magic_quotes_gpc())
{ $value = stripslashes($value);
}
// Quote if not a number
if (!is_numeric($value))
{ $value = "'" . mysql_real_escape_string($value) . "'";
}
return $value;
}
$con = mysql_connect("mydatabase","username","password");
if(!$con)
{ die('Could not connect: ' . mysql_error());
}
mysql_select_db("nameofdb", $con);

$1stname = check_input($_POST['1stname']);
$2ndame = check_input($_POST['2ndame']);
$kindofcar = check_input($_POST['kindofcar']);
$comments = check_input($_POST['comments']);

$sql="INSERT INTO NameofTable(FirstName, LastName, TypeOfCar, Comments)
VALUES($1stname, $2ndame, $kindofcar, $comments)";
if(!mysql_query($sql,$con))
{ die('Error: ' . mysql_error());
}
echo "1 record added";
mysql_close($con);
?>
</body> </html>

I do not know if I have used the above code correctly or there is a better way of doing things, but I am getting the following error:

Internal Server Error
The server encountered an internal error or misconfiguration and was unable to complete your request.
Please contact the server administrator and inform them of the time the error occurred, and anything you might have done that may have caused the error.

I wonder if you could tell me where I am going wrong or point me in the right direction. I would be very grateful.
 
Upvote 0 Downvote
Hi

That PHP code seems correct.

I am afraid, nobody can help you for now. To find out the reason of the failure we must see the error messages provided by PHP. Ask it to display them by adding this at the beginning of your PHP script :
Code: 
ini_set('display_errors','on');
error_reporting (E_ALL);

Feherke.
 
Upvote 0 Downvote
@OP
post your code within [ignore]
Code:
[/ignore] tags.
 
Upvote 0 Downvote
Either client-side element's name or server-side php variable name cannot begin with numeric like 1st, 2nd. Client-side: browser will tolerate it until problem surfaces. Server-side: no. Rename those variables.
 
Upvote 0 Downvote
... and then you have this:
>method="get"
change it to this.
[tt]method="[red]post[/red]"[/tt]
 
Upvote 0 Downvote
It's a syntax error, I had a play with it and if you swap the single ' for a double and vice-versa.
Code: 
  $sql='INSERT INTO NameofTable(FirstName, LastName,   TypeOfCar, Comments)
  VALUES("$_POST[1stname]","$_POST[2ndname]","$_POST[kindofcar]","$_POST[comments]")';
Try that but also take into account what other people have said about the code.
 
Upvote 0 Downvote
@ingresman
your code will not work as variables are not expanded within single quotes. the net effect will be to insert, eg, the actual text $_POST[lstname] rather than the value of that array element.
 
Upvote 0 Downvote
yep, should have tested it !, could try building up the string e.g.
$sql = "insert....values(" . $_POST[1stname] . "," . $_POST[2ndname] etc
With all the quites needed to satisfy SQL of course
 
Upvote 0 Downvote
apart from the lack of escaping, the OP's method is fine.
if it were an escaping issue then an error would be returned by MySql (I believe)
the get -vs- post issue that tsuji spotted looks like a killer though.
 
Upvote 0 Downvote
Thanks, feherke! But I must confess spotting $1st $2nd gave myself more cynical satisfaction.
 
Upvote 0 Downvote
Status
Not open for further replies.

Similar threads

SashaBuilder3
  • Locked
  • Question
import CSV file into MySql table in PhpMyAdmin
Replies
0
Views
228
SashaBuilder3
SashaBuilder3
PeDa
  • Locked
  • Question
unwanted spaces inserted into the middle of php-generated html email body 1
Replies
2
Views
115
PeDa
PeDa
Brianfree
  • Locked
  • Question
split file name into different values
Replies
1
Views
116
spamjim
spamjim
waubain
  • Locked
  • Question
Add an new JSON array into an existing JSON array 2
Replies
7
Views
456
waubain
waubain
waubain
  • Locked
  • Question
Insert a key:value pair into an array based on another key:value pair 2
Replies
2
Views
88
waubain
waubain

Part and Inventory Search

Sponsor

Back
Top