tutorial2中非法的情况是不是没判断完全 about json-tutorial

[oreo-lp]

lept_parse_number()函数的其中一个判断应该改成这样吧：
if (*p == '0'){ p++; if (*p != '\0' && *p != '.' ) return LEPT_PARSE_INVALID_VALUE; }

[oreo-lp]

再补充一下，如果测试用例中添加了这个测试用例，原来的处理逻辑会认为这是一个合法的数字，实际上这并不是一个合法的数字：
TEST_ERROR(LEPT_PARSE_INVALID_VALUE, "-1E-10k");
所以，应该需要将判断语句修改成这样：
if (*p == 'e' || *p == 'E') { p++; if (*p == '+' || *p == '-') p++; if (!ISDIGIT(*p)) return LEPT_PARSE_INVALID_VALUE; for (p++; ISDIGIT(*p); p++); if(*p != '\0'){ return LEPT_PARSE_INVALID_VALUE; } }

[Zhirui-Zhang]

再补充一下，如果测试用例中添加了这个测试用例，原来的处理逻辑会认为这是一个合法的数字，实际上这并不是一个合法的数字： TEST_ERROR(LEPT_PARSE_INVALID_VALUE, "-1E-10k"); 所以，应该需要将判断语句修改成这样： if (*p == 'e' || *p == 'E') { p++; if (*p == '+' || *p == '-') p++; if (!ISDIGIT(*p)) return LEPT_PARSE_INVALID_VALUE; for (p++; ISDIGIT(*p); p++); if(*p != '\0'){ return LEPT_PARSE_INVALID_VALUE; } }

你这个测试用例应该改为：
TEST_PARSE_ERROR(LEPT_PARSE_ROOT_NOT_SINGULAR, "-1E-10k");，是可以通过测试的（个人测试成功），lept_parse_number函数解析到k处退出，此时v->u.n = strtod()结果为-1E-10。到最外层的if (*c.json != '\0')判断为true，最后返回 LEPT_PARSE_ROOT_NOT_SINGULAR，综上，我个人认为作者代码应该不需要修改

[1250890838]

没问题啊，只解析了0

[1797818494]

是的，我想错了