minggeJS icon indicating copy to clipboard operation
minggeJS copied to clipboard

继续打脸, 1.7.3 版本中 $.toJSON 实现有问题, 导致字符串中的内容丢失

Open xqin opened this issue 9 years ago • 18 comments

<script type="text/javascript">

    var a = {x:",}"};

    alert($.toJSON(a));

</script>

转换后 内容里的 逗号 丢了.

image

xqin avatar Dec 17 '15 14:12 xqin

你觉得你加了 $ 之后还正确吗? 你有试过加了 $ 之后的代码吗?

xqin avatar Dec 17 '15 16:12 xqin

把你改后的代码提交上来, 你在这里发的代码,没有按 markdown 的格式写, 我不确定我这边复制出来的代码是否与你粘贴的时候一致.

所以请 push 你改后的代码到仓库里.

xqin avatar Dec 17 '15 16:12 xqin

var a = {x:'\u0010'},
b = $.toJSON(a),
c = JSON.stringify(a);

console.log(b, c, b === c);

不要觉得你那个处理好了, 你的 $.toJSON 就万事大吉了.

image

xqin avatar Dec 17 '15 16:12 xqin

呵呵。修了5次了……还嘴硬,README中居然说是别人阴你。

TooBug avatar Dec 18 '15 00:12 TooBug

做技术就好好做技术吧,都是台面上的东西,嘴硬没用,考虑不周没什么,修复即可,至于那个shearphoto,还是别提了,用的腾讯的类库做个上层应用,并不能叫自己的东西.

tabooc avatar Dec 18 '15 03:12 tabooc

如果不是太严格的话,我现在这样其实差不多过关了 你以为这样的态度, 何时能占有 jQuery 50% 的份额?

tree new bee搏出位, 请问你是 凤姐第二 吗?

xqin avatar Dec 18 '15 14:12 xqin

我在和你讨论这个 toJSON 的时候,有说过这个是和 jQuery 在进行对比吗?

xqin avatar Dec 18 '15 14:12 xqin