PHP Array building with foreach loop

Question

I'm trying to build an array that I can output using JSON. For whatever reason, it will only add one element to the array, even if there are multiple email addresses that come back with the object. I know this because the loop executes the correct amount of times. However the new values are not added to the array. I tried array_push but it mal forms the array and jQuery wont process it correctly.

        $load_model = $this->loadModel("LoadModel");
        $x = $load_model->updateLoad($LoadRate, $carrID, $comment, $temp, $loadStatus, $contactID, $load_id);
            if($x==true){
                $arr=array();
                foreach($load_model->loadEmailAddresses($load_id) as $val){
                   $arr['Email']= $val->Email;
                   $arr['ContactId']= $val->ContactId;
                }
                echo json_encode($arr);
            }

any ideas anyone?

slapyo · Accepted Answer · 2014-10-10 18:50:36Z

4

You are setting the same element over and over in the loop. Every time it loops it will set $arr['Email'] and $arr['ContactId']. In the end you'll only ever get the last value that went through the loop. You'll need to create an array for the email and contact id, then add that array to $arr.

$load_model = $this->loadModel("LoadModel");
$x = $load_model->updateLoad($LoadRate, $carrID, $comment, $temp, $loadStatus, $contactID, $load_id);
    if($x==true){
        $arr=array();
        foreach($load_model->loadEmailAddresses($load_id) as $val){
           $arr[] = array('Email' => $val->Email, 'ContactId' => $val->ContactId);
        }
        echo json_encode($arr);
    }

answered Oct 10, 2014 at 18:50

slapyo

2,9911 gold badge18 silver badges26 bronze badges

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

2 Comments

jon.r Over a year ago

Ah dhur, dunno how I missed that. Now I have a multidimensional array passed to the Jquery. Do I just iterate over the returned data?

slapyo Over a year ago

Yup, you'll have to handle that data in jQuery knowing it could be any number of records.

Jerome · Accepted Answer · 2014-10-10 18:52:00Z

1

You should change the lines in your foreach loop to

$arr[] = array(
    'Email'=>$val->Email,
    'ContactId'=>$val->ContactId,
);

Your array should grow properly at this point.

answered Oct 10, 2014 at 18:52

Jerome

2911 silver badge9 bronze badges

Collectives™ on Stack Overflow

PHP Array building with foreach loop

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related