m13253 / BiliDan

Play videos on Bilibili.com with MPV and Danmaku2ASS
http://m13253.blogspot.com/2014/06/watch-bilibili-with-biligrab-danmaku2ass.html
Other
513 stars 69 forks source link

Add a simple search tool #1

Closed 910JQK closed 10 years ago

910JQK commented 10 years ago

一個簡單的搜索腳本,還可以繼續擴展。

m13253 commented 10 years ago
  1. 你引入了一个新的依赖,Python有原生XML库,这是不必要的。(如果原生XML库真的达不到你的要求的话你可以用bs)
  2. 你没有处理解压deflate的情形。biligrab.py 里有URL获取函数,可以直接拿来用。
  3. 注意变量名。
  4. if __name__ == '__main__': 这个在你学Python的时候没有学过吗?
910JQK commented 10 years ago

改用 API ,已辦理這些問題。

910JQK commented 10 years ago

今週就到這,還有啥問題下週再辦。

910JQK commented 10 years ago

低級失誤……

m13253 commented 10 years ago

你的缩进有问题,过不了 PEP8。

m13253 commented 10 years ago

你给一个变量起名为 str,和内置类重名。还好你没有用内置的 str 类,不过会导致潜在的 bug。

910JQK commented 10 years ago

先不搞了,以後再說(挖坑太多真不好)