[Resolved] Need checkbox array to go into DB

[kilbey1]

For some reason, I can't get checkbox array to go into my database properly. Ideas?

			
			foreach($_POST['advertising'] as $value) {
				$advertising = $value . ", ";
			}
			foreach($_POST['jobboard'] as $value) {
				$jobboard = $value . ", ";
			}		

Echoing out these variables doesn't work either. I need my data to appear like so for each (where value 1 and 3 are checked):

value 1, value 3

How can I go about this?

[kilbey1]

Btw, checkbox array is set up correctly, ie...

    <input type="checkbox" name="jobboard[]" value="good matches">
    Good matches 
    <input type="checkbox" name="jobboard[]" value="confidentiality">
    Confidentiality 
    <input type="checkbox" name="jobboard[]" value="readable data"> 
    Easily readable data 
    <input type="checkbox" name="jobboard[]" value="other">
    other: 
    <input name="jobboard[]" type="text">

[bodzan]

..that if you use this code you get that variables will hold the value of the last checkbox clicked. Instead it would be wise to use this one:

$advertising .= $value . ", ";

This line (the difference is in .=) wuill ad the value of a chackbox to the variable and not replacing the old one...

[TheDefender]

Too much work. No need to cycle through the array, because you would then have to trim it and remove the end comma like below...

To accomplish it the way you are trying:

// Create empty variable to concatenate the values to when you iterate through the loop.
$jobboard = "";
foreach($_POST['jobboard'] as $value) { 
  $jobboard .= $value . ", "; 
}
$jobboard = substr($jobboard, 0, strlen(trim($jobboard))-1);
echo $jobboard;

The MUCH easier way is to use [man]implode[/man]...

$jobboard = implode(", ", $_POST['jobboard']);
echo $jobboard;

One step, no mess...

[kilbey1]

Super! But why, then, is nothing entering my DB?

See code...

//AUTHORIZE
		$jobboard = preg_match('/[a-z\'\-]/', $_POST['jobboard']);		
			if(!empty($_POST['jobboard']) && !jobboard) {
				$error[] = "Job board data not valid";
		//} 
		//elseif(!empty($_POST['jobboard'])) {
		//
		} elseif(!empty($_POST['jobboard']) && $jobboard[4] != "")  {
				$jobboard_implode = implode(", ", $jobboard);
		}

	$advertise = preg_match('/[a-z\'\-]/', $_POST['advertise']);		
	if(!empty($_POST['advertising']) && !advertise) {
			$error[] = "Comments concerning advertising not valid";
	} elseif (!empty($_POST['advertising'])) {
		$advertise_implode = implode(", ", $advertising);
	}	

//IF AUTHORIZED, INSERT INTO DB...
			$query = "INSERT INTO questionnaire (jobtype, search, searchmethod, experience, experiencewhy, jobboard, advertise, addlfeatures, brainbond, jobs24by7, mycooljob, wowwe, contact, name, address, city, state, country, zip, date_entered) values ('$jobtype', '$search', '$searchmethod', '$experience', '$experiencewhy', '$jobboard_implode', '$advertising_implode', '$addlfeatures', '$brainbond', '$jobs24by7', '$mycooljob', '$wowwe', '$contact', '$name', '$address', '$city', '$poststate', '$countries', '$zip', now())";

[TheDefender]

This:

if(!empty($_POST['jobboard']) && !jobboard) {

probably needs to look like this:

if(!empty($_POST['jobboard']) && !$jobboard) {

[kilbey1]

Well, this appears to work. Any better ideas, let me know!

		if(!empty($_POST['jobboard'])) {
			$jobboard_match = preg_match('/[a-z\'\-]/', $_POST['jobboard']);
			if(!$jobboard_match) {
				$error[] = "Job board data not valid";
			} else {
				$jobboard_implode = implode(", ", $jobboard);
			}
		}


	if(!empty($_POST['advertising'])) {
		$advertise_match = preg_match('/[a-z\'\-]/', $_POST['advertising']);
		if(!$advertise_match) {
			$error[] = "Comments concerning advertising not valid";
		}  else {
			$advertise_implode = implode(", ", $advertising);
		}
	}