topic if statement in Application Development and Automation Discussions

if statement

Former Member — Tue, 04 Dec 2007 08:43:29 GMT

Hi All,

Please let me know is there anything wrong with following if statement.

LOOP AT t_bsid_zpp_zpr.

IF t_bsid_zpp_zpr-zterm = 'ZPP '.

IF ( t_bsid_zpp_zpr-blart = 'SS' OR t_bsid_zpp_zpr-blart = 'SR' ).

t_bsid = t_bsid_zpp_zpr.

APPEND t_bsid.

ENDIF.

ENDLOOP.

IF NOT t_bsid[] IS INITIAL.

record-blart_002 = 'P7'.

PERFORM sub_process_records.

ENDIF.

Regards,

Sai Prasad

Re: if statement

former_member386202 — Tue, 04 Dec 2007 08:50:33 GMT

Hi,

nothing but if u r facing any problem on that then try like this

LOOP AT t_bsid_zpp_zpr.

IF ( t_bsid_zpp_zpr-zterm = 'ZPP ' ) AND

( t_bsid_zpp_zpr-blart = 'SS' ) OR

( t_bsid_zpp_zpr-blart = 'SR' ).

t_bsid = t_bsid_zpp_zpr.

APPEND t_bsid.

ENDIF.

ENDLOOP.

IF NOT t_bsid[] IS INITIAL.

record-blart_002 = 'P7'.

PERFORM sub_process_records.

ENDIF.

Regards,

Prashant

Re: if statement

JozsefSzikszai — Tue, 04 Dec 2007 08:53:24 GMT

"Please let me know is there anything wrong with following if statement."

syntactically it is ok, the question is if you get, what you expect...

however you can make simplier:

LOOP AT t_bsid_zpp_zpr WHERE zterm = 'ZPP ' AND

( blart = 'SS' OR blart = 'SR' ).

t_bsid = t_bsid_zpp_zpr.

APPEND t_bsid.

ENDLOOP.

Re: if statement

Former Member — Tue, 04 Dec 2007 08:54:41 GMT

hi sai

this if statement is correct . if condition is true then loop is working other wise it's not work .

check ur itab fields its correct r not .

Re: if statement

Former Member — Tue, 04 Dec 2007 08:57:50 GMT

there is nothing wrong , tell me what problem facing you

Re: if statement

johndeconinck — Tue, 04 Dec 2007 09:03:44 GMT

Absolutely correct. Maybe something's wrong in your form sub_process_records?

Regards,

John.

Re: if statement

matt — Tue, 04 Dec 2007 09:08:26 GMT

First, it isn't surrounded by CODE blocks, so I'll fix that.

LOOP AT t_bsid_zpp_zpr.

  IF t_bsid_zpp_zpr-zterm = 'ZPP '.
    IF ( t_bsid_zpp_zpr-blart = 'SS' OR t_bsid_zpp_zpr-blart = 'SR' ).
      t_bsid = t_bsid_zpp_zpr.
      APPEND t_bsid.
    ENDIF.
  ENDIF.
ENDLOOP.

IF NOT t_bsid[] IS INITIAL.
  record-blart_002 = 'P7'.
  PERFORM sub_process_records.
ENDIF.

Second, it is overly complex.

LOOP AT t_bsid_zpp_zpr WHERE zterm = 'ZPP ' AND ( blart = 'SS' OR blart = 'SR' ).
  APPEND z_bsid_zpp_zpr TO t_bsid.
ENDLOOP.

IF NOT t_bsid[] IS INITIAL.
  record-blart_002 = 'P7'.
  PERFORM sub_process_records.
ENDIF.

matt

Re: if statement

matt — Tue, 04 Dec 2007 11:14:51 GMT

There is a possible issue I've just noticed. Is the space after ZPP necessary?

matt