Laravel loop through associative array then save to database

Question

I'm a beginner and currently learning Vue js and Laravel. So I'm trying to figure out how to create a single POST request via axios then insert multiple entries into my database.

I have this array for example:

{
  "comakers": [
    {
      "name": "Sample",
      "email": "[email protected]"
    },
    {
      "name": "Test",
      "email": "[email protected]"
    }
  ]
}

then on my laravel controller I want to do a foreach loop like this

public function update(Request $request, $id)
{
    $data = $request->all();

    foreach ($data as $comaker){
        $nominate = new Nominate();
        $nominate->loan_application_id = $id;
        $nominate->comaker_name = $comaker->name;
        $nominate->comaker_email = $comaker->email;

        $nominate->save();
    }

}

But I get an error "Trying to get property 'name' of non-object". Can someone please give me some idea on how to achieve this?

And btw I'm trying to use the update function from my controller. Basically, I want to (insert new if not existing) or (edit when theres an existing data). Am I doing the right approach here?

(UPDATE)

laravel dd or dump shows this when I try to get the value of $data = $request->all();

array:1 [
  "comakers" => array:2 [
    0 => array:2 [
      "name" => "Sample"
      "email" => "[email protected]"
    ]
    1 => array:2 [
      "name" => "Test"
      "email" => "[email protected]"
    ]
  ]
]

Error messages like that normally have a line number to give you a clue where to find the error. Can you show ALL the error message when you post here please — RiggsFolly
– RiggsFolly, Commented Dec 12, 2018 at 0:43
Hi Noah, can you update your question with the axios post, I think you're missing to find the array in the request — Walter Cejas
– Walter Cejas, Commented Dec 12, 2018 at 0:43

Cyclonecode · Accepted Answer · 2021-02-24 19:32:02Z

2

I don't think all() will return an array of objects but instead an associative array. So you could try the following:

// Notice that you should use `json()` since the data is in json format
$data = $request->json()->all();

foreach ($data['comakers'] as $comaker) {
  $nominate = new Nominate();
  $nominate->loan_application_id = $id;
  $nominate->comaker_name = $comaker['name'];
  $nominate->comaker_email = $comaker['email'];

  $nominate->save();
}

of course you could also cast the result to an object like this:

foreach ($data['comakers'] as $item) {
  $comaker = (object) $item;
  $nominate = new Nominate($data);
  ...

I would also suggest that you simply print out the contents of the result in order to see what it actually contains, using var_dump() for instance:

var_dump($data);

edited Feb 24, 2021 at 19:32

answered Dec 12, 2018 at 0:54

Cyclonecode

30.4k11 gold badges77 silver badges97 bronze badges

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

2 Comments

Noah Over a year ago

Hey Cyclonecode, I added the line $comaker = (object) $comaker; after the foreach and it actually solved the problem. Can you add it back so I can accept your answer? Thanks a lot for the help!

Cyclonecode Over a year ago

@Noah - Yes I added it again, but the only reason that is working is because the data is not an array of object as I stated at the top of my answer. The data is an associative array, so the first solution should work as well.

Sasa Blagojevic · Accepted Answer · 2018-12-13 10:12:19Z

1

You are forgetting the comaker key when you try to loop, you should do it like this

$data = $request->all();

foreach ($data['comakers'] as $comaker){
    $nominate = new Nominate();
    $nominate->loan_application_id = $id;
    $nominate->comaker_name = $comaker['name'];
    $nominate->comaker_email = $comaker['email'];

    $nominate->save();
}

edited Dec 13, 2018 at 10:12

answered Dec 12, 2018 at 0:56

Sasa Blagojevic

2,23019 silver badges26 bronze badges

2 Comments

Cyclonecode Over a year ago

I don't think this will work, since there are no objects but associative arrays in the result.

Sasa Blagojevic Over a year ago

You are right, Laravel decodes json as associative array, I will update my answer

Waren · Accepted Answer · 2018-12-12 00:54:12Z

0

I think your doing a foreach on the Json variable not the comakers object inside it.

foreach ($data->comakers as $comaker) {
        $nominate = new Nominate();
        $nominate->loan_application_id = $id;
        $nominate->comaker_name = $comaker->name;
        $nominate->comaker_email = $comaker->email;

        $nominate->save();
}

answered Dec 12, 2018 at 0:54

Waren

3352 silver badges9 bronze badges

2 Comments

Noah Over a year ago

Unfortunately, doing this shows another error "Trying to get property 'comakers' of non-object" instead

Waren Over a year ago

Right, I saw your vardump, you cant get the property of a non-object. Since its an Array, you should try $data['comakers'] for the foreach and $comaker['name'] inside the loop

Collectives™ on Stack Overflow

Laravel loop through associative array then save to database

3 Answers 3

2 Comments

2 Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

2 Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related