Select id_value from db into array checkbox and Insert multiple values from array checkbox to one column with comma separated

Question

I have an input array checkbox with id values. (I'm not sure if the array is working)

<?php $resource=mysql_query("Select * from material_rec",$con); ?>
    <?php while($result=mysql_fetch_array($resource))
        { ?>

    <input type="checkbox"  name="checkbox1" id="checkbox1[]" value="<?php echo $result['id']?>" />

    <?php };?>

when i insert the value of my checked checkboxes. it only insert one value and doesn't insert my selected checkboxes.

How can i insert the array values from the selected checkboxes into one colum with comma separtor?

score 2 · Accepted Answer · answered Jan 23 '14 at 16:22

2

It should be

name="checkbox1[]"

Whereas you have

id="checkbox1[]"

Then when inserting the values in the database, you can do

$values = implode(",", $_POST["checkbox1"]);

answered Jan 23 '14 at 16:22

Ben Fortune

31,623
10
79
80

+1 for the answer, -1 for suggesting to just implode/insert... never forget about sql injection attacks. – Marc B Jan 23 '14 at 16:25
My bad, missed the fact that OP was still using `mysql_*`. Though there's usually some generic "Switch to PDO/MYSQLi" in the comments. – Ben Fortune Jan 23 '14 at 16:26
what do you mean by sql injection attacks sir Marc? – user3117337 Jan 23 '14 at 16:30
@user3117337 http://stackoverflow.com/questions/60174/how-can-i-prevent-sql-injection-in-php?rq=1 – Ben Fortune Jan 23 '14 at 16:31

digitai · Answer 2 · 2014-01-23T16:33:36.507

0

For inserting values of a checkbox can also loop through the array:

$check=filter_var($_POST['checkbox1'], FILTER_SANITIZE_STRING);

foreach($check as $value) 
  {
     $insert_table=$conn->prepare("insert into table values (?);");
     $insert_table->execute(array($value)); 
  }

edited Jan 23 '14 at 16:33

answered Jan 23 '14 at 16:26

digitai

1,870
2
20
37

You only need to prepare the statement once, even then you don't want to be referencing the variable directly in the query. – Ben Fortune Jan 23 '14 at 16:28
This is interesting and new to me, I already do this way, but as your comment suggests it should better prepare once. I'm eager to kmow more about this. Do you have a link where I can check this. And thanks, this comment will help me to improve my queries. – digitai Jan 23 '14 at 16:31
The accepted answer [here](http://stackoverflow.com/questions/9168562/pdo-cost-of-calling-prepare-in-a-loop) explains it pretty well. – Ben Fortune Jan 23 '14 at 16:35

Select id_value from db into array checkbox and Insert multiple values from array checkbox to one column with comma separated

2 Answers2