abap code if& else

[Former Member]

Hi experts how to change the below code with one if and else statemennnt

FORM SUB_CHOOSE_SELECTION.

IF P_FULL = C_RADIO_VALUE. "Choice of Total Invoice

PERFORM FULL_INVOICE.

ENDIF.

IF P_HOLD = C_RADIO_VALUE. "Choice of Hold Invoice

PERFORM HOLD_INVOICE.

ENDIF.

IF P_BLOCK = C_RADIO_VALUE. "Choice of Block Invoice

PERFORM BLOCK_INVOICE.

ENDIF.

IF P_DELETE = C_RADIO_VALUE. "Choice of Delete Invoice

PERFORM DELETE_INVOICE.

ENDIF.

IF P_REJECT = C_RADIO_VALUE. "Choice of Reject invoice

PERFORM REJECT_INVOICE.

ENDIF.

regards

siri

[Former Member]

Hi,

If P_FULL = C_RADIO_VALUE or

P_HOLD = C_RADIO_VALUE ....

else.

Regards

Amole

[Former Member]

thank u

[Former Member]

Hi

Use This,,,

IF P_FULL = C_RADIO_VALUE. "Choice of Total Invoice

PERFORM FULL_INVOICE.

ELSEIF P_HOLD = C_RADIO_VALUE. "Choice of Hold Invoice

PERFORM HOLD_INVOICE.

ELSEIF P_BLOCK = C_RADIO_VALUE. "Choice of Block Invoice

PERFORM BLOCK_INVOICE.

ELSEIF P_DELETE = C_RADIO_VALUE. "Choice of Delete Invoice

PERFORM DELETE_INVOICE.

ELSE "Choice of Reject invoice

PERFORM REJECT_INVOICE.

ENDIF.

OR Please use CASE statement. It will be faster.

CASE C_RADIO_VALUE.

WHEN P_FULL.

PERFORM FULL_INVOICE.

WHEN P_HOLD.

PERFORM HOLD_INVOICE.

-

---

-

---

ENDCASE.

Sreedhar

[Former Member]

thank u

[Former Member]

Hai

FORM SUB_CHOOSE_SELECTION.

IF P_FULL = C_RADIO_VALUE. "Choice of Total Invoice

PERFORM FULL_INVOICE.

ELSEIF P_HOLD = C_RADIO_VALUE. "Choice of Hold Invoice

PERFORM HOLD_INVOICE.

ELSEIF P_BLOCK = C_RADIO_VALUE. "Choice of Block Invoice

PERFORM BLOCK_INVOICE.

ELSEIF P_DELETE = C_RADIO_VALUE. "Choice of DeleteInvoice

PERFORM DELETE_INVOICE.

ELSEIF P_REJECT = C_RADIO_VALUE. "Choice of Rejectinvoice

PERFORM REJECT_INVOICE.

ENDIF.

Thanks & regards

Sreeni

[Former Member]

thank u

[Former Member]

Best way is to make as below using CASE.

FORM SUB_CHOOSE_SELECTION.

Case C_RADIO_VALUE.

when P_FULL  "Choice of Total Invoice
PERFORM FULL_INVOICE.



when P_HOLD  "Choice of Hold Invoice
PERFORM HOLD_INVOICE.

when P_BLOCK  "Choice of Block Invoice
PERFORM BLOCK_INVOICE.


when P_DELETE  "Choice of Delete Invoice
PERFORM DELETE_INVOICE.
ENDIF.

when P_REJECT "Choice of Reject invoice
PERFORM REJECT_INVOICE.

ENDCASE.

ENDFORM.

rgds,

TM.

Please mark points if helpful.

[Former Member]

thank u

[Former Member]

Hi,

u have to use case statement.

Case C_RADIO_VALUE.

when P_FULL = . "Choice of Total Invoice
PERFORM FULL_INVOICE.

when P_HOLD  "Choice of Hold Invoice
PERFORM HOLD_INVOICE.

when P_BLOCK "Choice of Block Invoice
PERFORM BLOCK_INVOICE.

when P_DELETE  "Choice of Delete Invoice
PERFORM DELETE_INVOICE.
when P_REJECT  "Choice of Reject invoice
PERFORM REJECT_INVOICE.

ENDASE.

<b>Please Mark Helpful Answers</b>

.

[Former Member]

thank u

[Former Member]

Hi,

Check this way...

IF C_RADIO_VALUE = p_full. "Choice of Total Invoice
PERFORM FULL_INVOICE.
esleif C_RADIO_VALUE = p_Hold.
PERFORM HOLD_INVOICE.
esleif C_RADIO_VALUE = p_block.
PERFORM BLOCK_INVOICE.
esleif C_RADIO_VALUE = p_delete
PERFORM DELETE_INVOICE.
esleif C_RADIO_VALUE = p_reject
PERFORM REJECT_INVOICE.
ENDIF.

Regards

vijay

[Former Member]

thank u