topic Re: Effective Loop in Application Development and Automation Discussions

Effective Loop

Former Member — Thu, 30 Oct 2008 20:19:45 GMT

Hi Guys,

can you please let me know which is less time consuming LOOP selection?

LOOP at BKPF where belnr = ws-Belnr.

IF i_bkpf-blart = 'A1'. ...........................check the condition and enter in loop ITAB

loop at itab into ws_itab where a = b.

delete record.

endloop.

ELSEIF i_bkpf-blart = 'B1' . check the condition and enter in loop ITAB

loop at itab into ws_itab where a = b.

delere record.

endloop.

ENDIF.

ENDLOOP.

OR**************************************************************************

LOOP at BKPF where belnr = ws-Belnr.

loop at itab . -

Enter the LOOP ITAB and then check condtion

IF i_bkpf-blart = 'A1'.

delete record.

elseif i_bkpf-blart = 'B1' .

delere record.

endif.

endloop.

ENDLOOP.

Re: Effective Loop

Former Member — Thu, 30 Oct 2008 20:32:17 GMT

Hi,

The second loop is more efficient.

Thanks,

Preetham S

Re: Effective Loop

Former Member — Thu, 30 Oct 2008 20:35:14 GMT

Hi ,

But the second loop LOOPs across all the records of ITAB and then check condition.

And first one just LOOPs when condition is met.

What do you say?

Regards,

Yogita

Re: Effective Loop

Former Member — Thu, 30 Oct 2008 20:37:41 GMT

LOOP at BKPF where belnr = ws-Belnr.

IF i_bkpf-blart = 'A1'. ...........................check the condition and enter in loop ITAB

loop at itab into ws_itab where a = b.

delete record.

endloop.

ELSEIF i_bkpf-blart = 'B1' . check the condition and enter in loop ITAB

loop at itab into ws_itab where a = b.

delere record.

endloop.

ENDIF.

ENDLOOP.

**************************************OR*************************************************************************

LOOP at BKPF where belnr = ws-Belnr.

loop at itab . Enter the LOOP ITAB and then check condtion

IF i_bkpf-blart = 'A1'.

delete record.

elseif i_bkpf-blart = 'B1' .

delere record.

endif.

endloop.

ENDLOOP.

Re: Effective Loop

Former Member — Thu, 30 Oct 2008 20:44:28 GMT

Both are bad. You should use READs with BINARY SEARCH instead. Please see:

[Performance of Nested Loops|]

Rob

Re: Effective Loop

Former Member — Thu, 30 Oct 2008 20:53:43 GMT

Hi Rob,

But as per the requirement, I need to select multiple records and hence I need to use LOOP instead of READ which selects single record.

Please advice.

Regards,

Yogita

Re: Effective Loop

Former Member — Thu, 30 Oct 2008 21:16:02 GMT

The blog shows one way to do exactly that.

Rob