topic Re: if condition in Application Development and Automation Discussions

if condition

Former Member — Thu, 12 Jun 2008 05:05:15 GMT

Hi all there,

I am using if condition like this

IF JTAB-AUART = 'ZODO' OR JTAB-AUART = 'ZNET' OR

JTAB-AUART = 'ZOID' OR JTAB-AUART = 'ZDRO' OR

JTAB-AUART = 'ZNFO' OR JTAB-AUART = 'ZNIP' OR

JTAB-AUART = 'ZPDO' OR JTAB-AUART = 'ZPFO'.

is there any options to make it short

Regards

Shashikant

Re: if condition

Former Member — Thu, 12 Jun 2008 05:07:04 GMT

Hi,

Its highly recommended that you make use of brackets. If your code requires these conditions then its unavoidable. It cant be made short.

Thanks

Nayan

Re: if condition

Former Member — Thu, 12 Jun 2008 05:08:09 GMT

Shashikant,

I think if you want to go only with IF than no way to do a short.but either you can you CASE statement instead if IF.It will look better also in coding.

Amit.

Re: if condition

Former Member — Thu, 12 Jun 2008 05:22:09 GMT

Instead you can write:

CASE JTAB-AUART.

WHEN 'ZODO' or 'ZNET' or 'ZOID' or.....

ENDCASE.

Re: if condition

kesavadas_thekkillath — Thu, 12 Jun 2008 05:29:34 GMT

case JTAB-AUART.

when 'ZODO' or 'ZNET' or 'ZOID' or 'ZDRO' or 'ZNFO'

or 'ZNIP' or 'ZPDO' or 'ZPFO'.

endcase.

Re: if condition

Former Member — Thu, 12 Jun 2008 05:30:55 GMT

either use case statement but that wil not make your code shorter but if you want use if statement like this , please use braces .

Re: if condition

Former Member — Thu, 12 Jun 2008 05:58:01 GMT

i see in your if statement is checked 8 values beginning with 'Z***'.

you can make code shorter, if jtab-auart has maximum accepted values equals

to, for example, 9 (9 value is 'Z123'). Then you will write:

if not jtab-auart = 'Z123'.

*----

your logic

endif.

Re: if condition

Former Member — Thu, 12 Jun 2008 06:09:08 GMT

You can make it short by using Ranges.

Example:

ranges: r_range for vbak-auart.

data: w_auart type auart.

r_range-low = '1234'.

r_range-option = 'EQ'.

r_range-low = 'I'.

append r_range.

r_range-low = '5678'.

r_range-option = 'EQ'.

r_range-low = 'I'.

append r_range.

r_range-low = '9101'.

r_range-option = 'EQ'.

r_range-low = 'I'.

append r_range.

IF w_auart in r_range.

write 'hi'.

endif.

If the same condition need to be evaluated in multiple places, then you can build a range and use it in the IF statement.

Regards,

Lakshmi.

Re: if condition

Former Member — Thu, 12 Jun 2008 06:12:26 GMT

You can very well use in Operator to make this short and effective.

if JTAB-AUART in ( 'VAL1', 'VAL2', 'VAL3'... )

endif.