Appending Array in For Loop

Question

I have a a loop that loops through data, what I am trying to achieve is for each loop an array is appended to another array before it is returned.

Here is what I have so far-

$response = array();
$response["CorrelationId"] = $correlationId;

This code sits outside the loop as I only want to return the "CorrelationId" once... This all works fine... for now I suppose!

foreach ($products as $value){
            $pid = mysql_escape_string($value['ProductId']);
            $dname = mysql_real_escape_string($value['departmentName']);
            $cname = mysql_real_escape_string($value['categoryName']);
            $pname = mysql_real_escape_string($value['productName']);

            //Insert Product data into DB
            $insert_product = "SQL Goes Here";
            $insert_result = mysql_query($insert_product);
            if(mysql_insert_id() >  0){
                $response["Messages"] = array("$pid Has been added.");
            }

        }
        return $response;

All that this is achieving so far is to add the last loop to the array. If I send 2 piecies of data to the loop, it will only have the last one in the array...

Am I missing something very basic here?

All I want if I where to send two pieces of data is to have the CorrelationId and 2 ["Messages"] returned.

Thanks

Sudhir Bastakoti · Accepted Answer · 2012-03-09 09:29:42Z

1

Change:


$response["Messages"] = "$pid Has been added.";
//To
$response["Messages"][] = "$pid Has been added.";

answered Mar 9, 2012 at 9:29

Sudhir Bastakoti

100k15 gold badges163 silver badges169 bronze badges

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

Comments

Nicola Peluchetti · Accepted Answer · 2012-03-09 09:28:35Z

0

That's because you are overrriding $response["Messages"]. you should do (inside the loop)

 $response["Messages"][] ="$pid Has been added.";

and outside the loop

 $response["Messages"] = array();

answered Mar 9, 2012 at 9:28

Nicola Peluchetti

76.9k32 gold badges150 silver badges197 bronze badges

1 Comment

user1124378 Over a year ago

It's always something blatantly obvious! Thanks I have been banging my head against a wall over this!

Collectives™ on Stack Overflow

Appending Array in For Loop

2 Answers 2

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related