ریفکتر

@abbasmomeny1994
سلام
میتونی از کوئری های پویا استفاده کنی

$user = User::query();
$user->where('state', true);
$user->where('age', 12);

داخل متد showDetails دو تا حلقه تو در تو داره که داخل هردوش هم کوئری زده میشه
این کد، کد بهینه ای نیست
میتونی به جاش از whereIn استفاده کنی

نه که کلا حلقه رو برداری
کوئری رو بیار بیرون حلقه و بعد توی حلقه فقط اندیس ها رو به آرایه اضافه کن
هر چند که با متدهای Collection هم تا حد زیادی میشه اون ساختار داده رو ساخت و کلا foreach رو حذف کرد

توی متد های checkTransactionId و payWithWallet هم میتونی از متد transaction کوئری بیلدر استفاده کنی
تا اگر مشکلی پیش اومد بقیه کوئری ها هم rollback بشند