Coding convention eager loading relationships

By @mratiebatie · 2021-09-07 12:03

Coding convention eager loading relationships

Back to thread

  • By @captbrogers · 2021-09-07 15:44

    My advice is that fat belongs somewhere and you can't always distribute it in an elegant way, especially when modifying an existing system. It may not be a bad thing to have some extra fat in the repository class or model class. Please understand that I'm not asking this to question your methods, but merely to understand the problem better: can you move some of the eager loading to the model and always include it? Relationships like employee tags, department, or isPresent all seem like simple enough data that it may be negligible on performance to list them in the 'with' array on the model if those relationships are on most of your other eager loading queries.

    • By @mratiebatie · 2021-09-13 12:01

      Thanks for your answer. I've considered this as well but since it's quite some data it would slow down page load. Also, some eager loading queries have a extra scope on a date field which I excluded for brevity.

      A couple of days ago I came up with the idea to start using ViewModels more. I think this is the best place to eager load extra data for a specific page. Thanks though!