关于CRM one order函数的一些代码审查和改进建议
写作于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的原创文章,请关注公众号"汪子熙":
相关文章
- C++到Python全搞定,教你如何为FastDeploy贡献代码
- 生态学建模:增强回归树(BRT)预测短鳍鳗生存分布和影响因素|附代码数据
- 使用Git向码云提交代码
- 低代码开发平台有哪些比较好用的?
- 比较精简的Javascript拖动效果函数代码
- 根据地区不同显示时间的javascript代码
- aspbase64加解密函数代码
- js变量类型转换常用函数与代码[比较全]
- php输出表格的实现代码(修正版)
- 返回对象在当前级别中是第几个元素的实现代码
- javascript客户端生成MD5值的函数代码
- asp中将相对路径转换为绝对路径的函数代码
- C#jpg缩略图函数代码
- 用正则表达式判断字符串是汉字还是拼音的js函数代码
- asp去除html的函数代码分析附实例说明
- javascript中将Object转换为String函数代码(jsonstr)
- 用类的继承关系(重写父类的方法)实现简易后台代码模板
- php数组去重的函数代码
- 使用php判断浏览器的类型和语言的函数代码
- JS代码判断IE6,IE7,IE8,IE9的函数代码
- JS短路原理的应用示例精简代码的途径
- JavaScript代码应该放在HTML代码哪个位置比较好?