collect

[Former Member]

hi,

is this logic correct?

thanks

sort it_tab by vbeln vgbel vgpos.

loop at it_tab into wa_tab.

move wa_tab-vbeln to wa_report-vbeln_fd.

move wa_tab-vgbel to wa_report-vbeln.

move wa_tab-vgpos to wa_report-posnr.

move wa_tab-matnr to wa_report-matnr.

move wa_tab-charg to wa_report-charg.

move wa_tab-lfimg to wa_report-lfimg.

collect wa_report into it_report.

endloop.

[Former Member]

HI EL,

UR CODE IS CORRECT AND IF U STILL WANT TO IMPROVE THE PERFORMANCE THEN MAKE THIS CHANGES.

sort it_tab by vbeln vgbel vgpos.

loop at it_tab into wa_tab.

move wa_tab-vbeln to wa_report-vbeln_fd.

move wa_tab-vgbel to wa_report-vbeln.

move wa_tab-vgpos to wa_report-posnr.

MOVE-CORRESPONDING WA_TAB TO WA_REPORT

collect wa_report into it_report.

endloop.

[Former Member]

hi,

thanks

may i know why i need to have move and move-corresponding together?

[Former Member]

You dont have to use them together. Basically MOVE-CORRESPONDING takes more time than just MOVE, so it is not recommended.

Hope this helps.