problem with loop statement

[Former Member]

Hi Experts,

My loop statement doesn't kindly suggest what am missing here.

Thanks, will appreciate your response.

TYPES: BEGIN OF lty_matnr,

matnr TYPE /sapapo/matnr,

END OF lty_matnr.

DATA: lt_matnr TYPE STANDARD TABLE OF lty_matnr WITH HEADER LINE,

ls_matnr TYPE lty_matnr.

Loop at lt_matnr[]. " THIS DOESN'T WORK"

ls_content_query-matnr_rng-high = lt_matnr-high.

ls_content_query-matnr_rng-low = lt_matnr-low.

ls_content_query-matnr_rng-sign = lt_matnr-sign.

ls_content_query-matnr_rng-option = lt_matnr-option.

append ls_content_query-matnr_rng.

Clear ls_content_query-matnr_rng.

Endloop.

[former_member195698]

use Loop at lt_matnr.

[Former Member]

if i use lt_matnr instead of lt_matnr[] it gives me an error.

Error: lt_matnr doesn;t have a component high

[Former Member]

loop at table without the [] is the right way to do it.

Loop at lt_matnr[]. " THIS DOESN'T WORK"

ls_content_query-matnr_rng-high = lt_matnr-high. < your code is erroring here

ls_content_query-matnr_rng-low = lt_matnr-low.

ls_content_query-matnr_rng-sign = lt_matnr-sign.

ls_content_query-matnr_rng-option = lt_matnr-option.

You are already inside the loop. Does the table have a column named high?

[Former Member]

it gives me an error at loop it_matnr[] . It says i need to use an assign ,transportation or into wa statement along with it

[Former Member]

doh, made a mistake in my last post. I forgot to remove the []. The code should be:

Loop at lt_matnr.

ls_content_query-matnr_rng-high = lt_matnr-high. < your code is erroring here

ls_content_query-matnr_rng-low = lt_matnr-low.

ls_content_query-matnr_rng-sign = lt_matnr-sign.

ls_content_query-matnr_rng-option = lt_matnr-option.

append ls_content_query-matnr_rng.

Clear ls_content_query-matnr_rng.

Endloop.

Can you post the definition for the table lt_matnr?

[Former Member]

Kevin is correct. You do not need the Brackets at yout LOOP it_matnr. You do get inside the loop and you error is at the next line of code. Check your struture for it_matnr, does it include a component named HIGH, as well as the other fields?

[Former Member]

Hi,

*TYPES: BEGIN OF lty_matnr,

*matnr TYPE /sapapo/matnr,

*END OF lty_matnr.

DATA: lt_matnr TYPE range of matnr,

ls_matnr like line of lt_matnr.

Loop at lt_matnr into ls_matnr.

ls_content_query-matnr_rng-high = ls_matnr-high.

ls_content_query-matnr_rng-low = ls_matnr-low.

ls_content_query-matnr_rng-sign = ls_matnr-sign.

ls_content_query-matnr_rng-option = ls_matnr-option.

append lt_content_query-matnr_rng from ls_content_query. "Declare an internal table of structure ls_content-query.

Clear ls_content_query-matnr_rng.

Endloop.

Regards,

Subramanian

Edited by: Subramanian PL on Jun 27, 2008 3:50 PM

[Former Member]

Only ranges have the high low option and sign

[Former Member]

Hi,

When you declare an object as select-options and ranges

then only HIGH and LOW values are possible.

Declare your object type of ranges.

Regards,

Rajitha.

[kesavadas_thekkillath]

TYPES: BEGIN OF lty_matnr,

matnr TYPE /sapapo/matnr,

END OF lty_matnr.

DATA: lt_matnr TYPE STANDARD TABLE OF lty_matnr ,

ls_matnr TYPE lty_matnr.

Loop at lt_matnr[] into ls_matnr. " THIS DOESN'T WORK"

ls_content_query-matnr_rng-high = lt_matnr-high.

ls_content_query-matnr_rng-low = lt_matnr-low.

ls_content_query-matnr_rng-sign = lt_matnr-sign.

ls_content_query-matnr_rng-option = lt_matnr-option.

append ls_content_query-matnr_rng.

Clear ls_content_query-matnr_rng.

Endloop.

[bpawanchand]

TYPES: BEGIN OF lty_matnr,

matnr TYPE /sapapo/matnr,

END OF lty_matnr.

DATA: lt_matnr TYPE STANDARD TABLE OF lty_matnr WITH HEADER LINE,

ls_matnr TYPE lty_matnr.

Loop at lt_matnr.

ls_content_query-matnr_rng-high = lt_matnr-high.

ls_content_query-matnr_rng-low = lt_matnr-low.

ls_content_query-matnr_rng-sign = lt_matnr-sign.

ls_content_query-matnr_rng-option = lt_matnr-option.

append ls_content_query-matnr_rng.

Clear ls_content_query-matnr_rng.

[Former Member]

Hi,

you are looping the body of your internal without any header, so the basic of internal table is that to read a internal table you always req a header line or work area so you have to specify a work area.

say

loop at itab. instead of loop at itab[].

with luck,

pritam.

[Former Member]

hi

u r Using

loop at i_iatab[] which signifies whole body of the table not the header line.

PLz use

loop at i_itab.

<statements>

endloop.

This will definately work.

Reward if helpful.

Sumit Agarwal

[Former Member]

HI ,

declare your structure as below

TYPES: BEGIN OF lty_matnr,

Sign type c,

option(2) type c,

low TYPE /sapapo/matnr,

high TYPE /sapapo/matnr,

END OF lty_matnr.

or declare

data : lt_matnr type range of matnr.

and check it will work.

Regards

Raj

[Former Member]

TYPES: BEGIN OF lty_matnr,

matnr TYPE /sapapo/matnr,

END OF lty_matnr.

DATA: lt_matnr TYPE STANDARD TABLE OF lty_matnr WITH HEADER LINE,

ls_matnr TYPE lty_matnr.

Loop at lt_matnr. " Just Remove the Square Braces it will work"

ls_content_query-matnr_rng-high = lt_matnr-high.

ls_content_query-matnr_rng-low = lt_matnr-low.

ls_content_query-matnr_rng-sign = lt_matnr-sign.

ls_content_query-matnr_rng-option = lt_matnr-option.

append ls_content_query-matnr_rng.

Clear ls_content_query-matnr_rng.

Endloop.