PHP : $_POST['choice$i'] doesnt work

WindowsBurner · 12-17-2004, 11:27 AM

Hey,

Ok I've got an html page that has about 100 variables to be passed to a php script which emails all of these variables to me. All of them are named choice1,choice2,choice3,etc..

Instead of writing like 100 lines or more of code, I thought I'd use a for loop, only I'm not sure how to do this... I have tried :

$post_array[$i] = $_POST['choice' . $i];
and
$post_array[$i] = $_POST['choice$i'];
and
$post_array[$i] = $_POST['choice'$i];

I'm really lost. Any ideas?

Thanks,

WB

Cedrik · 12-17-2004, 11:35 AM

I would name the field like this :
<input type="text" name="choice[0]" value="" />
<input type="checkbox" name="choice[1]" checked="true" />
<input type="radio" name="choice[2]" />
...

Then :

PHP Code:



if(isset($_POST["choice"])) { 
    $post_array = $_POST["choice"]; 
} 
// checkin' 
print_r($post_array);

WindowsBurner · 12-17-2004, 12:00 PM

Umm...sorry. I dont follow, I'm trying to ..but I dont quite understand how I could use that with like a for loop, if say I had 100 variables being passed. The answers probably right there...but I'm not that great at PHP, only been at it a few months.

Cedrik · 12-17-2004, 12:09 PM

If you name the fields like choice[0], choice[1]... no need to loop to assign the array $_POST["choice"] to the $post_array variable

It is equivalent to do :

PHP Code:



foreach($_POST["choice"] as $choice) { 
    $post_array[] = $choice; 
}

and :

PHP Code:



    $post_array = $_POST["choice"];

WindowsBurner · 12-17-2004, 12:42 PM

Holy crap, thats so cool! Thanks 1,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,00 0,000,000
Cedrik. I understand now, that really helps alot. Thanks!

cormander · 12-17-2004, 08:42 PM

Quote:

Originally posted by WindowsBurner

$post_array[$i] = $_POST['choice' . $i];
and
$post_array[$i] = $_POST['choice$i'];
and
$post_array[$i] = $_POST['choice'$i];

As you already know, those won't work, but this will:

Code:

$key = "choice" . $i;

$post_array[$i] = $_POST[$key];

Hope that helps.

-Corey

sigsegv · 12-17-2004, 09:14 PM

Why not change the variable names in the HTML to something useful and then do away with all that nonsense by just emailing yourself with something like this:

Code:

$recipient = "you@your.com";
$headers = "From: WebServer on " . $_SERVER['HTTP_HOST'] . " <you@your.com>\n";
$subject = "A really cool subject\n";

$redirectHeader = "Location: http://" . $_SERVER['HTTP_HOST'] . "/thankYou.php";

$body = "The form was filled out with the following:\n\n";

while(list($key, $val) = each($_POST)) {
        $body .= $key . ":     " . $val . "\n";
}

mail($recipient, $subject, $body, $headers);
header($redirectHeader);

You already have $post_array in $_POST. Reassigning all the values into another array is just a waste of CPU cycles and memory.

WindowsBurner · 12-18-2004, 08:06 AM

Thanks corcommander, I knew I was close...but I hadn't thought of that.

sigsegv, your right I could do it that way. I hadn't thought of that either, what I have is a form for a survey and
there is literally 100 or more options. And I wanted a way for it to send me the info using as few lines of code as possible.

So, thanks a lot guys. I hadn't expected my post to get all these wonderful posts, I figured I'd quite one or two quick ways around the problem. Thanks alot.