Add values of multiple selected checkboxes with comma

[vinpkl]

hi all

i want to insert all selected checkboxes values with a comma separator in
database in one single column.

i m able to echo all the checkboxes selected through the below code

<input  type="checkbox" value="Battery"  name="checkbox[]" id="checkbox[]" />

for ($i=0; $i<count($_REQUEST['checkbox']);$i++) {
echo "chosen<br />$i. ".$_REQUEST['checkbox'][$i];
}

ouput is

0. battery
1. rocket
2. pineapple

i want to insert all selected checkboxes values with a comma separator in
database in one single column.

so what should i write in my query

$query = "insert into checkbox table (checkboxes) values()"

vineet

[CoryThompson]

Something like this should work. (haven't tested)

for ($i=0; $i<count($_REQUEST['checkbox']);$i++) {
 $values .= $_REQUEST['checkbox'][$i].", ";
}
$query = "INSERT INTO table ('columnid', `columnvalue`) VALUES ('1', '".$values."');";

[vinpkl]

thank CoryThompson

that works perfect as needed.

just a last question can i replace comma with numbers 1,2,3

like

1. first checkbox
2. second checkbox

vineet

[Atli]

i want to insert all selected checkboxes values with a comma separator in
database in one single column.

This is not a good idea. The first rule of relational database design is to never put more than a single piece of data into a single field. When you do that, the database server considers all the values as a single string, and thus can not process them in any other way than as a single string. You can't search them, sort them, or even return them without needing front-end code or an overtly complex query to do so.

The consequences of using a non-normalized database may not be immediately apparent, but it will likely come back to bite you if you ever need to add or alter the functionality of your application.

Something like this should work. (haven't tested)

for ($i=0; $i<count($_REQUEST['checkbox']);$i++) {
 $values .= $_REQUEST['checkbox'][$i].", ";
}
$query = "INSERT INTO table ('columnid', `columnvalue`) VALUES ('1', '".$values."');";

You can use the implode function there instead of the for loop:
[code=php]$list_str = implode(",", $array);[/code]