关于SAP UI5 CRM Reuse Fiori应用的代码审查

2024-04-01 05:26:46

目录

1. Question about maven build output

If we open the target folder:
What is exactly the usage of these four parts?

下图这四种资源有什么用处？

Reason why I asked: the content between 1 and 2 are the same, 3 and 4 are the same.

2. Useless CSS style

Are these CSS styles still used?

上面的CSS似乎没有用到。

I don’t think so since a pure text search shows no *.js are using them.

3. Why we choose sap.m.panel as aggregation container?

I know the children element contained in the aggregation would also be rendered in the runtime:

包含在这些aggregation的子控件在运行时被渲染。

However, why we choose panel as placeholder but not other one like XXXlayout?

4. Unnecessary require statement about “…NonLogTypeNoteListItem”

5. Duplicate code in two IF branch

6. Unused comment

It could be deleted now

7. Confusing variable name

Should be noteEditDialog instead.

8. Magic number still exists

9. Better constant definition approach

Follow this design:

Use this instead:

10. Hard code text cannot be translated

11. Performance of this.getProperty

Take this code for example:

In the runtime lifecycle, the value of these properties can never change:

Do a performance test on this.getProperty:

在this.getProperty方法加上性能数据的采集：

After clicking add button, there are more than 1200 times getProperty call, actually most can be avoided.

发现随便点击一个Add按钮，会触发一千多次的getProperty调用。而大多数其实都可以避免的。

If we call getProperty on these properties once in init, we can save 1200 * 0.5 = 0.6s

如果我们在onInit里只调用一次getProperty，然后把结果缓存起来，就可以节省1200 * 0.5 = 0.6s的时间。

12. Unused argument

13. Lengthy code

Didn’t try in js yet, can we use “return ( oValue1 – oValue2 )”?

要获取更多Jerry的原创文章，请关注公众号"汪子熙":

关于SAP UI5 CRM Reuse Fiori应用的代码审查相关推荐

SAP UI5 CRM Reuse Fiori应用 note.js代码审查结果
目录 1. Unused require statement Line4 could be deleted. 2. Unused code _dateBound = true Is this _dat ...
SAP UI5 的本地 Fiori sandbox Launchpad
源代码如下: <!DOCTYPE html> <html><head><meta http-equiv="X-UA-Compatible" ...
SAP UI5 应用开发教程之五十四 - 如何将本地 SAP UI5 应用配置到本地 Fiori Launchpad 中的试读版
一套适合 SAP UI5 初学者循序渐进的学习教程教程目录 SAP UI5 本地开发环境的搭建 SAP UI5 应用开发教程之一:Hello World SAP UI5 应用开发教程之二:SAP U ...
使用 SAP Fiori Tools 部署 SAP UI5 应用到 ABAP 服务器时遇到的各种错误和解决办法
错误1 (node:28340) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'customMiddleware ...
SAP UI5 Tools 运行了 fiori add deploy-config 命令之后，工程文件发生了哪些变化
我使用这篇文章使用 generator-easy-ui5 快速创建 SAP UI5 应用的工程结构介绍的工具,创建了一个 SAP UI5 应用,然后运行了 SAP UI5 Tools 提供的命令 ...
关于有朋友询问，Spartacus，Fiori，SAP UI5这些术语的关系
有朋友听了Jerry在2020年12月15日SAP全球技术大会上的演讲后,在我的微信群里提问: Spartacus用Angular来实现Fiori风格了?以后不用SAP UI5了? 一些问题:fior ...
SAP Fiori图标(icon)设计原理：一个可以查看 SAP UI5 所有可用图标的工具
When you are creating a new button and assign an icon to it, there is no way for you to know exactly ...
如何将SAP UI5应用配置到SAP云平台Fiori Launchpad里
在SAP云平台WebIDE里新建一个UI5应用并本地运行通过后,在webapp文件夹下打开manifest.json文件,进入navigation区域,semantic object维护成data,a ...
如何把SAP UI5应用部署到SAP云平台的Fiori Launchpad里去
(1) 首先在WebIDE里把开发好的SAP ui5应用部署到SAP Cloud Platform上: 在弹出窗里维护部署明细: 成功部署后,点击Register to SAP Fiori Launc ...

最新文章

热门文章