current community

your communities

more stack exchange communities

Stack Exchange
tour help
stack overflow careers
Take the 2-minute tour ×
Stack Overflow is a question and answer site for professional and enthusiast programmers. It's 100% free, no registration required.
up vote 20 down vote favorite
4

I was wondering if it is possible within a foreach loop to edit the current object thats being handled.

I'm working with an array of objects $questions and I want to go through and look for the answers associated with that question object in my db. So foreach question go fetch the answer objects and update the current $question inside my foreach loop so I can output/process elsewhere.

    foreach($questions as $question){
        $question['answers'] = $answers_model->get_answers_by_question_id($question['question_id']);
    }
share|improve this question
    
how about references? –  Artjom Kurapov Apr 12 '12 at 10:08
    
As both ArtjomKurapov and @topener suggested I was looking for 'pass by reference' using the & sign. Thanks chaps :) have a nice day –  Garbit Apr 12 '12 at 10:12
    
@Garbit awesome. Could you please mark my answer as best if it helped you? –  Rene Pot Apr 12 '12 at 12:05

1 Answer 1

up vote 38 down vote accepted

There are 2 ways of doing this

   foreach($questions as $key => $question){
        $questions[$key]['answers'] = $answers_model->get_answers_by_question_id($question['question_id']);
    }

This way you save the key, so you can update it again in the main $questions variable

or

foreach($questions as &$question){

Adding the & will keep the $questions updated. But I would say the first one is recommended even though this is shorter (see comment by Paystey)

share|improve this answer
    
Nice! so the & sign will pass by reference? is that the correct terminology? –  Garbit Apr 12 '12 at 10:09
    
you are missing part with passing entire $question variable into get_answers_by_question_id() –  Artjom Kurapov Apr 12 '12 at 10:10
    
Correct. I just added a link to the docs about that –  Rene Pot Apr 12 '12 at 10:11
    
@ArtjomKurapov what do you mean? What part –  Rene Pot Apr 12 '12 at 10:11
10  
References in foreach is really not recommended, the way the foreach passes around the value part of the loop results in unpredicatble behaviour. It may be longer but you're far safer using method 1 here. –  Paystey Apr 12 '12 at 10:11

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.