ace_lite_jsfwk /src/core/base/key_parser.cpp
太多 if (!strcmp(s, "***")) { return..} 语句了,满屏都是看着就头疼,圈复杂度大。。 能否考虑表驱动方式,用 map 来匹配返回值?
感谢你的意见。 由于部分轻量级设备空间受限,对应产品没有引入 STL 库支持,所以无法使用map,vector等C++数据结构,为了保持代码一致性,统一使用了目前的实现。 但确实可以考虑区分开来,在部分target上提供类表驱动实现以便提高可读和可维护性。
后续有计划会对该代码进行整改,您也可以提交PR帮助我们一起改进,谢谢~
Sign in to comment