寫作于2017年2月17日

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？
![clipboard15](https://user-ima

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

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

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

ges.githubusercontent.com/5669954/54063458-7f750f00-4247-11e9-9966-0a4fea02c7cd.png)

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

總結

以上是生活随笔為你收集整理的关于CRM one order函数的一些代码审查和改进建议的全部內容，希望文章能夠幫你解決所遇到的問題。

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