topic Re: if statement in Application Development and Automation Discussions

if statement

Former Member — Fri, 03 Nov 2006 17:05:07 GMT

Hi,

can you please correct my code:

select-options: s_werks for marc-werks.

.......................

......................

form get_data.

if s_werks EQ 'KNX'.

*-- Here the if statement is not working. Can you please correct this IF statement, if it is wrong.

select * from mdkp

where matnr = i_mara-matnr.

......................

.....................

endif.

Thanks,

Re: if statement

Former Member — Fri, 03 Nov 2006 17:06:34 GMT

if s_werks-LOW eq EQ 'KNX'

Re: if statement

Former Member — Fri, 03 Nov 2006 17:09:01 GMT

sey,

u can't use if s_werks EQ 'KNX'.

as s_Werks is a select-option and it will be having multiple entries so can't use 'EQ'.

where u can use

loop at s_werks.

if s_werks-low eq 'KNX'.

endif.

endloop.

-Anu

Message was edited by: Anupama Reddy

Re: if statement

Former Member — Fri, 03 Nov 2006 17:09:29 GMT

Thnx John.

It worked!!!! But still there's a problem

if I have multiple values for S_WERKS, it wont work, right?

Thnx.

Message was edited by: sey ni

Re: if statement

Clemenss — Fri, 03 Nov 2006 17:10:58 GMT

Sey ni,

s_werks is a range table. It can be used with comparison operator IN.

ist it

<pre>

select * from mdkp

where PLWRK in s_werks

and matnr = i_mara-matnr.

</pre>

Sorry, I don't know your context.

Regards,

Clemens

Re: if statement

Former Member — Fri, 03 Nov 2006 17:11:07 GMT

I don't think I'bve seen it done this way, but try:

if 'KNX' in s_werks.

Rob

Re: if statement

Former Member — Fri, 03 Nov 2006 17:15:15 GMT

Bear in mind that SELECT-OPTIONS are range values associated with an internal table. The use of IN might be necessary... not sure of your needs. Do you need a select-options or will a parameter do the trick?

Re: if statement

Former Member — Fri, 03 Nov 2006 17:17:44 GMT

John,

i have a small doubt,

is the statement

if s_werks-LOW eq EQ 'KNX'

checks only the first value or the entire range?

If only first value, lets assume 'KNX' exist in middle then how ?

sailesh

Re: if statement

Former Member — Fri, 03 Nov 2006 17:18:10 GMT

sey,

loop at s_werks and in the loop check with each s_werks-low as given by me above post.

-Anu

Re: if statement

Former Member — Fri, 03 Nov 2006 17:22:04 GMT

Checks only first value... that is why I asked if you truly need a SELECT_OPTIONS statement. If the user can enter multiple values, then use "IN s_werks".

Or simply LOOP thru the internal table S_WERKS.

Re: if statement

RichHeilman — Fri, 03 Nov 2006 17:22:23 GMT

Right you will need to loop at the select-option and check each one.

But that will only help when the user does single values, he could do ranges also, so you should take care of that as well. Suggestion, get all of the plants in the select-option from table t001w, then check each one.

data: it001w type table of t001w with header line.

select-options: s_werks for it001w-werks.

select * into table it001w from t001w
            where werks in s_werks.

loop at it001w.
<b>  if it00w1-werks = 'KNX'.
 

  endif.</b>

endloop.

Regards,

Rich Heilman

Re: if statement

Former Member — Fri, 03 Nov 2006 17:29:59 GMT

Thanks Folks,

Yeah. I got it now. As it is select-option, i hve to do the loop.

Thanks again for ur valuable responses, I appreciate that.

But I can give 10 points to only one of you :)-

Re: if statement

RichHeilman — Fri, 03 Nov 2006 17:31:22 GMT

Remember, just looping the select-option will not cover all of the bases, in order to cover the bases, you need to process as I show above.

Regards,

Rich Heilman

Re: if statement

Former Member — Fri, 03 Nov 2006 17:31:24 GMT

HI Sey,

Please give the points to appropriate person and close the thread. It also encourages the other persons.

sailesh

Re: if statement

Former Member — Fri, 03 Nov 2006 17:34:35 GMT

Yes, Rich. That's true. Thanks a lot for the help.