topic IN operator problem in Application Development and Automation Discussions

IN operator problem

Former Member — Thu, 18 Sep 2008 10:49:01 GMT

Hi,

I have a function module.

FUNCTION z_read.

RANGES zmfrpn FOR zp-mfrpn.

CONCATENATE mfrpn '*' INTO zmfrpn-low.

zmfrpn-sign = 'I'.

zmfrpn-option = 'CP'.

APPEND zmfrpn.

CLEAR zmfrpn.

ENDFUNCTION.

FORM stock.

SELECT mfrpn matnr meins normt

INTO (tp-mfrpn, tpmatnr,tp-meins, tp-normt)

FROM mara WHERE mfrpn IN zmfrpn.

ENDSELECT.

ENDFORM.

And i receive following error: The IN operator with "ZMFRPN" is followed neither by an internal table nor by a value list.

What should i do?

Thank you!

Re: IN operator problem

Former Member — Thu, 18 Sep 2008 10:50:46 GMT

i tink u have to use the field name for the ranges as its an internal table.

Re: IN operator problem

Former Member — Thu, 18 Sep 2008 10:52:43 GMT

in where clause you use mfrpn in low.

Re: IN operator problem

Former Member — Thu, 18 Sep 2008 10:55:59 GMT

use these two

zmfrpn-high = '100'.
zmfrpn-low = '001'.

append.

regards,

kaushik

Re: IN operator problem

Former Member — Thu, 18 Sep 2008 10:59:46 GMT

not working

Re: IN operator problem

Former Member — Thu, 18 Sep 2008 11:01:28 GMT

hi,

now check it.

RANGES zmfrpn FOR zp-mfrpn.

zmfrpn-low = '100'

zmfrpn-high = '200'.

zmfrpn-sign = 'I'.

zmfrpn-option = 'CP'.

APPEND zmfrpn.

CLEAR zmfrpn.

FORM stock.

SELECT mfrpn matnr meins normt

I NTO (tp-mfrpn, tpmatnr,tp-meins, tp-normt)

FROM mara WHERE IN zmfrpn .

ENDSELECT.

ENDFORM.

Regards,

Mahesh,

Edited by: Mahesh Reddy .Pocha on Sep 18, 2008 1:04 PM

Re: IN operator problem

Former Member — Thu, 18 Sep 2008 11:01:55 GMT

Hello

If you fill ranges in functional module, then you need:

1. before select statement declare new ranges like ranges in FM

2. before select statement call FM

3. FM must return filled ranges to your new ranges.

4. use in select statement new ranfes.

Re: IN operator problem

Former Member — Thu, 18 Sep 2008 11:02:37 GMT

please you let me know use of

CONCATENATE mfrpn '*' INTO zmfrpn-low.

zmfrpn-sign = 'I'.

zmfrpn-option = 'CP'.

APPEND zmfrpn.

CLEAR zmfrpn.

Re: IN operator problem

Former Member — Thu, 18 Sep 2008 11:04:06 GMT

Hi,

Please check for 2 things in your code....

Must: Pass the range as passing param in the Subroutine...

1. How is the range declared?

2. Insert [] after the range in the SELECT statement.

Hope this solves your problem.

Regards,

Kunjal

Re: IN operator problem

Former Member — Thu, 18 Sep 2008 11:06:07 GMT

Hi,

How are the function module and the Subroutine joined???

You have declared the ranges in FM and trying to use that in a form I guess in a different program.

If the form stock is with in the FM then declare the ranges in the TOP include of the Function group.

santhosh

Re: IN operator problem

matt — Thu, 18 Sep 2008 11:09:02 GMT

This is a question of scope.

You've defined zmfrpn in the Function Module. You are attempting to use it in form stock. Data declared in a function module is local to the function module - i.e. the bits between FUNCTION... ENDFUNCTION.

You could define zmfrpn in the global data of the function group (don't do this though, it's REALLY BAD programming, to deal with global data in subroutines, unless you really have to )

What you should do is pass zmfrpn to the form stock as a parameter.

Further, can I suggest that you adopt a more meaningful naming convention for your variable names. E.g. it's a range, start it with the prefix rng_ for example.

Also, you are using SELECT ... ENDSELECT to select a single row from the database. Why not use SELECT SINGLE. Or SELECT ... UP TO 1 ROWS?

matt

Re: IN operator problem

Former Member — Fri, 19 Sep 2008 13:21:42 GMT

FORM stock TABLES tp STRUCTURE zp.

ENDFORM.                    "stock

Thank you to all... i have made it

Re: IN operator problem

Former Member — Fri, 19 Sep 2008 13:23:50 GMT

Further, can I suggest that you adopt a more meaningful naming convention for your variable names. E.g. it's a range, start it with the prefix rng_ for example.

OK.

Also, you are using SELECT ... ENDSELECT to select a single row from the database. Why not use SELECT SINGLE. Or SELECT ... UP TO 1 ROWS?

No, that select does not select a single row ...

Thank you for your advices ... .