潜入python书中对kgp.py程序的具体怀疑

这里我指的是kgp.py程序的一部分 –

def getDefaultSource(self):
  xrefs = {}
  for xref in self.grammar.getElementsByTagName("xref"):
    xrefs[xref.attributes["id"].value] = 1
  xrefs = xrefs.keys()
  standaloneXrefs = [e for e in self.refs.keys() if e not in xrefs]
  if not standaloneXrefs:
    raise NoSourceError, "can't guess source, and no source specified"
  return '<xref id="%s"/>' % random.choice(standaloneXrefs)

self.grammar：解析的XML表示(使用xml.dom.minidom) –

<?xml version="1.0" ?>
<grammar>
<ref id="bit">
  <p>0</p>
  <p>1</p>
</ref>
<ref id="byte">
  <p><xref id="bit"/><xref id="bit"/><xref id="bit"/><xref id="bit"/>\
<xref id="bit"/><xref id="bit"/><xref id="bit"/><xref id="bit"/></p>
</ref>
</grammar>

self.refs：是由他们的id缓存上述XML密钥的所有引用

我对这段代码有两个疑问：

疑惑1：

  for xref in self.grammar.getElementsByTagName("xref"):
    xrefs[xref.attributes["id"].value] = 1
  xrefs = xrefs.keys()

eventaully外部参照将id值保存在列表中.难道我们不能简单地通过 –

  xrefs = [xref.attributes["id"].value 
           for xref in self.grammar.getElementsByTagName("xref")]

疑惑2：

  standaloneXrefs = [e for e in self.refs.keys() if e not in xrefs]
  ...
  return '<xref id="%s"/>' % random.choice(standaloneXrefs)

在这里,我们从self.refs中保存ref,我们在计算的外部参照中没有看到.但接下来不是创建一个< ref> element,我们正在创建一个< xref>具有相同的ID.这使我们向后退了一步,因为以后我们无论如何都要找到这个计算的< xref>的交叉引用.并最终到达< ref>.我们刚刚开始使用这个< ref>首先.

放弃

我绝不想试着在这本书上发表评论.我甚至没有资格.

我很喜欢读这本书的每一刻.我意识到很少有章节已经过了outdated,但我喜欢Mark Pilgrim’s的写作风格,我不能停止阅读.

解决方法:

Dive Into Python现在已经七年了(2004年出版),并不总是包含最现代的代码.所以你需要轻松一点：Dive Into Python 3可能是一个更好的选择.

你怀疑的建议1改变了代码的含义：将id放入字典的键中然后再将它们删除可以消除重复,而列表理解包括重复.现代的方法是使用集合理解：

 xrefs = {xref.attributes["id"].value 
          for xref in self.grammar.getElementsByTagName("xref")}

但这在2004年没有.

关于你的疑问2,我不完全确定我看到了问题.是的,在某种意义上说这是浪费,但另一方面,代码已经有了外部参照案例的处理程序,因此重新使用该处理程序而不是添加额外的特殊情况是有意义的.

该示例中还有其他几个代码可以进行现代化.例如,

source and source or self.getDefaultSource()

现在将是source或self.getDefaultSource().和线

standaloneXrefs = [e for e in self.refs.keys() if e not in xrefs]

将更好地表达为设置差异操作,例如：

standaloneXrefs = set(self.refs) - set(xrefs)

但随着语言变得更具表现力,就会发生这种情况：旧代码开始看起来相当不优雅.

潜入python书中对kgp.py程序的具体怀疑

相关推荐