Application Development and Automation Discussions
Join the discussions or start your own on all things application development, including tools and APIs, programming models, and keeping your skills sharp.
cancel
Showing results for 
Search instead for 
Did you mean: 
Read only

syntax error

Former Member
0 Likes
1,075

Dear abap experts,

TABLES: SFLIGHT,SPFLI.

DATA: BEGIN OF ITAB_SFLIGHT,

CARRID LIKE SFLIGHT-CARRID,

CONNID LIKE SFLIGHT-CONNID,

FLDATE LIKE SFLIGHT-FLDATE,

PRICE LIKE SFLIGHT-PRICE,

CURRENCY LIKE SFLIGHT-CURRENCY,

CITYFROM LIKE SPFLI-CITYFROM,

AIRPFROM LIKE SPFLI-AIRPFROM,

PLANETYPE LIKE SFLIGHT-PLANETYPE,

END OF ITAB_SFLIGHT.

SELECT ACARRID AFLDATE APRICE BCITYFROM B~AIRPFROM INTO

ITAB_SFLIGHT-CARRID,ITAB_SFLIGHT-FLDATE,ITAB_SFLIGHT-PRICE,

ITAB_SFLIGHT-CITYFROM,ITAB_SFLIGHT-AIRPFROM

FROM SFLIGHT AS A INNER JOIN SPFLI AS B

ON SFLIGHT-CARRID = SPFLI-CARRID .

in the above syntax the error is

comma without preecding colon(after select?)line no 16

1 ACCEPTED SOLUTION
Read only

RaymondGiuseppi
Active Contributor
0 Likes
1,046

Remove the comma in the select add brackets

SELECT A~CARRID A~FLDATE A~PRICE B~CITYFROM B~AIRPFROM INTO
(ITAB_SFLIGHT-CARRID, ITAB_SFLIGHT-FLDATE, ITAB_SFLIGHT-PRICE,
ITAB_SFLIGHT-CITYFROM, ITAB_SFLIGHT-AIRPFROM)
FROM SFLIGHT AS A INNER JOIN SPFLI AS B
ON SFLIGHT-CARRID = SPFLI-CARRID .

Try to use a INTO CORRESPONDING FIELDS OF TABLE ITAB, to reduce use of SELECT...ENDSELECT is usually a good idea. (in your case, use the same sequence of fields remove the need to use a CORRESPONDING FIELD clause)

Regards

10 REPLIES 10
Read only

Former Member
0 Likes
1,046

Hi,

Dont use comma without using Braces.

SELECT ACARRID AFLDATE APRICE BCITYFROM B~AIRPFROM INTO

(ITAB_SFLIGHT-CARRID,ITAB_SFLIGHT-FLDATE, ITAB_SFLIGHT-PRICE,ITAB_SFLIGHT-CITYFROM ,ITAB_SFLIGHT-AIRPFROM)

FROM SFLIGHT AS A INNER JOIN SPFLI AS B

ON SFLIGHT-CARRID = SPFLI-CARRID .

endselect.

i think it is useful for you.

Reward points if useful.

Thanks & regards

Deepika.

Read only

RaymondGiuseppi
Active Contributor
0 Likes
1,047

Remove the comma in the select add brackets

SELECT A~CARRID A~FLDATE A~PRICE B~CITYFROM B~AIRPFROM INTO
(ITAB_SFLIGHT-CARRID, ITAB_SFLIGHT-FLDATE, ITAB_SFLIGHT-PRICE,
ITAB_SFLIGHT-CITYFROM, ITAB_SFLIGHT-AIRPFROM)
FROM SFLIGHT AS A INNER JOIN SPFLI AS B
ON SFLIGHT-CARRID = SPFLI-CARRID .

Try to use a INTO CORRESPONDING FIELDS OF TABLE ITAB, to reduce use of SELECT...ENDSELECT is usually a good idea. (in your case, use the same sequence of fields remove the need to use a CORRESPONDING FIELD clause)

Regards

Read only

amit_khare
Active Contributor
0 Likes
1,046

You miss the brackets after INTO

SELECT ACARRID AFLDATE APRICE BCITYFROM B~AIRPFROM INTO

(ITAB_SFLIGHT-CARRID,ITAB_SFLIGHT-FLDATE,ITAB_SFLIGHT-PRICE,

ITAB_SFLIGHT-CITYFROM,ITAB_SFLIGHT-AIRPFROM )

FROM SFLIGHT AS A INNER JOIN SPFLI AS B

ON SFLIGHT-CARRID = SPFLI-CARRID .

Read only

Former Member
0 Likes
1,046

Hi,

Use like this

SELECT ACARRID AFLDATE APRICE BCITYFROM B~AIRPFROM INTO

(TAB_SFLIGHT-CARRID,ITAB_SFLIGHT-FLDATE,ITAB_SFLIGHT-PRICE,

ITAB_SFLIGHT-CITYFROM,ITAB_SFLIGHT-AIRPFROM)FROM SFLIGHT AS A INNER JOIN SPFLI AS B

ON SFLIGHT-CARRID = SPFLI-CARRID .

Reward if useful.

Regards,

Swetha.

Edited by: swetha r on Jun 20, 2008 8:00 AM

Read only

Simha_
Product and Topic Expert
Product and Topic Expert
0 Likes
1,046

Hi,

Use this below code.

