I mean the main purpose of repository pattern is to abstract the database layer like this:
But I think it is already done by Eloquent. What do you think about it?
For further actions, you may consider blocking this person and/or reporting abuse
Sorry, it's true.
Camilo Micheletto -
Nathan Orris -
Murtaza Joo -
Once suspended, muizabdul29 will not be able to comment or publish posts until their suspension is removed.
Once unsuspended, muizabdul29 will be able to comment and publish posts again.
Once unpublished, all posts by muizabdul29 will become hidden and only accessible to themselves.
If muizabdul29 is not suspended, they can still re-publish their posts from their dashboard.
Once unpublished, this post will become invisible to the public and only accessible to Abdul Muiz.
They can still re-publish the post if they are not suspended.
Thanks for keeping DEV Community safe. Here is what you can do to flag muizabdul29:
Unflagging muizabdul29 will restore default visibility to their posts.
Top comments (8)
The repository pattern and Eloquent are not competitors, you can use Eloquent inside a repository, it's particularly useful when you need more functionality that makes the Eloquent Query weird, it can also make your code more legible with stuff like getAdmins or getActiveUsers instead of the direct query – also, having the query in one place instead of multiple will allow you to reduce errors
If you don't need to re-use the queries then a Repository is overkill anyway – Even though, it can still provide value depending on the complexity of the queries.
I've seen it used in Laravel apps before, in conjunction with the Service Pattern, this was a massive codebase though.
YAGNI and KISS always in mind!
Rails works the same way, where ActiveRecord already acts as the standard object mapping on top of the database. But I saw a talk where someone described eschewing ActiveRecord for the repository pattern and it was working just fine for them.
I don't remember all the details, but my thoughts are that this is probably totally possible, but if Laravel is anything like Rails, swimming against the current is just a little awkward. You won't be facing the same problems as most of the rest of the community.
The repository pattern seems like a wonderful approach, but I wouldn't break from the framework's norm without a lot of conviction for doing so.
Would love to see some answers from folks with more insight into these things.
@ben Yep. I was thinking the same to avoid it. I tried to implement the repository class and it looked like this:
Apart from the flexibility that
getBymethod gives, there does not seem to be any reason to follow this approach. This link also seems to be very helpful regarding this.
In my understanding, this is not how Repository Pattern works. If you create a Repository class in Laravel then you should never return Eloquent models from that class because it makes your whole application depended on Eloquent. You may ask how?
By returning Eloquent models, we end up using many Eloquent related functions in our controllers or views. For example, if you want paginated data then you end up using LengthAwarePaginator's functions to get information like total pages, total data, current page and others. There are other scenarios which you may be well aware of if you are into Laravel.
What if in future for some reason, you want to ditch Eloquent and use other ORMs? You expect your application to run smoothly after the switch but it won't because your controllers and views are dependent on the functions that were provided via Eloquent. Your application is tightly coupled with Eloquent. Instead of returning Eloquent, you should return domain object. Please read "Patterns of Enterprise Application Architecture" by Martin Fowler for more information on this.
Saying all this, in my experience, I don't see any benefits of using Repository class in Laravel. I have not seen anyone ditching Eloquent in favour of other ORMs. If you have valid reason to switch then repository pattern may help but you have to design it well at first.
If you are sticking with Eloquent then why add an extra layer? If you really want a place where you can create nice APIs for long queries that requires multiples query builder functions then you can define it in model itself by astracting the methods in a trait and using it model and use it in some Service class accordingly.
We have built a PHP package for Laravel and use it for our projects.
Please check it out!