Home/ Questions/Q 531011
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-13T09:15:23+00:00

I’m reading programming best practices and it is said that when creating a function

  • 0

I’m reading programming best practices and it is said that when creating a function we should make it do a only single specific task.

I got model functions that retrieves data and it’s related data. Example:

$this->Student->StudentAssignments();

Currently this function retrieves the student’s assignments plus the question for each assignment and data about the student. I use them all. My dilemma is if I try to make separate functions that retrieves the related data (student and question datas) it’s taxing since I’m producing more calls to the DB.

What would you guys suggest?

Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    Something to keep in mind when doing this sort of refactoring…

    I typically will have a Model->getSomethingAndSomethingElse functions in my models.
    These functions are public and meant to be called as a substitute for doing complicated (or any) find calls from the Controller.

    What I will usually do is then build up a small collection of private functions in the model.
    In your case I might have something along the lines of…

    Student->getStudentAssigmentsWithQuestions

    that then calls some private functions i.e.

    Student->getStudent which might call Student->joinStudentAssignment which in turn might call Assignment->joinAssignmentQuestion etc.

    The double underscore prefixes have been removed since markdown wants to bold things because of them. If you are using php5 the underscores aren’t really important anyways as long as you use the “private” or “proteced” keywords.

    Basically I use the public method as a container for a group of very specific query building or association building private functions within the models. This allows me to have an api that has complex data returned, but I build the query or the result set (depending on the type of data, relationships involved or query complexity) from small pieces – that can ideally be purposed and used in more than one public function call.

    • 0