topic Re: Looping and using Do Varying in Application Development and Automation Discussions

Looping and using Do Varying

Former Member — Wed, 06 Jul 2011 12:10:02 GMT

Hi,

I got a program which is using a Loop at IT PA0167 and building few tables based on conditions and also using Do 20 times varying statement as well couple of times. When the Program is run with complete data it takes around 20000 records in the IT PA0167 and looping for those and using Do varying is taking lot of time.

What I can think is that I can split IT PA0167 into different sets of employees and then execute, not sure how effective that will be.

Any Idea on how to resolve this ?

Thanks

Manoj

Re: Looping and using Do Varying

Former Member — Wed, 06 Jul 2011 14:54:10 GMT

Hi Manoj, Try replacing the Do varying with the actual workarea, like if you are summing up in the Do varying, try replacing with actual variable sum like WA-FLD1 + WA-FLD2...WA_FLD20 or Check if the Assign Increment variant improves the performance.


DO.
  inc = sy-index  - 1.
  ASSIGN WA_DATA-FLD1 INCREMENT inc TO <EACH_FLD> RANGE WA_DATA.
  IF sy-subrc = 0.
* ...Logic based on <EACH_FLD>    
  ELSE.
    EXIT.
  ENDIF.
ENDDO.

Re: Looping and using Do Varying

Former Member — Wed, 06 Jul 2011 15:18:12 GMT

It's unlikely that you have to DO 20 TIMES VARYING... for each record in PA0167. Most or all of the fields will be empty. You should determine when you have processed all of the fields and then EXIT or CONTINUE.

Rob

Re: Looping and using Do Varying

Former Member — Thu, 07 Jul 2011 04:00:03 GMT

Hi Suman,

Tried out but nothing changed much..

thanks anyways..

Manoj

Re: Looping and using Do Varying

Former Member — Thu, 07 Jul 2011 04:00:46 GMT

Hi Rob,

I am already using Exit in case of blank records.

Thanks

Manoj

Re: Looping and using Do Varying

Former Member — Thu, 07 Jul 2011 12:56:55 GMT

Use EXIT in case of blank fields, not records.

Rob

Re: Looping and using Do Varying

Former Member — Fri, 08 Jul 2011 04:30:43 GMT

I mean the same Rob.. I am using Exit for blank field only.. that was a casual statement..