topic Re: What is wrong with this code? in Application Development and Automation Discussions

What is wrong with this code?

Former Member — Mon, 21 Jan 2008 20:30:51 GMT

I am modifying a sapscript print program to suppress certain characteristics; we don't want them to print. To do this I am looping through the characteristic table and doing a case statement on the name. If I wish to suppress the characteristic I delete it from the table. When it runs I get a run-time error (no dump, one of those message errors). It doesn't like the delete statement. I am also putting several characteristics on one line to save space, you will see in the code below:

LOOP AT tkomcon.

CASE tkomcon-atnam.

WHEN 'Z_LENGTH'.

temp_length = tkomcon-atwtb.

CONCATENATE ztempstring tkomcon-atwtb INTO ztempstring.

WHEN 'Z_WIDTH'.

temp_width = tkomcon-atwtb.

DELETE tkomcon.

WHEN 'Z_THICKNESS1'.

temp_height1 = tkomcon-atwtb.

DELETE tkomcon.

WHEN 'Z_THICKNESS2'.

temp_height2 = tkomcon-atwtb.

DELETE tkomcon.

WHEN 'Z_THICKNESS3'.

temp_height3 = tkomcon-atwtb.

DELETE tkomcon.

WHEN 'Z_THICKNESS4'.

temp_height4 = tkomcon-atwtb.

DELETE tkomcon.

WHEN 'Z_PCS'.

temp_pcs = tkomcon-atwtb.

WHEN 'Z_BFT_PCE'.

DELETE tkomcon.

WHEN 'Z_BOARD_FT'.

DELETE tkomcon.

WHEN 'Z_PRICE_PASS'.

DELETE tkomcon.

WHEN 'Z_PRICE_PCE'.

temp_price = tkomcon-atwtb.

WHEN 'Z_GRIND'.

DELETE tkomcon.

WHEN 'Z_BORATE'.

DELETE tkomcon.

ENDCASE.

ENDLOOP.

READ TABLE tkomcon INDEX 1.

tkomcon-atwtb = ''.

CONCATENATE temp_length ' x ' temp_width ' x ' temp_height1 ' x ' temp_height2 ' x ' temp_height3

' x ' temp_height4 ' x ' INTO ztempstring.

tkomcon-atbez = ztempstring.

MODIFY tkomcon.

Regards,

Davis

Re: What is wrong with this code?

former_member194669 — Mon, 21 Jan 2008 20:36:23 GMT

Try this way


LOOP AT tkomcon.
v_tabix = sy-tabix.     "<<<

CASE tkomcon-atnam.
WHEN 'Z_LENGTH'.
temp_length = tkomcon-atwtb.

* CONCATENATE ztempstring tkomcon-atwtb INTO ztempstring.

WHEN 'Z_WIDTH'.
temp_width = tkomcon-atwtb.
DELETE tkomcon index v_tabix. "<<
WHEN 'Z_THICKNESS1'.
temp_height1 = tkomcon-atwtb.
DELETE tkomcon index v_tabix. "<<
WHEN 'Z_THICKNESS2'.
temp_height2 = tkomcon-atwtb.
DELETE tkomcon index v_tabix. "<<
WHEN 'Z_THICKNESS3'.
temp_height3 = tkomcon-atwtb.
DELETE tkomcon index v_tabix. "<<
WHEN 'Z_THICKNESS4'.
temp_height4 = tkomcon-atwtb.
DELETE tkomcon index v_tabix. "<<
WHEN 'Z_PCS'.
temp_pcs = tkomcon-atwtb.
WHEN 'Z_BFT_PCE'.
DELETE tkomcon index v_tabix. "<<
WHEN 'Z_BOARD_FT'.
DELETE tkomcon index v_tabix. "<<
WHEN 'Z_PRICE_PASS'.
DELETE tkomcon index v_tabix. "<<
WHEN 'Z_PRICE_PCE'.
temp_price = tkomcon-atwtb.
WHEN 'Z_GRIND'.
DELETE tkomcon index v_tabix. "<<
WHEN 'Z_BORATE'.
DELETE tkomcon index v_tabix. "<<
ENDCASE.
ENDLOOP.

READ TABLE tkomcon INDEX 1.
v_tabix = sy-tabix.
tkomcon-atwtb = ''.
CONCATENATE temp_length ' x ' temp_width ' x ' temp_height1 ' x ' temp_height2 ' x ' temp_height3
' x ' temp_height4 ' x ' INTO ztempstring.
tkomcon-atbez = ztempstring.
MODIFY tkomcon  index v_tabix.

a®

Re: What is wrong with this code?

Former Member — Mon, 21 Jan 2008 20:41:07 GMT

Thanks for that. I wasn't sure if I had to use an idex when a table had a header line.

Thanks again,

Davis.