Zmień swój kod na ten, aby był bezpieczny i funkcjonalny:
<?php
// Connect to the database
mysql_connect ("localhost","Username","Password")
or die ('Error: ' . mysql_error());
echo "connected to database!";
mysql_select_db ("Database");
// Insert data into table
$Email= mysql_real_escape_string($_POST['Input2']);
$Name= mysql_real_escape_string($_POST['Input3']);
$Company= mysql_real_escape_string($_POST['Input4']);
$Price= mysql_real_escape_string($_POST['Input5']);
$action = mysql_real_escape_string('insert php code for button here');
$query = "INSERT INTO CustomerInformation
(Email,Name,Company,Price,Tab Count,Action)
VALUES
('$Email', '$Name', '$Company', '$Price', '$action') ";
mysql_query($query) or die ('Error updating database');
echo "Database updated successfully!";
?>
Pamiętaj, że nie musisz wstawiać id
do stołu. Jeśli masz pole autoinkrementacji id
wtedy MySQL automatycznie utworzy dla Ciebie identyfikator.mysql_real_escape_string()
wymyka wartości dla Was. Zawsze otaczaj $var
w zapytaniu z '
pojedyncze cudzysłowy lub mysql_real_escape_string()
nie praca! I nigdy nie używaj go do nazw kolumn/tabeli lub baz danych, tylko do wartości.
Zobacz:te pytania, aby uzyskać więcej informacji:
Ogólnie wstrzyknięcie SQL:Jak działa wstrzyknięcie SQL z „Tabel Bobby'ego Komiks XKCD?
ochrona przed wstrzyknięciem SQL podczas używania dynamicznych nazw tabel:Jak zapobiec wstrzykiwaniu SQL za pomocą dynamicznych nazw tabel?