foreach loop is better or should use collection?

we have items array want to attach

this code is working fine for me.

i just want to know this approach is better or something else better solution?

foreach ($this->request->items as $item) {
    $sale->items()->attach([
        $item['item']['id'] => ['quantity' => $item['quantity']]
    ]);
}

i used with collection too but foreach method is neat and clean.

$collection = collect($this->request->items);

$items = $collection->mapWithKeys(function ($item) {
    return [$item['item']['id'] => ['quantity' => $item['quantity']]];
})->all();

$sale->items()->attach($items);

i'm asking just for best practices.

both are working fine.