PHP foreach loop inside foreach loop

Question

I'm trying to do a foreach loop inside a foreach loop.

I have a form where the user type in some text and hit send. On the server site I loop through an array with email addresses and send out the text message.

Now I also want the user to be able to use variables in the textarea, like $name. So my loop should first loop through the emails, and then str_replace the userinput $name variable, with the names in my array.

The loop works fine with the email part ($tlf), but not the replace $name part.

Could some spot what I am doing wrong?

$message = stripslashes(strip_tags($_POST['message2']));
$tlf=array("name1","name2");
$test=array("mname1", "mname2");
$subject = "Hello world";
$from = "[email protected]";
$headers = "From: $from";

foreach($tlf as $name){
    $to = $name. "@gmail.com";
    foreach($test as $navn){
        $message = str_replace('$navn', $navn, $message);}
    mail($to,$subject,$message,$headers);
}

Thanks a lot.

EDIT: The output is an email sent. Say the user type in "hello $name". I want it to first loop through the $tlf array, in this case creating 2 emails. This goes in as $to in the first loop. This works.

Now the next loop should recognize the user input "hello $name" and loop through the $test array replacing the user $name variable.

The output would be 2 emails send.

Mail output: to: [email protected] message: hello mname1
Mail output: to: [email protected] message: hello mname2

Let me know if I need to explain better, its hard for me to explain sorry.

I forgot to mention that it does output the first name in the array "mname1", but it does it not loop through the array. It ouput $test[0] each time.. — Bolli
– Bolli, Commented Jul 19, 2012 at 22:19
please tell us all your issues. It's really hard to diagnose a ghost. what do you expect and what do you get instead? — sdjuan
– sdjuan, Commented Jul 19, 2012 at 22:21

mario · Accepted Answer · 2012-07-19 22:24:58Z

2

When you do the following:

str_replace('$navn', $navn, $message)

Then all literal occurences of $navn will be replaced (the first, the second, the third, ...). So running through that loop a second time can't possibly replace anything more.

You will need to define two placeholders, or make some distinction, or use preg_replace_callback if you want to declare in which order (or other logic) the possible replacement strings are applied.

If you had told us (but you haven't) you only wanted to replace the first occurence in each iteration, then a normal preg_replace(.., .., .., 1) would work.

answered Jul 19, 2012 at 22:24

mario

146k20 gold badges243 silver badges293 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Bolli Over a year ago

Thanks a lot! I get it now. Will update my post as soon as I got it working.

Michael Petrotta · Accepted Answer · 2013-10-13 06:36:50Z

1

Is this what you want?

$message = stripslashes(strip_tags($_POST['message2']));
$tlf=array(
            array("email" => "name1", "name" => "mname1"),
            array("email" => "name2", "name" => "mname2")
          );
$subject = "Hello world";
$from = "[email protected]";
$headers = "From: $from";

foreach($tlf as $contact){
    $to = $contact["email"] "@gmail.com";
    $replacedMessage = str_replace('$navn', $contact["name"], $message);
    mail($to,$subject,$replacedMessage,$headers);
}

edited Oct 13, 2013 at 6:36

Michael Petrotta

61.1k27 gold badges153 silver badges181 bronze badges

answered Jul 19, 2012 at 22:24

valentinas

4,3371 gold badge23 silver badges27 bronze badges

4 Comments

Bolli Over a year ago

Thanks for the reply. My problem is that the second foreach loop does not loop through the array. It simply outputs "mname1" each time. Does the second foreach loop "start over" each time, because its inside another loop?

Bolli Over a year ago

Yes that looks very much like it. I just tested your code, didnt know I could do arrays in arrays :) - but it still only outputs "mname2" each time. EDIT: Checking it out

valentinas Over a year ago

you were assigning it back to the same variable. I've updated it. try it now.

Bolli Over a year ago

Thanks a lot! It works now, thats exactly what I wanted. I dont have my DB yet, but the point is to create the arrays from mysql db and then through this loop. Thanks a lot you helped me alot! Cheers

Collectives™ on Stack Overflow

PHP foreach loop inside foreach loop

2 Answers 2

1 Comment

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related