84 Star 470 Fork 156

OpenHarmony / ace_lite_jsfwk

太多 if 语句

Task
Backlog
pengxiaochuan  Opened this issue

ace_lite_jsfwk /src/core/base/key_parser.cpp

太多 if (!strcmp(s, "***")) { return..} 语句了,满屏都是看着就头疼,圈复杂度大。。
能否考虑表驱动方式,用 map 来匹配返回值?

+1 5
Attachments
1706307 pengxiaochuan 1578958200 total 3 participants

Comments (2)

piggyguy_jdx 2020-09-11 18:17 member

感谢你的意见。
由于部分轻量级设备空间受限,对应产品没有引入 STL 库支持,所以无法使用map,vector等C++数据结构,为了保持代码一致性,统一使用了目前的实现。
但确实可以考虑区分开来,在部分target上提供类表驱动实现以便提高可读和可维护性。

crazyracing0726 2020-09-14 10:37 member

后续有计划会对该代码进行整改,您也可以提交PR帮助我们一起改进,谢谢~

Sign in to comment

Assignees
Labels
Not set
Projects
Milestones
Branches
Planed to start
Not set
Planed to end
Not set
Top level
Priority
C++
1
https://git.oschina.net/openharmony/ace_lite_jsfwk.git
git@git.oschina.net:openharmony/ace_lite_jsfwk.git
openharmony
ace_lite_jsfwk
ace_lite_jsfwk

Search

103611 48b8ff67 1899542 103622 4d02230c 1899542