Php help!

This is some php code for a Uni project! It dont work! Can anyone see any errors! Cheers!




<?php

$connect = mysql_connect("localhost","root","live@49");
$db = mysql_select_db("flashphptask_zxq_customers", $connect);

$sql = mysql_query("INSERT INTO comments (ID, Name, eMail, Comments)
VALUE ('$ID', '$Name','$eMail','$Comments')")or die
(mysql_error());

?>

[Kalessin]

What error are you getting?

There is no error - it just wont write anything to the database I have set up! been annoying me all night long!

[christopher]

Assuming you have given a value to the variables you're using within the query then it will work... you have given the variables a value haven't you? I.e.

$ID = 'This goes in the ID column';
$Name = 'And this in the Name column';

I think something slightly worse is awry if you are not getting any error messages at all, but you could try these suggestions.

Why are you inserting an ID? is that the id for your item that will be incremented each time you add a new item? if so you should have it set as an auto_increment column in mysql, Then you wont need to insert it manually each time.

What is the column type for name, email and comments? If they are set to text you need to be adding them in the SQL query as such:

("INSERT INTO comments (ID, Name, eMail, Comments)
VALUE ('$ID', \''$Name'\',\''$eMail'\',\''$Comments'\')")

Hope that helps.

[Kalessin]

Originally Posted by Wester

I think something slightly worse is awry if you are not getting any error messages at all

Not necessarily. If using an integrated LAMP stack such as XAMPP or MAMP, PHP errors may be written to an error log rather than displayed on screen.

Pudsey, check your error logs!

[SoulKiss]

Originally Posted by Kalessin

Not necessarily. If using an integrated LAMP stack such as XAMPP or MAMP, PHP errors may be written to an error log rather than displayed on screen.

Pudsey, check your error logs!

Check your mySQL logs too, could be a permissions thing on the database.

Can you log into mySQL using those credentials and do an insert?

[Kalessin]

Also, do you have phpMyAdmin installed? If so, is it working?

Originally Posted by pudsey

This is some php code for a Uni project! It dont work! Can anyone see any errors! Cheers!




<?php

$connect = mysql_connect("localhost","root","live@49");
$db = mysql_select_db("flashphptask_zxq_customers", $connect);

$sql = mysql_query("INSERT INTO comments (ID, Name, eMail, Comments)
VALUE ('$ID', '$Name','$eMail','$Comments')")or die
(mysql_error());

?>

what is this...i dont even...

[TSM]

$con = mysql_connect("localhost","root","live@49");
if(!$con){
  die("Error conn - ".mysql_error());
}else{
  $db = mysql_select_db("flashphptask_zxq_customers", $con);
  if(!$db){
    die("Error changedb - ".mysql_error());
  }else{
    $sql = mysql_query("INSERT INTO comments (ID, Name, eMail, Comments) VALUE ('$ID', '$Name','$eMail','$Comments')",$db)or die
(mysql_error());
  }
}

Saftey points...
make sure you use mysql_real_escape_string() on your insert values, dont forget the ; at the end as an additional saftey, mabey wrap your query up with a sprintf such as...

$con = mysql_connect("localhost","root","live@49");
if(!$con){
  die("Error conn - ".mysql_error());
}else{
  $db = mysql_select_db("flashphptask_zxq_customers", $con);
  if(!$db){
    die("Error changedb - ".mysql_error());
  }else{
    $sqlstring=sprintf("INSERT INTO comments (ID, Name, eMail, Comments) VALUE ('%u', '%s',%s','%s');",
    $ID,
    mysql_real_escape_string($Name),
    mysql_real_escape_string($eMail),
    mysql_real_escape_string($Comments));
    $sql = mysql_query($sqlstring,$db)or die ("Error query - ".mysql_error());
  }
}

Its not OOP but should work fine if you just want a simple connection and query.
You could just use PDO or get a DB Class to do most of it for you in OOP standard.