mySQL_queryテーブルを更新しない

Question

フォームからエントリを読み込み、対応するデータテーブルを更新するだけでなく、ログインしたユーザのユーザ名を添付することになっている次のビットのPHPがありますが、更新する。

$Date = $_POST["Date"]; 
$Description = $_POST["Description"]; 
$Hour = (float)$_POST["Lat"]; 
$Min = (float)$_POST["Lon"]; 
$Username = $_SESSION['username']; 
mysql_connect("localhost", "dbusername", "dbpass"); 
mysql_select_db("dbname",$connect) or die("Unable to select database"); 

$sql = ("INSERT INTO Table (Date, Description, Hour, Min, Username) 
VALUES ($Date, $Description, $Lat, $Lon, $Username)"); 

if (!mysql_query($sql)) 
    { 
    die('Error: ' . mysql_error()); 
    } 
echo "1 record added"; 

mysql_close($con); 

echo $Date; 
echo $Description; 
echo $Hour; 
echo $Min; 
echo $Username; 

あなたは、単一引用符でクエリの入力変数を同封していない任意の助け

Answer 1

のために事前にありがとうございます。あなたは、クエリの失敗にmysql_error()と呼ばれている

$sql = ("INSERT INTO Table (Date, Description, Hour, Min, Username) 
    VALUES ('$Date', '$Description', '$Lat', '$Lon', '$Username')"); 

は、おそらく、それは最初の日付の入力値の近くに構文エラーを印刷しました。

mysql_real_escape_string()の変数をmysql_real_escape_string()でエスケープしてから、SQLインジェクション攻撃から身を守る必要があります。

$Date = mysql_real_escape_string($_POST["Date"]); 
$Description = mysql_real_escape_string($_POST["Description"]); 
// You might want to check sane bounds for these float values 
// in addition to casting them as floats 
$Hour = (float)$_POST["Lat"]; 
$Min = (float)$_POST["Lon"]; 
// Hopefully $Username is already sane, but escape to be safe 
$Username = mysql_real_escape_string($_SESSION['username']); 

mysql_select_db()を呼び出して、2番目のパラメータはオプションであり、そして実際にあなたのコード内で$connectを定義していません。私はあなたが意味を疑う：

$connect = mysql_connect("localhost", "dbusername", "dbpass"); 
mysql_select_db("dbname",$connect) or die("Unable to select database"); 

Answer 2

あの、ここに偉大な提案がたくさんいますが、それを掲載されたコードと1つの明白な問題があります。あなたは、クエリを記述するときに、それぞれ$緯度および$経度としてそれらの値を参照している、しかし... $時間と、それぞれ$分という名前のPHP変数へのポストから

$Hour = (float)$_POST["Lat"]; 
$Min = (float)$_POST["Lon"]; 

を緯度と経度の値を代入していますあなたが実際にLAT $または$ロンに値を代入、まだあなたは、クエリのパラメータにその値を割り当てたいん投稿のコードでどこにも....

$sql = ("INSERT INTO Table (Date, Description, Hour, Min, Username) 
VALUES ($Date, $Description, $Lat, $Lon, $Username)"); 

。本質的には、存在しないクエリで変数を呼び出すことになります。