python3 中 json 的一个 bug?

2020-11-18 11:06:56 +08:00
 deplives

目前发现这样一个问题:

自定义一个 dict 子类,并且实现自定义 __bool__ 方法后 json.dumps() 会产生和预期不符的结果

话不多说,上代码

import json

class InnerDict(dict):
    def __init__(self, **kwargs):
        super().__init__(**kwargs)

    def __bool__(self):
    # 原始这里的逻辑是只要字典为空才返回 False,非空都返回 True
    # 但是如果改掉比如这里所有都返回 False
        return False
        
dct = InnerDict()
dct["hello] = "world"
print(dct) # 到这里都没有问题
json.dumps(dct, ensure_ascii=False, indent=4) # 但是这里,dumps 的结果就是 {}

原因分析:

jsonencoder.py 文件中 _make_iterencode 方法内有个闭包的方法 _iterencode_dict

在这方法中第一件事去判断 dct 是否为空,用了

if not dct:
    yield '{}'
    return

截图在这里 https://www.v2ex.com/i/5Xjw6YFt.jpeg

但是问题就在这里,这里会调用 __bool__ 判断,但是 __bool__ 的逻辑已经被改了,所以在上面的代码里这里一直为 {}

而且追踪了 _iterencode_dict 方法,发现三处调用处均判断了 dct 对象为 dict

这里的 if not dct 仅仅做了判断 dct 是否为空 dict

目前就是这么个情况,不知道这算不算一个 bug

2828 次点击
所在节点    Python
16 条回复
ruanimal
2020-11-18 11:12:46 +08:00
这不算 bug 吧。。

而且你这个继承违反了里氏代换原则
acmore
2020-11-18 11:13:55 +08:00
如果 `_iterencode_dict` 只管按自己的逻辑去判断是否为空,而不管你怎么实现 `__bool__`,这才是一个 Bug,因为实际上对 `__bool__` 的重载是无效的。

你没有正确实现 `__bool__` 的功能,自然也得不到期待的结果。
huangzhiyia
2020-11-18 11:15:32 +08:00
不算,我认为你破坏了设计上的传递性。
Vinty
2020-11-18 11:15:53 +08:00
对象为空,所以 dump 也是空的,这很符合逻辑啊
ofooo
2020-11-18 11:27:42 +08:00
不要弄 dict 的子类,确实有 bug,python 猫公众号前几天还讲过。

不要弄 python 官方类的子类的魔法函数,他继承顺序有一定的不一致性
ofooo
2020-11-18 11:29:27 +08:00
不过感觉你这个是自作自受吧。。。。人家这个逻辑没问题啊
xuboying
2020-11-18 12:54:44 +08:00
看了半天没理解 op 觉得哪里是 bug ?是对那个 if 判断的条件有异议么? op 期望是怎样的结果。
cz5424
2020-11-18 13:01:01 +08:00
这个问题类似于,我修改了支付宝客服端,加了个一百万,但是我买不了东西
Wincer
2020-11-18 13:01:12 +08:00
楼主意思是 if not dct 应该改成 if dct is not {}?
SjwNo1
2020-11-18 14:38:22 +08:00
那就再改 __bool__ 知道你满意为止哈哈
mringg
2020-11-18 14:40:42 +08:00
我想说人生已经够难了,不要难为自己了,就用些基本的用法不好么
no1xsyzy
2020-11-18 14:40:55 +08:00
dict 的 __bool__ 含义就是这个 dict 是否为空
_iterencode_dict 说:既然你说它是空,那它就是空咯,不反驳,'{}' 来
timothyqiu
2020-11-18 14:41:39 +08:00
说白了就是 garbage in, garbage out
cnrting
2020-11-18 15:35:23 +08:00
都用 python 了何苦再为难自己
lithbitren
2020-11-18 17:01:28 +08:00
只有在算法题里写字典树的时候会继承字典,其他情况下原则上都不会继承这些基础数据结构
krixaar
2020-11-18 17:11:02 +08:00
这是不是可以类比成后端偷偷改了接口返回值,前端崩了说是前端有 bug ?

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/726591

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX