v8-code-style/bundles/com.e1c.v8codestyle.bsl/markdown/query-in-loop.md

Query in loop

It is recommended that you merge queries that address related data into a single query.

Noncompliant Code Example

// BanksToProcessing - contains an array of banks

InidividualQuery = New Query;
InidividualQuery.Text =
"SELECT
|	BankAccounts.Ref AS Account
|FROM
|	Catalog.BankAccounts AS BankAccounts
|WHERE
|	BankAccounts.Bank = &Bank");

For Each Bank From BanksToProcess Do
	InidividualQuery .SetParameter("Bank", Bank);
	AccountsSelection = InidividualQuery .Execute().Select();
	While AccountsSelection.Next() Do
		ProcessBankAccounts(AccountsSelection.Account);
	EndDo;
EndDo;

Compliant Solution

// BanksToProcess - contains an array of banks

MergedQuery = New Query;
MergedQuery.Text =
"SELECT
|	BankAccounts.Ref AS Account
|FROM
|	Catalog.BankAccounts AS BankAccounts
|WHERE
|	BankAccounts.Bank B(&BanksToProcess)";

MergedQuery.SetParameter("BanksToProcess", BanksToProcess);

AccountsSelection = MergedQuery.Execute().Select();
While AccountsSelection.Next() Do
	ProcessBankAccounts(AccountsSelection.Account);
EndDo;

See

https://support.1ci.com/hc/en-us/articles/360011001620-Multiple-execution-of-the-similar-queries

https://support.1ci.com/hc/en-us/articles/360011001540-Rounding-arithmetic-results-in-queries