Please make suggestions on how to improve this code.

Question

Hello Experts,
I am currently modifying a report where a certain code is giving
me run time error. When I debugged the program, the internal table
has 20,000+ records. Now, is there any alternative to this code?
I really need to speed this up. Help would be greatly appreciated and rewarded.
LOOP AT it_cdpos INTO wa_cdpos.
    l_matnr = wa_cdpos-objectid+00(18).
    l_charg = wa_cdpos-objectid+22(10).
    l_zustd = wa_cdpos-value_new+00(01).
    READ TABLE it_batch INTO wa_batch
         WITH KEY matnr = l_matnr
                  charg = l_charg.
    IF sy-subrc EQ 0.
      l_tabix = sy-tabix.
      wa_batch-zustd = l_zustd.
      MODIFY it_batch FROM wa_batch INDEX l_tabix
             TRANSPORTING zustd.
    ELSE.
      wa_batch-matnr = l_matnr.
      wa_batch-charg = l_charg.
      wa_batch-zustd = l_zustd.
      wa_batch-code  = 'A'.              "selected within period
      APPEND wa_batch TO it_batch.
    ENDIF.
  ENDLOOP.

Answer 1

Check the SQL statement from which u get the result otherwise
the code which u sent is optimized or u can use the binary search option for the Read statement for MATNR
while the rest of the code is optimized....
or u can use
loop at it_cdpos assinging <fs>.
             <FS>-cols = value.
<b>note: in this case <FS>-col = value</b>
u wont have to use <b>modify</b> statement for internal table for updating the line contents
since assinging it to field-symbol will dynamically modifies the contents.
<b>this method is faster than the below modify process </b>
endloop.