uknownothingsnow / JsBridge

android java and javascript bridge, inspired by wechat webview jsbridge
9.76k stars 2.01k forks source link

建议优化 Message toJson 多余的代码 #177

Open j1453 opened 5 years ago

j1453 commented 5 years ago

发现一个挺有意思的事,#102 的修改没有生效,因为 jsonObject.put(RESPONSE_DATA_STR, getResponseData()); 并没有删除。

然后好在没有生效,测试了下 #102 的修改生效的情况下,解析的时候会出错,例如:

@Test
public void hasSimpleResponseData_toJson() {
    Message m = new Message();
    m.setResponseData("simple Str");
    String excepted = "{\"responseData\":\"simple Str\"}";
    assertEquals(excepted, m.toJson());
}

Expected :{"responseData":"simple Str"}
Actual   :{"responseData":"simple"}
j1453 commented 5 years ago

如果可以的话还是增加点测试用例吧,另外一开始 toJson 内部实现是借助 gson 来实现的,处于减少依赖项(兼容性更好一点,本身库就很小)的角度,去掉了 gson,但在测试的时候可以和 gson 对比结果