topic Performance hit with 'ASSIGN COMPONENT' in Application Development and Automation Discussions

Performance hit with 'ASSIGN COMPONENT'

Former Member — Thu, 18 May 2006 14:34:05 GMT

Hi,

I am using 'Assign Component' to pass values between 2 dynamically created tables. However it seems the subroutine where this is occuring is taking long to run.

Is ASSIGN COMPONENT abc of STRUCTURE .... statement Performance intensive ?

Thanks

Re: Performance hit with 'ASSIGN COMPONENT'

franois_henrotte — Thu, 18 May 2006 15:16:47 GMT

be sure to do ASSIGN only once, not each pass in your LOOP

Re: Performance hit with 'ASSIGN COMPONENT'

Former Member — Thu, 18 May 2006 15:19:13 GMT

Thanks. The component to assign is inside a Loop.

Loop at itab

Assign Component itab-comp of structure struc..

Endloop.

itab has the name of the components (fields) of the structure.

Re: Performance hit with 'ASSIGN COMPONENT'

RichHeilman — Thu, 18 May 2006 15:23:12 GMT

I'm not sure if using the INDEX of the field of the structure is faster than using the name of the field or not. Maybe you can try it.

Regards,

Rich Heilman

Re: Performance hit with 'ASSIGN COMPONENT'

franois_henrotte — Thu, 18 May 2006 15:27:15 GMT

if you can give the complete code between LOOP and ENDLOOP we could see if optimization is possible...

Re: Performance hit with 'ASSIGN COMPONENT'

Former Member — Thu, 18 May 2006 18:20:00 GMT

Hi Francois... Here is the code

Loop at <it_select_tab> assigning <ls_select_tab>.

INSERT INITIAL LINE INTO <it_select_global> index cnt assigning

<ls_select_global>.

Loop at ifc_local into xfc.

assign component xfc-fieldname of structure <ls_select_tab> to

<ls_select_field>.

read table itg_map into itl_map with key

zsyst = lv_syst

zmapfield = xfc-fieldname

binary search.

if sy-subrc = 0.

assign component itl_map-fieldname of structure

<ls_select_global> to

<ls_select_global_field>.

<ls_select_global_field> = <ls_select_field>.

endif.

Endloop.

Unassign: <ls_select_field>,

<ls_select_global_field>.

Endloop.

Re: Performance hit with 'ASSIGN COMPONENT'

franois_henrotte — Wed, 24 May 2006 12:34:16 GMT

OK in this case you cannot do it else, but you should try to remove nested loops. For example, why don't you do the loop directly on itg_map in place of xfc ?

but anyway beware your UNASSIGN statements are not placed well, they should be placed like this:

Loop at ifc_local into xfc.

assign component xfc-fieldname of structure <ls_select_tab> to

<ls_select_field>.

read table itg_map into itl_map with key

zsyst = lv_syst

zmapfield = xfc-fieldname

binary search.

if sy-subrc = 0.

assign component itl_map-fieldname of structure

<ls_select_global> to

<ls_select_global_field>.

<ls_select_global_field> = <ls_select_field>.

UNASSIGN <ls_select_global_field>.

endif.

UNASSIGN <ls_select_field>.

Endloop.

Re: Performance hit with 'ASSIGN COMPONENT'

Former Member — Tue, 30 May 2006 19:37:50 GMT

Thanks Francois. It helped with the performance.