107 Star 794 Fork 442

OpenHarmony / arkui_ace_engine_lite

 / 详情

太多 if 语句

已拒绝
任务
创建于  
2020-09-11 17:00

ace_lite_jsfwk /src/core/base/key_parser.cpp

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

评论 (2)

pengxiaochuan 创建了任务
展开全部操作日志

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

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

borne 任务状态待办的 修改为已拒绝

登录 后才可以发表评论

状态
负责人
项目
里程碑
Pull Requests
关联的 Pull Requests 被合并后可能会关闭此 issue
分支
开始日期   -   截止日期
-
置顶选项
优先级
预计工期 (小时)
参与者(3)
1706307 pxc89 1695955349
C++
1
https://gitee.com/openharmony/arkui_ace_engine_lite.git
git@gitee.com:openharmony/arkui_ace_engine_lite.git
openharmony
arkui_ace_engine_lite
arkui_ace_engine_lite

搜索帮助