寫作于2017年2月17日

Issue1

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

Question1
這個API里面的實現(xiàn)和ORGMAN有什么關(guān)系？？放在這的意圖是？

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

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

Issue3
這里要加error handling的TODO:

Issue4
怎么能夠在LOOP里調(diào)用get_instance?!

Issue5
在我們的生產(chǎn)代碼里，還是會有可能進(jìn)入到這個COM API的分支：

Issue6
沒有必要用嵌套的TRY CATCH

放在一個層級結(jié)構(gòu)：

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錯誤處理不對。如果在方法調(diào)用后用sy-subrc evaluate, 必須要求方法里用non-OO的方式declare了exception。如果用TRY CATCH，要求方法體里raise OO的exception。

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

要獲取更多Jerry的原創(chuàng)文章，請關(guān)注公眾號"汪子熙":

總結(jié)

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

如果覺得生活随笔網(wǎng)站內(nèi)容還不錯，歡迎將生活随笔推薦給好友。