MySQL и PHP multiple checklist db insert

Question

Я пытаюсь вставить значение этого множественного контрольного списка в столбец db. Этот код не работает. Может ли кто-нибудь обнаружить проблему?

Моя база данных состоит из таблицы под названием «цвета» и одного столбца с названием «цвет».

<?php 
// connect to database 
require "mysql_connect.php"; 
?> 


<?php 
// get value from the form 
$color = $_POST['color']; 

foreach($_POST['color'] as $colors){ 
$insert = mysql_query("INSERT INTO colors (color) VALUES ('$color')"); 
} 
?> 

<form action="add_color.php" method="post" enctype="multipart/form-data" name="colorform"  id="colorform"> 

<input type="checkbox" name="color[]" value="black" /> Black 
<input type="checkbox" name="color[]" value="red" /> Red 
<input type="checkbox" name="color[]" value="blue" /> Blue 
<input type="checkbox" name="color[]" value="white" /> White 

<input name="submit" type="submit" value="Add color" /> 

</form> 

Благодаря

Answer 1

Это хороший способ, чтобы добавить ваших цвета

<?php 

     require "mysql_connect.php"; 
    // connect to database 
    $colors=array(); 
    // get value from the form 
    if (isset($_POST['color'])) $colors = $_POST['color']; 

    foreach($colors as $color) 
    { 
     mysql_query ("INSERT INTO colors ('color') VALUES ('$color')"); 
    } 
?> 

<form action="add_color.php" method="post" enctype="multipart/form-data" name="colorform"  id="colorform"> 

<input type="checkbox" name="color[]" value="black" /> Black 
<input type="checkbox" name="color[]" value="red" /> Red 
<input type="checkbox" name="color[]" value="blue" /> Blue 
<input type="checkbox" name="color[]" value="white" /> White 

<td><input name="submit" type="submit" value="Add color" /> 
</form> 

если (IsSet ($ _POST [ «цвет»])) Это условие является важным, поскольку это предотвратит ошибку индексации в случае массив пуст

$ цветов = массив(); Кроме того, не объявлять переменные, чтобы предотвратить получение незадекларированных varibles, ранее, в вашем коде, это произойдет, если пользователь не определяет цвет

Запомнить PHP на стороне сервера и, таким образом получая ошибки на PHP создавать лазейки для атаки.Попробуйте прочитать о PHP Best Practices, его очень impotant

надеется, что это помогает :-)

Answer 2

Я хотел бы также предположить, что вы дезинфицировать ваш из входов, прежде чем вставить в вашу базу данных. Вы не указываете, какой тип столбца цвета, может быть и несоответствие.

Когда вы говорите INSERT INTO $colors - это то, что вы имеете в виду? Имя вашей таблицы является переменной? Вероятно, у вас должно быть правильное имя таблицы вместо $colors.

Кроме того, вы использовали $ цвет, который я не вижу определенную, вы, вероятно, имел в виду использовать $ цвета, так что должно быть больше, как это:

INSERT INTO tblColors (color) VALUES ('$colors') 

Чтобы проверить возвращаемое значение, чтобы увидеть, что ошибки вы получаете:

$query = "INSERT INTO tblColors (color) VALUES ('$colors')"; 
$insert = mysql_query($query) or die("A MySQL error has occurred.<br />Your Query: " . $query . "<br /> Error: (" . mysql_errno() . ") " . mysql_error()); 

Answer 3

$insert = mysql_query("INSERT INTO $colors (color) VALUES ($color)"); 

Изменить его:

$insert = mysql_query("INSERT INTO colors_table_name (color) VALUES ($color)"); 

Кроме того, пожалуйста, проверьте возвращаемое значение вставки, возможно, вы получаете ошибки? Первой очевидной проблемой было то, что имя таблицы заменялось цветом из-за переменной, является ли это желаемым эффектом?

Answer 4

<?php 
// connect to database 
require "mysql_connect.php"; 
?> 


<?php 
// get value from the form 
$colors = $_POST['color']; 

foreach($colors as $color){ 
    $insert = mysql_query("INSERT INTO colors (color) VALUES ($color)"); 
} 


<form action="add_color.php" method="post" enctype="multipart/form-data" name="colorform"  id="colorform"> 

<input type="checkbox" name="color[]" value="black" /> Black 
<input type="checkbox" name="color[]" value="red" /> Red 
<input type="checkbox" name="color[]" value="blue" /> Blue 
<input type="checkbox" name="color[]" value="white" /> White 

<td><input name="submit" type="submit" value="Add color" /> 

</form> 

Answer 5

У вас есть ваши переменные задом, ошибки синтаксиса SQL, SQL инъекций уязвимости, а также полное отсутствие обработки ошибок

$color = $_POST['color']; <---stuff the POST data array into $color 

foreach($_POST['color'] as $colors){ <--- loop over the POST data directly 

$insert = mysql_query("INSERT INTO colors (color) VALUES ($color)"); 
                  ^^^^^^---insert the array 
                  ^^^^^^---no quotes 

Вы можете использовать $ цвета (с S), чтобы сохранить лицо цвета, но затем вставьте $ color, который является массивом.

Никогда не предполагайте, что запрос завершился. Если бы голая минимальная обработка or die(...) ошибки, вы видели, почему ваши запросы потерпели неудачу:

foreach($_POST['color'] as $color) { 
    $safe_color = mysql_real_escape_string($color); 
    $result = mysql_query("INSERT INTO colors (color) VALUES ('$safe_color');") or die(mysql_error()); 
}