Issue1

這個函數的signature是別人寫好的,大家研究過consumer怎么消費這個FM的?
如果et_product_detail有多個entry,則es_product_detail只包含et_product_detail的最后一行. 這個…

Question1

這個API里面的實現和ORGMAN有什么關系？？放在這的意圖是？

Issue2

既然這個FM內只需要product header的division field，然后get_single_h_with_desc_by_guid確實支持只取required field， 為什么不用？

這個和上面那個CRMS4_ORGMAN_CHECK_API_OBJ的實現是完全一樣的，必須要分開實現背后有什么故事？

Issue3

這里要加error handling的TODO:

Issue4

怎么能夠在LOOP里調用get_instance?!

Issue5

在我們的生產代碼里，還是會有可能進入到這個COM API的分支：

Issue6

沒有必要用嵌套的TRY CATCH

放在一個層級結構：

Issue7

這行語句有什么用？

Question2

那為什么line 451不直接用cs_orderadm_i_wrk-ordered_prod？

Issue8

只需要product，為什么CDS view里要把所有field全部讀出來?

Issue9

從語義上說，為什么只取第一個uom？它是main uom？

Issue10

這種ABAP錯誤處理不對。如果在方法調用后用sy-subrc evaluate, 必須要求方法里用non-OO的方式declare了exception。如果用TRY CATCH，要求方法體里raise OO的exception。

要獲取更多Jerry的原創文章，請關注公眾號"汪子熙":

總結

以上是生活随笔為你收集整理的一些SAP S4CRM On-Premises开发的代码审查，2017-02-16的全部內容，希望文章能夠幫你解決所遇到的問題。

如果覺得生活随笔網站內容還不錯，歡迎將生活随笔推薦給好友。