Коллеги, что вы думаете о данном коде?

[ZVV]

Цитата:

Сообщение от MikeR

Коллеги, вы не поверите, но суть в том, что цикл вообще не нужен.

WHILE(TRUE)

это стоит для того, что бы просто "запустить" выполнение.
Я подумал, что это очевидно.....

Так ли?
Он нужен для того что бы запускать и следующие итерации тоже, а не только первую.
Цикл будет выполнятся, кока не закончится дерево (не будет парента) и сработает брэйк.
Ну собственно такая эмуляция рекурсии...

[MikeR]

Собственно да, человек запускает перебор (чтобы более корректно было), но в других местах это вообще не требуется.
Используется конструкция

while select forupdate ... {}

что само по себе плохо, так как более корректно

while select
{
	ttsbegin;
	select forupdate 
	ttscommit;
}

То есть делаем маленькие транзакции.

[MikeR]

Кстати, а действительно кто-нибудь пишет рекурсию через

while (true)

в Dynamics Ax?

[AndyD]

Цитата:

Сообщение от MikeR

Собственно да, человек запускает перебор (чтобы более корректно было), но в других местах это вообще не требуется.
Используется конструкция

while select forupdate ... {}

что само по себе плохо, так как более корректно

while select
{
	ttsbegin;
	select forupdate 
	ttscommit;
}

То есть делаем маленькие транзакции.

Что значит более корректно?
Вы хотите поломать целостность данных?

[MikeR]

Цитата:

Сообщение от AndyD

Что значит более корректно?
Вы хотите поломать целостность данных?

table		t1,t2;
ttsbegin;
while select t1
{
	
	select forupdate t2 where t2.recid == t1.RecId;
	.....
	t2.update(); 
	
}
ttscommit;

[AndyD]

Цитата:

Сообщение от MikeR

table		t1,t2;
ttsbegin;
while select t1
{
	
	select forupdate t2 where t2.recid == t1.RecId;
	.....
	t2.update(); 
	
}
ttscommit;

И тем не менее.
В чем в этом варианте большая корректность перед первоначальным?

[MikeR]

Цитата:

Сообщение от AndyD

И тем не менее.
В чем в этом варианте большая корректность перед первоначальным?

Модификация огромного количества (сотни тысяч) записей в Axapta 3.0 SP4
aEremenko: Ресурс заблокирован, ждите...

[AndyD]

Цитата:

Сообщение от MikeR

Модификация огромного количества (сотни тысяч) записей в Axapta 3.0 SP4

Вы не находите, что приведенная ссылка не имеет никакого отношения к исходному утверждению (потому как в нем не было никакой речи о закрытии/открытии транзакции внутри цикла)?

В вашем же примере получается только увеличение времени выполнения выполнения за счет лишних запросов к базе.

[MikeR]

Цитата:

Сообщение от AndyD

И тем не менее.
В чем в этом варианте большая корректность перед первоначальным?

Таки вот что гласит документация

While select Statement with Large Transaction Scope 



ttsBegin; 
 
while select ItemId 
    from salesLine 
{ 
    select firstOnly forUpdate ItemType, ItemBuyerGroupId  
       from inventTable 
           where inventTable.ItemId == salesLine.ItemId; 
 
    If (inventTable && (inventTable.ItemType == InventItemType::Item)) 
    { 
         inventTable.ItemBuyerGroupId = ; 
         inventTable.update(); 
    } 
} 
 
ttsCommit; 


To limit the transaction scope, the code should be written to begin the transaction 
when the following criteria is met: 

While select Statement with Appropriate Transaction Scope 


while select ItemId 
    from salesLine 
{ 
    select firstOnly forUpdate ItemType, ItemBuyerGroupId  
       from inventTable 
           where inventTable.ItemId == salesLine.ItemId; 
 
    If (inventTable && (inventTable.ItemType == InventItemType::Item)) 
    { 
        ttsBegin; 
        inventTable.ItemBuyerGroupId = ; 
        inventTable.update(); 
        ttsCommit; 
    } 
}

Для AndyD DO NOT: Hold database locks for longer than is necessary.
DO: Limit the transaction scope by grabbing locks at the latest possible time and
releasing locks as early as possible.

[ZVV]

Цитата:

Сообщение от MikeR

Собственно да, человек запускает перебор (чтобы более корректно было), но в других местах это вообще не требуется.
Используется конструкция

while select forupdate ... {}

что само по себе плохо, так как более корректно

while select
{
	ttsbegin;
	select forupdate 
	ttscommit;
}

То есть делаем маленькие транзакции.

Сомнительное утверждение... (по крайней мере без контекста )