optimize performance on loop at screen

[Former Member]

Hiii

on my selection screen i have lot's of condition where when i select one radio button i must grey a field

In my at selection screen on output i have at least 4 loop at screen.

please advise is there a way to optimise or group the loop at screen

a sample of my code is as follow

  IF RB_rep_ut IS NOT INITIAL.  

    LOOP AT SCREEN.
      IF screen-group1 EQ gc_gpb.
        screen-input = gc_zero  .
        MODIFY SCREEN.
      ENDIF.
    ENDLOOP.
  ENDIF.

  IF RB_rep IS NOT INITIAL.  

    LOOP AT SCREEN.
      IF screen-group1 EQ gc_gpC.
        screen-input = gc_zero  .
        MODIFY SCREEN.
      ENDIF.
    ENDLOOP.
  ENDIF.

  IF s_comp[] IS NOT INITIAL.  
    LOOP AT SCREEN.
      IF screen-group1 EQ gc_gpa.
        screen-input = gc_zero.
        MODIFY SCREEN.
      ENDIF.
    ENDLOOP.
  ENDIF.

[Sandeep_Panghal]

As you are using radio buttons so only one of the LOOPs will be processed.

With the select-otion , you have to do this anyway.

[former_member242255]

you have to do the same way how you are doing...anyway as you are separating the logic based on radio buttons it should not be aproblem as any one of the loop at screen will be executed.

[Former Member]

ok the radio button is fine

but what about the check box

if cb_b1 EQ 'X'.
 loop at screen 
 endloop.
endif.

if cb_b2 EQ 'X'.
 loop at screen 
 endloop.
endif.

if cb_b3 EQ 'X'.
 loop at screen 
 endloop.
endif.

[I355602]

Hi,

Try using it this way:

loop at screen.
  if cb_b1 = 'X'.
  endif.

  if cb_b2 = 'X'.
  endif.

  if cb_b3 = 'X'.
  endif.

  modify screen.
endloop.

Hope this helps you.

Regards,

Tarun