SELECT A~CARRID A~FLDATE A~PRICE B~CITYFROM B~AIRPFROM INTO ITAB_SFLIGHT
(ITAB_SFLIGHT-CARRID,  ITAB_SFLIGHT-FLDATE,  ITAB_SFLIGHT-PRICE ,
ITAB_SFLIGHT-CITYFROM, ITAB_SFLIGHT-AIRPFROM)
FROM SFLIGHT AS A INNER JOIN SPFLI AS B 
ON a~CARRID = b~CARRID .
ENDSELECT.

Cheers,

SImha.

Read only

Former Member
0 Likes
1,046

Hi,

You write code like this.

TABLES: SFLIGHT,SPFLI.

DATA: BEGIN OF ITAB_SFLIGHT,

CARRID LIKE SFLIGHT-CARRID,

CONNID LIKE SFLIGHT-CONNID,

FLDATE LIKE SFLIGHT-FLDATE,

PRICE LIKE SFLIGHT-PRICE,

CURRENCY LIKE SFLIGHT-CURRENCY,

CITYFROM LIKE SPFLI-CITYFROM,

AIRPFROM LIKE SPFLI-AIRPFROM,

PLANETYPE LIKE SFLIGHT-PLANETYPE,

END OF ITAB_SFLIGHT.

SELECT ACARRID AFLDATE APRICE BCITYFROM B~AIRPFROM INTO

( ITAB_SFLIGHT-CARRID,ITAB_SFLIGHT-FLDATE,ITAB_SFLIGHT-PRICE,

ITAB_SFLIGHT-CITYFROM,ITAB_SFLIGHT-AIRPFROM )

FROM SFLIGHT AS A INNER JOIN SPFLI AS B

ON SFLIGHT-CARRID = SPFLI-CARRID .

append itab_sflight.

endselect.

Mainly for the field list put the brackets. Observe in the code. You have to write the endselect.

Reward.

Read only

Former Member
0 Likes
1,046

Hi,

I corrected your code, it is working fine.

TABLES: sflight,spfli.

DATA: BEGIN OF itab_sflight,

carrid LIKE sflight-carrid,

connid LIKE sflight-connid,

fldate LIKE sflight-fldate,

price LIKE sflight-price,

currency LIKE sflight-currency,

cityfrom LIKE spfli-cityfrom,

airpfrom LIKE spfli-airpfrom,

planetype LIKE sflight-planetype,

END OF itab_sflight.

SELECT a~carrid

a~fldate

a~price

b~cityfrom

b~airpfrom

INTO (itab_sflight-carrid,

itab_sflight-fldate,

itab_sflight-price,

itab_sflight-cityfrom,

itab_sflight-airpfrom)

FROM sflight AS a INNER JOIN spfli AS b

ON acarrid = bcarrid .

ENDSELECT.

Reward points if it is helpful.

Read only

Former Member
0 Likes
1,046

Hi,

Please change the code as below:



SELECT A~CARRID A~FLDATE A~PRICE B~CITYFROM B~AIRPFROM INTO
(ITAB_SFLIGHT-CARRID,ITAB_SFLIGHT-FLDATE,ITAB_SFLIGHT-PRICE,
ITAB_SFLIGHT-CITYFROM,ITAB_SFLIGHT-AIRPFROM)
FROM SFLIGHT AS A INNER JOIN SPFLI AS B 
ON SFLIGHT-CARRID = SPFLI-CARRID .

Thanks,

Sriram POnna.

Read only

Former Member
0 Likes
1,046

Hi Ansuman Parhi,

TABLES: SFLIGHT,SPFLI.
 
DATA: BEGIN OF ITAB_SFLIGHT1, " This is only Work Area
CARRID LIKE SFLIGHT-CARRID,
CONNID LIKE SFLIGHT-CONNID,
FLDATE LIKE SFLIGHT-FLDATE,
PRICE LIKE SFLIGHT-PRICE,
CURRENCY LIKE SFLIGHT-CURRENCY,
CITYFROM LIKE SPFLI-CITYFROM,
AIRPFROM LIKE SPFLI-AIRPFROM,
PLANETYPE LIKE SFLIGHT-PLANETYPE,
END OF ITAB_SFLIGHT1.
 
DATA : ITAB_SFLIGHT LIKE ITAB_SFLIGHT1 OCCURS 0 WITH HEADER LINE. " Declare internal table if u want to fill all the data in one table
 
SELECT A~CARRID A~FLDATE A~PRICE B~CITYFROM B~AIRPFROM INTO
(ITAB_SFLIGHT-CARRID, "Put the small brackets here ( )
 ITAB_SFLIGHT-FLDATE,
 ITAB_SFLIGHT-PRICE,
ITAB_SFLIGHT-CITYFROM, 
ITAB_SFLIGHT-AIRPFROM ) "Put the small brackets here ( )
FROM SFLIGHT AS A INNER JOIN SPFLI AS B
ON A~CARRID = B~CARRID . 
* Write ur logic here...Begin
  WRITE : ITAB_SFLIGHT-CARRID, ITAB_SFLIGHT-FLDATE, ITAB_SFLIGHT-PRICE,
ITAB_SFLIGHT-CITYFROM, ITAB_SFLIGHT-AIRPFROM.
 
APPEND ITAB_SFLIGHT.
* Write ur logic here...End
 
ENDSELECT.

Hope it will solve your problem..

Reward points if useful..

Thanks & Regards

ilesh 24x7

Read only

Former Member
0 Likes
1,046

Hi,

Try using into corresponding fields of...

Regards,