Home/ Questions/Q 92789
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-10T23:14:33+00:00

I have a function that looks something like this: //iteration over scales foreach ($surveyScales

  • 0

I have a function that looks something like this:

//iteration over scales foreach ($surveyScales as $scale) {     $surveyItems = $scale->findDependentRowset('SurveyItems');      //nested iteration over items in scale     foreach ($surveyItems as $item)     {         //retrieve a single value from a result table and do some stuff         //depending on certain params from $item / $scale     } }

QUESTION: is it better to do a db query for every single value within the inner foreach or is it better to fetch all result values into an array and get the value from there?

Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. One query that returns a dozen pieces of data is almost 12x faster than 12 queries that return 1 piece of data.

    Oh, and NEVER EVER NEVER put a SQL inside a loop, it will always lead in a disaster.

    Depending on how your app works, a new connection might be opened for each query, this is especially bad as every DB server has a limit on the number of connections. Then also realize this will happen for each user, so 50 queries with 5 users and you already have 250 queries at any given moment. But even if all the queries do share just 1 connection, you’re taxing the DB server X times more, slowing it down for everything else, every page, because users are hogging the DB server on this page, and everybody has to share.

    I’ve seen an entire application fail in the past because of this 1 design flaw, just don’t do it.

    • 0