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

[ZVV]

Цитата:

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

Вы тоже так пишите?
Я хочу услышать мнения коллег, насколько так вообще корректно писать.
Или может у меня одного глаз замылился.

Рекурсивный метод тут лучше бы смотрелся.
Ну по крайней мере я бы использовал рекурсию...

OFFTOP: не знал, что PL\SQL Formatter можно к Х++ прикрутить...

[MikeR]

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

WHILE(TRUE)

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

[samolalex]

Цитата:

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

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

WHILE(TRUE)

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

Кстати, да. О чем и говорит firstonly. А теперь и мне стало интересно, зачем так было сделано - "запускать выполнение" таким образом?

[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;

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

[ZVV]

Цитата:

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

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

while select forupdate ... {}

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

while select
{
	ttsbegin;
	select forupdate 
	ttscommit;
}

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

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

[oip]

Цитата:

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

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

WHILE(TRUE)

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

Почему? По-моему, это такой обход дерева. И без "while true" работать не будет. При каждой итерации меняется InvGUID и цикл прервется только тогда, когда не найдется parent. И "while true" при таком написании кода очень даже нужен. Понятно, что код можно переписать иначе.

[MikeR]

Цитата:

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

Понятно, что код можно переписать иначе.

Да дело в том, что не для всех это понятно.

[samolalex]

ИМХО, рекурсию лучше организовывать через отдельно написанную функцию внутри метода.