Bimit написал(а):
Лучше проверять заполнение поля VBRK-FKART чем привязываться к sy-subrc
В данном случае не факт. Функция называется check, что предполагает проверку уже внутри нее, иначе надо было бы назвать select.
Также использование глобальных переменных vbrk, vbdkr для такой проверки тоже не очень хорошо.
Почему бы из это проверки не сделать pure метод? Подаем на вход номер, получаем sy-subrc.
А чтобы было удобней его использовать, можно вообще вынести в статический метод и результат возвращать через return параметр, тогда можно сразу в условия запихивать:
Code:
if zcl_billing_utils=>check_cancel( lv_vbeln ) IS INITIAL.
...
endif.