ABAP Clean Code : 변수와 리터럴 - 변수(Variables)

이 글은 『Clean ABAP – A Style Guide for Developers』 서적 (written by Klaus Haeuptle, Florian Hoffmann, Rodrigo Jordão, Michel Martin, Anagha Ravinarayan, Kai Westerholz) 을 참고하여 작성하였습니다. 해당 내용을 좀 더 쉽고 이해할 수 있도록 요약/정리 및 개인적인 의견을 붙여 재구성한  포스팅 입니다. 

 

 

모든 프로그래밍 언어에서는 변수는 중간 결과를 저장하고 재사용하기 위한 핵심 요소이다. 이 글에서는 ABAP에서 변수 선언을 어떻게 깔끔하게 할 수 있는지 알아보도록 하자.

---

✅ 변수 선언은 코드 상단이 아닌 사용하는 위치에 선언하자.

ABAP에서는 전통적으로 코드의 맨 위에 필요한 변수를 미리 선언했었다. 이 방법은 모든 변수를 한눈에 볼 수 있다는 장점과 코드가 정돈된 느낌은 있지만, 코드가 길어지면 이 변수가 어디서 어떻게 쓰이는지 기억하기 어려워진다는 단점이 있다. 장기간 유지보수된 프로그램을 보게되면  사용하지 않은 변수들이 그대로 남아있는 경우도 종종 보게 된다. 

---

✅ 인라인 선언을 활용하자.

ABAP 7.40부터는 인라인 변수 선언이 도입되었다. 인라인 선언은 변수를 사용하는 그 자리에서 바로 선언하는 방식이다.

인라인 선언을 하면 다음과 같은 장점이 있다.

• 코드 가독성 증가: 코드의 흐름을 읽기 쉽게 만든다.
• 복잡성 감소: 변수 타입을 일일이 알지 않아도 된다.
• 불필요한 선언 제거: 쓰지 않는 변수를 미리 선언할 필요가 없다.

DATA lv_total TYPE i.
lv_total = 100.   "전통 방식

DATA(lv_total) = 100.  "인라인 선언 방식

 

두 방식 모두 같은 결과를 내지만, 인라인 방식은 읽기 쉽고 유지보수가 편리하다.

---

✅ IF 문 같은 분기 로직 중간에 인라인을 선언하지 말자.

조건문 안에서 변수를 선언하면 가독성이 떨어지고, 디버깅 시 혼란을 줄 수 있다.

❌ Bad Case
IF condition = abap_true.
   DATA(result) = 'YES'.
ELSE.
   result = 'NO'. "
ENDIF.

✅ Good Case
DATA(result) TYPE string.
IF condition = abap_true.
   result = 'YES'.
ELSE.
   result = 'NO'.
ENDIF.

---

✅ 선언문을 무조건 묶지( " : " Chaining ) 말자.

❌ Bad Case
DATA: lv_user_name TYPE string,
           lv_user_age TYPE i,
           lv_sales_order TYPE vbeln,
           lv_sum_total TYPE i.

✅ Good Case
DATA: lv_user_name TYPE string,
           lv_user_age TYPE i.           
DATA lv_sales_order TYPE vbeln.
DATA lv_sum_total TYPE i.

 

보통 선언문을 묶는 Chaining 방식은 정의된 변수들이 논리적으로 관련이 있을시 사용되는 것으로, 무조건 선언을 묶었을 경우 변수들이 관련되어 있다고 잘못 이해 할 수 있기 때문에, 웬만하면 개별적으로 선언하는 것이 좋다.

---

✅ LOOP 안에서의 변수 선언 ASSIGNING FIELD-SYMBOL을 사용하자.

다음은 전통적인 ABAP 루프 방식이다. LOOP INTO는 테이블에서 한 줄씩 읽어오고, 읽은 값을 복사해서 변수에 담게된다. 사실 필요 없는 데이터 복사가 일어나기 때문에 성능 측면에서 비효율적이다. 또한 인터널 테이블의 데이터 수가 많고 컬럼 수도 많을 경우 LOOP의 성능저하가 생길 수도 있다. 

DATA lt_mara TYPE TABLE OF mara.
DATA ls_mara TYPE mara.
DATA lv_matnr TYPE mara-matnr.

LOOP AT lt_mara INTO ls_mara.
   "IF 조건절 
      lv_matnr = ls_mara-matnr.
ENDLOOP. 

 

이번엔 클린코드 방식인 ASSIGNING FIELD-SYMBOL(<fs>)을 사용하는 것으로, 복사 대신 테이블의 한 줄을 직접 참조한다.

LOOP AT lt_mara ASSIGNING FIELD-SYMBOL(<fs_mara>).
   "IF 조건절 
      lv_matnr = <fs_mara>-matnr.
ENDLOOP. 

• 성능 향상: 복사 없이 직접 참조하므로 빠르다. 
• 데이터 변경 용이: 별도의 MODIFY구문 없이 변경 값이 바로 반영된다.
• 가독성 개선: 변수 선언과 동시에 사용하여 흐름이 자연스럽다.
• 유지보수 용이: 필요한 곳에서만 변수 선언, 불필요한 전역 선언 감소한다.